Reader - brauche Hilfe

[fh13]

Hallo,

ich hoffe das ich mit eurer Hilfe nun weiter kommen ... mein Problem is folgendes :

Ich habe mehrere BufferedReader (weil mehrer Dateien)...

nun will ich von jedem Reader über die Methode *.readLine(), den Text Zeile für Zeile aus der jeweiligen .txt Datei auslesen
nur is das Problem das jedes mal wenn ich die Methode "load" (siehe Quelltext) aufrufe wird der BufferedReader neu geladen und beim Befehl *.readLine() fängt das Prog wieder von vorn an die .txt Datei zu lesen...

WIE KANN ICH DAS PROBLEM BEHEBEN ??

//Variablen blabla

public void load (String loadstuff) 
 { 
   BufferedReader in1 = new BufferedReader(new FileReader("C:\\Bcom\\1.txt")  );
   BufferedReader in2 = new BufferedReader(new FileReader("C:\\Bcom\\2.txt")  );
   BufferedReader in3 = new BufferedReader(new FileReader("C:\\Bcom\\3.txt")   );

   if(loadstuff== "1"  ){ cur1  = in1.readLine();  };
   if(loadstuff== "2"  ){ cur2  = in2.readLine();  };
   if(loadstuff== "3"  ){ cur3  = in3.readLine();  };

 }
void doit()
{
  while(cur1 != null) 
  {
   load("1");
   load("2");
   load("3");
   System.out.println(cur1);
   System.out.println(cur2);
   System.out.println(cur3);
  }
}

// restliches BLABLA

Danke schon mal,
Flo

PS: ich natürlich nicht die String dann einfach ausgeben wie in dem Beispiel hier das dient nu zur Veranschaulichung...

PS

 

[Volvagia]

1) Streams IIIIIIIIIIIMMER schließen. *dir einhämmer*
2) Warum erzeugst du dann jedes mal einen neuen Reader, wenn du in Wahrheit vom alten weiterlesen willst?
3) http://www.java-forum.org/top-fragen/1350-vergleichen-strings.html

 

[fh13]

ja ich das mit dem Schließen normal müsste da auch noch try und catch hin... aber darum gehts ja nich...

und ich mach deshalb immer nen neuen Reader auf weil ich nicht weis wie man den nur einmal öffnet dann immer wieder so aufruft mit readLine so wie ich das jetzt habe ....

 

[Volvagia]

Du intialisierst einfach 1 mal mit new, und speicherst sie in eine Klassenvariable zwischen.

 

[fh13]

ok ich hätte wahrscheinlich dazu sagen sollen das ich ein ganz schöner Anfänger bin ....

kann mir das was Volvagia meint mal kurz anhand eines Codes erklären ... BITTE

 

[Noctarius]

class Foo {
  private BufferedReader in1 = new BufferedReader(new FileReader("C:\\Bcom\\1.txt"));

  public void load(String loadstuff) {
    ...
  }
}

Trotzdem solltest du daran denken nachdem du fertig bist mit laden die Reader per close() zu schließen.

 

[fh13]

Wow ich bin beeindruckt wie schnell hier einem geholfen wird ....

@ Noctarius: die Buffered Reader einfach mit in die Class zu schreiben funktioniert leider nicht weil dann nämlich beim Compilieren

immer kommt das das es die "Variable r1,r2 und r3 gar nicht gibt" also (Cannot find Symbol...) oder so

 

[Noctarius]

Zeig noch mal deinen Code

 

[fh13]

import java.io.*;

class Booo
{
 
 String datei; 
 int loadstuff; 

 String cur1";  
 String cur2;
 String cur3;
 

 
 public void load (int loadstuff) throws Exception
 { 
 
   BufferedReader r1 = new BufferedReader(new FileReader("C:\\Bcom\\1.txt")  );
   BufferedReader r2 = new BufferedReader(new FileReader("C:\\Bcom\\2.txt")  );
   BufferedReader r3 = new BufferedReader(new FileReader("C:\\Bcom\\3.txt")   );
  
   if(loadstuff==1 ){ cur1  = emailin.readLine();  };
   if(loadstuff==2 ){ cur2  = vnamein.readLine();  };
   if(loadstuff==3 ){ cur3   = namein.readLine();  };

 }
 
 void doit()throws Exception
 {

  
  while(cur1 != null) 
  {
   load(1);
   load(2);
   load(3);

   System.out.println(cur1)  ;
   System.out.println(cur2);
   System.out.println(cur3);
  }
  System.out.println("End");
 }
}

public class Mainprog
{
 public static void main(String[] args)
 {
  Booo doitnow = new Booo();
}}

UND ich schätze mal er meinte es so:

import java.io.*;

class Booo
{
 
 String datei; 
 int loadstuff; 

 String cur1";  
 String cur2;
 String cur3;
 
   BufferedReader r1 = new BufferedReader(new FileReader("C:\\Bcom\\1.txt")  );
   BufferedReader r2 = new BufferedReader(new FileReader("C:\\Bcom\\2.txt")  );
   BufferedReader r3 = new BufferedReader(new FileReader("C:\\Bcom\\3.txt")   );
 


