Objektorientierter Ansatz fehlt

[rastaman]

HAllo Leute

Also ich habe ein Problem...

Ich code seit einiger zeit an einem projekt... dabei habe ich mir mühe gegeben nach dem objektorientierten ansatz vorzugehen.

Endziel: Verwaltungssystem für eine Videothek

Hauptklasse: Die klasse, in der ich das ganze Gui per Netbeans gebastelt habe

Unterklassen: (unter anderem)
sql (hier ist der ganze Sql teil abgelegt)
kunde (verarbeite ich die result sets von der klasse sql in vectoren etc)

ich habe nun also diese klasse kunde mit der "new instance of Kunde"

public kunde(String n, String v, String st, String or, java.util.Date gd, int ku, int pl, int tel, int pw, String em ) {
        this.name = n;
        this.vorname = v;
        this.strasse = st;
        this.ort =or;
        this.geburtsdatum = gd;
        this.kundennummer = ku;
        this.plz = pl;
        this.telefonnr = tel;
        this.passwort = pw;
        this.email = em;

und dem teil, der die gewünschten Daten in einen Vector füllt,

    public Vector vektorfüllen(){
         Sql verbindung = new Sql();
         verbindung.connect();
         Vector abfragek = new Vector();
         try {
           rSet = verbindung.abfrage("SELECT * FROM kunde ORDER BY Name");
           //rSet = verbindung.abfrage("SELECT Name, Vorname, Vorname, Strasse, Ort, Geburtsdatum, Kundennummer, PLZ, Telefonnummer, Passwort, Email FROM kunde ORDER BY Name");

           while ( rSet.next() )
                abfragek.add( new kunde( rSet.getString("Name"), rSet.getString("Vorname"), rSet.getString("Strasse"), rSet.getString("Ort"), rSet.getDate("Geburtsdatum"), rSet.getInt("Kundennummer"), rSet.getInt("PLZ"), rSet.getInt("Telefonnummer"), rSet.getInt("Passwort"), rSet.getString("Email") ));
            }
         catch ( SQLException e )
            {
            System.out.println( "Fehler bei Tabellenabfrage" + e );
            }
        verbindung.abfrageende();
        verbindung.trennen();
        return abfragek;
    }

und nun versuche ich von meiner mainklasse aus auf die klasse kunde zuzugreiffen

    public static void main(String args[]) {
        //zugriff auf die klasse Kunde um anschliessend die methode vector füllen auszuführen
        kunde k = new kunde();
        k-vectorfüllen();
        
        //Zeichnen des Gui
        new guidatenbank().show();

und den Vector zu füllen.
Dies klappt aber nicht und der Compiler meldet:

guidatenbank.java [1'060:1] kunde(java.lang.String,java.lang.String,java.lang.String,java.lang.String,java.util.Date,int,int,int,int,java.lang.String) in videothek2.kunde cannot be applied to ()
new kunde();

also ich habe keinen plan wie ich weiter machen soll :Bahnhof:

 

[Roar]

der konstruktor von kunde (Klassennamen werden großg eschrieben) erwartet die bestimmten parameter. du hast nur einen konstruktor, also musst du den auch verwenden. du kannst nicht einfach parameter weglassen.

 

[Beni]

Du hast den Konstruktor von "kunde" als

public kunde(String n, String v, String st, String or, java.util.Date gd, int ku, int pl, int tel, int pw, String em ) {

definiert, und willst nun

new kunde();

aufrufen.

Aber diesen Konstruktor, ohne Argumente, hast du nicht definiert :arrow: er existiert nicht. Du musst entweder all die Argumente einfüllen, oder noch einen zusätzlichen Konstruktor definieren:

public class kunde{
  public kunde(){ ... }
  public kunde(String n, String v, String st, String or, java.util.Date gd, int ku, int pl, int tel, int pw, String em ){ ... }
}

P.S. eigentlich schreibt man Klassen gross ("Kunde").

[edit: nur zweiter, dafür ausführlicher]

 

[nollario]

das problem ist, dass die methode verktorfüllen (keine umlaute in Methodennamen und ausserdem: vektorFuellen) falsch plaziert ist. im moment ist es eine instanz methode der klasse kunde. du solltest sie aber entweder statisch machen (public static void vektorFuellen) oder teil einer anderen (beispielsweise deiner hauptklasse)....

die fehlermeldung ist entstanden, da du einen konstruktor benutzt der gar nicht vorhanden ist. du hast ja einen eigenen konstruktor geschrieben mit den ganzen parametern

 

[Beni]

Dieses vektorfüllen hat ja keinen direkten Zusammenhang mit einer Instanc von "kunde". Darum kannst du das Keyword "static" hinzufügen. (Damit wird vektorfüllen zu einer Klassenmethode, im Gegensatz zu einer Instanzmethode).

public static Vector vektorfüllen(){...}

So, und jetzt kannst du ganz einfach schreiben:

Vector datas = kunde.vektorfüllen();

 

[rastaman]

Funzt 1A

wenn ich jetzt folgendermassen vorgehe:

public static void main(String args[]) {
        
        start();
        
        
        new guidatenbank().show();
          
    }

    public static void start(){
       Vector kunde = Kunde.vektorfüllen(); 
       Vector filme = Filme.vektorfüllen();
       
       return; 
    }

sind dann die beiden vectoren sozusagen im ram gespeichert und können jederzeit von anderen methoden aus durchsucht werden?

 

[Beni]

Die werden nach dem "return" wieder gelöscht. Denn es gibt ja keine Variablen mehr, die auf diese Vectoren zeigen.

Guck doch mal hier nach:
http://www.java-forum.org/de/viewtopic.php?t=4080
http://www.java-forum.org/de/viewtopic.php?t=1531

 

[rastaman]

hm aber wenn ich das so mache:

    //Variablen
    private static Vector kunde;
    private static Vector filme;

    public static void main(String args[]) {
        
        start();
        
        
        new guidatenbank().show();
          
    }

    public static void start(){
       kunde = Kunde.vektorfüllen(); 
       filme = Filme.vektorfüllen();
       
       return; 
    }

dann sollte es ja gehen...
oder eben nicht weil die variablen static sind?

 

[bygones]

prinzipiell schon - es sind nun Klassenvariablen und somit werden die "gemerkt". Wobei sie private sind und wenn du von einer anderen Klasse zugreifen willst musst du dann noch getter und setter Methoden wählen..

Aber warum machst du alle Variablen bzw. Methoden statisch ?
Meiner Ansicht nach wäre es (OOP)-mäßig besser sie als instanzvariablen bzw -methoden zu schreiben ![/code]

 

[rastaman]

na ja die methode start ist statisch, weil mein Compiler sonst meckert...

wie müsste ich das ganze denn sonst schreiben?

 

[bygones]

wie müsste ich das ganze denn sonst schreiben?

public class Main {//Variablen
    private Vector kunde;
    private Vector filme;

    public Main() {
       kunde = Kunde.vektorfüllen();
       filme = Filme.vektorfüllen();
       new guidatenbank().show();
    }

    public static void main(String args[]) {
        new  Main();         
    }
}

wobei - Klassennamen immer groß und eher guidatenbank().setVisible(true)

 

[rastaman]

Hier mal der ganze code von Klasse

public class Kunde {
    
    static ResultSet rSet; 
    
    int anz;
    
    private String name;
    private String vorname;
    private String strasse;
    private String ort;
    private int kundennummer;
    private java.util.Date geburtsdatum = new java.util.Date();
    private int plz;
    private int telefonnr;
    private int passwort;
    private String email;

    
    /** Creates a new instance of kunde */
    public Kunde(String n, String v, String st, String or, java.util.Date gd, int ku, int pl, int tel, int pw, String em) {
        this.name = n;
        this.vorname = v;
        this.strasse = st;
        this.ort =or;
        this.geburtsdatum = gd;
        this.kundennummer = ku;
        this.plz = pl;
        this.telefonnr = tel;
        this.passwort = pw;
        this.email = em;
    }
       
    
    public static Vector vektorfüllen(){
         Sql verbindung = new Sql();
         verbindung.connect();
         Vector abfragek = new Vector();
         try {
           rSet = verbindung.abfrage("SELECT * FROM kunde ORDER BY Name");
           //rSet = verbindung.abfrage("SELECT Name, Vorname, Vorname, Strasse, Ort, Geburtsdatum, Kundennummer, PLZ, Telefonnummer, Passwort, Email FROM kunde ORDER BY Name");

           while ( rSet.next() )
                abfragek.add( new Kunde( rSet.getString("Name"), rSet.getString("Vorname"), rSet.getString("Strasse"), rSet.getString("Ort"), rSet.getDate("Geburtsdatum"), rSet.getInt("Kundennummer"), rSet.getInt("PLZ"), rSet.getInt("Telefonnummer"), rSet.getInt("Passwort"), rSet.getString("Email") ));
            }
         catch ( SQLException e )
            {
            System.out.println( "Fehler bei Tabellenabfrage" + e );
            }
        verbindung.abfrageende();
        verbindung.trennen();
        return abfragek;
    }
    
         
     public String getName(){
         return name;
     }
   
}

und meine Klasse Sql:

public class Sql {
    
    Connection conn;
    Statement stmt;
    ResultSet rSet;
    int update;

    
  public boolean connect()
  {
    try {
      Class.forName( "com.mysql.jdbc.Driver" );
    }
    catch ( ClassNotFoundException e ) {
      System.out.println( "Fehler bei ODBC-JDBC-Bridge" + e );
      return false;
    }
    try
    {
      String url = "jdbc:mysql://localhost/videothek";
      conn = DriverManager.getConnection( url, "root", "224181" );
      //conn.close();
    }
    catch ( SQLException e )
    {
      System.out.println( "Fehler bei Tabellenabfrage" + e );
      return false;
    }
    return true;
  }
   
  public ResultSet abfrage(String query){
    try
    {
      stmt = conn.createStatement();
      String sqlQuery = query;
      rSet = stmt.executeQuery( sqlQuery );

     }
    catch ( SQLException e )
    {
      System.out.println( "Fehler bei Tabellenabfrage" + e );

    }
     return rSet;  
  }
  
    public boolean abfrageende(){
    try
    {
        stmt.close();
     }
    catch ( SQLException e )
    {
      System.out.println( "Fehler bei Tabellenabfrage" + e );
      return false;
    }
     return true;  
  }
  
  
  public int update(String query){
    try
    {
      stmt = conn.createStatement();
      String sqlUpdate = query;
      update = stmt.executeUpdate( sqlUpdate );
      stmt.close();
     }
    catch ( SQLException e )
    {
      System.out.println( "Fehler bei Tabellenabfrage" + e );
    }
     return update;  
  }
  
  public boolean trennen(){
    try
    {
      conn.close();
    }
    catch ( SQLException e )
    {
      System.out.println( "Fehler bei Tabellenabfrage" + e );
      return false;
    }
    return true;   
  }
}