Werte in Klasse übergeben

[Shao]

Guten Abend,
ich hab mich langsam so ein bisschen reingefuchst was Klassen angeht und das Grundprinzip, glaube ich, ansich verstanden.
Ich habe nun mir eine Klasse erstellt (die auch super funktioniert) und wollte um noch mehr Code zu sparen die Abfrage der IP,Username,Password auch mit in die Klasse packen. Dazu habe ich momentan einfach die Werte einzeln übergeben ->

        test.ip = "jdbc:mysql://localhost/test";
        test.username = "root";
        test.password = "admin";

Klasse ->

package datenbank;

import java.sql.*;


public class verbindung {
    private Connection con;
    String ip = "";
    String username = "";
    String password = "";

    public void getData(){

    }
    void connect(){
        try
        {
            Class.forName("com.mysql.jdbc.Driver");
            con = DriverManager.getConnection(ip, username, password);
        }
        catch (ClassNotFoundException | SQLException e1)
        {
            e1.printStackTrace();
        }
    }
    void close(){
        try {
            con.close();
        }
        catch (SQLException e) {
            e.printStackTrace();
        }
    }
}

so funktioniert das auch super nun will ich habe irgendwie mit meiner eigenen Methode getData
alle die drei Werte direkt bekommen um mir noch mehr zeilen Code in der Hauptklasse zu sparen (Werte sollen von textFeldern ausgelsen werden aber das ist ja nicht das Problem).
nun habe ich mir die irgendwie was vorgestellt wie (reine Idee, klappt also nicht) ->

test.getData(dbip.gettext, dbusername.gettext, dbpassword.gettext)

Nun bin ich überfragt wie ich das realisere wie dafür die getData Methode aussehen muss und was ich dann nach test.getData schreiben muss.

MfG Shao

 

[MoxxiManagarm]

Das wird so nicht gehen da String immutable ist. Du müsstest StringBuilder beispielsweise verwenden.

package javaforum.org;

public class Example {
  
    public static void getData(StringBuilder refA, String refB) {
        refA.delete(0, refA.length()).append(":)");
        refB = ":)";
    }

    public static void main(String[] args) {
        StringBuilder getterA = new StringBuilder(":(");
        String getterB = new String(":(");
      
        getData(getterA, getterB);
      
        System.out.println(getterA + " & " + getterB);
    }
}

-->

&

Aber bevor du alles mit einem Builder machst bist du mit gettern für die 3 Werte doch bestimmt besser dran.

 

[Shao]

Wie genau muss ich das nur auserhalb und innerhalb der Klasse schreiben vllt hast du da nen kleines Beispiel oder so für mich. Ich habe erhlich gesagt noch nie mit Gettern gearbeitet.

 

[temi]

Ich habe erhlich gesagt noch nie mit Gettern gearbeitet.

Daran ist nichts geheimnisvolles dran. Ein getter ist einfach nur eine Methode, die den Wert einer privaten Eigenschaft zurück gibt.

class Foo
{
   private string bar;

   public Foo(string bar)
   {
      this.bar = bar; // private Eigenschaft per Konstruktor setzen
   }

   public string getBar() // getter Methode für die Eigenschaft "bar"
   {
      return bar;
   }

   // und der Vollständigkeit halber noch der passende setter
   public setBar(string bar)
   {
      this.bar = bar;
   }
}

Generell sollte eine Klasse soviel Information wie möglich nach außen verbergen (private Eingenschaften) und nur das außen geben (per getter) was zum Arbeiten mit der Klasse notwendig ist.
Ob es einen getter/setter für eine Eigenschaft gibt, hängt von der Funktion der Klasse ab. Es muss nicht zwangsweise ein getter/setter für jede Eigenschaft existieren. Genauso könnte es getter-artige Methoden geben, die als Rückgabewert mehrere private Eigenschaften kombinieren.

Außerdem sollte eine Instanz der Klasse, nachdem sie erzeugt wurde funktionstüchtig sein, also alle dazu notwendigen Parameter per Konstruktor übergeben werden.

 

[Shao]

So war leider die letzten Tage nicht am PC.
Ich habe mir nunmal diese getter/setter angeschaut und ansich hat es im simplen rahmen funktioniert.
Leider aber nicht in der "komplexeren Anwendung in meinem SQL Programm.

    private Connection con;
    String ip = "";
    String username = "";
    String password = "";

    mainframe kaese = new mainframe();


    public void getData(){
        ip = kaese.getdbip();
        username = kaese.getdbusername();
        password = kaese.getdbpassword();
    }
    void connect(){
        try
        {
            System.out.println(username);
            Class.forName("com.mysql.jdbc.Driver");
            con = DriverManager.getConnection("jdbc:mysql://" + ip, username, password);
        }
        catch (ClassNotFoundException | SQLException e1)
        {
            e1.printStackTrace();
        }
    }

