Char vergleich zu Int

[Mcburn]

Hallo liebe Community,

mein Problem heute ist das ich durch die Suche auf euren Forum etwas gefunden hab mit char at und der vergleich mit int. Dies habe ich auch gleich angewendet.
Problem ist leider das der Code unterhalb alle Daten rauswirft. Das will ich ja nicht ich möchte wenn am anfang des Strings eine Zahl steht die Zeile behalten.
Das Kuriose ist es funktiert wenn ich statt ungleich alle gleich setzte außer die Code zeile

line.charAt(0) != 3 dort gibt er mir alle zeilen die mit 3 anfangen aus. Negiere ich aber nur EIN zweites line.char dann zeigt er mir gar nichts mehr an.

Ich hoffe ich konnte so gut wie möglich das ganze erklären. Sollte euch was unklar sein in meiner Formulieren weist mich drauf hin und ich versuche es nochmal besser zu erklären.

if(line.equals(null) || line.equals("") ||line.charAt(0) != '0' ||line.charAt(0) != '1' 
||line.charAt(0) != '2' ||line.charAt(0) != '3'||line.charAt(0) != '4'||line.charAt(0) != '5'
||line.charAt(0) != '6'||line.charAt(0) != '7'||line.charAt(0) != '8'||line.charAt(0) != '9'       ){ 
// Hier wird geprüft ob die Zeile keine Leerzeile ist
     array.remove(line);                               
        // ist dies der Fall wird die Zeile rausgeschmissen
     }  //if(line.equals(null) || line.equals(""))
     else
     {
         array.add(line);                                  
         //ist dies nicht der Fall wird die Zeile dem Array hinzugefügt
      }//else

 

[SlaterB]

das war ja wohl der schlechtest-formatierte Code, den ich bisher gesehen habe,
bitte bisschen darauf achten, bevor du das im Forum postest

----------

line.charAt(0) != '0' ||line.charAt(0) != '1'

jedes Zeichen erfüllt entweder die eine oder die andere Bedingung, mit oder verknüpft ergibt das immer true

 

[Mcburn]

Das erste Zeichen darf ja eine Zahl sein aber kein Buchstabe.
Deswegen das oder. Bei und würde es ja heisen "Wenn das erste Zeichen 0 und 1 usw, dann mache dies" Problem ist das ein Zeichen nicht mehre Werte annehmen kann.

 

[SlaterB]

"Wenn das erste Zeichen 0 und 1 usw, dann mache dies"
ergibt keinen Sinn


"Wenn das erste Zeichen 0 oder 1 usw, dann mache dies"
+
"Wenn das erste Zeichen nicht 0 und nicht 1 usw, dann mache dies"
dagegen schon, beides könntest du umsetzen

line.charAt(0) == '0' || line.charAt(0) == '1' ... ergibt was sinnvolles
line.charAt(0) != '0' && ine.charAt(0) != '1' ... ebenso,

nur
line.charAt(0) != '0' || line.charAt(0) != '1' ...
ist Quatsch

 

[SDE]

Ist array am Anfang mit allen lines gefüllt, oder leer? Falls gefüllt, kannst du dir das add sparen, falls leer, das remove.

Die Bedingung würd ich so formulieren:

if ((int)line.charAt(0)<0x30||(int)line.charAt(0)>0x39){
array.remove(line);
}

falls array voll

bzw

if ((int)line.charAt(0)>=0x30&&(int)line.charAt(0)<=0x39){
array.add(line);
}

falls array leer.

 

[MiDniGG]

line.equals(null) ist blödsinn, oder?!... entweder "line == null" oder line.equals("");

Ich würde es an Deiner Stelle in etwa so machen:

if(Character.isDigit(line.charAt(0))) {
   removen...
} else if(line.charAt(0) != null && !"".equals(line.charAt(0)) {
   adden
} else {
   removen, glaub ich ^^
}

Im Browser getippt...

Edit: Argh! Bin ich langsam :-D

 

[Der Müde Joe]

>line.equals(null) || line.equals("")

line == null || line.length() == 0

char c = line.charAt(0);
if(c < '0' && c > '9') //keine Zahl

 

[SDE]

char c = line.charAt(0);
if(c < '0' && c > '9') //keine Zahl

Äh stimmt geht auch... muß nur oder heissen statt und sonst kanns nicht erfüllt werden.

 

[Der Müde Joe]

>muß nur oder heissen

 

[Mcburn]

Jeder hat seinen eigenen Stil auch wenn meiner noch sehr stümperisch ist. Aber er funktioniert das mit dem negieren hätte mir gleich auffallen müssen. Aber danke für die Hilfe.
Ich könnte auch langsam mal das line.equales(null) rausschmeisen.

 

[Der Müde Joe]

>Ich könnte auch langsam mal das line.equales(null) rausschmeisen.

Ich hoffe mal, dass es immer false ist. Hier gibts nur false und NullPointerException