Как получить случайное значение с двух диапазонов?

Question

[CodeInside]

пишу функцию, которая получает символ с диапазона 'a'-'d' и возвращает любой другой символ с этого же диапазона.

char randomResponse(char correctAnswer)//принимает правильный вариант ответа, возвращает неверный
{
	srand ( time(NULL) );//сбросить "random"
	switch(correctAnswer)
	{
	case 'a': return (char)rand() %(int)'d' + (int)'b';//получение случайного символа с диапазона 'b'-'d'
	}
}

Как организовать для case 'b'?

Answer 1

[andrewjabber]

Это решение подходит для любого диапазона, например
range[] = "asdfjkl";

#include <stdio.h>
#include <time.h>
#include <stdlib.h>
#include <string.h>
#include <conio.h>

char range[] = "abcd";
int rangeLen;

char randomResponse(char correctAnswer)//принимает правильный вариант ответа, возвращает неверный
{
	int r = rand() % (rangeLen-1);
	if (r >= correctAnswer)
		++r;
	return range[r];
}

int main()
{
	srand(time(0));
	rangeLen = strlen(range);
	// test
	for (int i = 0; i < rangeLen; i++)
		printf("%c ", randomResponse(range[i]));

	_getch();
	return 0;
}

Comments

[Максим Мосейчук]

Какая-то белеберда. Зачем вы сравниваете индекс массива и символ ответа? Вы же килограммы с метрами не складываете? r всегда будет меньше, чем correctAnswer, т.к. минимальное значение correctAnswer == 'a' == 97, а максимальное r == 3.

Answer 2

[Дмитрий Сныткин]

Мне кажется лучше так

#include <iostream>
#include <ctime>
#include <cstdlib>

using namespace std;

char randomeResponse(char a)
{
    int r = rand()%4;
    while(r == (a - 'a')) r = rand() % 4;
    return char(r + 'a');
}

int main()
{
    srand(time(NULL));
    cout << randomeResponse('b');
    return 0;
}

Comments

[CodeInside]

Решил сделать вот так.
string randomResponse(char correctAnswer)//принимает правильный вариант ответа, возвращает строку с неверным вариантом ответа и его идентификатором (буква)
{
char responseIdent;//буква, идентификатор варианта ответа
srand ( time(NULL) );//сбросить "random"
responseIdent = (char)rand() %(int)'d' + (int)'a';//получение случайного символа с диапазона 'a'-'d'
if(responseIdent == correctAnswer)//свопадает ли случайный символ с верным вариантом?
{
randomResponse(correctAnswer);//рекурсия, пока символы не будут совпадать
}
else
{
switch(responseIdent)//возврат соответствующего варианта ответа с идентификатором
{
case 'a': return "a)" + response[qNum][0] + '\n'; break;
case 'b': return "b)" + response[qNum][1] + '\n'; break;
case 'c': return "c)" + response[qNum][2] + '\n'; break;
case 'd': return "d)" + response[qNum][3] + '\n'; break;
}
}
}
Надеюсь и такой вариант будет нормально работать. Беспокоит только то, что вероятность совпадения 25% и может много рекурсий прокрутиться, пока не выпадет левое значение.

[Дмитрий Сныткин]

Кстати srand достаточно вызвать едиожды. Ну и по поводу вашего решения - имхо, слишком громоздко. А что будете делать, если диапазон увеличиться?) Да и switch как то некошерно.

[Олег Цилюрик]

CodeInside:

Надеюсь и такой вариант будет нормально работать.

Не нужно надеяться...

Беспокоит только то, что вероятность совпадения 25% и может много рекурсий прокрутиться, пока не выпадет левое значение.

1. не нужна там рекурсия - вы точно то же могли и циклом сделать....
2. любая рекурсия должна иметь отчётливо выписанную ветку гарантированного завершения...
3. а в вашем случае рекурсия может запросто быть бесконечной, сожрёт, в конце концов, всю доступную память компьютера, и завершится аварийно.

[Олег Цилюрик]

CodeInside:

responseIdent = (char)rand() %(int)'d' + (int)'a';

а вас не смущает, что (int)'d' - это число 100, и что ожидается, если к 'a' добавить случайное число в диапазоне [0....100) ... скажем 'a' + 99 ?

Answer 3

[Александр Латышев]

Попробуйте вот так:

char x = 'a' + (char)rand() % ( 'd' - 'a' );
if( x >= correctAnswer )
{
    x++;
}
return x;

Answer 4

[Максим Мосейчук]

#include <boost/random/mersenne_twister.hpp>
#include <boost/random/uniform_int.hpp>

char randomResponse(const char correctAnswer) {
    static boost::random::mt19937 rng;
    static boost::random::uniform_int_distribution<char> uni('a', 'd');

    char result;

    while((result = uni(rng)) == correctAnswer);

    return result;
}

Для c++11:

#include <random>

char randomResponse(const char correctAnswer) {
    static std::mt19937 rng;
    static std::uniform_int_distribution<char> uni('a', 'd');

    char result;

    while((result = uni(rng)) == correctAnswer);

    return result;
}

Comments

[andrewjabber]

Господи, ну а BOOST-то зачем?! В С++11 есть свой mt19937 !
Да и зачем С++, если задача отлично решается на Си?

[Максим Мосейчук]

andrewjabber: откуда у всех такая боязнь буста? Автор просил на с++, а не си.