(BufferedReader) inLine() "verschluckt" zeilen ?

[dh81]

hallo,
folgendes problem: ich habe in einer datei eine wortliste mit tausenden von einträgen. diese sollen in ein listen-array gesteckt werden (29 felder entsprechend alphabet samt umlauten). an und für sich klappt das, was ich da so zusammengebastelt hab, recht prima, allerdings ist mir beim nachzählen in excel aufgefallen, dass da einige einträge fehlen (z.b mit "A" beginnend 733:258, also um die zwo drittel). waran könnte das liegen, wie könnte man das fixen???

 public static int charToIndex(char c){
  int output = (int)c;
  switch(output){
    case 196: output = 26; break;
    case 214: output = 27; break;
    case 220: output = 28; break;
    default: output = output -65;  
  }
  return output;
  }
  
  /* testEinlesenUndSortieren liest aus einer unicode-txt-datei zeilenweise als string ein
   * in abhängigkeit zum anfangsbuchstaben wird das wort in einem 29-feldigen List-array gespeichert
   * der index für das array-feld wird über charToIndex beschafft
   * zurückgegeben wird befüllt das übergebene List-Array
   */ 
  
  
  public static void textEinlesenUndSortieren(java.util.List[] list){
  String tmp;
   try{
      BufferedReader in = new BufferedReader(new FileReader("muh.txt"));
      while(in.readLine() != null){
        tmp = (String)in.readLine();
        tmp.trim(); 
        if(tmp.length()>=5 && tmp.length()<=10){
        tmp = tmp.toUpperCase(); 
        list[Quellen.charToIndex(tmp.charAt(0))].add(tmp); 
        }
      }
      in.close();
    }
   catch (Exception e) {};
  }

 

[madboy]

Ich vermute mal, dass ziemlich genau die Hälfte deiner Einträge fehlt ;-)

      while(in.readLine() != null){ //liest eine Zeile, die wird aber nirgends gespeichert
        tmp = (String)in.readLine(); //lese nächste Zeile, die wird nach tmp geschrieben

Besser:

while((tmp = in.readLine()) != null)
...
tmp.trim(); //bringt nix. Strings sind unveränderlich.  Ersetze durch
tmp = tmp.trim();

Was grundsätzliches noch: warum testest du mit einer Datei mit MEHREREN TAUSEND Einträgen? Wenn da was fehlt/schief läuft, wirst du das nur schwerlich rausfinden... Nimm ne Datei mit 1 Eintrag, schau ob es klappt. Dann nimm 2 Einträge. Dann meinetwegen 10 und dann 100. Schau auch obs mit ungeraden Anzahlen klappt (101 Einträge) usw.


Ach ja: für sowas würde ich eine Map (HashMap oder TreeMap falls sortiert werden soll) nehmen. Als key den Anfangsbuchstaben (Character evtl. geht auch char, weiß ich grad nicht), als Value eine List der Wörter.

Beispiel (evtl. Fehler mit drinne ;-) )

HashMap <Character, ArrayList<String>>map = new HashMap<Character, ArrayList<String>>();
while(...) {
if(map.get(tmp.CharAt(0)) == null) {
  map.put(tmp.CharAt(0), new ArrayList<String>());
}
map.get(tmp.CharAt(0)).add(tmp);
}

 

[dh81]

moin, danke erstmal
wie vorher beschrieben fehlen 2/3 der einträge. dank deines ersten korrekturvorschlags bin ich jetzt von 233 auf 504 (von 733).

komisch:
beim .toUpperCase() hab ichs richtig gemacht und beim .trim() den anfänger raushängen lassen. vielleicht sollte ich nachts besser schlafen und nich am rechner sitzen...

zum umfang:
zum testen nehme ich nur jene mit a. dann aber auch alle, damit ich sehen kann ob auch wirklich nur die der bedingung length in [5;10] genügenden in die liste geraten sind und ob auch alle in großbuchstaben sind.

das mit der map muss ich ma genauer angucken. all die listen sind für mich mühsam erlesenes neuland. hatten sowas im studium nicht.


nachtrag:
übernächtigter trottel am werk. hab noch ma richtig nachgezählt (hatte die bedingung <=10 vergessen). das ergebnis ist nun doch wunschgemäß und es fehlte so ziemlich die hälfte.

problem gelöst, thema abgehakt...