Durchschnitt ist nicht korrekt; Eingaben löschen

[Antoras]

Hallo,

ich hab gerade nochmal ein Problem, genauer zwei:

Ich will den Durchschnitt aller eingegebenen Zahlen in einer bestimmten Anzahl an Textfeldern ermitteln; Nullen werden dabei nicht berücksichtigt:

private void jButton1ActionPerformed(ActionEvent evt) {
		for (int i = 1; i < jTextField.length; i++) {
			temp = Integer.parseInt(jTextField[i].getText());				//Speichervariable
			summe = summe + temp;												//Summenberechnung
			if (temp == 0) {																//Häufigkeiten ermitteln
				h0++;
			} else if (temp == 1) {
				h1++;
			} else if (temp == 2) {
				h2++;
			} else if (temp == 3) {
				h3++;
			} else if (temp == 4) {
				h4++;
			} else if (temp == 5) {
				h5++;
			} else {
				h6++;
			}
		}		
	}
	
	private void jButton2ActionPerformed(ActionEvent evt) {
		temp2 = jTextField.length - h0;										//Speichervariable 2
		average = summe / temp2;												//Durchschnitt
		jTextField21.setText(String.valueOf(average));
		jTextField22.setText(String.valueOf(h0));
		jTextField23.setText(String.valueOf(h1));
		jTextField24.setText(String.valueOf(h2));
		jTextField25.setText(String.valueOf(h3));
		jTextField26.setText(String.valueOf(h4));
		jTextField27.setText(String.valueOf(h5));
		jTextField28.setText(String.valueOf(h6));
	}

Wenn ich z.b fünf zweien eingebe kommt als Durchschnitt 1 raus obwohl ja eigentlich wieder eine 2 rauskommen sollte.

Zweites Problem:
Wenn ich andere Zahlen eingebe werden diese zu den bereits bestehenden Ergebnissen hinzuaddiert. Gibt es hier eine Möglichkeit mit der man alle bereits eingelesenen Werte beim betätigen der Buttons wieder löschen kann?

 

[Gast]

Bau doch bitte mal ein kleines lauffähiges Beispiel

 

[Antoras]

Bitteschön:

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class asdf extends JApplet {
	int summe, 				
		temp,					
		temp2,
		h0;		
	float average;
	
	private JTextField jTextField[] = new JTextField[10];
	private JTextField jTextField11;
	private JButton jButton1;
	
	public asdf() {
		GUI();
	}	
	
	private void GUI() {
		setLayout(new FlowLayout());
		for (int i = 0; i < jTextField.length; i++) {
			   jTextField[i] = new JTextField(6); 
			   add(jTextField[i]);
			   jTextField[i].setText("0");
		}
		{
			jButton1 = new JButton();
			add(jButton1);
			jButton1.setText("Einlesen");
			jButton1.setBounds(7, 144, 98, 22);
			jButton1.addActionListener(new ActionListener() {
				public void actionPerformed(ActionEvent evt) {
					jButton1ActionPerformed(evt);
				}
			});
		}
		{
			jTextField11 = new JTextField(6);
			add(jTextField11);
		}
	}
	
	private void jButton1ActionPerformed(ActionEvent evt) {
		for (int i = 1; i < jTextField.length; i++) {
			temp = Integer.parseInt(jTextField[i].getText());				//Speichervariable
			summe = summe + temp;												//Summenberechnung
			if (temp == 0) {																//Häufigkeiten ermitteln
				h0++;
			}
		}			
		temp2 = jTextField.length - h0;											//Speichervariable 2
		average = summe / temp2;													//Durchschnitt
		jTextField11.setText(String.valueOf(average));						
	}
}

 

[Frank_H]

also lauffähig is n bissel was anderes....

dein proble war, dass du die variablen
1) nicht vorinitialisiert hattest
2) nicht zurücksetzt, wenn du einen erneuten durchlauf machst, somit steigt h0 und summe irgendwann in den himmel...

hier die buttonActon-methode nochmal...

   private void jButton1ActionPerformed(ActionEvent evt) {
	   temp = 0;
	   summe = 0;
	   h0 = 0;
      for (int i = 0; i < jTextField.length; i++) {
         temp = Integer.parseInt(jTextField[i].getText()); //Speichervariable
         if (temp > 0) {                                                //Häufigkeiten ermitteln
        	 summe += temp;                                    //Summenberechnung
             h0++;
         }
      }         
      average = summe / h0;     //Durchschnitt
      jTextField11.setText(String.valueOf(average));                  
   }

 

[Antoras]

also lauffähig is n bissel was anderes....

