Question

I am trying to display a single UIPickerView in my app and depending on what button is selected a UIPickView will popup and populate with the relevant data for the button selected. No sure if it matters but one UIPickerView has three columns and one has two. I can get both of them to work perfectly when the are separate but the minute that I nest them into if statements it breaks.

On the third time when I select the button that fires the opposite UIPickerView it crashes and give me the error of NSRangeException and that it is trying to find the object at index 2 in an array [0…1]. My understanding is that the program thinks that there is an object in a spot in the array that doesn’t exist.

I have tried the [picker reloadAllComponents] method and have it called each time the button is selected but as stated above that only works once and then it quits. The strange thing is that if I select the button that I pressed for the second set of code it works perfectly. It is when I select a button that would make the UIPickerView have to change its view that it crashes.

I am really new to iOS development and Objective C and hope that someone out there knows what I am doing wrong. I will post any code that you guys need and will post a couple samples just to understand what is going on.

Here is one of the buttons methods that sets an int to 2 and also calls the method to show the picker.

-(IBAction)linePopupPicker{
picker = 2;
[self pickerShow];
}

This code is the other method which is pretty much the same as the other (I am aware that I haven’t implemented the UserDefaults into the second button yet)

-(IBAction)namePicker{
field  = 1;
picker = 1;
NSLog(@"The value of field is:%i", field);
NSUserDefaults *pickerViewSelectionDefaults = [NSUserDefaults standardUserDefaults];
[tipPicker selectRow:[pickerViewSelectionDefaults integerForKey:@"pickerViewSelectionKey"] inComponent:0 animated:NO];
[tipPicker selectRow:[pickerViewSelectionDefaults integerForKey:@"pickerViewNameColor"] inComponent:1 animated:NO];
[tipPicker selectRow:[pickerViewSelectionDefaults integerForKey:@"pickerViewNameFontSize"] inComponent:2 animated:NO];

[self pickerShow];
}

This is the method that calls the UIPickerView. I have it so that it emerges onto the view rather than being static. I thought that this would be the best place to reload the components.

-(IBAction)pickerShow{

[UIView beginAnimations:nil context:nil];
[UIView setAnimationDuration:0.5];
CGAffineTransform transform = CGAffineTransformMakeTranslation(0, 240);
pickerView.transform = transform;
[self.view addSubview:pickerView];


[tipPicker reloadAllComponents];


[UIView commitAnimations];  
}

Finally here is just a sample of the code that is used throughout the various methods that need to be called for the UIPickerView.

- (UIView *)pickerView:(UIPickerView *)pickerView viewForRow:(NSInteger)row forComponent:(NSInteger)component reusingView:(UIView *)view {
if (picker ==2) {
    switch (component)
    {
        case 0:
        {
            UILabel *lineColorLabel = [[[UILabel alloc] initWithFrame:CGRectMake(0, 0, 130, 44)] autorelease];
            lineColorLabel.backgroundColor = [UIColor clearColor];
            NSString *lineTempColor = [colorArray1 objectAtIndex:row]; 
            //NSLog(@"The String Color Name:%@", tempColor);
            UIColor *myColor1 = [UIColor performSelector:NSSelectorFromString(lineTempColor)];
            lineColorLabel.backgroundColor =  myColor1;
            return lineColorLabel;
        }
        case 1:
        {
            UILabel *alphaLabel = [[[UILabel alloc] initWithFrame:CGRectMake(0, 0,40, 44)] autorelease];
            alphaLabel.text = [alphaArray objectAtIndex:row];
            alphaLabel.font = [UIFont systemFontOfSize:18];
            alphaLabel.textAlignment = UITextAlignmentCenter;
            alphaLabel.backgroundColor = [UIColor clearColor];
            alphaLabel.shadowColor = [UIColor whiteColor];
            return alphaLabel;
        }
        default:
            break;
    }
}

else if (picker == 1) 
 {
    switch (component)
    {
        case 0:
        {
            UILabel *fontLabel = [[[UILabel alloc] initWithFrame:CGRectMake(0, 0, 130, 44)] autorelease];
            fontLabel.backgroundColor = [UIColor clearColor];
            fontLabel.shadowColor = [UIColor whiteColor];
            fontLabel.text = [fontNames objectAtIndex:row];
            NSString *font = [fontNames objectAtIndex:row];
            fontLabel.font = [UIFont fontWithName:font size:18.0];
            fontLabel.textAlignment = UITextAlignmentLeft;
            return fontLabel;
        }
        case 1:
        {


            UILabel *colorLabel = [[[UILabel alloc] initWithFrame:CGRectMake(0, 0, 35, 44)] autorelease];
            NSString *tempColor = [colorArray objectAtIndex:row]; 
            //NSLog(@"The String Color Name:%@", tempColor);
            UIColor *myColor = [UIColor performSelector:NSSelectorFromString(tempColor)];
            colorLabel.backgroundColor =  myColor;


            return colorLabel;
        }
        case 2:
        {
            UILabel *sizeLabel = [[[UILabel alloc] initWithFrame:CGRectMake(0, 0,40, 44)] autorelease];
            sizeLabel.text = [fontSizeArray objectAtIndex:row];
            sizeLabel.font = [UIFont systemFontOfSize:18];
            sizeLabel.textAlignment = UITextAlignmentCenter;
            sizeLabel.backgroundColor = [UIColor clearColor];
            sizeLabel.shadowColor = [UIColor whiteColor];
            return sizeLabel;
        }
        default:
            break;
    }

}

    return 0;
}

Here are the data methods:

- (NSInteger)numberOfComponentsInPickerView:(UIPickerView *)thePickerView { 
if (picker == 2) {
    return 2;
}else if (picker == 1) {
    return 3;
}

//return 3; 
}

- (NSInteger)pickerView:(UIPickerView *)thePickerView numberOfRowsInComponent:(NSInteger)component { // This method also needs to be used. This asks how many rows the UIPickerView will have.
if (picker == 2) {
    if (component == 0) {
        return [colorArray1 count];
    }
    return [alphaArray count];
}else if (picker == 1) {
    if (component == 0) {
        return [fontNames count];
    }else if (component == 1) {
        return [colorArray count];
    }
    return [fontSizeArray count];
}

}

I hope this post provides all of the information that is need to help me get this fixed. I really appreciate it and hope that I am just missing something really simple. Thank you in advanced.

I am really new to iOS development and Objective C so this problem is really stumping me.

Answer 1

case 1 in the if ( picker == 2) switch statement has an opening brace but no closing one.

EDIT

- (void) savePickerOne {
    [pickerViewSelectionDefaults setInteger:[tipPicker selectedRowInComponent:0] ForKey:@"pickerViewSelectionKey"];
    [pickerViewSelectionDefaults setInteger:[tipPicker selectedRowInComponent:1] ForKey:@"pickerViewNameColor"];
    [pickerViewSelectionDefaults setInteger:[tipPicker selectedRowInComponent:2] ForKey:@"pickerViewNameFontSize"];
}