Discussion :

[jalane]

Salut à tous,

J'ai un petit souci, vraiment petit je pense... J'ai une fonction et un main, ma fonction est correcte mais elle ne s'exécute pas comme je veux...

Fonction :

Code :

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

1
2
3
4
5
6
7
8
9
10
void afficheannonce(char annonce[][20], int taille){
    int i,nbreal;
     for(i=1;i<=taille;i++){
        printf("[%d] %s\n",i,annonce[i]);
    }
 
    srand(time(NULL));
    nbreal = (rand() % (5 - 1 + 1)) + 1;
    printf("[%d] %s\n",nbreal,annonce[nbreal]);
}

Dans le main je l'appelle 2 fois :

Code :

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

1
2
3
4
5
6
int main (int argc, char *argv[]){
    char just[10][10];
    char dmde[10][10];
 
    afficheannonce(just,5);
    afficheannonce(dmde,5);

Mon problème ce trouve au niveau du nombre aléatoire. Le nombre aléatoire marche bien mais le souci est qu'elle est toujours la même pour les 2 appellations de la fonction dans le main...

Code :

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

1
2
    afficheannonce(just,5);
    afficheannonce(dmde,5);

J'aimerai bien qu'elle change de temps en temps. Par exemple pour la 1ère, elle peut être 2 et pour la 2ème 3. pour afficher de annonces aléatoirement...

[Bktero]

Bonjour,

Essaye de mettre cette ligne srand(time(NULL)); dans le main() pour ne pas réinitialiser le générateur de nombre à chaque fois.

[jalane]

Merci beaucoup...

[Médinoc]

En complément d'information, le problème se produisait parce que srand(time(NULL)); était (dans l'immense majorité des cas) appelé deux fois dans la même seconde, et donc initialisait à chaque fois le générateur avec les deux mêmes valeurs.

[Sve@r]

Code c :

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

 nbreal = (rand() % (5 - 1 + 1)) + 1;

Salut

Déjà les parenthèses en rouge sont inutiles vu que le modulo est du même rang que la multiplication et division et donc qu'il est prioritaire sur l'addition.

Ensuite pour avoir un aléatoire plus régulièrement réparti, vaut mieux éviter le modulo et remplacer rand() % (MAX - MIN + 1) + MIN par rand() / RAND_MAX * (MAX - MIN + 1) + MIN...

[Médinoc]

Sve@r, sans un cast, ta formule vaudra zéro...

[Sve@r]

Sve@r, sans un cast, ta formule vaudra zéro...

Exact, j'avais oublié que rand() renvoyait un int. J'ai écrit en pensant "float". Désolé