kompletten Datensatz statt nur ein Feld auslesen lassen,wie?

[spaci76]

Hi@ll,

so wie im beispiel habe ich es geschaft einzelne Datensätze per Klasse zu ziehen und zu verwenden. Könnte mir jemand auf die Sprünge helfen wie ich es schaffe nicht nur ein Feld sondern die komplette Line von id 1 (line 34) ala fahrzeug1.getString("Name") und fahrzeug1.getString("Farbe"), welche dann in Line 35 verfügbar sind.

Ich gehe mal davon aus das es in der LINE 22 liegt.

Ich bedanke mich im voraus für eure Vorschläge.

spaci76

import java.sql.ResultSet;
import java.sql.SQLException;
public class Sql{

        protected String selectstring;
public Sql (){}

public Sql( String e ){
        selectstring = e;
}

public String getSelectString() throws ClassNotFoundException{
        
  try {
            Class.forName( Constants.DBDRIVER ); 
            Connection conn = (Connection) DriverManager.getConnection( Constants.DBURL, Constants.DBUSER, Constants.DBUSERPW );
            Statement st = (Statement) conn.createStatement();
            String s = "SELECT * FROM configuration where configuration_id="+selectstring;
            ResultSet erg = st.executeQuery(s);
            while(erg.next()) 
                {
                selectstring = erg.getString("configuration_key");                
               }
            erg.close();
            st.close();
         }
       catch(SQLException e) {
       }
  return selectstring;
  
       }

static public void main( String args[]) throws ClassNotFoundException{
        Sql fahrzeug1 = new Sql("1"); //id from get.SelectString
        System.out.println( "F1: "+fahrzeug1.getSelectString());
}
}

 

[Prismapanda]

Oje, das ist aber ein übles Design.... :shock:
Also mal das was ich so sehe: Es werden Fahrzeuge in der Tabelle configuration gespeichert? Dann erstelle auch eine Klasse, die diese Tabellen (bzw. einen Datensatz daraus) repräsentiert. Also bspw. eine Klasse Fahrzeug. Dann kannst du ein Objekt dieser Klasse erstellen und auf die gewünschten Methoden zugreifen (z.B. getColor(), getName()).
So separierst du die Datenbank von deinem zu erstellenden Objekt. Egal wie klein dein Projekt sein mag, aber soweit sollte das Design mindestens gehen. Kannst dich auch gerne mal über DAOs informieren.
Wenn dir das alles nicht zusagt kannst du auch deine Methode getSelectString() ein Array oder eine Map zurückgeben lassen in der dann alle Werte indiziert zur Verfügung stehen, aber das wäre wohl die schlechteste und unsauberste Variante...

 

[spaci76]

hi,

vielen dank für die Antwort. Zwecks Design, nun zum Glück bin ich ja hier im Anfängerforum
Ich will das was ich mache Hand und Fuss hat, wenn das mit dem Array so wie es in php der Fall wäre nicht korrekt ist, dann will ich mir selbst verständlich eine korrekte Programmierung angewöhnen.

Ich bin hier weil ich will, nicht weil ich muss!!

nun die Frage, was bei diesem 2. Versuch zu verbessern wäre, bin für alle Tipps, gerne auch kurze Beispiele zur Veranschaulichung, sehr dankbar.

import java.sql.ResultSet;
import java.sql.SQLException;
public class Fahrzeug{

protected String selectid;
private String selectid_value;
    private String select_id;
public Fahrzeug (){}

public Fahrzeug( String e ){
        selectid = e;
        
}

public String getTyp() throws ClassNotFoundException{
        
  try {
            Class.forName( Constants.DBDRIVER ); 
            Connection conn = (Connection) DriverManager.getConnection( Constants.DBURL, Constants.DBUSER, Constants.DBUSERPW );
            Statement st = (Statement) conn.createStatement();
            String s = "SELECT configuration_key FROM configuration where configuration_id="+ selectid;
            ResultSet erg = st.executeQuery(s);
            while(erg.next()) 
                {
                select_id = erg.getString("configuration_key");                
               }
            erg.close();
            st.close();
         }
       catch(SQLException e) {
       }
  return select_id;
  
       }

public String getColor() throws ClassNotFoundException{
        
        
  try {
            Class.forName( Constants.DBDRIVER ); 
            Connection conn = (Connection) DriverManager.getConnection( Constants.DBURL, Constants.DBUSER, Constants.DBUSERPW );
            Statement st = (Statement) conn.createStatement();
            String s = "SELECT configuration_value FROM configuration where configuration_id="+ selectid;
            ResultSet erg = st.executeQuery(s);
            while(erg.next()) 
                {
                selectid_value = erg.getString("configuration_value");                
               }
            erg.close();
            st.close();
         }
       catch(SQLException e) {
       }
  return selectid_value;
  
       }

static public void main( String args[]) throws ClassNotFoundException{
        Fahrzeug fahrzeug1 = new Fahrzeug("1"); //id from get.SelectString
        System.out.println( "F1: "+fahrzeug1.getTyp());
        System.out.println( "F1: "+fahrzeug1.getColor());
}
}

 

[Prismapanda]

Ich denke dein Problem ist, dass du ziemlich viel vermischt und dich nicht ganz an die Objektorientierung anpasst. Die Vorgehensweise wie sie in php meistens benutzt wird (Array) führt vielleicht zum Ziel, verfehlt aber das OO Paradigma.
Grundsätzlich versuche es mal mit folgender Struktur:

Klasse Connection: Kapsel hier den Datenbankzugriff. Du kannst schon an deinem letzten Beispiel sehen, dass die DB Verbindung bei dir erheblichen Code verursacht. Die gibts denke ich mal auch fertig, aber schreib sie dir ruhig mal selber. Methoden wie query(String) wren sinnvoll...

Klasse Fahrzeug: Hier kommen nur die fahrzeugrelevanten Sachen rein. Farbe, Typ als Instanzvariablen etc. aber keine! Datenbankangelegenheiten

Klasse FahrzeugDAO: Ein DataAccessObject. EInfach mal danach googlen oder so. Hier übergibst du die DB Connection und beziehst über die ID eine Instanz der Klasse Fahrzeug.

Und dann der Ablauf:

Connection aufbauen. (z.B. con = connection.connect(host,port,user,pass,db))
Über DAO Fahrzeug instanzieren (z.B. Fahrzeug fahrzeug = dao.getById(id))
Benötigte Werte auslesen (z.B. fahrzeug.getColor())


Klingt vielleicht komplizierter als der prozedurale Ansatz aber wird dir so einige Vorteil schaffen...