2010-08-20 3 views
0

L'application se bloque uniquement lors de la première rotation de mon téléphone (EXC_BAD_ACCESS). N'importe quand après tout sa sauce. Il se bloque seulement sur l'appareil aussi. Simulateur tout est bon.L'application se bloque lorsque je fais pivoter le téléphone pour la première fois

#import "ImageViewController.h" 
#define degreesToRadian(x) (M_PI * (x)/180.0) 
#define ZOOM_VIEW_TAG 100 
#define ZOOM_STEP 1.5 

@interface ImageViewController (UtilityMethods) 
- (CGRect)zoomRectForScale:(float)scale withCenter:(CGPoint)center; 
@end 

@implementation ImageViewController 
@synthesize imageView,url, enableLandscapeOrientation; 

/* 
// The designated initializer. Override if you create the controller programmatically and want to perform customization that is not appropriate for viewDidLoad. 
- (id)initWithNibName:(NSString *)nibNameOrNil bundle:(NSBundle *)nibBundleOrNil { 
    if ((self = [super initWithNibName:nibNameOrNil bundle:nibBundleOrNil])) { 
     // Custom initialization 
    } 
    return self; 
} 
*/ 


// Implement viewDidLoad to do additional setup after loading the view, typically from a nib. 
- (void)viewDidLoad { 
    [super viewDidLoad]; 

    // set up main scroll view 
    imageScrollView = [[UIScrollView alloc] initWithFrame:[[self view] bounds]]; 
    [imageScrollView setBackgroundColor:[UIColor blackColor]]; 
    [imageScrollView setDelegate:self]; 
    [imageScrollView setBouncesZoom:YES]; 
    [[self view] addSubview:imageScrollView]; 

    // add touch-sensitive image view to the scroll view 
    imageView = [[AsyncImageView alloc]initWithFrame:[self.view bounds]]; 
    if (enableLandscapeOrientation) { 
     imageView.showGhost = YES; 
    } 
    [imageView loadImageFromURL:self.url]; 
    [imageView setTag:ZOOM_VIEW_TAG]; 
    [imageView setUserInteractionEnabled:YES]; 
    [imageScrollView setContentSize:[imageView frame].size]; 
    [imageScrollView addSubview:imageView]; 
    [imageView release]; 

    UITapGestureRecognizer *singleTap = [[UITapGestureRecognizer alloc] initWithTarget:self action:@selector(handleSingleTap:)]; 
    UITapGestureRecognizer *doubleTap = [[UITapGestureRecognizer alloc] initWithTarget:self action:@selector(handleDoubleTap:)]; 
    UITapGestureRecognizer *twoFingerTap = [[UITapGestureRecognizer alloc] initWithTarget:self action:@selector(handleTwoFingerTap:)]; 

    [doubleTap setNumberOfTapsRequired:2]; 
    [twoFingerTap setNumberOfTouchesRequired:2]; 

    [imageView addGestureRecognizer:singleTap]; 
    [imageView addGestureRecognizer:doubleTap]; 
    [imageView addGestureRecognizer:twoFingerTap]; 

    [singleTap release]; 
    [doubleTap release]; 
    [twoFingerTap release]; 

    // calculate minimum scale to perfectly fit image width, and begin at that scale 
    float minimumScale = [imageScrollView frame].size.width/[imageView frame].size.width; 
    [imageScrollView setMinimumZoomScale:minimumScale]; 
    [imageScrollView setMaximumZoomScale:4.0f]; 
    //[imageScrollView setZoomScale:minimumScale]; 

    //[imageScrollView setMinimumZoomScale:0.5f]; 
    //[imageScrollView setMaximumZoomScale:2.0f]; 



    //[self.imageView loadImageFromURL:self.url]; 
} 



