Programm läuft nicht wie vorgesehen

Question

Ich versuche ein Java-Programm zu machen, das Eingaben hinzufügt, bis die Summe 100+ entspricht oder der Benutzer 5 Zahlen eingibt. Ich versuche auch, einen höchsten Lauf hinzuzufügen, der die höchste Eingabe verfolgt. Momentan läuft es nach 5 Eingaben weiter, wenn es weniger als 100 ist und mein höchster Lauf funktioniert nicht. ? Wie würde ich dieses Problem beheben (ich bin zu Java neu, wenn Sie sagen können)

import java.io.*; 

public class HighScoreTest { 

    public static void main(String[] args) { 
     // input streams. 
     InputStreamReader input = new InputStreamReader(System.in); 
     BufferedReader reader = new BufferedReader(input); 

     // constant declarations 
     final Integer MAX = 100; 
     final Integer MAX_NUMBER = 4; 

     // variable declarations 
     String sName; 
     Integer currentTotal; 
     Integer currentNumber; 
     Integer numbersInputed = 0; 
     Integer count; 
     Integer maxRunToDate = 0; 
     // we catch exceptions if some are thrown. 
     // an exception would be entering a string when a number is expected 
     try { 
       System.out.println("What is your name?"); 

       // reading string from the stream 
       sName = reader.readLine(); 
       currentTotal = 0; 

       for(count = 0; count < MAX_NUMBER; count++) { 
        numbersInputed += count; 
       } 

       do { 
        System.out.println("Please enter a number"); 
        currentNumber = Integer.parseInt(reader.readLine()); 
        currentTotal = currentTotal + currentNumber; 
       }while(currentTotal < MAX || numbersInputed == MAX_NUMBER); 

       if (maxRunToDate < currentTotal) { 
        maxRunToDate = currentTotal; 
       } 

       System.out.println(sName +", the total for this run is "+ currentTotal); 
       System.out.println("The highest run is "+ maxRunToDate); 

     } catch (IOException e){ 
      System.out.println("Error reading from user"); 
     } 

    } 

} 

Answer 1

Einige helfen Sie loslegen:

currentRun = currentRun + currentNumber; 

einfach keinen Sinn macht!

Ich nehme an, dass currentRun die Anzahl der Läufe so weit zählen sollte. Damit können Sie nach der 5. Runde aufhören.

Also: Sie sollten diesen Zähler nur für jede Runde erhöhen.

Mit anderen Worten: versuchen Sie separate Dinge. Gehen Sie zurück und überlegen Sie, welche Art von Informationen Sie "verfolgen" möchten und wie viele Variablen Sie wirklich benötigen.

Und bitte verstehen: wir werden Ihren Auftrag nicht für Sie lösen. Wenn überhaupt, wird es Hinweise geben, wie man Fortschritte machen kann. Aber erwarten Sie nicht, dass wir alle die Fehler in Ihrem Code darstellen und sie für Sie auflösen.

Answer 2

Es gibt mehrere Dinge, die Sie wissen müssen:

Zuerst Wrap-Klassen wie Integer vermeiden, wenn Sie es verwenden zusammen mit dem Colection FrameWork oder sogar Streams beabsichtigen. Wenn Ihr Problem die Tatsache ist, dass die Ausgabe der Analyse die Klasse Integer ist, machen Sie sich keine Sorgen, denn es wird automatisch aus der Box entfernt. Etwas wie dieses:

int currentNumber = Integer.parseInt(reader.readLine()); //Auto Unboxing 

Zweitens, warum haben Sie sogar die for-Schleife am Anfang? Entfernen Sie es. Wenn Sie die Variable numbersInputed initialisieren wollen, tun Sie es einfach.

Und drittens, wenn Sie zu erhöhen oder verringern whant, können Sie nur ++ oder `-

und überprüfen Sie die Oracle-Tutorials: https://docs.oracle.com/javase/tutorial/

Ich hoffe, dass ich dazu beigetragen haben.

Einen schönen Tag noch. :)