Discussion :

[Deallyra]

Bonjour,

Pour l'un de mes projets universitaire, j'ai pour sujet de faire la comparaison de deux textes afin de récupérer la séquence commune la plus grande.

Pour ce programme, j'ai souhaité générer aléatoirement des mots dont la taille serait saisie au lancement.

Cependant, je n'arrive pas à obtenir des " rand() " différents.

Pourriez vous jeter un oeil à mon code pour m'expliquer ce qui ne va pas ?

Voici une archive contenant mon code

[diogene]

srand() ne doit être appelé qu'une fois, en général au début du programme. Il sert à initialiser le générateur aléatoire avec la "graine" spécifiée en argument.

Tu l'as mis dans la boucle for de integerGeneration(). Celle-ci s'exécute si rapidement que time(NULL) renvoie toujours la même valeur et le générateur esr réinitialisé toujpours à la même valeur.

[ok.Idriss]

Salut.

D'après ce que j'ai vu au premier coup d'œil :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
/**
 * Function returning the position of the letter
 *   who was crossed by parameter               
 *@param char letter, the letter                
 *@return int, the position of the letter in the alphabet
 */ 
int getPositionByLetter(char letter){                    
  //to do                                                
  letter=letter;                                         
  return 0;                                              
}

Cette fonction est censée te renvoyer sous la forme d'un int la position dans l'alphabet d'une lettre si j'ai bien compris. Seulement cette fonction renvoie toujours 0, donc toujours la même position ...

Edit : je viens aussi de m'apercevoir que cette fonction n'est pas utilisé ...
Pour régler le problème du srand, comme l'a expliqué diogene : enlève le de ta fonction integerGeneration () et met le au début du programme (dans le main).

[Deallyra]

Bonjour,

diogene,

Donc je dois "externaliser" le srand...
Est-ce une bonne façon de le faire dans le main ?

ok.Idriss

Cette fonction n'est pas encor implémentée à vrai dire ^^
Du fait que je n'en ai pas besoin (mais que je comptais me faire une mini librairie plus ou moins complète) je n'ai pas encor écrit son code.

[Melem]

Donc je dois "externaliser" le srand...
Est-ce une bonne façon de le faire dans le main ?

Du moment que tu appelles srand une fois seulement et ni plus ni moins dans ton programme et que cet appel se fait avant la première utilisation de rand, c'est bon. Donc tu peux faire l'appel dans main si l'appel remplit ces conditions.

A part ça, appeler exit juste devant le return de main est absolument inutile, tu peux l'enlever.

[Deallyra]

mon exit, enfin tout le code qui n'est pas commenté dans le else, c'est pour le débug.

Il me suffit d'ajouter ou enlever un slash pour que j'obtienne mon code "en production" ou mon code "en debug"
à la base, je n'avais pas ce système pour commenter/décommenter

edit : ok, j'ai compris ce que tu voulais dire ^^

[Deallyra]

J'ai donc fait la modification ( archive du premier lien mise à jour)

Cependant, bien qu'il me génère deux mots différents, il me génère les même mots d'un lancement du programme à un autre.

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
 
sasa@sasaki:/media/Lapinator/Complexite/_Sources/test$ ./rand 5 10
 
word: nwlrb
word: bmqbhcdarz
Words :
	nwlrb
	bmqbhcdarz
 
sasa@sasaki:/media/Lapinator/Complexite/_Sources/test$ ./rand 5 10
 
word: nwlrb
word: bmqbhcdarz
Words :
	nwlrb
	bmqbhcdarz

Puisque la graine est basée sur l'heure, ne devrais-je pas obtenir des résultats différents ?

[Melem]

Puisque la graine est basée sur l'heure, ne devrais-je pas obtenir des résultats différents ?

Exact, sauf que tu as oublié d'appeler srand (ou randInitialize) ...

[Deallyra]

... Tout à fait -_-"

Maintenant, je n'ai plus de répétition sauf si je lance le programme moins d'une second après.

Ce qui n'arrivera pas ^^

Merci beaucoup pour votre aide et vos remarques :3