iPhone app development: UIProgressView won't update within a for loop

Hi AGoodKeenMan,

Thank you for your comment. Do you think you might elaborate just a bit on exactly how to use performSelectorInBackground:@selector() and performSelectorOnMainThread:@selector() ? I'm not quite sure how to use those. Thanks for your help!

Ooh, thank you for the code. I just popped into work (I don't check this account at home) so give me a little while to try out what you have (I'm not ignoring your comments).

Hey,

Thanks again for the code. I tried it out and I'm not getting any changes in the result. I had a couple questions:

1) How come you're never using the variable "pool"? Does that variable have any purpose?
2) How come you're setting waitUntilDone to NO, isn't the whole point to make it wait until the update is done and only then continue? However, I tried it with it set to YES and it didn't make any difference at all.
3) Did that code work for you?

I'm very frustrated. This seems to be too hard for what it is - if this is as hard as it appears to be then Apple made a huge mistake with this UIProgressView class. People should be able to update a progress bar without having to - sleep the main thread - or any other voodoo.

Here's the basic code as I have it. Any other ideas?

float fProgressValue;
 
 
 
-(IBAction)PushDownloadImageButton:(id)sender
{
	fProgressValue = 0.0;
	[self performSelectorOnMainThread:@selector(SetProgressBar) withObject:nil waitUntilDone:YES];
	
	[self performSelectorOnMainThread:@selector(DownloadNow) withObject:nil waitUntilDone:YES];
}
 
 
 
-(void)DownloadNow
{
	NSAutoreleasePool* pool = [[NSAutoreleasePool alloc] init];
 
	// Other code here...
 
		for (iDownloadIteration = 0; iDownloadIteration < iIterations; iDownloadIteration++)
		{
			//[NSThread sleepForTimeInterval:0.1]; //pause the background thread
			fProgressValue = (((float)iDownloadIteration)/((float)iIterations));
			//[self SetProgressBar];
			[self performSelectorOnMainThread:@selector(SetProgressBar) withObject:nil waitUntilDone:NO];
 
			// Other stuff here...
		}
 
	fProgressValue = 0.5;
	[self performSelectorOnMainThread:@selector(SetProgressBar) withObject:nil waitUntilDone:NO];
	
	[pool release];
}
 
 
 
-(void)SetProgressBar
{
	[DownloadProgressView setProgress:fProgressValue];
}
                                              

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:
27:
28:
29:
30:
31:
32:
33:
34:
35:
36:
37:
38:
39:
40:
41:
42:

Sorry I didn't mean to overwhelm you. Please don't get put off, its probably my fault for not explaining things well enough.

1. Every new thread we start should have an autorelease pool and since your downloadNow method is run in  a background thread it needs its own autorelease pool. You can read more about it here:
http://developer.apple.com/DOCUMENTATION/Cocoa/Conceptual/MemoryMgmt/Articles/mmAutoreleasePools.html#//apple_ref/doc/uid/20000047

2. If waitUntilDone is set to YES, the background thread will wait until setProgressBar returns before it resumes execution. Which would halt the download for a short period of time and we don't want that. You can read more about NSObject here:
http://developer.apple.com/DOCUMENTATION/Cocoa/Reference/Foundation/Classes/NSObject_Class/Reference/Reference.html#//apple_ref/occ/instm/NSObject/performSelectorOnMainThread:withObject:waitUntilDone:

3. Yes, I just tested it in a new project.

The reason I sleep the background thread is just to simulate your download process and give you some visual feedback, it is not necessary for your solution, however if you don't have some sort of delay in this example the progress bar will appear to go from 0 to 1 instantly, because without anything for the for loop to do, it will execute very fast.

In your code above what is the value of iIterations?

I am not at work at the moment, but in the morning I will post a link so you can download myTest example.

EE won't allow me to attach the example project, so you can download it from my iDisk here:
files.me.com/robcarruthers/dbhrbq
(this link will expire in 30 days)