Java - Beim Abspeichern Redundanzen vermeiden!

[Aslan1905]

Hallo, einen schönen guten Tag alle zusammen!

Ich würde gerne wisssen, wie ich vor dem Speichern eines Wertes in eine Datenbanktabelle prüfen kann, ob dieser Eintrag bereits existiert oder nicht. Existiert der Wert bereits schon in der Datenbanktabelle soll kein Eintrag erfolgen und somit das Speichern verhindern.


Die Tabelle Mitarbeiter hat folgende Attribute:

- ID (Primärschlüssel), Vorname, Nachname, Geschlecht, Geburtsdatum, EMail, Mobilnummer


Die Tabelle Login hat folgende Attribute:

ID (Primärschlüssel), Benutzername, Passwort, Typ, MitarbeiterID (Fremdschlüssel)



Die Tabelle Mitarbeiter & Person stehen in Relation. Meine Quellcode sieht folgendermaßen aus:

try{
        
            String sql = "INSERT INTO mitarbeiter (Vorname,Nachname,Geburtsdatum,Geschlecht,EMail,Mobilnummer) VALUES (?,?,?,?,?,?)";
            
            pst1 = conn.prepareStatement(sql, pst1.RETURN_GENERATED_KEYS);
           
            pst1.setString(1, txt_vorname.getText());
            pst1.setString(2, txt_nachname.getText());
            pst1.setString(3, txt_geb.getText());
            
            String combobox_geschl = combobox_geschlecht.getSelectedItem().toString();
            pst1.setString(4, combobox_geschl);
            
            pst1.setString(5, txt_email.getText());
            pst1.setString(6, txt_mobilnr.getText());
      
            pst1.execute();
            pst1.close();
           
            //JOptionPane.showMessageDialog(null, "Die Daten wurden gespeichert!");
        
        }   catch(Exception e){
        
                JOptionPane.showMessageDialog(null, e);
        
            }   finally {
                        
                        try {
                            
                             String sql1 = "INSERT INTO login (Benutzername,Passwort,Typ,MitarbeiterID) VALUES(?,?,?,(SELECT MAX(ID) FROM mitarbeiter))";
            
                             pst2 = conn.prepareStatement(sql1, pst2.RETURN_GENERATED_KEYS);
            
                             pst2.setString(1, txt_benutzername.getText());
                             pst2.setString(2, txt_passwort.getText());
                             String cbox_typ = combobox_typ.getSelectedItem().toString();
                             pst2.setString(3, cbox_typ);
                             
            
                             pst2.execute();
            
                             pst2.close();
                             
            
                             JOptionPane.showMessageDialog(null, "Die Daten wurden erfolgreich gespeichert!");
                       
                        }   catch (Exception e) {
                       
                                JOptionPane.showMessageDialog(null, e);
                           
                            }
            
                }

Momentan erfolgt die Speicherung eines Datensatzes nach dem Klicken auf das Button "speichern" unabhängig davon ob dieser bereits existiert oder nicht.

1. Wie kann ich jetzt einen Eintrag vorher prüfen?
2. Wie würde der Quellcode dann aussehen?

Ich habe schon einiges ausprobiert und komme da einfach kein Stück weiter - bin echt am zweifeln. Wäre echt lieb, wenn mir jemand da weiterhelfen könnte. Vielen Dank im Voraus!

Liebe Grüße

 

[turtle]

Prüfen ob ein Eintrag bereits existiert, kannst du, indem du schaust, ob es den Eintrag gibt:

select count(*) from Mitarbeiter where id=?

Kommt da ein nicht leeres Resultset zurück, gibt es den Mitarbeiter bereits und du musst ihn nicht mehr Einfügen.

Dieses kann aber auch zu Performance-Problemen führen, weil du jetzt vor jedem Einfügen erst die Abfrage an die DB stellen musst, also zwei Round-trips zur DB machst.

Eine andere Möglichkeit wäre, auf die unique ID-Verletzung zu reagieren, weil die DB nicht zulässt, zwei Datensätze mit gleichem primary key zu haben.

Musst du entscheiden, wie du das löst und bewerten wie häufig die Aktion "neuen Mitarbeiter einfügen" wirklich vorkommt.

 

[Aslan1905]

Vielen Dank für deine Antwort!

Die ID wird automatisch bei jedem Datensatzeintrag erhöht (Auto-Increment), daher kann keine ID-Verletzung vorkommen. Ich müsste irgendwie schauen können, ob die Werte in den Spalten Vorname, Nachname & Geburtsdatum vorkommen.

Würde der folgende SQL-Befehl klappen?

SELECT COUNT(*) FROM mitarbeiter WHERE (Vorname='"+Value1+"' AND Nachname= '"+Value2+"' AND Geburtsdatum='"+Value3+"');

Count liefert ein Integer. Wenn die Zahl == 0 ist soll der Eintrag erfolgen, weil es keinen Datensatz gibt der die Werte enthält.

Würde das so gehen?
Und wie würde dann die if-Schleife genau aussehen?

Vielen Dank im Voraus!

 

[turtle]

Ich würde IMMER PreparedStatements verwenden, da dein Code dahingehend anfällig für SQLInjection-Angriffe ist, GEFÄHRLICH!!!

SELECT COUNT(*) FROM mitarbeiter WHERE (Vorname='? AND Nachname= ? AND Geburtsdatum=?
U

Und das laden des Resultsets und Prüfen, ob da 0 zurück kommt, würde ich natürlich in eine kleine Funktion notExists schreiben. Dann sähe die if-Zeile wahrscheinlich so aus:

if (mitarbeiter.notExists(vorname, nachname, geburtsdatum)) {
 einfuegen_in_db
}

 

[Aslan1905]

Ich habe es jetzt nun versucht folgendermaßen zu lösen:

SELECT * FROM mitarbeiter WHERE Vorname=? AND Nachname=? AND Geburtsdatum=?;

int zeile = rs2.getRow();

if( zeile >= 1){

JOptionPane.showMessageDialog(null, "Datensatz existiert.");   

} else if (zeile == 0){

(Der Eintrag soll erfolgen)

}

Leider gibt mir INT aber den Wert 0 zurück, obwohl ein Datensatz existiert!

Kann mir da vielleicht jemand weiterhelfen, wieso das jetzt nicht geht?!

 

[turtle]

int zeile = rs2.getRow()

Das sieht für mich verdächtig...falsch aus.

SELECT COUNT(*) FROM mitarbeiter...

Wenn du das ausführst, kriegst du IMMER ein Resultset zurück. Dieses musst du ebenfalls iterieren, wie du dies mit jeder Query machen musst. In diesem Fall ist die Iteration einfach, weil du weisst das da nur eine Zeile zurückkommen kann.

rs.next()

;
Und dann fragst du den Wert des count ab und der ist entweder 0 oder 1.

rs.getInt(1)

 

[stg]

Eine andere Möglichkeit wäre, auf die unique ID-Verletzung zu reagieren, weil die DB nicht zulässt, zwei Datensätze mit gleichem primary key zu haben.

Die ID wird automatisch bei jedem Datensatzeintrag erhöht (Auto-Increment), daher kann keine ID-Verletzung vorkommen. Ich müsste irgendwie schauen können, ob die Werte in den Spalten Vorname, Nachname & Geburtsdatum vorkommen.

Dann setz doch einfach ein unique-constraint auf eben diese drei Spalten... Diesen Weg würde ich immer bevorzugen. Somit ist auch sichergestellt, dass auch auf anderem Wege (als deine Java-Anwendung) keine ungültigen Daten in die Datenbank gelangen können.