STILL struggling with Crystal Ball extra credit

I created an NSArray property in the View Controller called predictionColors;

In the viewDidLoad method, I initialized the array with:

- (void)viewDidLoad {
     self.predictionColors = [[NSArray alloc] initWithObjects:[UIColor greenColor],[UIColor redColor],[UIColor blueColor],
                             [UIColor cyanColor],[UIColor purpleColor],[UIColor grayColor],[UIColor orangeColor],[UIColor magentaColor],
                             [UIColor brownColor],nil];
}

In the makePrediction method in the View Controller, I used the following to get the random colors:

- (void)makePrediction {
    NSInteger randomColorIndex = arc4random()%[self.predictionColors count];
    self.predictionLabel.textColor = [self.predictionColors objectAtIndex:randomColorIndex];
}

It may not be the proper way, but it worked well for me.

Hi Steve!

Thank you so much for your help! I thought the assignment asked for an NSArray in the "button pressed method" He never showed us anything called makePrediction. I'm still so lost. Thinking of starting at the beginning and taking the intro to programming course. Thank you anyway.

No problem! It depends on where you are at in the assignment. Initially you start out with a button, and the button pressed method would generate the prediction. Later in the assignment the code is refactored into a standalone method "makePrediction" which is called upon by both a motion gesture (shaking the phone) or by touch.

If you are using the button and buttonPressed method to generate a prediction, you can combine the above statements into one method to generate the different colors.

- (IBAction)buttonPressed {
    NSArray *predictonColors = [[NSArray alloc] initWithObjects:[UIColor greenColor],[UIColor redColor],[UIColor blueColor],
                             [UIColor cyanColor],[UIColor purpleColor],[UIColor grayColor],[UIColor orangeColor],[UIColor magentaColor],
                             [UIColor brownColor],nil];

    NSInteger randomColorIndex = arc4random()%[predictionColors count];
    self.predictionLabel.textColor = [predictionColors objectAtIndex:randomColorIndex];
}

You'll want to eventually refactor the above code so that the predictionColors is initialized in the viewDidLoad method and not each time the button is pressed for better memory management, but the above code should get you the random colors :).

I am working on this same extra credit code challenge Steve Livingston . Having a memory allocation issue? I chose to make color a property.

here is my property i created

@property (strong, nonatomic) NSArray *colors;

here is my implementation under -(void)[viewDidLoad]

 self.colors = [[NSArray alloc]initWithObjects:[UIColor redColor], [UIColor greenColor], [UIColor blueColor], [UIColor yellowColor], [UIColor orangeColor],  nil];

here is my implementation under [buttonPressed]

int randColors = arc4random_uniform(self.colors.count); self.predictionLabel.text = [self.colors objectAtIndex:randColors];

any issues with my code? Apologies I haven't figured out how to make those black code snippets in the comments like you guys did.