 public void load (int loadstuff) throws Exception
 { 
 
   if(loadstuff==1 ){ cur1  = emailin.readLine();  };
   if(loadstuff==2 ){ cur2  = vnamein.readLine();  };
   if(loadstuff==3 ){ cur3   = namein.readLine();  };

 }
 
 void doit()throws Exception
 {

  
  while(cur1 != null) 
  {
   load(1);
   load(2);
   load(3);

   System.out.println(cur1)  ;
   System.out.println(cur2);
   System.out.println(cur3);
  }
  System.out.println("End");
 }
}

public class Mainprog
{
 public static void main(String[] args)
 {
  Booo doitnow = new Booo();
}}

aber da kommt der genannte Feheler

 

[ARadauer]

ahh ... hier stand blödsinn

 

[ARadauer]

*doppelpost*.. omg es ist noch zu früh für mich ;-)

 

[fh13]

aha und was soll das jetzt.?!

 

[ARadauer]

ok mal was konstruktives...

String cur1";  
String cur2;
String cur3;

na wo ist der fehler?

Generell ist der Code nicht so besonders schön, was willst du eigentlich genau machen?

 public void load (int loadstuff) throws Exception
 {     
   
   if(loadstuff==1 ){ cur1  = emailin.readLine();  };
   if(loadstuff==2 ){ cur2  = vnamein.readLine();  };
   if(loadstuff==3 ){ cur3   = vnamein.readLine();  };
 
 }

so würde ich keine funktionalität kapseln...

aber naja, du deklarierst oben r1 und nimmst unten emailin ??

so würde es laufen

class Booo
{
 
 String datei; 
 int loadstuff; 
 
 String cur1;  
 String cur2;
 String cur3;
 
 BufferedReader emailin;
 BufferedReader vnamein;
 BufferedReader namein;
 

 
 public void load (int loadstuff) throws Exception
 {     
   
   if(loadstuff==1 ){ cur1  = emailin.readLine();  };
   if(loadstuff==2 ){ cur2  = vnamein.readLine();  };
   if(loadstuff==3 ){ cur3   = vnamein.readLine();  };
 
 }
 
 void doit()throws Exception
 {
    try {
       emailin=  new BufferedReader(new FileReader("C:\\Bcom\\1.txt")  );
       vnamein = new BufferedReader(new FileReader("C:\\Bcom\\2.txt")  );
       namein = new BufferedReader(new FileReader("C:\\Bcom\\3.txt")   );
     
     while(cur1 != null) 
     {
      load(1);
      load(2);
      load(3);
    
      System.out.println(cur1)  ;
      System.out.println(cur2);
      System.out.println(cur3);
     }
     System.out.println("End");
    
   } finally{
      if(emailin != null)
         emailin.close();
      if(vnamein != null)
         vnamein.close();
      if(namein != null)
         namein.close();
   }
   
}

aber wie schon gesagt, so würde ich methoden nicht aufbauen. Hast du wirklich 3 Listen wo die Personen verteilt sind? Nicht sehr sauber oder?

Ich würde eher einen Methode machen die ein gesamtes PersonenObjekt lädt

 

[fh13]

ok das ist ne Möglichkeit
ja der code is nicht schön keine frage aber ... hauptsache es geht aber kann ich die Buffreader nich einfach in ein object machen oder so ... weil dann wäre die doit()-Methode übersichtlicher

ich weis halt nicht ...

 

[ARadauer]

Im Grunde fängt es ganz vorne an... da kann nix gutes daraus werden
doit ist als name falsch, was soll maches machen? es machen... toll ;-)
Booo was ist booo? Wenn man Dinge nach dem benennt was sie sind, ist man schon einen schritt weiter das zu programmieren was sie machen sollen ...
1.txt, 2.txt, 3.txt Da sind ja Personen/Benutzer oder was drinnen, oder? Steht wirklich in 1 die ganzen Vornamen und in 2 die ganzen Nachnamen? das ist ja selsam, warum nicht in einer Datei und dann mit ; getrennt?

 

[ARadauer]

was willst du den machen? eine liste von benutzern lesen und mit denen das was machen? zb ausgeben?
wir brauchen einen benutzer:

public class User {
   
   private String name;
   private String surname;
   private String password;
   
   public User(String name, String surname, String password) {
      this.name = name;
      this.surname = surname;
      this.password = password;
   }

   public String getName() {
      return name;
   }
   
   public void setName(String name) {
      this.name = name;
   }
   
   public String getSurname() {
      return surname;
   }
   
   public void setSurname(String surname) {
      this.surname = surname;
   }
   
   public String getPassword() {
      return password;
   }
   
   public void setPassword(String password) {
      this.password = password;
   }
   
}

