IOException bei BufferedReader

[Sindbad1983]

Hi!


Folgende Ausgabe soll es geben:

Zaehlerstand:0 j
Zaehlerstand:1 j
Zaehlerstand:2 n
Zaehlerstand:1 .......


je nachdem, ob der Benutzer j oder n eingibt, soll die Zahl erhöht bzw. erniedrigt werden!
Ich komm aber einfach nicht auf den Fehler drauf!
Bitte um Hilfe!
Glaub der Fehler liegt im BufferedReader!...oder in der Schleife!

public class Zaehler{


 	private int number;

 	public int getNumber(){
		return number;
	}

	public void erhoehe(){

		++number;

	}

	public void senke(){
		--number;

	}
}

import java.io.*;


class Zaehlertest{

public static void main(String [] args) throws IOException {

	String s="";

	Zaehler z=new Zaehler();


	BufferedReader br=new BufferedReader(new InputStreamReader (System.in));


	do{

		System.out.println("Zaehlerstand:" +z.getNumber() + ", (j/n/e)?");

		do{
			try{

					s=br.readLine();



			}catch(IOException ioe){}
		}while(s!="j" && s!="n");


	if(s=="j"){
		z.erhoehe();
	}
	else if(s=="n"){
		z.senke();
	}


	}while(s!="e");



	}
}

Ich hoff, es findet jemand den Fehler!
danke

 

[Wildcard]

String werden mit

.equals()

verglichen.

deine abbruchbedingung muss aber

while (!s.equals("j") || !s.equals("n"))

statt

while (!s.equals("j") && !s.equals("n"))

heißen. Gibt ja sonst keinen Sinn.

 

[L-ectron-X]

import java.io.*;

public class Zaehlertest {
   private Zaehler z;

   public Zaehlertest() {
      z = new Zaehler();
      BufferedReader br = new BufferedReader(new InputStreamReader(System.in));

      String s = "";
      do {
         System.out.print("Zaehlerstand: " +z.getNumber() + ", (j/n/e)?  ");

         try {
            s = br.readLine();
         }
         catch(IOException ioe){}

         if(s.equals("j"))
            z.erhoehe();

         else if(s.equals("n"))
            z.senke();
      }
      while(!s.equals("e"));
   }

   public static void main(String[] args) throws IOException {
      Zaehlertest zt = new Zaehlertest();
   }
}

public class Zaehler {
   private int number;
   
   public Zaehler() {
      number = 0;
   }

   public int getNumber() {
      return number;
   }

   public void erhoehe() {
      number++;
   }

   public void senke() {
      number--;
   }
}

 

[Sindbad1983]

danke für deine Hilfe, aber deine Version läuft nicht..
da hab ich wieder dasselbe Problem!
Er gibt den aktuellen Zählerstand nicht aus!

 

[L-ectron-X]

Bist Du sicher, dass Du nicht wieder Deine alte Klasse ausgeführt hast?
Ich habe das Programm getestet, es läuft.

 

[Sindbad1983]

naja...ich hab deinen Source-Code reinkopiert

aber ich habs jetzt ein wenig umgebaut und es läuft!

Brauch ich den Konstruktor in der Klasse Zählertest unbedingt?
Das ist mir nicht ganz so klar...!
Danke für deine Hilfe!

 

[L-ectron-X]

Man kann es auch in der main()-Methode laufen lassen. Allerdings entspricht dann diese Klasse nicht mehr der OOP.
Man kann sie nicht wieder verwenden bzw. instanzieren und in ein anderes Programm mit einbauen.
Außerdem wird sie unübersichtlich, wenn etwas dazu kommt bzw. Du die Klasse erweiterst.
Ich habe meine Version Deines Codes nochmals leicht modifiziert. Siehe oben.

 

[Sindbad1983]

sehr gut...es läuft!!! Danke!!!!!!!
kann man das in einer einzigen Klasse auch laufen lassen?
oder geht das nicht? brauch ich unbedingt 2?

und das ist mir nicht ganz klar:

mit z=new Zaehler() ... erzeugst du ein Objekt der Klasse Zaehler -> die hat nur number = 0 drinnen stehn!

aber wozu braucht man dann noch
Zaehlertest zt = new Zaehlertest(); ??
was mach ich damit?
was erzeugt ich damit?welches Objekt mit welchem Inhalt?

 

