DBConnection als Methode in mehreren Klassen

[coo123]

Hallo liebe Gemeinde,

ich habe vor kurzem begonnen Java zu programmieren und bin nun auf ein Problem gestoßen bei dem ich nicht mehr weiterkomme und die Hilfe einer netten Community brauche .

Und zwar möchte ich eine Methode in einer Klasse erstellen, auf der ich von allen anderen Klassen zugreifen kann.
Diese sollte lediglich die Verbindung zu meiner MySql Datenbank herstellen.
Beispiel:

import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.ResultSet;

import javax.swing.JComboBox;

public class dbconnect {
	
	public dbconnect() {
	
		  String url = "jdbc:mysql://localhost:3306/database";
		  String driver = "com.mysql.jdbc.Driver";
		  String userName = "root"; 
		  String password = "MyNewPassword";
		  try {
				Class.forName(driver).newInstance();
				Connection con = (Connection)DriverManager.getConnection(url,userName,password);
					
			  } 
		  
	}catch (Exception ex) {
		  System.out.println("Error: "+ex);
	}
}
}

Nun möchte ich in einer anderen Klasse, wo ich ein GUI mit Comboboxen programmiert habe, diese Query in meiner Combobox anzeigen lassen. Wie mache ich das? Das sollte doch irgendwie so funktionieren oder nicht?
Muss der Code in die Klasse rein wo ich das GUI programmiert habe? Wenn ja, wo kommt dann meine Methode zum Verbinden der Datenbank hin?

public void getcombo (){
	String sql = "SELECT Expense.Code FROM expense_code_master";
	ResultSet rs = con.prepareStatement(sql).executeQuery();
	
	while (rs.next()) {
		String name = rs.getString("Expense Code");
		comboBox.addItem(name);
		
  }

Vielen Dank für eure Hilfe

 

[stg]

Der GUI ist es egal, wo die Daten herkommen, und was später mit diesen passiert. Sie ist nur dafür die Eingaben vom Benutzer entgegen zu nehmen und dem User Daten anzuzeigen. Datenbankabfragen haben in der GUI daher rein gar nichts verloren.

Ich würde aber unabhängig davon empfehlen, dass du erst mal klein anfängst. Ein Programm mit GUI und Datenbank-Verbindung ist mMn nichts für Anfänger. Wenigstens die OOP-Grundlagen sind Vorraussetzung, dass da überhaupt was brauchbares bei rumkommt.
Wenn du gerade anfängt mit der Programmierung, dann beschränke dich zunächst mal auf die Grundlagen und schreibe ein paar einfache Konsolenanwendungen, auch wenn das Ergebnis dann vielleicht nicht direkt so hübsch aussieht und Spaß macht. Aber letzten Endes hast du mehr davon, wenn du nicht gleich alles auf einmal versuchst und nur Kappes dabei herauskommt

 

[coo123]

Hallo stg,

mir ist schon klar das sowas nichts für Anfänger ist, ich habe auch schon kleine Anwendungen erfolgreich programmiert.
Mit meiner Datenbank habe ich auch schon erhebliche Fortschritte erreicht, aber leider ist das alles nicht sehr effizient.

Gibt es keinen der eine Lösung bieten will?

LG

 

[Joose]

public class dbconnect {
	
	public dbconnect() {
	
		  String url = "jdbc:mysql://localhost:3306/database";
		  String driver = "com.mysql.jdbc.Driver";
		  String userName = "root"; 
		  String password = "MyNewPassword";
		  try {
				Class.forName(driver).newInstance();
				Connection con = (Connection)DriverManager.getConnection(url,userName,password);
					
			  } 
	}catch (Exception ex) {
		  System.out.println("Error: "+ex);
	}
}
}

Gut sobald du ein Objekt dieser Klasse "dbConnect" machst, holst du dir ein Connection Objekt für die DB.
Und dieses wird gleich wieder im Nirvana verschwinden da es eine lokale Variable im Konstruktor ist.
--> Grundlagen der OOP

Nun möchte ich in einer anderen Klasse, wo ich ein GUI mit Comboboxen programmiert habe, diese Query in meiner Combobox anzeigen lassen. Wie mache ich das? Das sollte doch irgendwie so funktionieren oder nicht?
Muss der Code in die Klasse rein wo ich das GUI programmiert habe? Wenn ja, wo kommt dann meine Methode zum Verbinden der Datenbank hin?

public void getcombo (){
	String sql = "SELECT Expense.Code FROM expense_code_master";
	ResultSet rs = con.prepareStatement(sql).executeQuery();
	
	while (rs.next()) {
		String name = rs.getString("Expense Code");
		comboBox.addItem(name);
		
  }

So wie du den Datenbank Zugriff und den Aufbau der GUI hier mischt ist es ganz schlecht gelöst!
--> Schichtenmodell

mir ist schon klar das sowas nichts für Anfänger ist, ich habe auch schon kleine Anwendungen erfolgreich programmiert.
Mit meiner Datenbank habe ich auch schon erhebliche Fortschritte erreicht, aber leider ist das alles nicht sehr effizient.

Gibt es keinen der eine Lösung bieten will?

Was für kleine Anwendungen denn? Deine dbconnect Klasse zeugt nicht von viel Verständnis der OOP.
Ich würde dir vorschlagen kleine Schritte zu machen. Versuche zuerst einmal dein Program ohne GUI zu schreiben.
Sprich: Verbindung zur Datenbank herzustellen, Daten abfragen, (möglicherweise in Objekte packen) und diese Daten dann auf der Konsole auszugeben.

Danach kannst du immer noch eine GUI einbauen (und auf Grund eines Schichtenmodells sollte dies nicht all zu schwer sein)
Wir bieten hier keine Lösungen (und wenn kostest das etwas), wir helfen aber gerne dir zu einer Lösung finden -> Eigeninitiative

 

[turtle]

Ich stimme meinem Kollegen Joose voll zu

Ich ergänze und sage dir, das bereits in den wenigen Zeilen Code, die du gezeigt hast, Probleme enthalten sind, die du lösen solltest, BEVOR du dich mit einer GUI befasst.

Class.forName(driver).newInstance();

Braucht man nicht mehr und sollte entfernt werden.

Das du eine Connection nur in der Methode dbconnect aufbaust wurde schon gesagt. Aber praktisch immer nutzt man einen Connection-Pool für JDBC-Verbindungen. Damit sollest du dich mal beschäftigen.

Dabei wirst du auch bemerken, das die Kapselung einer dbconnect-Methode zu feingranular ist und einem Pool überlassen bleiben sollte.

Dann bleibt noch das Abschicken und Verwalten von Resourcen, wie Statements, Resultset und Connections. Dieses ist in der Tat nicht ganz ohne, insbesondere weil jederzeit Exception auftreten können. Daher werden oft ORM wie Hibernate, JPA, oder myBATIS eingesetzt, weil man von denen weiss, das sie es richtig machen.Apropos Statements: Hier solltest du dich unbedingt mal mit Prepared-Statements auseinandersetzen (Stichwort SQL-Injection)

Du siehst, selbst ohne GUI, gibt es noch eine Fülle von Dingen, die du machen kannst. DANACH sollte sich das Nutzen von DB aus einer GUI dir schon erschließen.
Einfach so ein Programm zusammenbasteln bringt es nicht, wenn man nicht bereit ist, sich tiefer in die Materie einzuarbeiten:rtfm: