kleines Würfelspiel

[schnosi]

Hallo an alle,
bin neu hier und studiere seit diesem Semester Elektrotechnik und muss mich jetzt gezwungener Maßen mit Java auseinandersetzen, aber das macht bisher eigentlich Spaß...nicht so schön ist es, wenn man dann halb am verzweifeln ist mit den Fehlern und einfach nicht weiter weiß...so wie jetzt.

Wir sollen ein kleines Würfelspiel schreiben mit 3 Integervariablen (a,b,c) die jeweils das Resultat eines Würfelwurfs darstellen.
Ausgewertet wird folgendermaßen:
- wenn alle 3 Würfel die gleiche Augenzahl haben, beträgt die Punktzahl 20
- wenn 2 der 3 Würfel das gleiche Ergebnis haben, -"- 10
- wenn alle Würfel ein verschiedenes Ergebnis haben, aber ein Würfel eine 6, dann ist die Punktzahl 5
... für alle anderen Fälle ist die Punktzahl -1
Die Punktzahl wird in einer int-Variable gespeichert.

Das ist mein bisheriges Programm:

public class WuerfelSpiel{
	public static void main (String[] args){
		int a = 4;
		int b = 4;
		int c = 4;
		int punkte;
			if ( a == b == c ){
				punkte = 20;
				System.out.println("Der Wurf ergab:"+punkte+" Punkte");
				}else if ( a == b || b==c || a==c ){
					punkte = 10;
					System.out.println("Der Wurf ergab:"+punkte+" Punkte");
						}else if ( (a|b|c) = 6 ){
							punkte = 5;
							System.out.println("Der Wurf ergab:"+punkte+" Punkte");
							}else{
								punkte = -1;
								System.out.println("Der Wurf ergab:"+punkte+" Punkte");
							}
	}
}

Wenn ich dieses Programm nun kompilieren will, dann gibt er mir ein Error in Zeile 7 aus ( if ( a == b == c ){ ) und sagt, dass da incomparable types sind ... boolean und integer ...was ich sehr komisch finde, denn weiter unten habe ich die selbe Operation auch nochmal und da gibt er kein Error aus ?!?!?!?!

Und ein zweiter Error erscheint in Zeile 13 ( }else if ( (a|b|c) = 6 ){ ) ...da sagt er mir, dass er eine Variable erwartet hat, aber eine Konstante gefunden hat... mhh weiß nicht, was ich da machen soll.

Bitte helft mir. Würde mich sehr über eine schnelle Antwort freuen. DAAAANKEEE!

Liebe Grüße,
Schnosi

 

[SlaterB]

> was ich sehr komisch finde, denn weiter unten habe ich die selbe Operation auch nochmal und da gibt er kein Error aus ?!?!?!?!

wo genau, nicht im geposteten Code?
ist allerdings auch möglich dass nach Auffinden eines Fehlers danach weniger oder anders geprüft werden,
wenn man z.B. paar } zuviel setzt gilt eine Klasse schnell als geschlossen und der Code danach wird ganz anders interpretiert als wenn er normal in einer Methode stehen würde

a == b == c
ist jedenfalls keine erlaubte Java-Syntax, immer nur zwei Werte vergleichen und die Ergebnisse mit oder/ und verknüpfen

> if ( a == b || b==c || a==c ){
schon viel besser

--------

es reicht übrigens,
System.out.println("Der Wurf ergab:"+punkte+" Punkte");
einmal ganz ans Ende nach allen ifs/else zu schreiben

 

[Painii]

Wenn ich dieses Programm nun kompilieren will, dann gibt er mir ein Error in Zeile 7 aus ( if ( a == b == c ){ ) und sagt, dass da incomparable types sind ... boolean und integer ...was ich sehr komisch finde, denn weiter unten habe ich die selbe Operation auch nochmal und da gibt er kein Error aus ?!?!?!?!

Das sieht für deinen Rechner etwa so aus:

boolean bool = a==b;
boolean bool2 = bool == c; //das ist dein fehler

Einfache Lösung:
Aus a==b und a==c folgt auch b==c (für reelle Zahlen zumindest).

Und ein zweiter Error erscheint in Zeile 13 ( }else if ( (a|b|c) = 6 ){ ) ...da sagt er mir, dass er eine Variable erwartet hat, aber eine Konstante gefunden hat... mhh weiß nicht, was ich da machen soll.

1. Du versuchst eine Zuweisung, keinen Vergleich. Also versucht Java der Variable a|b|c den Wert 6 zuzuweisen.

2. Was soll denn a|b|c geben? (da | der bitweise Operator ist gäbe es das bitweise or der 3 zahlen aus)

 

[Leroy42]

			if ( a == b == c ){

Falsch da a==b einen boolean liefert der dann mit einem int verglichen werden soll

						}else if ( (a|b|c) = 6 ){

Auch falsch da man dem Ausdruck a|b|c nichts zuweisen kann.

 

[schnosi]

danke erstmal für die super schnelle antwort!

also das erste hab ich verstanden und auch gleich in die tat umgesetzt... vielen dank!
aber bei meinem zweiten fehler komme ich nicht so recht weiter...
was ich möchte ist ja ihm sagen, dass ein würfelergebnis (entweder a oder b oder c) gleich 6 sein sollen,also nur eins der drei und deswegen muss ich doch da irgendwie eine konstante einbinden oder nicht? aber das macht er ja nicht, denn er will ja eine variable ...
könnt ihr mir da nochmal weiterhelfen? möchte es ja verstehen lernen!!

danke übrigens auch für den tip mit system.out.... ganz am ende reicht

 

[faetzminator]

[c]if (Math.max(a, Math.max(b, c)) == 6)[/c] oder [c]if (a == 6 || b == 6 || c ==6)[/c]

 

[schnosi]

vielen, vielen dank! ihr seid super! ... habe es gleichzeitig auch verstanden, woran es lag...

 

[Landei]

[c]if (Math.max(a, Math.max(b, c)) == 6)[/c] oder [c]if (a == 6 || b == 6 || c ==6)[/c]

if(a+b+c == 18){...}

 

[Painii]

if(a+b+c == 18){...}

Da müssten ja dann alle 3 Zahlen 6 sein, aber es reicht ja wenn eine Zahl 6 ist.

 

[Swoop]

Außerdem kann ja die Ausgabe einmal nur ganz am Schluss stattfinden ..!

 

[Landei]

Da müssten ja dann alle 3 Zahlen 6 sein, aber es reicht ja wenn eine Zahl 6 ist.

Ups, verguckt - das sind ja || und keine &&. Das kommt davon, wenn man zu clever sein will. Aber da ich lernresistent bin, hier der zweite Versuch:

 if((a%6)*(b%6)*(c%6) == 0){...}