Programm hängt sich auf

[Timo2010]

Hallo,
ich habe ein komisches Problem. Das Programm hängt sich auf, sobald unterschiedliche Operatoren gedrückt werden.
Der ActionListener:

if (e.getSource()==TaschenrechnerUIJWindow.mal){
	mal = "*";
	if (TaschenrechnerUIJWindow.operandenstrings.toString() != ""){
		addOperand(TaschenrechnerUIJWindow.operandenstrings);
		sizeoperanden++;
		}
	TaschenrechnerUIJWindow.anzeige.setText(TaschenrechnerUIJWindow.anzeige.getText()+mal);
	if (TaschenrechnerUIJWindow.operatoren.isEmpty() || istHoeherwertig(mal,TaschenrechnerUIJWindow.operatoren.top())){
		TaschenrechnerUIJWindow.operatoren.push(mal);
}
	else {
		while (!TaschenrechnerUIJWindow.operatoren.isEmpty()||!istHoeherwertig(mal,TaschenrechnerUIJWindow.operatoren.top())){
			if (sizeoperanden >=2){
				rechne(mal);
				}
			if (TaschenrechnerUIJWindow.operatoren.isEmpty()){
				TaschenrechnerUIJWindow.operatoren.push(mal);
				break;
			}
		}
		}
	}
	if (e.getSource()==TaschenrechnerUIJWindow.geteilt){
		geteilt = "/";
		if (TaschenrechnerUIJWindow.operandenstrings.toString() != ""){
			addOperand(TaschenrechnerUIJWindow.operandenstrings);
			sizeoperanden++;
			}
		TaschenrechnerUIJWindow.anzeige.setText(TaschenrechnerUIJWindow.anzeige.getText()+geteilt);
		if (TaschenrechnerUIJWindow.operatoren.isEmpty() || istHoeherwertig(geteilt,TaschenrechnerUIJWindow.operatoren.top())){
			TaschenrechnerUIJWindow.operatoren.push(geteilt);
	}
		else {
			while (!TaschenrechnerUIJWindow.operatoren.isEmpty()||!istHoeherwertig(geteilt,TaschenrechnerUIJWindow.operatoren.top())){
				if (sizeoperanden >=2){
					rechne(geteilt);
				}
				if (TaschenrechnerUIJWindow.operatoren.isEmpty()){
				TaschenrechnerUIJWindow.operatoren.push(geteilt);
				break;
			}
			}
			}		
	}

Die rechne Methode:

public void rechne(String eingabeoperator){
		if (!TaschenrechnerUIJWindow.operatoren.isEmpty() && sizeoperanden>=2){
		double operandaint = Double.valueOf((String)TaschenrechnerUIJWindow.operanden.top());
		TaschenrechnerUIJWindow.operanden.pop();
		double operandbint = Double.valueOf((String)TaschenrechnerUIJWindow.operanden.top());
		TaschenrechnerUIJWindow.operanden.pop();
		String stackoperator = (String)TaschenrechnerUIJWindow.operatoren.top();
		TaschenrechnerUIJWindow.operatoren.pop();
		if (stackoperator == "*"){
			zwischenergebnis = operandbint * operandaint;
		}
		if (stackoperator == "/"){
			zwischenergebnis = operandbint / operandaint;
		}
		if (stackoperator == "+"){
			zwischenergebnis = operandaint + operandbint;
		}
		if (stackoperator == "-"){
			zwischenergebnis = operandaint - operandbint;
		}
		zwischenergebnisstring = String.valueOf(zwischenergebnis);
		TaschenrechnerUIJWindow.operanden.push(zwischenergebnisstring);
		sizeoperanden-=1;
		}
		System.out.println(zwischenergebnisstring);
	}

Das komische ist, dass es bei den gleichen Operatoren (z.B. 4*4*4) funktioniert, aber bei unterschiedliche (z.B. 4*4/4) ab dem zweiten Operatoren hängt.

Wie kann es dazu kommen, dass ein Programm nicht mehr reagiert? Liegt es an der Schleife?

 

[Final_Striker]

gelesen was ich geschrieben habe?

http://www.java-forum.org/java-basics-anfaenger-themen/111861-ausgabe-findet-statt.html

 

[Timo2010]

Liegt das denn daran? Ich glaube eher nicht. In der Schule haben wir den Vergleich so gelernt. Denn die While Schleife scheint den Fehler zu verursachen.

 

[Murray]

In der Schule haben wir den Vergleich so gelernt.

Ist trotzdem grundfalsch.

 

[Timo2010]

Aber das ist doch nicht die Fehlerursache oder? Schließlich funktionierte es auch nur mit dem gleichen Operator.

Muss man also immer eine extra Variable erstellen?

 

[Murray]

Zunächst mal muss man Strings mit equals statt mit == vergleichen. Solange das nicht der Fall ist, lohnt es sich kaum, den Rest zu analysieren, weil die ersten elementaren Fallunterscheiden bereits nicht greifen.

 

[Final_Striker]

Was bring dir jetzt den Fehler zu suchen, wenn deine ganzen Vergleiche falsch sind???

nicht

stackoperator == "*"

sondern

stackoperator.equals("*")

In der Schule haben wir den Vergleich so gelernt.

Dann hast du es falsch gelernt.

 

[Timo2010]

So müsste es doch z.B. richtig verglichen sein:

		String operatormal = "*";
		String operatorgeteilt = "/";
		String operatorplus = "+";
		String operatorminus = "-";
		if ((operatora.equals(operatormal) || (operatora.equals(operatorgeteilt)) && (operatorb.equals(operatorplus))||(operatorb.equals(operatorminus)))){

 

[Timo2010]

Und da wundere ich mich, dass es alles nicht klappt. Vielen Dank für die wichtige Info. Eine Frage noch:
Wie sieht es aus, wenn man z.B. prüfen will, ob etwas != "" ist? Macht man es dann so:
!wert.equals("")?

 

[Murray]

Wie sieht es aus, wenn man z.B. prüfen will, ob etwas != "" ist? Macht man es dann so:
!wert.equals("")?

Das geht. Alternativ ginge auch wert.length() >0 oder (ab JAVA 6) !wert.isEmpty().

 

[Andi_CH]

stackoperator.equals("*")

"*".equals(stackoperator)

Ist noch besser, denn da gibt es keine NPE falls stackoperator mal null ist

 

[tfa]

"*".equals(stackoperator)

Ist noch besser, denn da gibt es keine NPE falls stackoperator mal null ist

Kommt drauf an, ob man null in stackoperator erwartet. Wenn nicht, ist die erste Variante die bessere.

 

[Andi_CH]

Ich hätte jetzt gedacht dann ist sie gleichwertig - was ist der Nachteil wenn die Kosntante zuerst kommt?

 

[Gastredner]

Ich hätte jetzt gedacht dann ist sie gleichwertig - was ist der Nachteil wenn die Kosntante zuerst kommt?

Ich habe immer den Eindruck, dass die Variante mit der führenden Konstante schlechter lesbar ist. Könnte allerdings auch einfach eine Sache der Gewohnheit sein.

 

[tfa]

Wenn ich mir sicher bin, dass stackoperator niemals null sein soll, dann verstecke ich durch die zweite Variante einen Fehler, wenn stackoperator (aus welchem Grund auch immer) doch irgendwann mal null sein sollte. In dem Fall möchte ich möglich früh eine Exception um die Ohren geworfen bekommen, und nicht, dass das Programm einfach so weiter läuft und am Ende unsinnige Ergebnisse liefert.