Discussion :

[mazertys17]

Bonjour,

J'ai un pb avec un mot que j'arrive a afficher mais qui ne s'utilise pas dans le 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
23
24
25
26
 
 
 try{
                fIn = context.openFileInput("map0");
                isr = new InputStreamReader(fIn);
                isr.read(inputBuffer);
 
                String word = new String() ;
                String wordEnd = new String("end") ;
 
 
 
                    for (int c = 0; c < 255; c++) {
                        word += inputBuffer[c];
                        if (inputBuffer[c] == ' ') {
                            cleaner = true;
                            word = new String();                         
                        }
 
                    }
                    canvas.draw(  word ) ; // mon texte lu depuis le fichier affiche "end"
 
                    si j'essaye en revanche d'utiliser le mot exemple :
                   if ( word == "end" ) {
                   dosomthing() ;
                   }//ca ne marche pas

Ai-je fais une erreur ?

Merci si vous pouvez m'aider.

[tchize_]

on utilise .equals() pour tester l'égalité d'objets en java

[joel.drigo]

Salut,

Tout d'abord, on ne teste pas l'égalité entre chaîne avec == mais avec equals : if ( "end".equals(word) ) {. == c'est pour l'identité.
Ensuite, ce serait bien plus simple et plus propre d'encapsuler ton InputStreamReader dans un BufferedReader et d'utiliser sa méthode readLine(), voire d'utiliser un Scanner, puisque tu cherches des mots séparés par des espaces. En tout cas ça t'éviterait d'une part écrire ça [c]word += inputBuffer;, soit une concaténation entre une chaîne et un int, ce ne risque pas de donner le mot "end" (""+65, ça donne "65", pas "A" !), et d'autre part d'écrire des boucles en dur de 0 à 255 alors qu'il n'y aucune garantie qu'il y ait bien 255 caractères dans ton tableau.
Au lieu d'écrire String wordEnd = new String("end");, autant écrire String wordEnd = "end";, plus simple et concis, et l'autre écriture n'est que (très) rarement utile.
Enfin, si tu veux absolument lire caractère par caractère et concaténer, utilise plutôt un StringBuilder qu'une concaténation :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
int read = isr.read(inputBuffer);
 
if ( read!=-1 ) { 
   StringBuilder wordBuilder = new StringBuilder() ;
   for (int c = 0; c < read; c++) {
       if (inputBuffer[c] == ' ') {
           wordBuilder.setLength(0);
       }
       else {
           wordBuilder.append( (char)c );
       }
   }
}
String word=wordBuilder.toString();

[mazertys17]

Merci pour tous ces conseils et précisions.

Je vais étudier tout cela.

En fait j'aimerais lire les choses mots par mots. L'idéal serait de pouvoir lire des valeurs. Le but est de charger une map d'objet.

Le pb effectivement avec mon système est qu'un inputBuffer va être sans doute insuffisant pour lire une grande quantité d'objets. mais je n'ai pas trouvé comment lire avec InputStreamReader autrement...

[joel.drigo]

Le pb effectivement avec mon système est qu'un inputBuffer va être sans doute insuffisant pour lire une grande quantité d'objets. mais je n'ai pas trouvé comment lire avec InputStreamReader autrement...

Ce n'est pas que le buffer est trop petit : on ne lit jamais tout d'un coup, mais en plusieurs fois, la capacité du buffer limitant le maximum, et le buffer n'étant pas forcément rempli à chaque lecture.

Le plus simple étant de faire :

Code :

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

1
2
3
4
5
6
7
8
try(BufferedReader reader = new BufferedReader(new InputStreamReader(context.openFileInput("map0")))) {
   for(String line = reader.readLine(); line!=null; line=reader.readLine()) {
      final String[] words = line.split(" "); // la ligne contient des mots séparés par des espaces
      for(String word : words ) {
          // tu peux traiter tes mots
      }
   }
}

[mazertys17]

Merci pour ta réponse, je vais essayer.

Ce que je ne comprends pas, c'est comment savoir où reprendre la lecture quand le buffer a atteint le max de char ?
Si mon fichier fait 10 000 mots par exemples,

Code :

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

1
2
3
 fIn = context.openFileInput("map0");
                isr = new InputStreamReader(fIn);
                isr.read(inputBuffer);

Je me contente de lire que ce que mon inputBuffer peut contenir, étant un char ca reste limité sauf erreur.
Comment faire pour lire en plusieurs fois ?

[joel.drigo]

Il faut appeler plusieurs fois isr.read(inputBuffer). Et cette méthode retourne un entier, qui vaut -1 lorsqu'il n'y a plus rien à lire, et sinon le nombre de valeurs lues. Donc il faut appeler read=isr.read(inputBuffer) tant que read ne vaut pas -1, et utiliser read comme borne supérieure de la boucle de traitement du tampon, comme dans l'exemple que j'ai mis dans ma première réponse.

[mazertys17]

Merci beaucoup !

Ca marche.

Autre chose j'aimerais lire aussi des données int pour construire les objets...Sais tu comment savoir si le mot lu est un int ou un String ?

[joel.drigo]

Le plus simple est de le convertir en int et si ça soulève une NumberFormatException, tu sais que ce n'est pas un int. Tu peux aussi tester s'il ne contient que des chiffres et qu'il commence éventuellement par un - (s'il peut y avoir des nombres négatifs).
Si tu utilises un Scanner, ce sera plus simple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
try(Scanner = new Scanner(new InputStreamReader(context.openFileInput("map0")))) {
   while ( scanner.hasNext() ) {
        if ( scanner.hasNextInt() ) {
             int value = scanner.nextInt();
             // traitement des entiers
        }
        else {
             String word = scanner.next();
             // traiterment des mots
        }
   }
}

[mazertys17]

Sois loué, Joel.Drigo !!

ca marche. Merci a toi !