Variablen in Klasse verpacken?

[michi2]

Hallo,
Ich hatte bisher in der Hauptklasse viele Variablen auf die ich von überall direckt zugegriffen habe:

   // globale Variablen innerhalb der Klasse
   public File einausgabedatei;           		// zu ladende/speichernde Datei
   public String dateiname_Name = "";          		// reiner Name
   public String dateiname_Path = "";           		// reiner Pfad
   boolean dateiname_gueltig = false;
   boolean gespeichert = false;

Jetzt dachte ich mir ich benütze stadt dessen eine Instanz folgender Klasse:

package pk;

public class FileInfo
   {
   private File einausgabedatei;           		// zu ladende/speichernde Datei
   private String dateiname_Name = "";          		// reiner Name
   private String dateiname_Path = "";           		// reiner Pfad
   private boolean dateiname_gueltig = false;
   private boolean gespeichert = false;
   
   
   public FileInfo(File datei)
      {
      einausgabedatei = datei;
      dateiname_Name = datei.getName();
      dateiname_Path = datei.getPath().substring(0,
      datei.getPath().length() - dateiname_Name.length());
      dateiname_gueltig = true;
      }
   
   public File getFile()
      {
      return einausgabedatei;
      }  
   public void setFile(File datei)
      {
      einausgabedatei = datei;
      }
   
   public String getName()
      {
      return dateiname_Name;
      }
   public void setName(String name)
      {
      dateiname_Name = name
      }
   public void generateName()
      {
      dateiname_Name = einausgabedatei.getName();
      }
   
   public String getPath()
      {
      return dateiname_Path;
      }
   public void setPath(Sting path)
      {
      dateiname_Path = path;
      }
   public void generatePath()
      {
      dateiname_Path = einausgabedatei.getPath().substring(0,
      einausgabedatei.getPath().length() - dateiname_Name.length());
      }
   
   public boolean isValidated()
      {
      return dateiname_gueltig;
      }  
   public void validate(boolean gueltig)
      {
      dateiname_gueltig = gueltig;
      }
   
   public boolean isSaved()
      {
      return gespeichert;
      }
   public void setSaved(boolean saved)
      {
      gespeichert
      }
   
   }

Und greife darauf dan auf deren Variablen, so wie auf die Instanz selbst, nur mit den get-/set-ern zu!

Ist das die richtige Angehensweise? Was sollte ich beachten/besser machen?

Vielen Dank,
Michi

 

[Gast]

Attribute public zu machen solltest du dir abgewöhnen. Lediglich bei Konstanten kann es u.U. ok sein. Ansonsten gilt prinzipiell: Alles private machen und den Zugriff über getter/setter regeln.

Merk dir: public == nicht gut *g*

 

[Gast]

noch was: Zur Benennung von Attributen sollten keine Unterstriche verwendet werden. Das ist auch nur bei Konstanten ok.

 

[michi2]

zu 1: deshalb mach ich diese Änderung ja!

 

[Beni]

Die Idee ist :toll:

Für mich stellen sich folgende Fragen:

• was sollen diese public generateX-Methoden? Wenn intern etwas zu berechnen ist, soll das FileInfo das selbstständig machen. Ansonsten vergisst du einmal den Aufruf, und schon suchst du Stunden nach dem Fehler.
• Sind diese "dateiname_X"-Variablen notwendig? Du kannst diese Werte jederzeit aus dem File neu berechnen (und es ist nicht zeitaufwendig)
• Wieso hat es in den generateX-Methoden und dem Konstruktor denselben Code? Das ist einfach nur schreibaufwendig und fehleranfällig.
• Bei den set-Methoden (und dem Konstruktor) wären kleine Prüfungen, ob das Argument korrekt ist (z.B. nicht null), angebracht. Das kann ein "if" sein, welches dann eine "IllegalArgumentException" mit einer aussagekräftigen Fehlermeldung versendet.

 

[michi2]

Die Idee ist :toll:

*Luftsprung*

[*]Sind diese "dateiname_X"-Variablen notwendig? Du kannst diese Werte jederzeit aus dem File neu berechnen (und es ist nicht zeitaufwendig)
[*]Wieso hat es in den generateX-Methoden und dem Konstruktor denselben Code? Das ist einfach nur schreibaufwendig und fehleranfällig.
[*]Bei den set-Methoden (und dem Konstruktor) wären kleine Prüfungen, ob das Argument korrekt ist (z.B. nicht null), angebracht. Das kann ein "if" sein, welches dann eine "IllegalArgumentException" mit einer aussagekräftigen Fehlermeldung versendet.[/list]

Ok, werd ich machen - Danke für den Tipp.
Weis jedoch nicht wie ich Exceptions werfe... :###

Für mich stellen sich folgende Fragen:

• was sollen diese public generateX-Methoden? Wenn intern etwas zu berechnen ist, soll das FileInfo das selbstständig machen. Ansonsten vergisst du einmal den Aufruf, und schon suchst du Stunden nach dem Fehler.

• 
  Nun ich dacht mir das so: Normalerweise hat die Datei immer seinen Dateinamen als Namen, allerdings soll es möglich sein der Datei auch einen anderen Namen zu geben (z.B: mian-Klasse WollMichSau.java - man kann Namen auf 'Main-Klasse' setzen) und um wieder den eigentlichen Dateinamen zu setzen niemt man generateName().
  Vieleicht ist es jedoch klüger 2 Variablen/Methodenpaare zu machen: 1 für den Dateinamen und eine für den Namen der Datei. (vileicht noch einen für ne Beschreibung)
  
  Vielen Dank,
  Michi
  
  PS1: ja ich weis das klingt verwierend
  PS2: Das einfügen von Schilden geht seit einiger Zeit nicht mehr!

 

[michi2]

