Create a non-repeating random number [duplicate] - ios

Question

This question already has answers here:
Non repeating random numbers in Objective-C
(6 answers)
Closed 8 years ago.
I would like to know how to make it so that the same picture is not chosen twice in a row. Lets say pictures are given a number 1-3. If picture 1 is chosen, then picture 1 will not be chosen next. If picture 3 is chosen, then picture 1 can be chosen again, and so on.
I know that I would have to use a while statement, except I don't know how. Heres what I have as of now:
- (void)chooseBackgroundImage{
if(thisNumber % 10 == 0){
int chooseBackgroundImage = arc4random() % 7;
switch (chooseBackgroundImage) {
case 0:
backgroundImage.image = [UIImage imageNamed:#"CyanToYellowBackground.png"];
break;
case 1:
backgroundImage.image = [UIImage imageNamed:#"GreenToBlueBackground.png"];
break;
case 2:
backgroundImage.image = [UIImage imageNamed:#"OrangeToGreenBackground.png"];
break;
case 3:
backgroundImage.image = [UIImage imageNamed:#"OrangeToPurpleBackground.png"];
break;
case 4:
backgroundImage.image = [UIImage imageNamed:#"PurpleToCyanBackground.png"];
break;
case 5:
backgroundImage.image = [UIImage imageNamed:#"RedToBlueBackground.png"];
break;
case 6:
backgroundImage.image = [UIImage imageNamed:#"YellowToRedBackground.png"];
break;
}
}
}
I've also tried using:
- (void)chooseBackgroundImage{
if(slogansGenerated % 10 == 0){
int chooseBackgroundImage = arc4random() % 7;
while(chooseBackgroundImage == oldChooseBackgroundImage){
switch (chooseBackgroundImage) {
case 0:
backgroundImage.image = [UIImage imageNamed:#"CyanToYellowBackground.png"];
break;
case 1:
backgroundImage.image = [UIImage imageNamed:#"GreenToBlueBackground.png"];
break;
case 2:
backgroundImage.image = [UIImage imageNamed:#"OrangeToGreenBackground.png"];
break;
case 3:
backgroundImage.image = [UIImage imageNamed:#"OrangeToPurpleBackground.png"];
break;
case 4:
backgroundImage.image = [UIImage imageNamed:#"PurpleToCyanBackground.png"];
break;
case 5:
backgroundImage.image = [UIImage imageNamed:#"RedToBlueBackground.png"];
break;
case 6:
backgroundImage.image = [UIImage imageNamed:#"YellowToRedBackground.png"];
break;
}
int oldChooseBackgroundImage = chooseBackroundImage
}
}
But nothing seems to work. Is there any way to create a non-repeating random number?

Answer 1

The following is probably random enough for you needs:
First add an instance variable, say, lastChosenBackgroundImage.
After:
int chooseBackgroundImage = arc4random() % 7;
add:
if(chooseBackgroundImage == lastChosenBackgroundImage)
chooseBackgroundImage = (chooseBackgroundImage + 1) % 7; // same as last time, move to next choice
lastChosenBackgroundImage = chooseBackgroundImage; // remember for next time
This does mean that picking the next image is twice as probable as picking any of the other ones, but I suspect that will not be a significant issue for your use case.

Answer 2

I recon you're looking for something like this.
-(int)getNonRepeatedRandom{
int randomNumber = -1;
do{
randomNumber = arc4random_uniform(7);
} while (randomNumber == oldRandomNumber);
oldRandomNumber = randomNumber; //set the random number to old random so you can check it on the next run.
return randomNumber;
}
oldRandomNumber will have to be an iVar for this to work though.
You should use arc4random_uniform instead of arc4random modulus to get rid of modulous bias.

Answer 3

Use this utility function to give you random integers between a range (in your case 0 and 6) -
#define MAX_ATTEMPTCOUNT 10
// Provides a random number between the range (both inclusive).
+ (int)randomIntegerInRange:(int)fromInt toInteger:(int)toInt excluding:(NSArray *)excludeNumbers {
NSAssert((toInt - fromInt) > 0 && (!excludeNumbers ? YES : (toInt - (fromInt - 1)) >= [excludeNumbers count]), #"Invalid range");
static int randomAttempts = 0;
srandom(time(NULL));
int randomInteger = fromInt + random() % (toInt - (fromInt - 1));
if (excludeNumbers != nil) {
for (NSNumber *number in excludeNumbers) {
if ([number intValue] == randomInteger) {
if (randomAttempts == MAX_ATTEMPTCOUNT) {
// Reached the maximum attempt count to get the random number but failed to find one.
break;
}
else {
// Recursive call to get obtain the next number.
++randomAttempts;
randomInteger = [self randomIntegerInRange:fromInt toInteger:toInt excluding:excludeNumbers];
}
break;
}
}
if (randomAttempts >= MAX_ATTEMPTCOUNT) {
// Pick up the first number that's not there in visited words.
randomAttempts = 0; // Reset the counter for next attempt.
randomInteger = fromInt;
for (; randomInteger <= toInt; ++randomInteger) {
bool found = NO;
for (NSNumber *number in excludeNumbers) {
if ([number intValue] == randomInteger) {
// Found the number.
found = YES;
break;
}
}
if (!found) break;
else continue;
}
}
}
return randomInteger;
}
Add the returned integer in an array (array of excluded numbers) which is actually passed as parameter (excludeNumbers).
Let me know if any issue (I made it for one of my project so may not completely suit your need so feel free to modify it or ask me)