if und else spielen verrückt

[feuerzeug]

hallo ihr

ich beschäfftige mich seit ein paar tagen mit java und will das durchziehen, nur bekomme ich jetzt schon kopfschmerzen :bahnhof:

zur info ich benutze netbeans 5.5

hier der code:

InputStreamReader i_stream = new InputStreamReader (System.in);
        BufferedReader buffer_r = new BufferedReader (i_stream);
        
               
        String name = buffer_r.readLine();
        String ich = "maik";
        
        if ( name == ich ) System.out.println("hallo maik");
        else System.out.println("wer bist du");

hier der gleiche nur bissl anders:

InputStreamReader i_stream = new InputStreamReader (System.in);
        BufferedReader buffer_r = new BufferedReader (i_stream);
        
               
        String name = buffer_r.readLine();
        
        
        if ( name == "maik" ) System.out.println("hallo maik");
        else System.out.println("wer bist du");

ich gebe immer maik ein

bei beiden kommt raus:

wer bist du

habe ich hier nen schusselfehler oder warum klappt das nicht?? wäre nett wenn mir jemand hilft.

mfg feuerzeug

 

[andre111]

statt

name == "maik"

musst du

name.equals( "maik" )

schreiben, weil du mit equals den inhalt des Strings vergleichst und mit == die Objektreferenz

 

[feuerzeug]

aha das wusste ich nicht. danke.

nur nochmal verständnisshalber:

String name = buffer_r.readLine();
        String ich = "maik";
       
        if ( name == ich ) System.out.println("hallo maik");

wenn ich der variablen -->ich<-- den string/wert -->maik<-- zuweise kann ich es auch nicht mit

name == ich

vergleichen???

mfg feuerzeug

 

[andre111]

ja, aber wenn du Strings vergleichen willst, mach es immer mit equals...
Dazu steht auch in der faq nochmal eine genaue erklärung dazu

 

[Guest]

Mit == machst du einen Referenzvergleich. Das heisst, Variable A zeigt auf die gleiche Speicherstelle, wie Variable B.
Ausnahme sind primitive Typen wie chat, int, long, float, double etc., bei diesen erfolgt ein Wertvergleich.
Halte dich einfach an die Regel: Bei Instanzen einer Klasse immer über equals vergleichen, bei primitiven Typen über
den Vergleichsoperator ==.
Seit JDK 1.5 sind ist noch Autoboxing dazugekommen, so dass auch bei den Wrappertypen Character,
Integer, Long, Float, Double ein solcher Vergleich möglich ist... aber lass das mal für's erste.

 

[Landei]

Das Problem mit dem String-Vergleich wird vielleicht so am deutlichsten

System.out.println(new String("bla") == new String("bla")); //false

 

[diel2001]

Hallo
ich erkläre es dir

String s1 = "hallo"; // String mit Zeichensatz "hallo"
String s2 = "hallo"; // String mit Zeichensatz "hallo"
String s3 = new String("hallo"); // neues Object
String s4 = new String("hallo"); // neues Object

s1.equals(s2); //true 
s1.equals(s3); //false
s3.equals(s4); //false

Bei s1 und s2 wird nur der Zeichensatz übergeben, dass heißt es ist der gleiche.
Die equals Methode ist für String nämlich überschrieben, dass wenn 2 zeichensätze verglichen werden,
sind sie gleich.

Bei s3 und s4 weden neue Objecte erstellt die zeigen alle auf ein anderes Object,
also sind Sie nicht gleich.

Ich mache es noch deutlicher und zwar :

MeineKlasse mk1 = new MeineKlasse(); // neues Object
MeineKlasse mk2 = new MeineKlasse(); // wieder neues Object
MeineKlasse mk3 = mk2;

mk1.equals(mk2); //false
mk2.equals(mk3); //true

mk1 und mk2 könnte man auch noch besser vergleichen .
Und zwar indem man die equals und die hashCode Methode
vernünftig implementiert in MeineKlasse.
Das ist aber ein etwas komplizierteres Thema.
Ich hoffe, dass es verständlich ist.
Falls noch Fragen sind einfach posten

 

[andre111]

MeineKlasse mk1 = new MeineKlasse(); // neues Object 
MeineKlasse mk2 = new MeineKlasse(); // wieder neues Object 
MeineKlasse mk3 = mk2; 

mk1.equals(mk2); //false 
mk2.equals(mk3); //true

ich glaube hier meinst du eher

MeineKlasse mk1 = new MeineKlasse(); // neues Object 
MeineKlasse mk2 = new MeineKlasse(); // wieder neues Object 
MeineKlasse mk3 = mk2; 

mk1 == mk2; //false 
mk2 == mk3; //true

ansonsten würde mk1.equals( mk2 ) true ergeben

 

[diel2001]

So habe nochmal ein Test gemacht um die letzte Aussage des Vorposters
zu widerlegen

public class Test02 {


	public static void main(String[] args) {
	
		NotEquals ne1 = new NotEquals();
		NotEquals ne2 = new NotEquals();
		NotEquals ne3 = ne2;
		
		System.out.println(ne1.equals(ne2));//false
		System.out.println(ne1 == ne2);//false
		System.out.println(ne2.equals(ne3));//true
		System.out.println(ne2 == ne3);//true
		
	}

}
class NotEquals{}

Ich wusste nämlich das ich recht haben muss und das hatte mich verwirrt

Das stimmt nämlich nicht :

ansonsten würde mk1.equals( mk2 ) true ergeben

P.S habe es gerade getestet
Ich werde die Tage mal versuchen zu dem Thema ein Tut zu schreiben
wenn es erwünscht ist .

 

[andre111]

Ich bin davon ausgegangen dass in MeineKlasse equals überschrieben wurde

 

[semi]

Ich bin davon ausgegangen dass in MeineKlasse equals überschrieben wurde

Jede Katastrophe beginnt mit einer Vermutung. :lol: :wink:

 

[Andreas29]

Hi,

ein Hinweis noch, zu dem, was Andre111 geschrieben hat:
Ich würde mir von vorneherein angewöhnen, in solchen Vergleichen mit equals IMMER die Konstante nach vorne zu schreiben. Also statt:

string.equals("maik")

lieber

"maik".euqals(string)

Hintergrund ist folgender:
Wenn string null ist, stirbt erste Version mit einer NullPointerException, während zweite Version einfach false liefert...

Ich weiß, für kurze Übungsprogramme mag das nicht notwendig erscheinen, weil man da meistens noch überblicken kann, ob eine Variable null sein kann oder nicht. Aber ich bin der Meinung, wenn man etwas lernt, dann am besten gleich richtig...

Liebe Grüße,
Andy

 

[Marco13]

Ich würde mir von vorneherein angewöhnen, in solchen Vergleichen mit equals IMMER die Konstante nach vorne zu schreiben.

Dem Argument an sich stimme ich prinzipiell zu, allerdings ist das IMMER hier ein bißchen zu stark hervorgehoben: Man sollte sich im Einzelfall schon überlegen, ob der String, um den es da geht, überhaupt 'null' sein KANN (und wenn nicht, dann ist eine NullPointerException an dieser Stelle genau das richtige Verhalten!) oder ob er null sein DARF, und WAS ggf. genau passieren soll, wenn er null ist....

Ich persönlich finde auch eine Abfrage wie

if (string != null && string.equals("Bla")) ...

nicht verkehrt (jaja, ich weiß, dass einige das anders sehen, aber) dort sieht man, dass der Fall, dass der string null ist, explizit berücksichtigt wurde.