Erste Schritte IF-Abfrage fehlerhaft

[freakyy]

Hallo liebe Community!

Ich wollte ein eigenes Knobelspiel programmieren, welches den User nach Schere, Stein oder Papier fragt. Der Computer generiert eine Random-Zahl von 0-2, und hat dann auch entweder Schere, Stein oder Papier. Das funktioniert alles sehr gut, ich habe jedoch 2 große Probleme mit dem Code, schaut ihn euch mal bitte an.

Spoiler: JAVA Code

import java.util.Random;
import javax.swing.JOptionPane;

public class Master {
	public static void main(String[] args) {
		Random rand = new Random();
		while (true) {
			int var = rand.nextInt(3);
			String eingabe, comp = null;

			eingabe = JOptionPane.showInputDialog("Bitte Schere/Stein/Papier eingeben:");

			switch (var) {
			case 0:
				comp = "Schere";
				break;
			case 1:
				comp = "Stein";
				break;
			case 2:
				comp = "Papier";
				break;
			}
			System.out.println("comp: \"" + comp + "\" eingabe: \"" + eingabe + "\"");
			if (comp==eingabe) {
				JOptionPane.showMessageDialog(null, "Der Computer hat auch " + comp + " genommen! Score bleibt gleich.");
				score(false);
			} else if ((eingabe == "Stein") && (comp == "Schere")) {
				score(true);
			} else if ((eingabe == "Stein") && (comp == "Papier")) {
				score(false);
			} else if ((eingabe == "Schere") && (comp == "Stein")) {
				score(false);
			} else if ((eingabe == "Schere") && (comp == "Papier")) {
				score(true);
			} else if ((eingabe == "Papier") && (comp == "Schere")) {
				score(false);
			} else if ((eingabe == "Papier") && (comp == "Stein")) {
				score(true);
			}
		}
	}
	public static void score(boolean gewonnen) {
		int[] score= new int[2];
		if (gewonnen) {
			score[0] += 1;
			score[1] += 1;
			JOptionPane.showMessageDialog(null, "Gluckwunsch, Gewonnen! Insgesamt " + score[0] + "/" + score[1]);
		} else {
			score[0] += 0;
			score[1] += 1;
			JOptionPane.showMessageDialog(null, "Leider verloren! Insgesamt " + score[0] + "/" + score[1]);
		}
	}
}

Diese Probleme sind:
1- Die IF-Abfragen funktionieren komischer Weise nicht... Ich habe einmal den Code debugt und habe durch gestept, und auch wenn die IF, bzw. die Else-If abfrage stimmt, überspringt er es einfach. Kann mir da wer helfen, bzw. sagen warum er die IF-Abfrage einfach als "false" sieht, obwohl es richtig ist?

Spoiler: 2- Erledigt

in der Methode score, recht weit unten, definiere ich ein Array namens score[], das Problem ist jedoch, dass diese Variable immer neu auf 0 gesetzt wird. Wo, bzw. wie kann ich die Variable anders setzen, dass er nur einmal die Variable definiert?

Danke schon mal im Vorraus!

PS: Bitte produktive Antworten, nicht soetwas wie: der Code ist ja ur schlecht, schäm dich!

 

[Kaibear]

Zum Score fällt mir auf, dass du jedes mal, nach jedem Spielergebnis die Score-Funktion neu aufrufst. Das ist auch kein Problem bis auf, dass er das Array immer neu erstellt, weshalb der Score immer 0 ist, wenn das zweite Spiel gemacht wird.

Du müsstest die Array-Felder global definieren und nicht nur in der Methode. Dann müsstest du dein Ziel erreichen.

Was die If-Anweisung angeht, kann ich dir nicht weiterhelfen. Ich vermute irgend ein Problem mit den "breaks" aus der If-Anweisung. Aber das ist nur Spekulation.

 

[freakyy]

Danke für die schnelle Antwort,

habe jetzt das Array außerhalb der Methoden erstellt, und es funktioniert jetzt, sprich Problem 2 ist gelöst

 

[nomuri]

Sorry hatte nen Bug. Antwort 2 bereits gegeben.

Zu 1. Man vergleicht Strings nicht mit == sondern mit .equals()

 

[freakyy]

Oh, das erklärt einiges...
Das heißt, wenn ich den Inhalt einer Variable überprüfen will, und diese als String definiert ist, muss ich statt

if(variable=="String"){}

if(variable.equals("String")){}

eingeben?

 

[MrClave]

== überprüft die Referenz

.equals( ) überprüft den Inhalt der Objekte

 

[shishigami]

== überprüft die Referenz

.equals( ) überprüft den Inhalt der Objekte

Prinzipiell macht equals() auch nur das

public boolean equals(Object obj) {
    return (this == obj);
}

wenn du es nicht selbst überschreibst

 

[ARadauer]

Prinzipiell macht equals() auch nur das

public boolean equals(Object obj) {
    return (this == obj);
}

wenn du es nicht selbst überschreibst

aber nicht von String!!

   /**
     * Compares this string to the specified object.  The result is {@code
     * true} if and only if the argument is not {@code null} and is a {@code
     * String} object that represents the same sequence of characters as this
     * object.
     *
     * @param  anObject
     *         The object to compare this {@code String} against
     *
     * @return  {@code true} if the given object represents a {@code String}
     *          equivalent to this string, {@code false} otherwise
     *
     * @see  #compareTo(String)
     * @see  #equalsIgnoreCase(String)
     */
    public boolean equals(Object anObject) {
	if (this == anObject) {
	    return true;
	}
	if (anObject instanceof String) {
	    String anotherString = (String)anObject;
	    int n = count;
	    if (n == anotherString.count) {
		char v1[] = value;
		char v2[] = anotherString.value;
		int i = offset;
		int j = anotherString.offset;
		while (n-- != 0) {
		    if (v1[i++] != v2[j++])
			return false;
		}
		return true;
	    }
	}
	return false;
    }

 

[shishigami]

aber nicht von String!!

Natürlich, war nur eine Anmerkung um klar zu stellen dass "equals( ) überprüft den Inhalt der Objekte" nicht ganz richtig ist.

 

[MrClave]

Ich meine selbstverständlich eine überschriebene equals Methode.