Abbruchbedingung funktioniert nicht

[redfalcon]

Hi,
ich habe die Aufgabe ein Programm zu entwerfen, welches nach Eingabe von Name, Personalnummer, Stundenlohn und Arbeitsstunden auswertet und ausgibt. Sobald als Personalnummer 0 eingegeben wird, soll die Eingabe von Daten abgebrochen werden und zur Auswertung/Ausgabe fortgeschritten werden. Der Code.

public class brutto {
public String[] persNr = new String[10];                //Array für Personalnummern
public String[] name = new String[10];                  //Array für die Namen 
public int[] stdlohn =new int[10];                          //Array für den Stundenlohn
public int[] arbstd =new int[10];                           //Array für die Arbeitstunden
public int[] brutto =new int[10];                           //Array für die Bruttosumme
public double summe;                                          //Lohnkosten insgesamt
public int pers=0;                                               //Hilfvariable für Personen
public double durchstd;                                       //Durchschnittliche Arbeitsstunden
public double durchlohn;                                      //Durchschnittlicher Lohn
public int p=0;                                                   //Hilfsvariable für Arrays

  public void eingabe() {
  while(pers<3){

    System.out.println("Geben sie die Personalnummer ein." );
	
		persNr[p]=StdIn.stringInput();
			if(persNr[p].equals("0")){
			this.verarbeitung();
			}
			
    else{
		System.out.println("Geben sie den Namen des Mitarbeiters ein." );
			name[p]=StdIn.stringInput();
		System.out.println("Geben sie die Arbeitsstunden des Mitarbeiters ein." );
			arbstd[p]=StdIn.intInput();
		System.out.println("Geben sie den Stundenlohn des Mitarbeiters ein." );
			stdlohn[p]=StdIn.intInput();
		brutto[p]=brutto[p]+(stdlohn[p]*arbstd[p]);
		p++;
		pers++;
    }
  }
  }

  public void verarbeitung() {
    if(pers==0)
      this.exit();
    else{
  
  for(int durch=0;durch<pers;durch++){
    durchstd=durchstd+arbstd[durch]/pers;
    durchlohn=durchlohn+stdlohn[durch]/pers;
  }
    }
    for(int durch=0;durch<pers;durch++){
      summe=summe+brutto[durch];
    }
  }



    public void ausgabe() {
      System.out.println("PersNr.\t Name\t Stunden\t Std. Lohn\t Brutto\t");
    for(int aus=0;aus<pers;aus++){
      System.out.println(persNr[aus]+"\t"+name[aus]+"\t"+arbstd[aus]+"\t"+stdlohn[aus]+"\t"+brutto[aus] );
    }
    System.out.println("Lohnsumme: "+summe);
    System.out.println("Durchscnittslohn: " +durchlohn );
    System.out.println("Durchschnittliche Arbeitsstunden: "+durchstd);
  }
    
    public void exit(){
      System.exit(1);
    }



  public static void main (String[]args) {
         brutto pro = new brutto();
         pro.eingabe();
         pro.verarbeitung();
         pro.ausgabe();
         pro.exit();

  }

}

Das Problem ist nun: Wenn ich eine 0 als Personalnummer eingeben, wird zwar die weitere Eingabe übersprungen, allerdings lande ich wieder in der while-Schleife drin, obwohl ich ja eigentlich ich verarbeitung() springe. Wo ist der Fehler?

 

[Beni]

In "verarbeitung" fragst du "pers==0" ab, aber du setzt ja "pers" niemals = 0, nachdem der Benutzer 0 eingegeben hat.

 

[Kian]

Sobald du "0" eingibst, wird nirgends der Wert von pers verändert, die while wird nur ausgeführt solange pers < 3 ist.
Ich würde das mit einem BOOL - Wert machen, aber es gibt sicher auch eine einfachere Lösung (es geht immer einfacher, meistens jedenfalls!). Das hier dürfte die WHILE mit Sicherheit stoppen:

public class brutto {
public String[] persNr = new String[10];                //Array für Personalnummern
public String[] name = new String[10];                  //Array für die Namen
public int[] stdlohn =new int[10];                          //Array für den Stundenlohn
public int[] arbstd =new int[10];                           //Array für die Arbeitstunden
public int[] brutto =new int[10];                           //Array für die Bruttosumme
public double summe;                                          //Lohnkosten insgesamt
public int pers=0;                                               //Hilfvariable für Personen
public double durchstd;                                       //Durchschnittliche Arbeitsstunden
public double durchlohn;                                      //Durchschnittlicher Lohn
public int p=0;                                                   //Hilfsvariable für Arrays
boolean run = true;     //!!! BOOL für WHILE !!!

  public void eingabe() {
  while(run == true){

    System.out.println("Geben sie die Personalnummer ein." );
   
      persNr[p]=StdIn.stringInput();
         if(persNr[p].equals("0")){
         run = false; // WHILE stoppen
         this.verarbeitung();
         }
         
    else{
      System.out.println("Geben sie den Namen des Mitarbeiters ein." );
         name[p]=StdIn.stringInput();
      System.out.println("Geben sie die Arbeitsstunden des Mitarbeiters ein." );
         arbstd[p]=StdIn.intInput();
      System.out.println("Geben sie den Stundenlohn des Mitarbeiters ein." );
         stdlohn[p]=StdIn.intInput();
      brutto[p]=brutto[p]+(stdlohn[p]*arbstd[p]);
      p++;
      pers++;
    }
  }
  }

  public void verarbeitung() {
    if(pers==0)
      this.exit();
    else{
 
  for(int durch=0;durch<pers;durch++){
    durchstd=durchstd+arbstd[durch]/pers;
    durchlohn=durchlohn+stdlohn[durch]/pers;
  }
    }
    for(int durch=0;durch<pers;durch++){
      summe=summe+brutto[durch];
    }
  }



