Zwei Strings vergleichen

[quiqum]

Hallo Leute
Ich habe ein ziemlich kniffliges Problem. (mind. für mich)

String foo = "1234";
String bar = "1234:4321";
String[] a = bar.splitt(":");
if(foo == a[1]) {
 System.out.println(":D");
}

Ich habe das Ganze schon mit equals versucht. Bei mir ist foo eigentlich eine Klasse, bzw. Methode, die einen String als Rückgabe liefert. Ich weiss nicht, ob das einen Unterschied ausmacht.

Wer kann mir helfen?

 

[Noctarius]

Versuchs mal mit equals und a[0]

Arrays sind 0 based

 

[quiqum]

Das bringt nichts. Ich muss den Wert im a[1] haben...

 

[Ark]

[HIGHLIGHT="Java"]foo == a[0] // false
foo == a[1] // false
foo.equals(a[0]) // true
foo.equals(a[1]) // false[/HIGHLIGHT]
So etwas sollte unter den aktuellen Umständen rauskommen, denke ich.

Ark

 

[Marco13]

Und wenn man jetzt sagt, "Mach einfach
String temp = a[0];
a[0] = a[1];
a[1] = temp;
" kommt der nächste Brocken.

 

[L-ectron-X]

a[1].equals(foo) :question:

 

[quiqum]

Ich habe glaube ein kleiner Fehler gemacht. Ich brauche bei mir a[0] für eine Funktion dessen return ich mit a[1] vergleichen möchte. Gibt es da eine Alternative? Oder muss ich mich mit diesen Strings herumschlagen...

 

[Ark]

Bevor wir hier anfangen, möglicherweise aneinander vorbeizureden: Zeig doch bitte das Stück Code, um das es konkret geht. Dein Code am Anfang ist schon ziemlich "verbraucht", so wie eine Variable beim Lösen eines Gleichungssystems.

Ark

 

[quiqum]

Das ist meine letzte "Fassung"... Sie funktioniert auch nicht...
Bei GetLine(pin,"",'\n',100); kommt logquiqum:1234
quiqum 1234 sind korrekte Logindaten

           String Temp1 = "";
            String Temp2 = "";
            String Temp3 = "";
            boolean gemacht = false;
           Temp1 = GetLine(pin,"",'\n',100);
          
           Temp2 = GetFirstThree(Temp1);
           Temp2 = Temp2.toUpperCase();
           Temp3 = RemoveFirstThree(Temp1);

           if(Temp2.equals("LOG")) {
               if(Temp3.length() < 3) {
                   out.write(("CANLOG").getBytes());
                   out.write('\n');
               }else{
                   String[] logindaten = Temp3.split(":");
                   //System.out.println(String.valueOf(logindaten.length));
                   
                   if(logindaten.length == 2) {
                       //System.out.println(logindaten[0] + " : " + logindaten[1]);
                       ConfigFile Benutzer = new ConfigFile("users.conf");
                       if(Benutzer.Load()) {
                           //System.out.println(Benutzer.Get(logindaten[0]));
                           String ta1 = new String(logindaten[0]);
                           String ta2 = new String(logindaten[1]);
                           if(!Benutzer.Get(ta1).equals("<error>")) {
                               if(ta2.equals(Benutzer.Get(ta1))) {
                                   L_Login = true;
                                   out.write(("BESLOG").getBytes());
                                   out.write('\n');
                               }else{
                                   out.write(("CANLOG").getBytes());
                                   out.write('\n');
                               }
                           }else{
                               out.write(("CANLOG").getBytes());
                               out.write('\n');
                           }
                       }else{
                           out.write(("CANLOG").getBytes());
                           out.write('\n');
                       }
                   }else{
                       out.write(("CANLOG").getBytes());
                       out.write('\n');
                   }
               }
               gemacht = true;
           }

 

[Ark]

Ich glaube, in diesem Stück Code sind mehr Fehler als Zeilen.

Wo fängt man da am besten an? Bezeichner sind nicht nach Namenskonventionen geformt, alles sieht sehr stark nach prozeduraler Programmierung aus, Code wiederholt sich mehrfach, Exceptions scheinen gänzlich unbekannt zu sein; der Umgang mit Strings (worum es ja hier eigentlich gehen sollte) scheint hier fast noch das kleinere Problem zu sein.

EDIT:
Teilweise scheinen auch nicht die Anforderungen an einen Algorithmus bekannt zu sein: allgemein, ausführbar, eindeutig, endlich. (So kenne ich das zumindest.) Dieser Code dagegen scheint teilweise nicht allgemein zu sein und neigt dazu, unendlich lang zu werden ...

Ark

 

[hdi]

Und woher sollen wir wissen, was GetLine() macht? Zeig mal die Methode.
Im Übrigen hat Ark völlig Recht: Es würde nicht schaden den Code erstmal gemäss Name Conventions zu überarbeiten.
Variablen- und Methodennamen schreibt man in camelCase.

 

[quiqum]

Ja, ich programmiere schon lange, jedoch habe ich nie Richtig gelernt, wie man "richtig" programmiert. Exception usw. sind natürlich drin, doch in diesem Teil nicht sichtbar. Der Code ist auch noch in der Anfangsphase und wird nur langsam Modularisiert...

Kennt jemand eine gute Seite (im Netz), wo man "schönes/ richtiges Programmieren" lernen/nachlesen kann?

Der Fehler lag daran, dass der Telnet Client noch ein (char)13 am Ende der Zeile einfügte (das bekannte '\r'). Dieses habe ich leider völlig vergessen, da ich nur '\n' nehme...
GetLine() gibt mir einfach eine Zeile, welche es von einer PipedInputStream herausliest.

 

[Ark]

Wie wäre es mit Folgendem?
[HIGHLIGHT="Java"]BufferedReader br = new BufferedReader(new InputStreamReader(new PipedInputStream(blubb)));
String line = br.readLine();[/HIGHLIGHT]
"Richtiges" Programmieren wird es wahrscheinlich auch nie geben: Geh mit einem Problem zu 10 Informatikern, und du bekommst 15 Lösungen. Man kann aber für eine Programmiersprachengeneration/Programmiersprache angemessen programmieren. Da gehört dein Code wohl leider nicht dazu. So etwas macht sich auch nicht über Nacht, man muss halt viele (leidvolle) Erfahrungen machen und auch Code von Erfahrenen nachvollziehen (hier im Forum gibt's einen Experten, der gerade das ganz besonders gut kann! *ironie*).

Kram doch einfach ein paar alte Programme/Codestücke aus und überlege, was du aus heutiger Sicht anders machen würdest. Auch dabei kann man eine Menge lernen.

Ark

 

[Leroy42]

Hallo Leute
Ich habe ein ziemlich kniffliges Problem. (mind. für mich)

String foo = "1234";
String bar = "1234:4321";
String[] a = bar.splitt(":");
if(foo == a[1]) {
 System.out.println(":D");
}

Ich habe das Ganze schon mit equals versucht. Bei mir ist foo eigentlich eine Klasse, bzw. Methode, die einen String als Rückgabe liefert. Ich weiss nicht, ob das einen Unterschied ausmacht.

Wer kann mir helfen?

Nein! foo ist ein String und keine Methode.

Sicher macht das einen Unterschied aus!

 

[quiqum]

So hatte ich es eigentlich. Jedoch wollte ich teilweise nur bis zu einem gewissen Zeichen auslesen...

Aber Danke für die Tipps!