Class als Datenspeicher

[Aerics]

Abend zusammen.

Und zwar möchte ich gerne die Struktur einer Tabelle in einem Objekt speichern. Als Attribute möchte ich gerne Tabellenname, Spaltenname mit Datentyp und Key hinterlegen.
Dafür habe ich eine Klasse erstellt. Um die 3 Attribute für eine Spalte hinterlegen zu können habe ich eine weitere Klasse erstellt.
Damit die Klasse getTableFields() MyFields übergeben kann, muss diese Klasse public sein. Dies empfinde ich recht unschön, da sie so von Außen erreichbar ist.
Desweiteren kann ich wiederum nur auf den Vector.Fieldname zugreifen, wenn ich wiedere get Methode schreibe (siehe Auskommertieren Bereich), doch dann kann ich mir die getTableFields() Methode auch irgend wie sparen...

Da ich wenig Erfahrung im Bereich Klassen habe würde mich interessieren, wie ich korrekt solch eine Klasse aufbaue, dass ich am Ende ein Object Tables habe wo ich einen Namen setzen kann sowie x Spalte mit je 3 Atributen und auf diese wieder zugreifen kann.

Schon einmal vielen Dank.

Meine Klasse:

import java.util.Vector;

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */

/**
 *
 * @author sl
 */
//------------------------------------------------------------------------------

public class Tables {
    private String sName;
    private Vector<MyFields> vFields = new Vector<MyFields>();
    
    //------------------------------------------------------------------------------

    public class MyFields {
        private String sFieldname;
        private String sDatatype;
        private byte bKey;

        /*public byte getBKey() {
            return bKey;
        }

        public String getSDatatype() {
            return sDatatype;
        }

        public String getSFieldname() {
            return sFieldname;
        }*/

        public MyFields(String Fieldname, String Datatype, byte Key){
            this.sFieldname = Fieldname;
            this.sDatatype = Datatype;
            this.bKey = Key;
        }
    }

//------------------------------------------------------------------------------
    
    //--------------------------------------------------------------------------
    
    public void setTableName(String Name){
        sName = Name;
    }
    
    //--------------------------------------------------------------------------
    
    public String getTableName(){
        return sName;
    }
    
    //--------------------------------------------------------------------------
    
    public void setTableFild(String Fieldname, String Datatype, byte Key){
        vFields.add(new MyFields(Fieldname,Datatype,Key));
    }
    
    //--------------------------------------------------------------------------
    
    public Vector<MyFields> getTableFields(){
        return vFields;
    }
    
    //--------------------------------------------------------------------------
}

Und hier mein Testaufruf:

        Tables test = new Tables();
        
        test.setTableName("Nudel");
        test.setTableFild("a", "b", (byte)1);
        
        System.out.println(test.getTableFields().get(0).getSFieldname());

 

[Aerics]

Ist meine Fragestellung unklar, oder zu umfangreich?
Würde mich sehr über Hilfe freuen.

 

[Landei]

- Schreib bitte, bitte Variablennamen klein
- Wenn das S und B für "String" und "byte" stehen sollte: Lass es, das ist völlig unnötig
- Wenn du mit "Datentyp" den Typ einer Java-Klasse meinst, nimm Class statt String, um selbigen zu speichern
- Vector ist out (ArrayList ist besser)
- Die Klasse MyFields sollte statisch sein (braucht ja keine Instanz von Tables)
- Vielleicht solltest du überlegen, ob du statt dem Vector eine Map (z.B. HashMap) verwendest, bei der du die MyFields über den sFieldName ansprichst. Dann kannst du MyFields auch private machen

...
Map<String, MyFields> map = new HashMap<String, MyFields>();
...
public void setTableFild(String fieldname, String datatype, byte key){
        map.put(fieldname, new MyFields(fieldname,datatype,key));
} 
...
public String getDataType(String fieldname) {
   MyFields myFields = map.get(fieldname);
   return myFields == null ? null : myFields.getSDataType();
}
...
public byte getKey(String fieldname) {
   MyFields myFields = map.get(fieldname);
   return myFields == null ? -1 : myFields.getBKey();
}

 