[L-ectron-X]

Man kann es auch in einer Klasse schreiben. Das kann dann so aussehen:

import java.io.*;

public class Zaehlertest {

   public Zaehlertest() {
      int zaehler = 0;
      BufferedReader br = new BufferedReader(new InputStreamReader(System.in));

      String s = "";
      do {
         System.out.print("Zaehlerstand: " +zaehler+ ", (j/n/e)?  ");

         try {
            s = br.readLine();
         }
         catch(IOException ioe){}

         if(s.equals("j"))
            zaehler++;

         else if(s.equals("n"))
            zaehler--;
      }
      while(!s.equals("e"));
   }

   public static void main(String[] args) throws IOException {
      Zaehlertest zt = new Zaehlertest();
   }
}

und das ist mir nicht ganz klar:

mit z=new Zaehler() ... erzeugst du ein Objekt der Klasse Zaehler -> die hat nur number = 0 drinnen stehn!

Du arbeitest doch innerhalb der Methoden der Klasse Zaehler mit einer Instanzvariablen. Wenn man ein Objekt einer Klasse erzeugt, setzt man meistens bestimmte Variablen auf Anfangswerte. In diesem Fall eben "0", weil der Zähler bei 0 anfangen soll zu zählen. Es stimmt, in diesem Fall müsste das nicht sein, weil eine Instanzvariable vom Typ int vom Compiler automatisch mit 0 initialisiert wird.

aber wozu braucht man dann noch
Zaehlertest zt = new Zaehlertest(); ??

Damit erzeuge ich eine Instanz der Klasse Zaehlertest. Im Prinzip wird nur ihr Konstruktor aufgerufen, der dann alles nötige abarbeitet. Wenn das Programm um Funktionalität wachsen soll, müsste das geändert werden.
Bei kleinen Klassen mag eine Implementation des Programmes in der main()-Methode noch übersichtlich erscheinen.
Sobald Du aber mehrere Klassen und Methoden verwenden willst, die auch noch gegenseitig miteinander kommunizieren, wirds schwierig bzw. unmöglich.

 

[Sindbad1983]

ok...danke!
Das hat mir sehr weitergeholfen!
Vielen Dank!
Gute Nacht!

 

[Sindbad1983]

Ich hab noch eine Frage...

import java.io.*; 

public class Zaehlertest { 
  ...
   public Zaehlertest() { 
      z = new Zaehler(); 
    ......
         try { 
            s = br.readLine(); 
         } 
         catch(IOException ioe){} 

      ......
   public static void main(String[] args) [b]throws IOException [/b]{ 
      Zaehlertest zt = new Zaehlertest(); 
   } 
}

Warum braucht man das IOException bei der main-Methode?
Warum funktioniert das Folgende nicht?

public class Zahlertest throws IOException{
..}

Was müsst ich dann verändern?

Danke!

 

[bygones]

du brauchst in der Main methode kein throws IOException (jedenfalls dem Code nach), da du im Konstruktor ja einen try catch Block hast

 

[Sindbad1983]

hast Recht..danke

 

[L-ectron-X]

throws dient dazu anzuzeigen, dass eine Methode eine bestimmte Exception werfen kann.
Dabei muss aber keine try-catch-Behandlung in der Klasse gemacht werden. Dann gibt die Methode die Exception weiter, an die aufrufende Instanz/Methode, die dann ihrerseits die Exception behandeln oder ebenfalls weiter leiten muss.

 

[Sindbad1983]

wenn ich ein Programm schreib, dann vergiss ich das Verwenden von Exceptions meistens!
Denn wann weiß ich, dass eine Methode eine bestimmte Exception werfen kann? Und welche Exception?

oder wann verwendet man try...catch? oft gehts auch ohne! :-(

 

[Bleiglanz]

musst du noch ein bisschen Java lernen, es gibt da nämlich zwei Arten von Exceptions

Denn wann weiß ich, dass eine Methode eine bestimmte Exception werfen kann? Und welche Exception?

aus der javadoc, wozu glaubst du gibts @throws?

oder wann verwendet man try...catch?

a) wenn man muss
b) wenn man nicht muss aber trotzdem eine Runtime-Exception anfangen will (typisches Beispiel Integer.parseInt)