Question

I have a method like the following:

- (void)add:(id)sender {

MyAddViewController *controller = nil;

controller = [[MyAddViewController alloc] initWithAddType:1];

//controller.delegate = self; 

// Do some animation. Slide it in from the right side of the screen.

    CGPoint initialCenter = self.view.center;
    controller.view.center = 
        CGPointMake(initialCenter.x + 320, initialCenter.y);

    // Add the subview now with it's center + 320 off the screen.
    [self.view addSubview:controller.view];

    [UIView beginAnimations:@"animation" context:NULL];
    [UIView setAnimationDuration:0.35];
    [UIView setAnimationDelegate:self];
    controller.view.center = CGPointMake(initialCenter.x, initialCenter.y); 
  //[UIView setAnimationDidStopSelector:@selector(aMethodToBeCalledAfterAnimationEnd:finished:context:)];
    [UIView commitAnimations];

  //[controller release];
    }

You see I have a controller release as the last line in my add method. If I uncomment this line the animation completely dies and it just dumps the view into place with no animation.

Is there a better way to do release without having to create another method which does cleanup via [UIView setAnimationDidStopSelect:@selector(aMethodToBeCalledAfterAnmiationEnd… ?

Or can I do something like setAnimationDidStopSelector:@selector([controller release]) ? 🙂

Thanks in advance!

Answer 1

Using setAnimationDidStopSelector: is the proper way to solve the generic problem of releasing resources after an animation completes.

Take a step back and reconsider what you’re doing here, though. Why are you looking to free the controller you just created? If you are only creating the controller for the purpose of getting at the view, that isn’t the right way to do it. Build yourself a factory method to create your view, or use the methods in NSBundle to load the view from a NIB.