[Aerics]

- Vector ist out (ArrayList ist besser)

Was bedeutet out? Wo liegt der Vorteil bei einer ArrayList?

Code...

Danke, werde ich nachher in der Firma mal an schauen, testen und mich noch einmal hier melden.

 

[Landei]

Wo liegt der Vorteil bei einer ArrayList?

Das ursprüngliche "Angebot" an Collections in Java bestand aus Vector und Hashtable.
Dann hat es Sun ordentlich gemacht: Eine Hierarchie mit Interfaces (List, Map...), Iteratoren statt den älteren Enumerationen, schnellere Implementierungen (durch Verzicht auf "Multithreadfähigkeit") und der Hilfsklasse Collections. Vector und Hashtable wurden zwar nachträglich umgebogen, um in das neue Collection-Framework zu passen, aber bei den neuen Klassen (ArrayList, HashMap...) hat man aus alten Fehlern gelernt. Jetzt sind noch die Implementierungen in der concurrent package hinzugekommen, womit der letzte Grund, die alten zu nehmen (synchronisierte Methoden -> multithreaded Zugriff möglich) weggefallen ist.

 

[Aerics]

Also dein Beispiel funktioniert soweit.
Was mir jedoch nun auffällt, dass ich auf die Spalten nur zugreifen kann, wenn ich den Spaltennamen weis.
Wenn ich nun 10 Tabellenobjekte mit jeweils 5 unterschiedlichen Spalten erstelle, weis ich ja nicht welche Spaltennamen in diesem Objekt existiert.
Wie löse ich das nun? Im Normalfall müsste ich ja mit einer For-Schleife z.B. durch mein Table.MyFields Objekt um über den Index alle Spalten mit dessen 3 Atributen zugreifen zu können.

import java.util.HashMap;

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
import java.util.Map;

/**
 *
 * @author sl
 */
//------------------------------------------------------------------------------

public class Tables {
    private String sName;
    private Map<String, MyFields> map = new HashMap<String, MyFields>();
    
    //------------------------------------------------------------------------------

    private static class MyFields {
        private String sFieldname;
        private String sDatatype;
        private byte bKey;

        private byte getBKey() {
            return bKey;
        }
        
        private String getSDatatype() {
            return sDatatype;
        }

        private MyFields(String Fieldname, String Datatype, byte Key){
            this.sFieldname = Fieldname;
            this.sDatatype = Datatype;
            this.bKey = Key;
        }
    }

//------------------------------------------------------------------------------
    
    //--------------------------------------------------------------------------
    
    public void setTableName(String Name){
        sName = Name;
    }
    
    //--------------------------------------------------------------------------
    
    public String getTableName(){
        return sName;
    }
    
    //--------------------------------------------------------------------------
    
    public void setTableFild(String fieldname, String datatype, byte key){
        map.put(fieldname, new MyFields(fieldname,datatype,key));
    }
    
    //--------------------------------------------------------------------------
    
    public String getDataType(String fieldname) {
       MyFields myFields = map.get(fieldname);
       return myFields == null ? null : myFields.getSDatatype();
    }
    
    //--------------------------------------------------------------------------    

    public byte getKey(String fieldname) {
       MyFields myFields = map.get(fieldname);
       return myFields == null ? -1 : myFields.getBKey();
    }     
    
    //--------------------------------------------------------------------------
}

        Tables test = new Tables();
        
        test.setTableName("Nudel");
        test.setTableFild("a", "b", (byte)1);
        
        System.out.println(test.getDataType("a"));

 

[Landei]

Du kannst dir alle Spaltennamen mit map.keySet() zurückliefern lassen, bzw. als Array mit map.keySet().toArray(new String[]{})

Du kannst auch die Anzahl der Spalten mit map.getSize() ermitteln. Ich weiß jetzt nicht, ob du auch über einen Index zugreifen willst, dazu könntest du z.B. zusätzlich noch eine List<MyFields> list = new ArrayList<MyFields>() nehmen.

Nebenbei: Was soll eigentlich der bKey sein?