richtiges Ergebnis zählen und übergeben?

[mobile]

Hi,
habe ne kleine Frage.
In der Methode hier:

void numberadd() {
	      BufferedReader eingabe = new BufferedReader(new InputStreamReader(System.in));
	      String eingabeZeile;
	      int anzahl;
	      try {
	         System.out.println("Wie viele Aufgaben willst du machen?");
	         eingabeZeile = eingabe.readLine(); //Eingabe Zeile wird eingelesen
	         StringTokenizer tokens = new StringTokenizer(eingabeZeile);
	         anzahl = Integer.parseInt(tokens.nextToken()); 
	         for(int i=0; i<=anzahl; i++) {
	            if(i != anzahl) {
	               CalculationMethods plus = new CalculationMethods();
	               plus.add();      
	            }
	            else if(i == anzahl) {
                        //HIER WIRD ANZAHL AUSGELESEN IN DIE KLAMMERN SOLLEN JETZT RICHTIGE
	            	System.out.println("Du hast von "+anzahl + " Aufgaben () richtig geloest."+"\n");
	            	System.out.println("Was moechtest du jetzt tun?");
	            	StartingScreen run = new StartingScreen();
	            	run.start();   
	            }
	         }
	      }
	      catch(IOException e) {
	         System.out.println("Fehler bei der Eingabe");
	      }
	      catch(NoSuchElementException e) {
	         System.out.println("Zuwenig Werte eingegeben");
	      }
	      catch(NumberFormatException e) {
				System.out.println("Bitte Ganzezahlen von 1 - 50 verwenden!");
	      }
	}

rufe ich eine Methode durch Eingabe des Nutzers verschieden oft auf. In Zeile 18, wird nun die Anzahl der Aufrufe ausgegeben und da wo die Klammern sind soll die Anzahl der richtigen Ergebnise angezeigt werden. Ob das Ergebnis richtig oder falsch ist, wird allerdings in dieser Methode entschieden.

final void add() {
		BufferedReader eingabe = new BufferedReader(new InputStreamReader(System.in));
		String eingabeZeile;
		int zahl;
		try {		
			Random generator = new Random();
			int zahl1 = generator.nextInt(30);
			int zahl2 = generator.nextInt(30);
			int ergebnis = zahl1+zahl2;
			System.out.println("Aufgabe: " + zahl1 + "+" + zahl2 + "=");
			eingabeZeile = eingabe.readLine(); 
			StringTokenizer tokens = new StringTokenizer(eingabeZeile);
			zahl = Integer.parseInt(tokens.nextToken());  
			if(zahl == ergebnis) {
				System.out.println("Richtig!"+"\n");
			}
			else if(zahl != ergebnis){
				System.out.println("Das war leider falsch!"+"\n");
			}
		}
		catch(IOException e) {
			System.out.println("Fehler bei der Eingabe");
		}
		catch(NoSuchElementException e) {
			System.out.println("Zuwenig Werte eingegeben");
		}
		catch(NumberFormatException e) {
			System.out.println("Bitte Ganzezahlen verwenden!");
		}
	}

Jetzt meine Frage: Wie kann ich erreichen, dass die richtigen Ergebnisse gezählt werden und dann an die erste Methode weitergegeben werden, damit ich sie hier auch in Zeile 18 ausgeben kann?

Hoffe ihr könnt mir helfen! Danke!

 

[kaie]

Gib doch einfach aus der add-Methode heraus einen boolean-Wert zurück, der Dir sagt, ob die Antwort richtig oder falsch war:

final boolean add() { 
         ....
         if(zahl == ergebnis) { 
            System.out.println("Richtig!"+"\n"); 
            return true;
         } 
         else if(zahl != ergebnis){ 
            System.out.println("Das war leider falsch!"+"\n"); 
            return false;
         } 
         ....
}

Dann musst Du in Deiner Hauptschleife nur die richtigen Antworten zählen:

