Erste Schritte Zufallszahl Spiel Problem

[MarleyBOSS]

Hallo,
Ich programmier erst seit neuem mit Java (seit 1 Woche) Ich habe das Problem in meinem Spiel, dass ich nie die richtige Zahl errate obwohl es nur 1/2 sein kann. Ich denke, dass der Fehler bei ,,if (input2 == zz){"liegt. Ich weiß nur nich was ich falsch gemacht habe :/ hier mein Code:

package de.Marley.java;
import java.util.Random;

import javax.swing.JOptionPane;
public class Marley {
    public static void main(String[] args) {
        int eingabe = JOptionPane.showConfirmDialog(null, "Spiel starten?", "Menü", JOptionPane.YES_NO_OPTION);
        if(eingabe == 0){
            Zahlenraten(true);
        }else if(eingabe == 0){
            System.exit(0);
        }
    }
    public static void Zahlenraten(boolean mm){
        while(mm = true){
            Random random = new Random();
            int zufallszahl = random.nextInt(3)+1;
            String zz = String.valueOf(zufallszahl);
            JOptionPane.showMessageDialog(null, "Errate die Zahl");
            String input2 = JOptionPane.showInputDialog(null);
            [B]if(input2 == zz){[/B]
                JOptionPane.showMessageDialog(null, "WOW Du hast die richtige Zahl erraten !");
                int zm = JOptionPane.showConfirmDialog(null, "Weiterspielen", "Zwischenenü", JOptionPane.YES_NO_OPTION);
                if(zm == 0 ){
                    mm = true;
                }else if(zm == 1){
                    break;
                }
            }else{
                JOptionPane.showMessageDialog(null, "Leider war das die Falsche Zahl ;(");
                int zm = JOptionPane.showConfirmDialog(null, "Weiterspielen", "Zwischenenü", JOptionPane.YES_NO_OPTION);
                if(zm == 0 ){
                    mm = true;
                }else if(zm == 1){
                    break;
                }
            }
        }
    }
}

 

[JStein52]

Ja, Strings vergleicht man mit equals, also.
if(input2.equals(zz)){

Edit:

und hier:
while(mm = true){
meinst du sicher
while(mm == true){

 

[InfectedBytes]

und hier:
while(mm = true){
meinst du sicher
while(mm == true){

Was man außerdem zu folgendem verkürzen sollte:

while(mm) {
...
}

 

[Meniskusschaden]

Wobei man sprechende Variablennamen verwenden sollte, z.B.:

while(weiterSpielen) {
...
}

 

[Xyz1]

Was man außerdem zu folgendem verkürzen sollte:

Müssen Anfänger nicht machen, es ist wichtig, das sie ihren Code verstehen, das Verständnis zählt.

Ich mag solche Oberlehrer-"Tipps" eigentlich nicht so.

Auch Konventionen sind erst einmal einerlei, wenn die Programmablauflogik verstanden wurde.

 

[Meniskusschaden]

Ich finde, das Weglassen von == true verbessert (in Verbindung mit guten Namen) nicht nur die Lesbarkeit, sondern fördert auch das Verständnis boolescher Ausdrücke. Das ist also alles andere als ein Oberlehrer-Tipp.
Wenn der Anfänger nicht darüber nachdenkt, warum das funktioniert, sollte er es meines Erachtens aber tatsächlich noch nicht benutzen, denn gerade zu Beginn sollte man generell nur Code verwenden, den man begreift. Das liegt aber letztendlich in der eigenen Verantwortung. Der Tippgeber kann erst einmal nicht wissen, inwieweit der Tippempfänger darüber reflektiert.

 

[Xyz1]

Es geht hier aber nicht um Produktivcode im professionellen Umfeld, sondern, wie ganz deutlich zu erkennen, das Thema ist Erste Schritte. Und ich werfe niemanden, der 1 Woche dabei ist, vor, was er erst in 1 Jahr lernen + verstehen wird können.

Wenn doch, müsste man alles andere auch kritisieren. Methode wird nicht regulär verlassen, JOP wird nicht geprüft, uvm.

mm = true ist allerdings eine Zuweisung. Darauf kann man sollte kenntlich machen. Alles andere find ich daneben. Aber ist ja gute Absicht.

 

[Meniskusschaden]

Ich will dir deine Meinung auch gar nicht ausreden. Ich finde es nur nicht angemessen, die Ratschläge von Leuten, die anderer Meinung sind, als Oberlehrer-"Tipps" zu bezeichnen.