Array count error

Question

I have two methods. From the first method I am sending an array to the second method. In my first method [array count] value is 2. But in the seconf method the value is 1. But it should be the same in both method. I know its a silly mistake. But I dont understand exactly where I have done the mistake.

First method:

-(void)uploadOverlayManually: (NSMutableArray *)path{

NSFileManager *fileManager = [NSFileManager defaultManager];

NSLog(@"Array count #1: %d",[path count]);

for (int i =0; i < [path count]; i++) {

    imagePath = [path objectAtIndex:i];

    NSString *infoPath = [[imagePath stringByDeletingPathExtension] stringByAppendingPathExtension:@"info"];
    NSData *infoData = [[NSMutableData alloc] initWithContentsOfFile:infoPath];

    NSKeyedUnarchiver *unarchiver = [[NSKeyedUnarchiver alloc] initForReadingWithData:infoData];
    MediaInformation *currentInfo = [unarchiver decodeObjectForKey:@"info"];
    [unarchiver finishDecoding];

    UIImage *baseImage = [UIImage imageWithContentsOfFile:imagePath];
    NSData *data = [root addImageOverlay:baseImage withInfo:currentInfo andPath:imagePath];

    [data writeToFile:imagePath atomically:YES];

    fileUpload = [[NSMutableArray alloc] init];

    [fileUpload addObject:imagePath];

}

[self upload:fileUpload];

}

Second method:

-(void)upload:(NSArray*)filePaths{

if (![[DBSession sharedSession] isLinked]) {

    [[DBSession sharedSession] linkFromController:root]; //root
}

NSLog(@"Array count #2: %d",[filePaths count]);

}

Wain · Accepted Answer · 2014-02-07 11:14:05Z

1

This line

fileUpload = [[NSMutableArray alloc] init];

inside your loop recreates the array.

It should be before the loop so you add an object on each iteration.

answered Feb 7, 2014 at 11:14

Wain

119k15 gold badges132 silver badges152 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

lukaswelte · Accepted Answer · 2014-02-07 11:16:11Z

You instantiate a new array on each loop so it can always be just 1

To achieve the intended behavior use (moved instantiation out of the loop):

-(void)uploadOverlayManually: (NSMutableArray *)path{

NSFileManager *fileManager = [NSFileManager defaultManager];

NSLog(@"Array count #1: %d",[path count]);

fileUpload = [[NSMutableArray alloc] init];

for (int i =0; i < [path count]; i++) {

    imagePath = [path objectAtIndex:i];

    NSString *infoPath = [[imagePath stringByDeletingPathExtension] stringByAppendingPathExtension:@"info"];
    NSData *infoData = [[NSMutableData alloc] initWithContentsOfFile:infoPath];

    NSKeyedUnarchiver *unarchiver = [[NSKeyedUnarchiver alloc] initForReadingWithData:infoData];
    MediaInformation *currentInfo = [unarchiver decodeObjectForKey:@"info"];
    [unarchiver finishDecoding];

    UIImage *baseImage = [UIImage imageWithContentsOfFile:imagePath];
    NSData *data = [root addImageOverlay:baseImage withInfo:currentInfo andPath:imagePath];

    [data writeToFile:imagePath atomically:YES];

    [fileUpload addObject:imagePath];

}

[self upload:fileUpload];

rahul · Accepted Answer · 2014-02-07 12:47:32Z

0

You are allocating Array inside of a loop please allocate it in viewDidLoad or in any function that only called once.

answered Feb 7, 2014 at 12:47

rahul

1785 bronze badges

Collectives™ on Stack Overflow

Array count error

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related