Klassen Ein Problem mit den Instanzvariablen

[Jesfreric]

Liebes Forum

Ich hätte einmal eine grundsätzliche Frage zum Thema Instanzvariablen.

Ich habe eine eigene Klasse erstellt (Name: FREAD( String strA ) mit einer eigenen Methode readFile().
In meiner Hauptklasse (also dort wo meine main() Methode ist) erstelle ich zwei neue Objekte über new FREAD und übergebe zwei verschiedene Strings an die Objekte:

// Code 1

FREAD no1 = new FREAD(String a);
FREAD no2 = new FREAD(String b);

im Anschluss rufe ich an beiden Objekten meine Methode auf:

//Code 2

String E1 = no1.readFile();
String E2 = no2.readFile();

Obwohl unterschiedliche Ergebnisse zu erwarten sind, liefern E1 und E2 die gleichen Ergebnisse.
Der Grund dafür liegt mit Sicherheit darin, dass die Instanzvariablen die von no1 in Code 1 beschrieben wurden sofort durch no2 überschrieben wurden.
Ist das normal? Muss ich das dann wirklich immer so schreiben?

FREAD no1 = new FREAD(String a);
String E1 = no1.readFile();

FREAD no2 = new FREAD(String b);
String E2 = no2.readFile();

Oder reicht es etwas an der Bezeichnung meiner Variablen zu ändern.
Momentan sind die Instanzvariablen mit "private static..." bezeichnet. Das static (so sagt der Compiler) wäre notwendig.


Viele Grüße
Richard

 

[X5-599]

Normal ist das nicht. Interessant wäre zu wissen wie deine FREAD Klasse aussieht. Ich nehme mal an das der übergebene String ein Dateiname/Pfad ist und du momentan zweimal dieselbe Datei ausliest, obwohl du zwei unterschiedliche Objekte (mit zwei unterschiedlichen Strings) von FREAD erstellst.

Ich vermute weiter, dass der übergebene String in der Klasse FREAD als statische Variable deklariert ist. Wenn das der Fall ist müsstest du nur das "static" daraus entfernen.

 

[VfL_Freak]

Moin,

so ganz habe ich deine Frage nicht verstanden!

Aber Folgendes ist IMHO falsch:

Ich habe eine eigene Klasse erstellt (Name: FREAD( String strA ) mit einer eigenen Methode readFile().
In meiner Hauptklasse (also dort wo meine main() Methode ist) erstelle ich zwei neue Objekte über new FREAD und übergebe zwei verschiedene Strings an die Objekte:

FREAD no1 = new FREAD(String a);
FREAD no2 = new FREAD(String b);

Du übergibst hier ja beim Aufruf keinen String, sondern deklarierst einen neuen, somit leeren String!

Korrekt sollte es so lauten:

String a = new String( "irgendwas");
String b = new String( "was anderes");
FREAD no1 = new FREAD( a );
FREAD no2 = new FREAD( b );

Gruß
Klaus

 

[Jesfreric]

Hi

Danke für deine Antwort.
Meine Klasse schaut so aus:

class FREAD{
   private static String fileSource;
   private static String Content = new String("");


   FREAD(String a){
      fileSource = a;
   }


   public static String readFile() throws IOException{
      FileReader fr = new FileReader(fileSource);
      BufferedReader br = new BufferedReader(fr);
      String line = "";

      while(  (line = br.readLine()   ) != null){
         Content = Content + "\n" + line;
      }
      br.close();
      return content;
   }
}

Wenn ich die static aus den Intanzvariablen raus nehme, dann beschwert sich der Compiler.
Ich sollte String a wahrscheinlich lieber an die Methode übergeben (dann sind sie ja automatisch nur temporär...)

Die "Quellstrings" meiner beiden FREAD Objekte sind auch auf jeden Fall verschieden (hab nochmal nach geschaut).=)

Viele Grüße
Richi

 

[Jesfreric]

Ich merke gerade...zumindest content könnte ich auch komplett in die Methode legen...

 

[VfL_Freak]

Moin,

sieht auf den ersten Blick korrekt aus, wobei ich "fileSource" auch initialisieren würde!

class FREAD{
   private static String fileSource;
   private static String Content = new String("");


   FREAD(String a){
      fileSource = a;
   }


   public static String readFile() throws IOException{
      FileReader fr = new FileReader(fileSource);
      BufferedReader br = new BufferedReader(fr);
      String line = "";

      while(  (line = br.readLine()   ) != null){
         Content = Content + "\n" + line;
      }
      br.close();
      return content;
   }
}

Wenn ich die static aus den Intanzvariablen raus nehme, dann beschwert sich der Compiler.
Ich sollte String a wahrscheinlich lieber an die Methode übergeben (dann sind sie ja automatisch nur temporär...)

Die "Quellstrings" meiner beiden FREAD Objekte sind auch auf jeden Fall verschieden (hab nochmal nach geschaut).=)

