Zamykanie BufferedReader - Java

Question

Cześć wszystkim :)

Na początku przepraszam, za styl w jakim jest to napisane, ale cały czas się uczę (uwagi, zrypki bardzo proszę!). W poniższym kodzie, nie wiem jak zamknąć BufferedReader. Pętla IF nie działa. Próbowałem zrobić też w while( i != -1), ale to także nie działa.

Dziekuje za zaangażowanie i pomoc.

Pozdrawiam

 

package Zadania;
import java.io.*;
public class sredniaArytmetyczna {

    public static void main(String[] args) {
    
    double wynik = 0;    
    double srednia = 0;
    double n = 1;
    double i = 0;    
    
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        
        try {
            
            while (true) {
                System.out.println("Podaj liczbę:");
                i = Double.parseDouble(reader.readLine());
                
                System.out.println("Podana liczba: " + i);
                
                wynik += i++;                
                System.out.println("Suma: " + wynik);
                
                srednia = (wynik/n);
                System.out.println("Średnia arytmetyczna: " + srednia);
                
                if(i == -1) {
                    reader.close();
                    System.out.println("Koniec"); break;
                }
                n++;
            }
            System.out.println(wynik);
            
            
        } catch (IOException e) {
            System.out.println("Błąd: " + e);
        } catch (NumberFormatException e) {
            System.out.println("Błąd formatu " + e);
        }
    }
}

 

Answer 1

Nie rozumiem czemu nie użyjesz po prostu klasy Scanner?

Ps: IF nie jest pętlą.

Pozdrawiam!

Comments

Fakt, moja pomyłka.

Chciałem skorzystać z tej klasy, bo słyszałem że często jest wykorzystywana.

---

Jeżeli chce czytać z pliku linia po linii to BufferedReader jest okej ale właśnie tu Scanner będzie lepszy :)!

http://docs.oracle.com/javase/6/docs/api/java/io/BufferedReader.html

Answer 2

W linii 22. robisz: wynik += i++; co zwiększa wartość zmiennej: i z minus jeden na zero. 
Musisz zmienić warunek w if'ie na i == 0 :) 

Pozdrawiam,

Comments

Dzięki :)

A mam pytanie, czy da się jakoś wprowadzić by przy łańcuchu np. ''quit'' kończył program? Próbowałem, ale wyskakuje błąd NumberFormatException.

Pozdrawiam :)

---

Hmmm... Mógłbyś na przykład tego: reader.readLine() nie rzutować od razu na double'a, tylko na początek przypisać do Stringa i robić coś w stylu: if ( twojString.equals("quit") ) { break; }

A dopiero później, jeśli powyższy warunek nie zajdzie, zamieniałbyś tego Stringa na double :)

Pozdrawiam,

Answer 3

System.out.println("Podaj liczbę:");
                i = Double.parseDouble(reader.readLine());
               
               if(i==-1){

              System.exit(0);//zrobiłbym to tak w sytuacji gdzy chodzi Ci o
                                        //zakonczenie dzialania programu
           }

wynik += i++;
wynika+=i;  //bez postinkrementacji i