Discussion :

[PastorTroy]

Qu'est-ce qui ne va pas dans ce code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
const char letters[27] = "abcdefghijklmnopqrstuvwxyz";
 
void rand_seed()
{
int seed = static_cast<int>(time(0));
srand(seed);
}
 
short inline RandomNbr(const int& lim_inf, const int& lim_sup)
{
return lim_inf + rand() % (lim_sup - lim_inf + 1);
}
 
char* GenerateString(const int& length)
{
char *String;
for (int i = 0; i <= length; i++)
String[i] = letters[RandomNbr(0, 27)];
String[length] = '\0';
return String;    
}

[droggo]

Soa,

Qu'est-ce qui ne va pas dans ce code :

Plein de choses.

Vérifie les limites, et ça ira mieux.

[FunK92]

Salut,

Quel est la réelle question ?

[PastorTroy]

En fait, lors de l'appel a GenerateString(), le programme plante et je ne sais pas vraiment ce qui se passe car ya pas de depasements de limites pour letters[] car c'est le nombre aleatoire est lim_inf <= x < lim_sup dans inclus dans [0, 27] avec le '\0'

[FunK92]

Salut,

Tu utilise String[i] sans lui allouer de mémoire (via malloc), alors que tu le déclare comme un pointeur :/

[Médinoc]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

static_cast<int>

Pas du C. C'est un bout de code C++.