OK, hab das jezt geändert:

package pk;

import java.io.*;

public class FileInfo
   {
   private File einausgabedatei;           	//die Datei
   private String name = "";		//ein freiwählbarer Name für die Datei
   private String description = "";	//eine freiwählbare Beschreibung der Datei
   private boolean dateiname_gueltig = false;	//ist diese FileInfo Korekt?
   private boolean gespeichert = false;	//ist die Datei schon gespeichert?
   
   public FileInfo(File datei)
      {
      if(datei == null)
         throw new IllegalArgumentException ("Arrgument 'null' is not allowed in FileInfo(File)");
      einausgabedatei = datei;
      dateiname_gueltig = true;
      generateDescription();
      generateName();
      }
   
   public File getFile()
      {
      return einausgabedatei;
      }  
   public void setFile(File datei)
      {
      if(datei == null)
         throw new IllegalArgumentException ("Arrgument 'null' is not allowed in FileInfo(File)");
      einausgabedatei = datei;
      generateDescription();
      generateName();
      }
   
   //Ändern des Pfades, falls die Datei in einen anderen Ordner verschoben wurde
   public String getPath()
      {
      return generatePath();
      }
   public void setPath(String path)
      {
      if(path == null)
         throw new IllegalArgumentException ("Arrgument 'null' is not allowed in FileInfo(File)");
      einausgabedatei = new File( path+getFilename() );
      }
   private String generatePath()
      {
      return einausgabedatei.getPath().substring(0,
      einausgabedatei.getPath().length() - getFilename().length());
      }
   
   //Ändern des Dateinamens, falls die Datei umbenennt wurde
   public String getFilename()
      {
      return generateFilename();
      }
   public void setFilenamen(String newName)
      {
      if(newName == null)
         throw new IllegalArgumentException ("Arrgument 'null' is not allowed in FileInfo(File)");
      einausgabedatei = new File( getPath()+newName );
      }
   private String generateFilename()
      {
      return einausgabedatei.getName();
      }
   
   //Gibt an ob die Informationen in dieser FileInfo korrekt sind
   public boolean isValidated()
      {
      return dateiname_gueltig;
      }  
   public void validate(boolean gueltig)
      {
      dateiname_gueltig = gueltig;
      }
   
   //Gibt an ob die Datei, seit dem letzten speichern, geändert wurde
   public boolean isSaved()
      {
      return gespeichert;
      }
   public void setSaved(boolean saved)
      {
      gespeichert = saved;
      }
   
   //set-, get- und generate- Methoden für den Namen
   public String getName()
      {
      return name;
      }
   public void setName(String newName)
      {
      if(newName == null)
         throw new IllegalArgumentException ("Arrgument 'null' is not allowed in FileInfo(File)");
      name = newName;
      }
   public void generateName()
      {
      name = generateFilename();
      }
   
   //set-, get- und generate- Methoden für die Beschreibung
   public String getDescription()
      {
      return description;
      }
   public void setDescription(String desc)
      {
      if(desc == null)
         throw new IllegalArgumentException ("Arrgument 'null' is not allowed in FileInfo(File)");
      description = desc;
      }
   public void generateDescription()
      {
      description = generateFilename();
      }
   
   } //Ende Klasse "FileInfo"

Was ich mich jetzt noch frage ob dateiname_gueltig überhaupt sinfoll ist, oder ob ich einfach die Instanz on FileInfo auf null setze wenn die Infos falsch sind.

[Edit: Fehler im Code -> korrigiert]

 

[Beni]

Nun ich dacht mir das so: Normalerweise hat die Datei immer seinen Dateinamen als Namen, allerdings soll es möglich sein der Datei auch einen anderen Namen zu geben (z.B: mian-Klasse WollMichSau.java - man kann Namen auf 'Main-Klasse' setzen) und um wieder den eigentlichen Dateinamen zu setzen niemt man generateName().
Vieleicht ist es jedoch klüger 2 Variablen/Methodenpaare zu machen: 1 für den Dateinamen und eine für den Namen der Datei. (vileicht noch einen für ne Beschreibung)

Achso, in diese Richtung habe ich nicht gedacht. Ich kann dir nicht sagen, welche Lösung die bessere ist. Das "generate" hat mich halt auf die falsche Fährte geführt.

Was ich mich jetzt noch frage ob dateiname_gueltig überhaupt sinfoll ist, oder ob ich einfach die Instanz on FileInfo auf null setze wenn die Infos falsch sind.

Ich würde das nicht auf null setzen. Null führt sehr schnell zu NullPointerExceptions, und du hast schon einige Methoden in FileInfo die davon ausgehen, dass nichts null ist.

 

[michi2]

Was ich mich jetzt noch frage ob dateiname_gueltig überhaupt sinfoll ist, oder ob ich einfach die Instanz on FileInfo auf null setze wenn die Infos falsch sind.

Ich würde das nicht auf null setzen. Null führt sehr schnell zu NullPointerExceptions, und du hast schon einige Methoden in FileInfo die davon ausgehen, dass nichts null ist.

Ich glaub du hast das falsch gelessen: ich will nicht dateiname_gueltig null-Setzen sondern die Instanz von FileInfo:

FileInfo datei;
datei = new FileInfo(File xx); //Datei ab hier gültig
//...
datei = null;  //Datei ab hier ungültig
//... - andere Methode:
if(datei != null)
datei.getName();

Und lieber eine NullPointerExceptions als ein Logik-Fehler, oder? :wink:

 

[Beni]

Achso, ja ich hab dich falsch verstanden. Ja, sowas funktioniert gut (und dann ist ein FileInfo-Objekt immer valid, das vereinfacht die Logik).

 

[michi2]

OK, werd das dan so machen!
Danke für den Test und die Tipps!