Das der Code meiner Klasse die die Verbindung steuern soll
und jetzt der meiner Hauptklasse wo die Werte übergeben werden sollen:

   String getdbip(){
        return dbip.getText();
    }
    public String getdbusername(){
        return dbusername.getText();
    }
    String getdbpassword(){
        char[] pass = dbpassword.getPassword();
        String passString = new String(pass);
        return passString;
    }

Leider kommt (getestet mit sysout im connect) garnichts an was in meinem TextAreas steht.
Offensichtlich habe ich irgendwo nen gedankenfehler aber trotz lauter getter/setter tutorials komm ich einfach nicht auf die lösung ^^

 

[Shao]

leer

 

[Robat]

mainframe kaese = new mainframe();

Hier erstellst du eine neues Objekt deiner Klasse mainframe. Kann es sein, dass es sich dabei nicht um das eigentliche Objekt handelt in der du die Eingaben machst?
Erstellst du irgendwo (in der main-Methode) schon mal so eine Instanz?


BTW:
Gewöhn' dir am Besten an Klassennamen im UpperCammelCase und Methoden/Variablennamen im lowerCammelCase zu schreiben.

 

[Shao]

Nein tue ich nicht. Ich hab um ehrlich gesagt diese Zeile nur drin weil er sonst meine drei get Methoden aus der Main klasse nicht finden konnte. Deshalb auch der komische Name

 

[Robat]

Wie sieht denn deine Main-Klasse aus?

 

[Shao]

So endlich am Rechner. Hier meine Main:

package datenbank;

import javax.swing.*;
import java.awt.*;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.sql.*;
import java.awt.color.*;

public class mainframe {
    private JPanel mainpanel;
    private JPanel loginpanel;
    private JProgressBar fortschritt_verbinden;
    private JTextArea dbip;
    private JTextArea dbusername;
    private JPasswordField dbpassword;
    private JButton btn_verbinden;
    private JComboBox drop_datum;
    private Connection con;

    String getdbip(){
        return dbip.getText();
    }
    public String getdbusername(){
        return dbusername.getText();
    }
    String getdbpassword(){
        char[] pass = dbpassword.getPassword();
        String passString = new String(pass);
        return passString;
    }

    public static void main(String[] args) {
        JFrame frame = new JFrame("mainframe");
        frame.setContentPane(new mainframe().mainpanel);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.pack();
        frame.setVisible(true);
    }

    public mainframe() {
        btn_verbinden.addActionListener(arg0 -> verbinden());
    }

    void verbinden(){
        verbindung test = new verbindung();
        test.getData();
        //test.ip = "jdbc:mysql://localhost/test";
        //test.username = "root";
        //test.password = "admin";
        test.connect();
        test.close();
        btn_verbinden.setBackground(Color.green);

    }

}

 

[Robat]

Ein recht ungewöhnliches Konstrukt.
Müsstest du deine mainframe Klasse nicht irgendwo mal instanzieren damit der Code im Konstruktor ausgeführt wird?
Der Klasse verbindung müsstest du dann die aktuelle Instanz übergeben damit du in der Klasse auf die Methoden zugreifen kannst.

 

[temi]

Schau dir noch mal mein einfaches Beispiel mit der Klasse "Foo" an und versuche das auf deine Klasse Verbindung umzusetzen. Ziel sollte sein, dass anstatt von:

verbindung test = new verbindung();

folgende steht:

Verbindung test = new Verbindung("jdbc:mysql://localhost/test", "root", "admin");

Damit erzeugst du eine gültige Instanz von "Verbindung" mit allen notwendigen Daten.

 

[Shao]

@temi so konnte ich es jetzt lösen:
In dem ich beim aufruf meine get methoden übergebe.

test.connect(getdbip(), getdbusername(), getdbpassword());

und dann so in der Klasse ->

 void connect(String ip, String username, String password){
        try
        {
            System.out.println(ip);
            Class.forName("com.mysql.jdbc.Driver");
            con = DriverManager.getConnection("jdbc:mysql://" + ip, username, password);
        }
        catch (ClassNotFoundException | SQLException e1)
        {
            e1.printStackTrace();
        }

Danke euch allen jetzt habe ich wieder einen Teil mehr verstanden