Lire toutes les lignes avec BufferedReader

Question

Je veux taper un texte multiligne dans la console en utilisant un BufferedReader et quand j'appuie sur "Entrée" pour trouver la somme de la longueur du texte entier. Le problème est qu'il semble que je suis dans une boucle infinie et quand j'appuie sur "Enter" le programme ne se termine pas. Mon code est ci-dessous:

InputStreamReader instream = new InputStreamReader(System.in); 
BufferedReader buffer = new BufferedReader(instream); 

    line= buffer.readLine(); 

    while (line!=null){ 
     length = length + line.length(); 
     line= buffer.readLine(); 
    } 

Pourriez-vous s'il vous plaît me dire ce que je fais mal?

Answer 1

La façon idiomatiques de lire toutes les lignes est while ((line = buffer.readLine()) != null). Aussi, je suggère un try-with-resources statement.Quelque chose comme

try (InputStreamReader instream = new InputStreamReader(System.in); 
     BufferedReader buffer = new BufferedReader(instream)) { 
    long length = 0; 
    String line; 
    while ((line = buffer.readLine()) != null) { 
     length += line.length(); 
    } 
    System.out.println("Read length: " + length); 
} catch (Exception e) { 
    e.printStackTrace(); 
} 

Si vous voulez mettre fin à la boucle lorsque vous recevez une ligne vide, ajoutez un test que dans la boucle while

while ((line = buffer.readLine()) != null) { 
    if (line.isEmpty()) { 
     break; 
    } 
    length += line.length(); 
} 

JLS-14.15. The break Statement dit

Une déclaration break transfère le contrôle d'une instruction englobante.

Answer 2

line ne sera pas nulle lorsque vous appuyez sur Entrée; Ce sera une chaîne vide.

Prenez note de ce que l'BufferedReader JavaDoc dit au sujet readLine():

Reads une ligne de texte. Une ligne est considérée comme terminée par un saut de ligne ('\ n'), un retour chariot ('\ r') ou un retour chariot suivi immédiatement d'un saut de ligne.

Et readLine() rendements:

une chaîne contenant le contenu de la ligne, sans compter tous les caractères de fin de lignes, ou une valeur nulle si la fin du flux est atteinte

Ainsi, lorsque vous appuyez sur [Entrée], vous donnez au BufferedReader une nouvelle ligne contenant uniquement \n, \r ou \r\n. Cela signifie que readLine() renverra une chaîne vide.

Donc, essayez quelque chose comme ceci:

InputStreamReader instream = new InputStreamReader(System.in); 
BufferedReader buffer = new BufferedReader(instream); 

line = buffer.readLine(); 

while((line != null) && (!line.isEmpty())){ 
    length = length + line.length(); 
    line = buffer.readLine(); 
} 

Answer 3

réponse Snarky: ce que vous faites mal crée seulement 2 objets Java pour faire quelque chose ... si vous cherchez, vous pouvez probablement trouver quelques plus de classes qui étendent BufferedReader ou ExtendedBufferReader etc, et puis il peut être vrai Enterprise Java.

Maintenant que je l'ai sorti de mon système: réponse plus utile. System.in est fermé lorsque vous entrez EOF, qui est Control-D sous Linux et je pense que MacOS, et je pense Control-Z plus entrer sous Windows. Si vous voulez vérifier pour entrer (ou plus précisément, deux entre ... un pour terminer la dernière ligne et un pour indiquer que vous avez terminé, qui est essentiellement la façon dont http gère la détermination lorsque les en-têtes http sont terminés et il est temps pour le corps de http, alors la solution de @dbank devrait être une option viable avec une correction mineure que je vais essayer de faire pour déplacer le! dedans du prédicat while au lieu de! while

(Edit # 2: réalisé readLine supprime la nouvelle ligne, donc une ligne vide remplacerait la nouvelle ligne par "", et maintenant mon code passe à une autre réponse avec le bit EOF comme réponse à la place du commentaire

Éditez ... c'est bizarre, @dbank avait J'ai répondu pendant que je tapais ma réponse, et je me serais arrêté si je n'avais pas mentionné l'alternative EOF. de la mémoire avec l'édition que j'allait faire:

InputStreamReader instream = new InputStreamReader(System.in); 
BufferedReader buffer = new BufferedReader(instream); 

    line= buffer.readLine(); 
    while (line != null && !line.equals("")){ 
     length = length + line.length(); 
     line= buffer.readLine(); 
    } 

Answer 4

Lorsque vous appuyez uniquement sur Entrée, le retour de buffer.readLine(); n'est pas nul, il s'agit d'une chaîne vide.

Par conséquent, vous devriez changer line != null à !line.equals("") (Vous pouvez également changer à line.length() > 0)

Maintenant, votre code ressemblera à quelque chose comme ceci:

InputStreamReader instream = new InputStreamReader(System.in); 
BufferedReader buffer = new BufferedReader(instream); 

line = buffer.readLine(); 

while (!line.equals("")){ 
    length = length + line.length(); 
    line = buffer.readLine(); 
} 

Cela devrait résoudre votre problème. J'espère que cela a aidé! :)

Answer 5

Depuis Java 8, vous pouvez utiliser la méthode BufferedReader#lines directement sur un lecteur tamponné.

try (InputStreamReader in = new InputStreamReader(System.in); 
     BufferedReader buffer = new BufferedReader(in)) { 
     final int length = buffer.lines().mapToInt(String::length).sum(); 
     System.out.println("Read length: " + length); 
    } catch (Exception e) { 
     e.printStackTrace(); 
    } 

Answer 6

Une ligne de code Java en utilisant 8:

line = buffer.lines().collect(Collectors.joining());