Bei mir läuft das ohne Probleme und ohne Fehlermeldungen - abgesehen von den beiden genannten. Aber das war ja klar, dass die hier wieder vorhanden sind.

1) nicht vorinitialisiert hattest

Das ist unnötig. Die sind standardmäßig doch sowieso immer null, oder?

2) nicht zurücksetzt, wenn du einen erneuten durchlauf machst, somit steigt h0 und summe irgendwann in den himmel...

Ich weiß, dass ich es nicht zurücksetze. Deswegen hab ich ja gefragt. Ich weiß nämlich nicht wie ich das machen soll. Mit setText("") geht es irgendwie nicht...

hier die buttonActon-methode nochmal...

So wie du es gemacht hast ist es deutlich einfacher und es funktioniert. Danke! Ich muss es immer umständlich machen.

Trotzdem verstehe ich nicht was an meiner Methode falsch war. h0 wir immer hochgezählt wenn eine 0 eingegeben wird. Gleichzeitig werden die eingegebenen Zahlen miteinander addiert. Danach wird von der Anzahl der Textfelder h0 subtrahiert. Der Durchschnitt berechnet sich dann aus der Summe der Zahlen dividiert durch den Rest. Erkennt da jemand den Denkfehler?

 

[Final_Striker]

das programm funktioniert doch jetzt, oder welche fehler sind jetzt noch drin???

du müsstest noch prüfen bevor du teilst ob h0 == 0 ist. teilen durch 0 ist nicht so gut. :noe:

edit:

außerdem ist es quatsch die avarage variable vom typ float zu machen. denn wenn du integer durch integer teilst bekommst du auch ein integer zurück. :wink:

 

[Antoras]

du müsstest noch prüfen bevor du teilst ob h0 == 0 ist. teilen durch 0 ist nicht so gut. noe.gif

Stimmt, leuchtet mir ein. Hab ich eingebaut.

außerdem ist es quatsch die avarage variable vom typ float zu machen. denn wenn du integer durch integer teilst bekommst du auch ein integer zurück. icon_wink.gif

Hab ich nicht gewusst. Aber jetzt weiß ich es. Schon wieder was gelernt. Das hat sich dann auch erledigt.


Das Zurücksetzten der Textfelder hat mir Probleme bereitet, ich hab jetzt aber eine Lösung gefunden. Jetzt werden mit "setText("")" einfach die Textfelder gelöscht und mit "variable = 0" setz ich die betroffenen Variablen wieder auf den Anfangswert zurück. Geht zwar bestimmt professioneller, aber das erfüllt ja seinen Zweck:

private void jButton1ActionPerformed(ActionEvent evt) {
		for (int i = 1; i < jTextField.length; i++) {
			temp = Integer.parseInt(jTextField[i].getText());		
			if (temp > 0) {
				summe += temp;
				haeufigkeit++;
			}
			if (temp == 0) h0++;				
			else if (temp == 1) h1++;
			else if (temp == 2) h2++;
			else if (temp == 3) h3++;
			else if (temp == 4) h4++;
			else if (temp == 5) h5++;
			else h6++;			
		}
		jTextField21.setText("");
		jTextField22.setText("");
		jTextField23.setText("");
		jTextField24.setText("");
		jTextField25.setText("");
		jTextField26.setText("");
		jTextField27.setText("");
		jTextField28.setText("");
	}
	
	private void jButton2ActionPerformed(ActionEvent evt) {
		if (haeufigkeit != 0) {
			average = summe / haeufigkeit;						
			jTextField21.setText(String.valueOf(average));
		} else jTextField21.setText("Kein Schüler hat eine Arbeit geschrieben");
					
		jTextField22.setText(String.valueOf(h0));
		jTextField23.setText(String.valueOf(h1));
		jTextField24.setText(String.valueOf(h2));
		jTextField25.setText(String.valueOf(h3));
		jTextField26.setText(String.valueOf(h4));
		jTextField27.setText(String.valueOf(h5));
		jTextField28.setText(String.valueOf(h6));
		temp = 0;
		haeufigkeit = 0;
		summe = 0;
		average = 0;
		h0 = 0;
		h1 = 0;
		h2 = 0;
		h3 = 0;
		h4 = 0;
		h5 = 0;
		h6 = 0;
	}

Nochmal Danke an alle, die mir geholfen haben.

 

[Guest]

außerdem ist es quatsch die avarage variable vom typ float zu machen. denn wenn du integer durch integer teilst bekommst du auch ein integer zurück. :wink:

das stimmt nicht. z.b.

9:5 ist nicht = 1

sondern 1,8 wenn du integer benutzt, schneidet er alles nach dem Komma ab! Dann hast du Rundungsfehler.