falscher if-Zweig

[exceble]

Nabend,

ich habe eine kleine Methode overwriteField() geschrieben, welche einfach ein Feld ersetzen soll, wenn dieses noch frei ist

 overwriteField()

    private static void overwriteField(String eingabe){
        int field;

        field = Integer.parseInt(eingabe) - 1;
        
        System.out.println(eingabe + " " + fields[field]);

        if (fields[field] == eingabe){
            System.out.println("abc");
        } else {
            System.out.println("xyz");
        }
    }

Abgesehen davon, dass es eine Exception geben würde, wenn die Eingabe keine Zahl ist, habe ich ein ganz anderes Problem. Und zwar, dass fields[field] und eingabe zwar gleich sind, das Programm aber dennoch in den else-Zweig springt und xyz ausgibt.

 Ausgabe

4 4
xyz

Das kann ich mir irgendwie nicht erklären, habe ich etwas ganz simples übersehen? Vielen Dank für die Hilfe

Cheers

[Syc]

So auf Anhieb würde ich sagen, dass eingabe noch immer ein String ist und du versuchst einen String mit einem Int zu vergleichen. Du müsstest eingabe auch noch als int parsen. Wie das geht weißt du ja.

[exceble]

Originally Posted by Syc

So auf Anhieb würde ich sagen, dass eingabe noch immer ein String ist und du versuchst einen String mit einem Int zu vergleichen. Du müsstest eingabe auch noch als int parsen. Wie das geht weißt du ja.

Oh ups, ich hatte vergessen zu erwähnen, dass fields[] ein String-Array ist, somit vergleiche ich einen String mit einem String, aber vielen Dank für die rasche Antwort!

[Syc]

Oh da hast du Recht. In meinem Fall würde der Compiler eh ne exception werfen.
Habe schon länger nicht mit Java gearbeitet. Aber wenn ich mich recht erinnere muss/soll man für Stringvergleiche .equals() verwenden, damit der tatächliche Wert geprüft wird. Vielleicht liegt da das problem.

[exceble]

Originally Posted by Syc

Oh da hast du Recht. In meinem Fall würde der Compiler eh ne exception werfen.
Habe schon länger nicht mit Java gearbeitet. Aber wenn ich mich recht erinnere muss/soll man für Stringvergleiche .equals() verwenden, damit der tatächliche Wert geprüft wird. Vielleicht liegt da das problem.

Tatsächlich, ich nämlich auch nicht Vielen dank!

[Mikesch01]

Nur als nachträglichen Hinweis:

[Menan]

Die beste Methode Strings in java zu Vergleichen ist die StringUtils Bibliothek von Apache.

Dann kannst du Strings einfach per:

boolean eq = StringUtils.equals(string1,string2);

vergleichen. Und hast gleichzeitig auch noch den Nullcheck dabei, Sprich wenn einer der beiden Strings Null wäre, würde fals zurückgegeben werden und nicht dein Programm abstürzen.