    public void ausgabe() {
      System.out.println("PersNr.\t Name\t Stunden\t Std. Lohn\t Brutto\t");
    for(int aus=0;aus<pers;aus++){
      System.out.println(persNr[aus]+"\t"+name[aus]+"\t"+arbstd[aus]+"\t"+stdlohn[aus]+"\t"+brutto[aus] );
    }
    System.out.println("Lohnsumme: "+summe);
    System.out.println("Durchscnittslohn: " +durchlohn );
    System.out.println("Durchschnittliche Arbeitsstunden: "+durchstd);
  }
   
    public void exit(){
      System.exit(1);
    }



  public static void main (String[]args) {
         brutto pro = new brutto();
         pro.eingabe();
         pro.verarbeitung();
         pro.ausgabe();
         pro.exit();

  }

}

EDIT:
@Beni:
selbst wenn er pers auf "0" setzt, würde die while weiterlaufen, zudem würde es später in der Verarbeitung zu Fehlern kommen, wenn er zuvor schon Daten eingegeben hat -> siehe FOR - Schleifen!

 

[Beni]

Ne, der Code springt ja dann von verarbeitung zu exit zu System.exit?

Ok, ein bisschen *hust* unübersichtlich finde ich das ganze ja schon, und KIAN's Lösung hat ein gutes Stück Sicherheit mehr.

Ich frage mich, wieso springst du zuerst zu Verarbeitung, und nicht direkt zu exit?

 

[redfalcon]

@Kian: Dann hab ich aber das Problem, dass die While-Schleife theoretisch unendlich lang gehen kann (wenn man nicht abbricht). Es sollen aber maximal 10 Mitarbeiter eingetragen werden.

 

[Kian]

@Kian: Dann hab ich aber das Problem, dass die While-Schleife theoretisch unendlich lang gehen kann (wenn man nicht abbricht). Es sollen aber maximal 10 Mitarbeiter eingetragen werden.

while(run == true && p < 10)

oder pers anstatt p, je nachdem was bei dir die Mitarbeiter zählt.

Wenn run == false ist und wenn p >= 10 ist wird abgebrochen!

 

[redfalcon]

Oh, stimmt hätte ich auch selbst draufkommen können :autsch:. Auf jedenfall funzt das Programm jetzt, wie es soll. Danke für eure Hilfe!

 

[redfalcon]

Doch noch was. Wenn ich die Eingabe für nur eine Person mache und dann abbreche, sind die Durchschnittswerte (Gesamt, Lohn und Stunden) verdoppelt. Wenn ich aber zwei oder mehr eingebe ist die Summe wieder korrekt. Woran liegt das? Hier mein aktueller Code:

public class brutto {
public String[] persNr = new String[10];    //Array für Personalnummern
public String[] name = new String[10];      //Array für Namen
public int[] stdlohn =new int[10];          //Array für Stundenlohn
public int[] arbstd =new int[10];           //Array für Arbeitsstunden
public int[] brutto =new int[10];           //Array für Bruttolohnsumme
public double summe=0;                      //Gesamtlohnsumme
public int pers=0;                          //Anzahl Personen
public double durchstd=0;                   //Durchschnittliche Arbeitsstunden
public double durchlohn=0;                  //Durchschnittlicher Lohn
public int p=0;                             //Hilfsvariable für While-Schleife
boolean run = true;                         //Boolean zum beenden der While-Schleife

//Eingabe

 public void eingabe() {
  while(run == true && pers < 3) {
    System.out.println("Geben sie die Personalnummer ein." ); 
    persNr[p]=StdIn.stringInput();
    
      if(persNr[p].equals("0")){
        run = false; 
        this.verarbeitung(); 
      }
      
      else{
        System.out.println("Geben sie den Namen des Mitarbeiters ein." );
          name[p]=StdIn.stringInput();
        System.out.println("Geben sie die Arbeitsstunden des Mitarbeiters ein." );
          arbstd[p]=StdIn.intInput();
        System.out.println("Geben sie den Stundenlohn des Mitarbeiters ein." );
          stdlohn[p]=StdIn.intInput();
        brutto[p]=brutto[p]+(stdlohn[p]*arbstd[p]);
        p++;
        pers++;
        StdIn.cls();
      }
  }
 }
  
//Verarbeitung

  public void verarbeitung() {
      for(int durch=0;durch<pers;durch++){
        durchstd=durchstd+arbstd[durch];
        durchlohn=durchlohn+stdlohn[durch];
      }
    
      for(int durch=0;durch<pers;durch++){
        summe=summe+brutto[durch];
      }
      durchlohn=durchlohn/pers;
	  durchstd=durchstd/pers;
  }


//Ausgabe
  
    public void ausgabe() {
      System.out.println("PersNr.\t Name\t\t Stunden  Std.Lohn Brutto");
        for(int aus=0;aus<pers;aus++){
          System.out.println(persNr[aus]+"\t"+name[aus]+"\t "+arbstd[aus]+"\t  "+stdlohn[aus]+"\t   "+brutto[aus]+"\n" );
        }
        System.out.println("Lohnsumme: "+summe);
      System.out.println("Durchscnittslohn: " +durchlohn );
      System.out.println("Durchschnittliche Arbeitsstunden: "+durchstd);
  }
    
//Main
    
  public static void main (String[]args) {
         brutto pro = new brutto();
         pro.eingabe();
         pro.verarbeitung();
         pro.ausgabe();      

  }

}

 

[redfalcon]

OK, gelößt. Nachdem ich this.verarbeitung(); entfernt hab, wird alles normal berechnet.