UIAlertView kind of contradicts the UIActivityIndicator

Question

I'm downloading the images from the online image for cache when offline. When I open application, The image was downloaded all completely before UIAlertView pop. It's finish incorrectly. I want to make the UIAlertView pop before images download. Here there are my code below.

- (void) operatePopupUpdating {
    myAlert = [[UIAlertView alloc] initWithTitle:@"Updating database.." message:nil delegate:self cancelButtonTitle:nil otherButtonTitles: nil];
    [myAlert show];

    UIActivityIndicatorView *indicator = [[UIActivityIndicatorView alloc] initWithActivityIndicatorStyle:UIActivityIndicatorViewStyleWhiteLarge];

    indicator.center = CGPointMake(myAlert.bounds.size.width / 2, myAlert.bounds.size.height - 50);
    [indicator startAnimating];
    [myAlert addSubview:indicator];

    [self operateUpdate];
    [self updateImage];

    [self operationCompleted];
}

In updateImage, operateUpdate and operationCompleted Method

- (void)updateImage {
NSString *snake_ico_file = @"snake_img_icn_";
NSString *filePath = [self dataFile:[snake_ico_file stringByAppendingString:@"0.jpg"]];
int max_count = [[self readData] count];
if (![[NSFileManager defaultManager] fileExistsAtPath:filePath]) {
    for (int i = 0; i < max_count; i++) {
        NSString *path = @"http://exitosus.no.de/drngoo/image/";
        path = [path stringByAppendingFormat:@"%d",i];
        NSString *ico_path = [path stringByAppendingFormat:@"/ico"];

        //icon            
        NSArray *paths = NSSearchPathForDirectoriesInDomains(NSDocumentDirectory, NSUserDomainMask, YES);
        NSString *documentsDirectory = [paths objectAtIndex:0];
        NSString *filename = [snake_ico_file stringByAppendingFormat:@"%d.jpg"];
        NSString *localFilePath = [documentsDirectory stringByAppendingPathComponent:filename];
        NSData *thedata = [NSData dataWithContentsOfURL:[NSURL URLWithString:ico_path]];;
        [thedata writeToFile:localFilePath atomically:YES];
       }
}

-(void) operationCompleted
{
       [myAlert dismissWithClickedButtonIndex:0 animated:YES];

       UIAlertView *alert = [[UIAlertView alloc] initWithTitle:@"Message form System" message:@"Database was updated successful" delegate:nil cancelButtonTitle:@"OK" otherButtonTitles: nil];

       [alert show];
}

- (void)operateUpdate {

     NSString *filePath = [self dataFileSettingPath];
     int updatedVer = 0;
     int version = [self checkDatabase];
     if (version == -1) {
         updatedVer = [self createDataBase:0];
     } else {
         updatedVer = [self updateDataBase:version];
     }

     [self.settingInfo setObject:[NSString stringWithFormat:@"%d",updatedVer] forKey:@"DBVersion"];
   [self.settingInfo writeToFile:filePath atomically:YES];
}

How I fix them and work correctly?

Answer 1

The alertView will show only after your image downloading complete. One Solution to solve this is,

Create another class ImageDownLoad.h

in .h

@property(nonatomic,assign)id delegate;
@property(nonatomic,retain) NSString *path;

in .m

@synthesize delegate;
@synthesize path;

Create a method namely,

-(void)startDownloadImageWithUrl:(NSURL*)imageUrl withLocalFilePath:(NSSTring*)filePath{
       path = filePath;
       receiveData = [NSMutableData data];
       NSURLRequest *request = [[NSURLRequest alloc]initWithURL:imageUrl];
       NSURLConnection *connection = [[NSURLConnection alloc]initWithRequest:request];
 }

- (void)connection:(NSURLConnection *)connection didReceiveData:(NSData *)data {
        [receiveData appendData:data];
}

- (void)connectionDidFinishLoading:(NSURLConnection *)connection{
        [(id)delegate performSelector:@selector(SaveData:inFilePath:) withObject:receiveData withObject:path];
}

This will create connection and start download your image in a separate class. Then in your mai ViewController add bit of code like below.

  - (void)updateImage {
          NSString *snake_ico_file = @"snake_img_icn_";
          NSString *filePath = [self dataFile:[snake_ico_file stringByAppendingString:@"0.jpg"]];
          int max_count = [[self readData] count];
          if (![[NSFileManager defaultManager] fileExistsAtPath:filePath]) {
                 for (int i = 0; i < max_count; i++) {
                 NSString *path = @"http://exitosus.no.de/drngoo/image/";
                 path = [path stringByAppendingFormat:@"%d",i];
                 NSString *ico_path = [path stringByAppendingFormat:@"/ico"];

                      //icon            
                 NSArray *paths =NSSearchPathForDirectoriesInDomains(NSDocumentDirectory, NSUserDomainMask, YES);
                 NSString *documentsDirectory = [paths objectAtIndex:0];
                 NSString *filename = [snake_ico_file stringByAppendingFormat:@"%d.jpg"];
                 NSString *localFilePath = [documentsDirectory stringByAppendingPathComponent:filename];

                 ImageDownLoad *imageDown = [[ImageDownLoad alloc]init];
                 imageDown.delegate = self;
                 [imageDown startDownloadImageWithUrl:[NSURL URLWithString:ico_path] withLocalFilePath:localFilePath];

            }
   }

This method will be called each time when image download completes.

   -(void)SaveData:(NSData*)data inFilePath:(NSString*)filePath{

          [data writeToFile:filePath atomically:YES];

   }

Your UI wont delay by doing so :)