Discussion :

[s2a07]

Bonjour,
je ne comprends pas le code de la classe word du projet avec le

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
	int i = (int)(Math.random() * 100000);
		while(i > 336529){
			 i /= 2;
		}
	try {
			LineNumberReader fnr = new LineNumberReader(new FileReader(new File("file/word.txt")));
			int carac;
			this.word = "";
			this.secretWord= "";
			while((carac = fnr.read()) != -1){
				if(fnr.getLineNumber() == (i+1))
					break;
 
				else{
					if(fnr.getLineNumber() == i){
						this.word += (char)carac;						
					}
				}
			}

merci d'avance

[dinobogan]

Quelles sont les lignes qui te posent problème ?

[joel.drigo]

Salut,

D'un autre côté ce code est tellement abscons que ne pas le comprendre n'est pas si étonnant.

1. Dans cette partie :
   

   Code :

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

   1
   2
   3
   4
   int i = (int)(Math.random() * 100000);
   		while(i > 336529){
   			 i /= 2;
   		}

   i ne peut être supérieur à 100_000 (puisque Math.random() est compris entre 0 et 1.0 inclus (cf la doc)), donc ne risque pas d'être supérieur à 336_529 (va savoir à quoi cette valeur peut correspondre...)
2. Quant à la lecture, le code qui suit, elle consiste à lire le fichier caractère par caractère jusqu'à ce qu'on atteigne la ligne de numéro immédiatement supérieur à i et lorsqu'on est sur la ligne de numéro i, on concatène les caractères lus. Chose qu'on pourrait simplement faire, et plus efficacement, comme ça :
   
   

   Code :

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

   String word = Files.lines(Paths.get("file/word.txt")).skip(i).findFirst().orElse("").

   qui correspond en clair à lire le fichier lignes à lignes en passant les i premières lignes et renvoyer la première ligne trouvée ou chaîne vide si en trouve pas.

[Drowan]

Vu qu'il s'agit d'un jeu de pendu et que le fichier s'appelle word.txt, je suppose que i est utilisé pour obtenir un mot aléatoire au sein d'un dictionnaire. Ce dictionnaire étant le fichier word.txt qui contient un mot par ligne. On prend donc la i-ème ligne = le i-ème mot.

i ne peut être supérieur à 100_000 (puisque Math.random() est compris entre 0 et 1.0 inclus (cf la doc)), donc ne risque pas d'être supérieur à 336_529 (va savoir à quoi cette valeur peut correspondre...)

Donc suivant cette logique, la valeur 336 529 doit correspondre au nombre de mots dans ce dictionnnaire. Et la boucle est sensée empecher i d'être plus grand que cette limite. Cependant, comme expliqué par Joel, Il n'est pas possible pour i d'être plus grand que 100 000. La boucle est donc inutile, de plus il y a 236 529 mots qui sont inatégnables.

Un bon design serait de lire le nombre de lignes du fichier (pour connaitre le nombre de mots) et de stocker cette valeur dans une variable. Puis de tirer un nombre i aléatoire entre 0 et la valeur de cette variable.