Was macht das hauptprogramm? User Lesen und ausgeben

   public static void main(String[] args) {
      UserReader userReader = new UserReader(new File("C:\\Bcom\\1.txt"));
      List<User> userList;
      try {
         //lesen
         userList = userReader.readUsers();
         
         //ausgeben
         for(User user: userList){
            System.out.println("User: "+user.getName()+" "+user.getSurname());
         }
      } catch (Exception e) {
         System.out.println("Fehler beim lesen..");
         e.printStackTrace();
      }      
   }

Gut und wie lesen wir jetzt?

import java.io.*;
import java.util.ArrayList;
import java.util.List;

class UserReader {
   
   private File fileToReadFrom;

   public UserReader(File fileToReadFrom) {
      this.fileToReadFrom = fileToReadFrom;
   }

   public List<User> readUsers() throws Exception {
      List<User> result = new ArrayList<User>();
      BufferedReader reader = null;
      try {
         reader = new BufferedReader(new FileReader(fileToReadFrom));
         String line = null;
         while ((line = reader.readLine()) != null) {
            result.add(createUser(line));
         }
      } finally {
         if (reader != null)
            reader.close();
      }
      return result;

   }

   public User createUser(String line) {
      String[] parts = line.split(";");
      return new User(parts[0], parts[1], parts[2]);
   }
}

So würde ich sowas machen...
das ganze setzt voraus das die liste als csv mit ; getrennt vorliegt...

 

[fh13]

So hab ichs jetzt...

import java.io.*;

class Botdo
{
 Botdo()  // Hauptmethode weil public static void main nicht die Methoden aufrufn kann
 {
  final();
 }  

 String datei; 
 String loadstuff; 

 String cureml=".";  	BufferedReader emlin;
 String curvnm;		BufferedReader vnmin;
 String curnme;		BufferedReader nmein;
 String curplz;		BufferedReader plzin;
 String curort;		BufferedReader ortin;
 String curstr;		BufferedReader strin;
 String curnmr;		BufferedReader nmrin;
 String curtnr;		BufferedReader tnrin;


 
 public void init() 
 {
  try
  {
   emlin = new BufferedReader(new FileReader("C:\\Bcom\\email.txt")  );
   vnmin = new BufferedReader(new FileReader("C:\\Bcom\\vname.txt")  );
   nmein = new BufferedReader(new FileReader("C:\\Bcom\\name.txt")   );
   plzin = new BufferedReader(new FileReader("C:\\Bcom\\plz.txt")    );
   ortin = new BufferedReader(new FileReader("C:\\Bcom\\ort.txt")    );
   strin = new BufferedReader(new FileReader("C:\\Bcom\\straße.txt") );  
   nmrin = new BufferedReader(new FileReader("C:\\Bcom\\nummer.txt") );
   tnrin = new BufferedReader(new FileReader("C:\\Bcom\\tnummer.txt"));
  }
  catch(IOException iox){System.out.println("Data File Error");};
 }

 public void closeStreams()
 {
  try
  {
   emlin.close(); 
   vnmin.close();
   nmein.close();
   plzin.close();
   ortin.close();
   strin.close();
   nmrin.close();
   tnrin.close();
  }
  catch(IOException iox){System.out.println("Error in closing Data Reader");};
 }

 public void load (String loadstuff)
 {
  try 
  {
   if(loadstuff.equals("em")){ cureml = emlin.readLine(); };
   if(loadstuff.equals("vn")){ curvnm = vnmin.readLine(); };
   if(loadstuff.equals("nm")){ curnme = nmein.readLine(); };
   if(loadstuff.equals("pl")){ curplz = plzin.readLine(); };
   if(loadstuff.equals("or")){ curort = ortin.readLine(); };
   if(loadstuff.equals("st")){ curstr = strin.readLine(); };
   if(loadstuff.equals("nr")){ curnmr = nmrin.readLine(); };
   if(loadstuff.equals("tn")){ curtnr = tnrin.readLine(); };
  }
  catch(IOException iox){System.out.println("Error in reading Data File");};
 }
 
 void final()
 {
  init();
  
  while(cureml != null) 
  {
   load("em");
   load("vn");
   load("nm");
   load("pl");
   load("or");
   load("st");
   load("nr");
   load("tn");
   System.out.println(cureml);
   System.out.println(curvnm);
   System.out.println(curnme);
   System.out.println(curplz);
   System.out.println(curort);
   System.out.println(curstr);
   System.out.println(curnmr);
   System.out.println(curtnr);
   System.out.println("");
  }
  System.out.println("End");
  closeStreams();
 }
}

public class Bot
{
 public static void main(String[] args)
 {
  Botdo doitnow = new Botdo();
}}

 

[Haave]

Merke: In Java darf kein Bezeichner einem Schlüsselwort gleichen.
Hast du das Gepostete schon mal ausprobiert? Das wird sich höchstwahrscheinlich nicht einmal kompilieren lassen.

 

[Noctarius]

Abgesehen davon, dass der Aufbau immer noch sehr unschön ist

 

[fh13]

@ Haave : doch geht einwandfrei ... aber werds trotzdem ändern

@Noctarius : nich schön aber für mehr reichen meine Kenntnisse nicht

@all : für meine Zwecke sollte es reichen