Programmfehler - Anfänger

[jekkt]

Hallo ,

ich habe hier 3 Fehler die ich beseitigen muss , kann mir jemand sagen was falsch ist und wie ich die beseitigen kann?

class Wuerfel {

	public static void main (String [] arg) {

		int a,b,c,d,e;

		

		Out.println(" Bitte kleinste Zahl eingeben: ");

		a = In.readInt();

		Out.println(" Bitte nachstgroessere oder gleiche Zahl eingeben: ");

		b = In.readInt();

		Out.println(" Bitte nachstgroessere oder gleiche Zahl eingeben: ");

		c = In.readInt();

		Out.println(" Bitte nachstgroessere oder gleiche Zahl eingeben: ");

		d = In.readInt();

		Out.println(" Bitte groessere Zahl eingeben: ");

		e = In.readInt();

		

		if ((a==b) & (b==c) & (c==d) & (d==e)); {

		

			Out.println("  Yahtzee ");

			

		}

		else if (((a==b) & (b==c) & (c==d)) && (d!=e)) || (!(a==b) && ((b==c) & (c==d) & (d==e)));

		

			Out.println("4 einer Art");

			

		else if (((a==b) && (c==d==e)) || ((a==b==c) && (d==e)))

		

		// Dadurch das Yahtzee zuerst ausgewertet wird , braucht man bei Full House nicht zu sagen das d==e ungleich a | b | c ist.

		

			Out.println(" Full House ");

			

		else if((b-1==a)&&(c-1==b) && (d-1==c) && (e-1)==d)

		

			Out.println(" Lange Strasse ");

			

		else if (((c-1)==b) && ((d-1)==c) && ( (d>e) || (b>a) ))

		

		// Dadurch das ich vorher die grosse strassee geprueft habe kann sie nicht mehr rauskommen.

			Out.println(" Kurze Strasse ");

			

		else

		

			Out.println("Leider keine Figur");

		

	}

}

compliler sagt : Wuerfel.java:21 : else without if
Wuerfel.java:21: illegal start of expression
Wuerfel.java:25: else without if

 

[Sergeant_Pepper]

Hi,
Java-Quelltexte solltest du im Forum mit
[noparse]

 ... dein Programm ...

[/noparse]
umgeben, ist wesentlich lesbarer.

Ein Fehler in deinem Programm, den ich spontan sehe:

if ((a==b) & (b==c) & (c==d) & (d==e));

Am Ende der Zeile darf kein Semikolon stehen.

Allgemein ist die Neigung, hier für andere Hausaufgaben zu machen, nicht so hoch.

 

[Noctarius]

Ich hab's mal geschönt

 

[Sergeant_Pepper]

wow

@jekkt
der gleiche Fehler, den ich im ersten Posting genannt habe, existiert auch in Zeile 41 (bezogen auf den Forum-Code oben)

 

[KrokoDiehl]

if ((a==b) & (b==c) & (c==d) & (d==e));

Sicher dass es

&

anstatt von

&&

sein soll? Der normale logische Operator ist nämlich das

&&

.
Außerdem sehe ich eine Variablenbenennung á la "a, b, c, d ..." auch als Fehler an :bae:

 

[eRaaaa]

Sicher dass es

&

anstatt von

&&

sein soll? Der normale logische Operator ist nämlich das

&&

.

naja, short circuit ist an sich schon zu bevorzugen, aber falsch ist der andere operator auch nicht oder? (er wertet halt immer alle ausdrücke aus)

 

[Painii]

naja, short circuit ist an sich schon zu bevorzugen, aber falsch ist der andere operator auch nicht oder? (er wertet halt immer alle ausdrücke aus)

Ich hab gelernt dass [c]&[/c]bitweise verknüpft (also beides auswertet) und deswegen bei Logik-Operationen[c]&&[/c] zu bevorzugen ist.

Vor allem wenn man nicht simple int-vergleiche macht sondern irgendwelche Funktionen aufruft die evtl. etwas länger zum rechnen brauchen.

Denke das war auch deine Aussage, ich weiss nur grad nicht welchen der beiden Operatoren du mit short circuit bezeichnest (mein Fehler)

 

[Sergeant_Pepper]

den jekkt interessiert das alles gar nicht mehr ...
:-( :-(
Perlen vor die S...

 

[bygones]

naja, short circuit ist an sich schon zu bevorzugen, aber falsch ist der andere operator auch nicht oder? (er wertet halt immer alle ausdrücke aus)

[c]if (true & ewigDauerendeBerechnung()) { }[/c] gegen [c]if (true && ewigDauerendeBerechnung()) { }[/c] ist schon ein kleiner Unterschied (eben weil & immer (!) beides auswertet)

 

[eRaaaa]

[c]if (true & ewigDauerendeBerechnung()) { }[/c] gegen [c]if (true && ewigDauerendeBerechnung()) { }[/c] ist schon ein kleiner Unterschied (eben weil & immer (!) beides auswertet)

ich hab doch auch garnichts anderes behauptet. (man beachtet das bevorzugen in meinem satz ;D )
falsch wird der ausdruck dadurch trotzdem nicht..(oder etwa doch ?? ) es gibt auch situationen, wo ich umbedingt möchte dass ewigDauerendeBerechnung() ausgeführt wird.
dein beispiel macht imo auch wenig sinn, da er ewigDauerendeBerechnung() ausführen MUSS, da du mit && vernüpfst (bzw vorne true stehen hast ???:L )

 

[Unregistriert]

else if (((a==b) & (b==c) & (c==d)) && (d!=e)) || (!(a==b) && ((b==c) & (c==d) & (d==e)));



Out.println("4 einer Art");

Ist diese Abfrage etwa richtig? Was ist, wenn alle Zahlen gleich sind mit Ausnahme z.B. der Zahl a? Und somit auch d = e ist.

 

[Nicer]

eine "IF" funktion ist eigentlich immer wie folgt aufgebaut :

if (zu-prüfender-faktor Operator boolean-( false / true )-oder-ne-zahl) { hier kommt das was passiert wenn die If funktion positiv ausgeht }

Wennich das richtig weiß

MfG Nicer

 

[HanKooR]

Um nochmal diese & oder && - Sache aufzugreifen:
& und && erfüllen die selbe Funktion auf unterschiedliche Weise.
Beispiel:

"Bedingung a & Bedingung b" prüft Bedingung b auch wenn a false ist und die Überprüfung von b demnach nicht mehr nötig wäre.

"Bedingung a && Bedingung b" prüft b halt nur wenn a true ist.

Der Vergleich mit && wird auch oft als "lazy Evaluation", "short circuit Evaluation" oder "McCarthy" Evaluation bezeichnet.