// Override to allow orientations other than the default portrait orientation. 
- (BOOL)shouldAutorotateToInterfaceOrientation:(UIInterfaceOrientation)interfaceOrientation { 

    if (interfaceOrientation == UIInterfaceOrientationLandscapeLeft ||interfaceOrientation == UIInterfaceOrientationLandscapeRight) { 

     imageScrollView.frame = CGRectMake(0,0,480,300); //self.view.bounds; 
     imageView.frame = CGRectMake(0,0,480,300); //self.view.bounds; 
    } 
    else { 
     imageScrollView.frame = CGRectMake(0,0,320,460); 
     imageView.frame = CGRectMake(0,0,320,460); 
    } 


    //return (interfaceOrientation == UIInterfaceOrientationPortrait); 
    if (enableLandscapeOrientation) { 
     [[self navigationController] setNavigationBarHidden:UIInterfaceOrientationIsLandscape(interfaceOrientation) animated:YES]; 
     return YES; 
    } 
    else { 
     return NO; 
    } 
} 


- (void)dealloc { 
    [imageView release];imageView=nil; 
    [url release];url=nil; 
    [super dealloc]; 
} 

#pragma mark UIScrollViewDelegate methods 

- (UIView *)viewForZoomingInScrollView:(UIScrollView *)scrollView { 
    return [imageScrollView viewWithTag:ZOOM_VIEW_TAG]; 
} 

/************************************** NOTE **************************************/ 
/* The following delegate method works around a known bug in zoomToRect:animated: */ 
/* In the next release after 3.0 this workaround will no longer be necessary  */ 
/**********************************************************************************/ 
- (void)scrollViewDidEndZooming:(UIScrollView *)scrollView withView:(UIView *)view atScale:(float)scale { 
    [scrollView setZoomScale:scale+0.01 animated:NO]; 
    [scrollView setZoomScale:scale animated:NO]; 
} 

#pragma mark Utility methods 

- (CGRect)zoomRectForScale:(float)scale withCenter:(CGPoint)center { 

    CGRect zoomRect; 

    // the zoom rect is in the content view's coordinates. 
    // At a zoom scale of 1.0, it would be the size of the imageScrollView's bounds. 
    // As the zoom scale decreases, so more content is visible, the size of the rect grows. 
    zoomRect.size.height = [imageScrollView frame].size.height/scale; 
    zoomRect.size.width = [imageScrollView frame].size.width/scale; 

    // choose an origin so as to get the right center. 
    zoomRect.origin.x = center.x - (zoomRect.size.width/2.0); 
    zoomRect.origin.y = center.y - (zoomRect.size.height/2.0); 

    return zoomRect; 
} 

- (void)handleSingleTap:(UIGestureRecognizer *)gestureRecognizer { 
    // single tap does nothing for now 
} 

- (void)handleDoubleTap:(UIGestureRecognizer *)gestureRecognizer { 
    // double tap zooms in 
    float newScale = [imageScrollView zoomScale] * ZOOM_STEP; 
    CGRect zoomRect = [self zoomRectForScale:newScale withCenter:[gestureRecognizer locationInView:gestureRecognizer.view]]; 
    [imageScrollView zoomToRect:zoomRect animated:YES]; 
} 

- (void)handleTwoFingerTap:(UIGestureRecognizer *)gestureRecognizer { 
    // two-finger tap zooms out 
    float newScale = [imageScrollView zoomScale]/ZOOM_STEP; 
    CGRect zoomRect = [self zoomRectForScale:newScale withCenter:[gestureRecognizer locationInView:gestureRecognizer.view]]; 
    [imageScrollView zoomToRect:zoomRect animated:YES]; 
} 




@end 
+0

Publiez le plantage. – bbum

+0

et également poster où vous initialisez imageView et imageScrollView – jmont

+0

code complet affiché. crash était dans un journal de plantage, avec aucune donnée lisible sauf pour EXC_BAD_ACCESS –

Répondre

1

Dans viewDidLoad vous allouez la imageView et à la fin vous désallouez, donc le conserver nombre de imageView est 0. imageView a été désallouée, et c'est la raison pour laquelle vous recevez un EXC_BAD_ACCESS dès que vous voulez accéder imageView la prochaine fois. Vous mettez le [imageView dealloc] sur votre viewDidLoad et tout devrait bien fonctionner.

Et juste une autre chose que j'ai remarqué: Vous avez oublié de libérer l'affichage de défilement dans -(void)dealloc.

+0

donc, fondamentalement, ne publie pas imageView dans dealloc() car je le libère déjà dans viewDidLoad? –

+0

Vous devez libérer imageView UNIQUEMENT dans dealloc(). – burki

Questions connexes