if else problematik

Zitat von oxypha

hallo, ich habe zwei fragen vielleicht kann mir ja hier jemand helfen (x

bei folgendem code:

Hoioi!

Zitat von oxypha

Code

public class ZRF extends ZRFrame {        public static String zahlenraten (int input, UnbekannteZahl zuErraten) {                    
                            while(zuErraten.gleich(input)==false){
                                if (input == 666) {
                                return ""+zuErraten;
                                }
                                else if (input < 0 || input > 99) {
                                //System.out.println("Nur Zahlen von 0 bis 99!");
                                return "Nur Zahlen von 0 bis 99!";
                                }
                                else if (zuErraten.gleich(input)) {
                                //System.out.println("Gratulation! Zahl erraten!");
                                return "Gratulation! Zahl erraten!";
                                }
                                else if (zuErraten.kleiner(input)) {
                                //System.out.println("Gesuchte Zahl ist kleiner.");
                                return "Gesuchte Zahl ist kleiner.";
                                }
                                else {
                                //System.out.println("Gesuchte Zahl ist größer.");
                                return "Gesuchte Zahl ist größer.";
                                }
                            }
                    return "Gratulation! Zahl erraten!";
        }
    }

Alles anzeigen

ich verstehe nicht wieso bei meinem einfachen code hier das letzte return nötig ist und wieso

Puh.. der Code.. ok..
Du hast ne Methode, die nen String zurück gibt. Als erstes in der Methode hast du eine while-Schleife. In die kommst du rein.. oder auch nicht (wenn die Bedingung von Anfang an falsch ist. Kommst du nicht rein, hast du nirgendwo ein return stehen. Deshalb brauchst du nach dem while das return (für den Fall, dass du ins while gar nicht rein kommst.). Interessanter wäre, wozu du das while brauchst. Das ist aus meiner Sicht absolut unnötig.

Zitat von oxypha

Code

else if (zuErraten.gleich(input)) {return "Gratulation! Zahl erraten!";}

einfach übergangen wird?

Liegt an der while-Schleife. Du betrittst die Methode mit zwei gleichen Zahlen als Parametern. Ins while kommst du aber nur, wenn diese beiden Zahlen nicht gleich sind --> ganze while-Schleife wird übersprungen, du kommst direkt zum return unten. Wie gesagt, die while-Schleife ist nutzlos..

Zitat von oxypha

genau wie später hier:

Code

if (x=="Gesuchte Zahl ist kleiner.") {...}
        if (x=="Gesuchte Zahl ist größer.") {...}
        if (x=="Gratulation! Zahl erraten!") 
        {...}
        else {...}

bzw.:

Code

if (x=="Gesuchte Zahl ist kleiner.") {...}
        else if (x=="Gesuchte Zahl ist größer.") {...}
               else if (x=="Gratulation! Zahl erraten!") {...}
                      else {...}

solange man else nicht als kommentar kennzeichnet immer else herauskommt und sobald man es // auskommentiert funktioniert es?

irgendwer eine idee warum das so ist?

Hier stand Schwachsinn.. ein gedanklicher Mix aus dem neuen Switch und equals()
Generell sollte man Strings in Java nur mit .equals() vergleichen.

Allerdings.. was immer das wird (ein grafisches Zahlenratespiel?).. der Code ist ziemlich grauenhaft. Wenn du später mit == Abfragen (was du mit equals() machen solltest, s.o.) musst, welche Usernachricht du zurückgegeben hast, um irgendetwas zu tun, schreit das geradezu nach Refactoring (Struktur deines Programms verbessern). Könntest beispielsweise ein Enum statt nem String zurückgeben, das ne toString() Methode hat. Enums kannst du sauber vergleichen, und den Text im Enum jederzeit nach belieben verändern, ohne dass es irgendwelche Auswirkungen hat.

Zitat von oxypha

liebe gruesse, oxypha

Liebe Grüße,
emptyvi

Zitat von emptyvi

Verwendest du Java 7 oder 6? Java 6 konnte Stringvergleiche mit == noch nicht. Ansonsten fällt mir spontan nichts ein.

Was verzapfst denn du da bitte? Schau dir mal an, wie String Interning funktioniert, und was der Unterschied zwischen dem Identitätsvergleich mit == und dem inhaltlichen Vergleich mit equals() ist.

Strings immer mit equals vergleichen.

Und allgemein ist der Wechsel des JDK von 6 auf 7 mit mehr Problemen als Lösungen verbunden. Java 6 32 Bit und man erspart sich eine Menge von Problemen .. aber das is ein anderes Thema.

Zitat von JohnFoo

Was verzapfst denn du da bitte? Schau dir mal an, wie String Interning funktioniert, und was der Unterschied zwischen dem Identitätsvergleich mit == und dem inhaltlichen Vergleich mit equals() ist.

Strings immer mit equals vergleichen.

Ich denke er wollte eigentlich darauf hinaus das es in Java 7 möglich ist Strings mit switch zu vergleichen.

Thomas

Zitat von JohnFoo

Was verzapfst denn du da bitte?

Ich begrüße dich auch mit Wohlwollen, mein herzallerliebster JohnFoo! Auch dir ein freundliches Hallo!

Aber ich gebe zu.. ich war wohl vorhin nicht ganz da. Was anwesender sagt, ist richtig.. Ich hatte an Switch mit Strings gedacht.. warum auch immer..

Zitat von JohnFoo

Schau dir mal an, wie String Interning funktioniert, und was der Unterschied zwischen dem Identitätsvergleich mit == und dem inhaltlichen Vergleich mit equals() ist.

Ist mir bekannt, aber danke :).

Zitat von JohnFoo

Strings immer mit equals vergleichen.

Dem ist nichts hinzuzufügen.

Zitat von JohnFoo

Und allgemein ist der Wechsel des JDK von 6 auf 7 mit mehr Problemen als Lösungen verbunden. Java 6 32 Bit und man erspart sich eine Menge von Problemen .. aber das is ein anderes Thema.

Ich weiß nicht genau, auf was du hinaus willst, aber ja.. ist ein anderes Thema.

Meine herzlichsten Abschiedsgrüße entbahrend,
emptyvi