void numberadd() { 
            ....
            int richtige = 0;
            for(int i=0; i<=anzahl; i++) { 
               if(i != anzahl) { 
                  CalculationMethods plus = new CalculationMethods(); 
                  if( plus.add()==true )
                  {
                    richtige++;
                  }
               } 
               else if(i == anzahl) { 
                        //HIER WIRD ANZAHL AUSGELESEN IN DIE KLAMMERN SOLLEN JETZT RICHTIGE 
                  System.out.println("Du hast von "+ anzahl + " Aufgaben "+ richtig +" richtig geloest."+"\n"); 
               ....
}

kaie

 

[mobile]

Morgen die Herren,

also ich habe das jetzt so umgeschrieben. Hier die Methode add():

final boolean add() {
		BufferedReader eingabe = new BufferedReader(new InputStreamReader(System.in));
		String eingabeZeile;
		int zahl;
		try {		
			Random generator = new Random();
			int zahl1 = generator.nextInt(30);
			int zahl2 = generator.nextInt(30);
			int ergebnis = zahl1+zahl2;
			System.out.println("Aufgabe: " + zahl1 + "+" + zahl2 + "=");
			eingabeZeile = eingabe.readLine(); 
			StringTokenizer tokens = new StringTokenizer(eingabeZeile);
			zahl = Integer.parseInt(tokens.nextToken());  
			if(zahl == ergebnis) {
				System.out.println("Richtig!"+"\n");
				return true;
			}
			else if(zahl != ergebnis){
				System.out.println("Das war leider falsch!"+"\n");
				return false;
			}
		}
		catch(IOException e) {
			System.out.println("Fehler bei der Eingabe");
		}
		catch(NoSuchElementException e) {
			System.out.println("Zuwenig Werte eingegeben");
		}
		catch(NumberFormatException e) {
			System.out.println("Bitte Ganzezahlen verwenden!");
		}
	}

und hier die Methode numberadd():

final void numberadd() {
	      BufferedReader eingabe = new BufferedReader(new InputStreamReader(System.in));//Instanz für EIngabeStream
	      String eingabeZeile;
	      int anzahl;
	      try {
	         System.out.println("Wie viele Aufgaben m\u00f6chtest du rechnen?");
	         eingabeZeile = eingabe.readLine(); //Eingabe Zeile wird eingelesen
	         StringTokenizer tokens = new StringTokenizer(eingabeZeile);
	         anzahl = Integer.parseInt(tokens.nextToken()); 
	         int richtige = 0;
	         for(int i=0; i<=anzahl; i++) {
	            if(i != anzahl) {
	               CalculationMethods plus = new CalculationMethods();
	               if( plus.add()==true ) {
	            	   richtige++;
	               }
	            }
	            else if(i == anzahl) {
	            	System.out.println("Du hast von " +anzahl + " Aufgaben " + richtige + "gelu00f6st."+"\n");
	            	System.out.println("Was m\u00f6chtest du jetzt tun?");
	            	StartingScreen run = new StartingScreen();
	            	run.start();   
	            }
	         }
	      }
	      catch(IOException e) {
	         System.out.println("Fehler bei der Eingabe");
	      }
	      catch(NoSuchElementException e) {
	         System.out.println("Zuwenig Werte eingegeben");
	      }
	      catch(NumberFormatException e) {
				System.out.println("Bitte Ganzezahlen von 1 - 50 verwenden!");
	      }
	}

Jetzt sagt er mir aber: This method must return a result of type boolean.
Aber die Methode add() liefert doch einen Typ von boolean zurück. Was ist falsch?

Danke für Hilfe!

 

[Guest]

Hallo,

Deine Methode hat nur einen Rückgabewert wenn try {} fehlerlos durchläuft!!
Du musst eben noch festlegen was zurückgegeben werden soll, wenn eine Exception auftritt.

 

[Murray]

Der Compiler ist nicht so "schlau" zu erkennen, dass eine der beiden Bedingungen (zahl == ergebnis) und (zahl != ergebnis) auf jeden Fall zutreffen muss.

Wenn Du schreibst:

         if(zahl == ergebnis) {
            System.out.println("Richtig!"+"\n");
            return true;
         } else { //--- hier kein if
            System.out.println("Das war leider falsch!"+"\n");
            return false;
         }

, dann sollte es klappen /EDIT: (aber nur, wenn Du auch im Falle der abgefangenen Exceptions auch einen Wert zurücklieferst)

 

[mobile]

Also irgendwie klappt das immer noch nicht. Habe das jetzt so gemacht.

if(zahl == ergebnis) {
				System.out.println("Richtig!"+"\n");
				return true;
			}
			else {
				System.out.println("Das war leider falsch!"+"\n");
				return false;
			}

Kommt immer noch die gleiche fehlermeldung. Hmm... .(

 

[Murray]

Siehe den Post des Gastes:

     catch(IOException e) {
         System.out.println("Fehler bei der Eingabe");
         return false;
      }
      catch(NoSuchElementException e) {
         System.out.println("Zuwenig Werte eingegeben");
         return false;
      }
      catch(NumberFormatException e) {
         System.out.println("Bitte Ganzezahlen verwenden!");
         return false;
      }

 

[Murray]

Da Du die Exception ja aber auch im aufrufenden Code behandelst, könntest Du im add die Exception auch einfach deklarieren:

final boolean add() throws IOException, NoSuchElementException, NumberFormatException {
      BufferedReader eingabe = new BufferedReader(new InputStreamReader(System.in));
      String eingabeZeile;
      int zahl;

      Random generator = new Random();
      int zahl1 = generator.nextInt(30);
      int zahl2 = generator.nextInt(30);
      int ergebnis = zahl1+zahl2;
      System.out.println("Aufgabe: " + zahl1 + "+" + zahl2 + "=");
      eingabeZeile = eingabe.readLine();
      StringTokenizer tokens = new StringTokenizer(eingabeZeile);
      zahl = Integer.parseInt(tokens.nextToken()); 
      if(zahl == ergebnis) {
          System.out.println("Richtig!"+"\n");
          return true;
      } else {
          System.out.println("Das war leider falsch!"+"\n");
          return false;
      }
}