Code source d'un jeu de pendu java

Version imprimable

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

Code:

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

03/05/2018, 11h20
dinobogan

Quelles sont les lignes qui te posent problème ?
03/05/2018, 12h03
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:
  
  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:
  
  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.
03/05/2018, 13h26
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.

Citation:

Envoyé par joel.drigo

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.