Passing in a function with an argument under @selector

You searched the entire internet?

Did you read this:
http://developer.apple.com/documentation/Cocoa/Reference/Foundation/Classes/NSTimer_Class/Reference/NSTimer.html#//apple_ref/occ/clm/NSTimer/scheduledTimerWithTimeInterval:target:selector:userInfo:repeats:

And especially this:

The aSelector method has the following syntax:

- (void)myTimerFireMethod: (NSTimer*)theTimer

 

There are a few things that are wrong. First of all, the method you're performing must have the following syntax (as PhoneyDeveloper said):

Code:

-(void)myTimerFireMethod:(NSTimer*)theTimer

The design problem this creates for you should be apparent - you need the UIImageView object that you want fade. You could solve this in one of two ways (that come to mind, there's probably infinitely many) - if you're only ever going to be starting one timer at a time, you can simply have an ivar named "imageViewToFade" or something and do this:

Code:

-(void)deal:(UIImageView*)cardImage {
	imageViewToFade = cardImage;
	[NSTimer scheduledTimerWithTimeInterval:timeTaken*10 target:self selector:@selector(alphaDealImage:) userInfo:nil repeats:NO];
}

-(void)alphaDealImage:(NSTimer*)timer {
	imageViewToFade.alpha = 0.0;
}

Or, you could opt to use NSInvocation and avoid the imageViewToFade (which I personally think is 'messy', but that's just me). This would also be necessary if you're starting multiple fade timers.

N.B. - I don't use NSInvocation much, so make sure to check this code.

Code:

-(void)deal:(UIImageView*)cardImage {
	NSInvocation *invoc = [NSInvocation invocationWithMethodSignature:[UIImageView instanceMethodSignatureForSelector:@selector(setAlpha:)];
	[invoc setTarget:cardImage];
	[invoc setSelector:@selector(setAlpha:)];
	[invoc setArgument:0.0 atIndex:2];
	[NSTimer scheduledTimeWithTimeInterval:timeTaken*10 invocation:invoc repeats:NO];
}

 

Please see the documentation. It says that the method you give in the selector must follow that pattern.

http://developer.apple.com/document...imeInterval:target:selector:userInfo:repeats:

Oh, and I forgot the easiest solution out of them all. Wow I feel silly now. This is how I would do it - no silly NSInvocation use or instance variable needed.

Code:

-(void)deal:(UIImageView*)cardImage {
	[NSTimer scheduledTimerWithTimeInterval:timeTaken*10 target:self selector:@selector(alphaDealImage:) userInfo:cardImage repeats:NO];
}

-(void)alphaDealImage:(NSTimer*)timer {
	((UIImageView*)timer.userInfo).alpha = 0.0;
}

There you go! Much cleaner.

 

I'm 99% certain that your timer method does not have to receive the NSTimer if it does not want to - just make sure your @selector is correct:

Code:

-(void)deal:(UIImageView*)cardImage {
	[NSTimer scheduledTimerWithTimeInterval:timeTaken*10 target:self selector:@selector(alphaDealImage) userInfo:cardImage repeats:NO];
}

-(void)alphaDealImage {
	// do something
}

The above will work - the important thing is to make sure that the selector has no trailing colon, which implies it takes an argument.

Of course, if you're method relies on the user info, you will need to receive the NSTimer.

 

In the solution where you make an instance variable, where and how do you declare it?

For whatever reason I always get an EXC_BAD_ACCESS when I try to call a method on the ivar, unless I pass it in as a userInfo.

But I cant use the userInfo method because other functions need to access this variable too.