Das "static" natürlich drin bleiben, dass es ja eben statische Klassenmember sind!
Wo rufst Du denn "readFile" auf?
Du könnstest natürlich den Pfad auch dort direkt übergeben und auf die Klassenvariablen verzichten !

Die "Quellstrings" meiner beiden FREAD Objekte sind auch auf jeden Fall verschieden (hab nochmal nach geschaut).=)

Schon möglich, nur war dein Aufruf falsch (s. o.) !

Gruß
Klaus

 

[Jesfreric]

Erst einmal danke für eure Antworten=)
Wäre es eigentlich möglich den Konstruktor leer zu lassen und einfach nur die Methode zu schreiben?
Weil wenn ich die Quelle für die Datei direkt an die Methode übergebe, wird der Konstruktor selbst irgendwie "überflüssig" für mein "readFile()".

 

[VfL_Freak]

Moin,

Wäre es eigentlich möglich den Konstruktor leer zu lassen und einfach nur die Methode zu schreiben?

Sicher, warum sollte es nicht ???:L

wenn ich die Quelle für die Datei direkt an die Methode übergebe, wird der Konstruktor selbst irgendwie "überflüssig" für mein "readFile()".

nein, durch den Aufruf instantiierst Du ja das zugehörige Objekt!

Gruß
Klaus

 

[Jesfreric]

Das stimmt...
es würde also so etwas hier gehen?

class FREAD{   

   FREAD(){      
   }     

   public static String readFile(String a) throws IOException{      
     //INHALT 
   }
}

 

[VfL_Freak]

Moin,

aber sicher ...
Ist das "Ausprobieren" bei dir kaputt ???:L

Gruß
Klaus

 

[Jesfreric]

Jep...leider irreparabel defekt.
Mein Computer akzeptiert das "Ausprobieren" durch meine ständigen Versuche nicht mehr...

P.S.: Dachte bis vorhin, dass im Konstruktor IMMER etwas stehen muss. Wollte nicht, dass mich dann später das böse Ende ereilt...

Es funktioniert übrigens jetzt tadellos=)

 

[VfL_Freak]

Jep...leider irreparabel defekt.
Mein Computer akzeptiert das "Ausprobieren" durch meine ständigen Versuche nicht mehr...

verständlich :lol:

Dachte bis vorhin, dass im Konstruktor IMMER etwas stehen muss

Nein, der Aufruf eines leeren Konstruktor erzeugt einfach nur das entsprechende Objekt!
Wenn aber bspw. die Klasse von was anderem abgeleitet wäre, dann müsste zumindest der entsprechende Konstruktor der vererbenden Klasse "super( ... ) " aufgerufen werden!
Galileo Computing :: Java ist auch eine Insel – 2.2 Anweisungen formen Programme

Es funktioniert übrigens jetzt tadellos=)

Prima :toll:

Gruß
Klaus

 

[X5-599]

Das stimmt...
es würde also so etwas hier gehen?

class FREAD{   

   FREAD(){      
   }     

   public static String readFile(String a) throws IOException{      
     //INHALT 
   }
}

Es sollte noch erwähnt sein, dass wenn du das so machst, du keine Instanzen von deiner Klasse erzeugen musst. Ein einfaches: FREAD.readFile("Pfad/zum/File.txt"); reicht dann aus. Der Konstruktor ist dann auch überflüssig.

 

[Natac]

class FREAD{
   private static String fileSource;
   private static String Content = new String("");

   FREAD(String a){
      fileSource = a;
   }

   public static String readFile() throws IOException{
    ...
   }
}

Der Compiler meckert weil deine methode auch statisch ist. Wenn du alle drei statics wegnimmst, sollte es so funktionieren, wie du es gedacht hast.

Bleibt natürlich die Frage, was "besser" ist:

String s1 = new FREAD("file.txt").readFile();// deine Version
String s2 = FREAD.readFile("file.txt"); // statische Version ohne Instanz