Problem mit Entfehnen eines Spielers (MYSQL)

[McWizzardDev]

Guten Tag,

ich schreibe gerade an einem AutoNick Plugin nun stellt sich aber folgendes Problem in den Weg.
Ich habe gemacht, das der Nickname eingetragen wird in die MySQL Datenbank eingetragen wird und wenn man den Befehl /unnick ausführt, wird man wieder ausgetragen, doch der Spieler wird nicht ausgetragen (Auch keine Fehlermeldung)

Dies ist meine Klasse dafür (In dieser liegt der Fehler)

package de.germandev.autonick.mysql;

import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.UUID;

import org.bukkit.Bukkit;

public class SQLMethoden {
   
    public static boolean isUserExists(UUID uuid) {
        try {
            PreparedStatement ps = MySQL.getConnection().prepareStatement("SELECT * FROM Nick WHERE UUID = ?");
            ps.setString(1, uuid.toString());
            ResultSet rs = ps.executeQuery();
            rs.next();
        } catch (SQLException e) {
            e.printStackTrace();
        }
        return false;
    }
   
    public static void update(UUID uuid, boolean remove, String playername) {
       
        if(isUserExists(uuid)) {
            try {
                PreparedStatement ps = MySQL.getConnection().prepareStatement("UPDATE Nick SET NickName = ? WHERE UUID = ?");
                ps.setString(1, uuid.toString());
                ps.executeUpdate();
            } catch (SQLException e) {
                e.printStackTrace();
            }
        } else {
            try {
                PreparedStatement ps = MySQL.getConnection().prepareStatement("INSERT INTO Nick (UUID,NickName) VALUES (?,?,?)");
                ps.setString(1, uuid.toString());
                ps.setString(2, playername);
                ps.executeUpdate();
            } catch (SQLException e) {
                e.printStackTrace();
            }
        }
    }
   
    public static void delete(UUID uuid) {
        if(isUserExists(uuid)) {
            try {
                PreparedStatement ps = MySQL.getConnection().prepareStatement("DELETE * FROM Nick WHERE UUID = ?");
                ps.setString(1, uuid.toString());
                ps.executeUpdate();
            } catch (SQLException e) {
                e.printStackTrace();
            }
        } else {
            Bukkit.getConsoleSender().sendMessage("§cMySQL der Spieler mit der UUID " + uuid.toString() + " §cist nicht in der Datenbank");
        }
    }
   
    public static String getPoints(UUID uuid) {
        try {
            PreparedStatement ps =  MySQL.con.prepareStatement("DELETE * FROM Nick WHERE UUID = ?");
            ps.setString(1, uuid.toString());
            ResultSet rs = ps.executeQuery();
            while(rs.next()) {
                return rs.getString("NickName");
            }
        } catch (SQLException e) {
            e.printStackTrace();
        }
        return null;
    }

}

Hier noch die erstellten Tabellen:

        PreparedStatement ps = MySQL.getConnection().prepareStatement("CREATE TABLE IF NOT EXISTS Nick (UUID VARCHAR(100),NickName VARCHAR(100))");

Bitte schreibt mir keine anderen Fehler die nicht zu meiner Frage passen, da der Rest noch nicht ausgetestet ist, außerdem kommt mir nicht mit anderen Empfehlungen.

 

[Baba43]

Dies ist meine Klasse dafür (In dieser liegt der Fehler)

Bist du zu 100% sicher, dass die Methode aufgerufen wird und das Statement funktioniert?

 

[skcxck]

Wenn du allgemein mit Datenbanken arbeiten möchtest, solltest du dich immer nach geeigneten Bibliotheken umschauen.
Ein Beispiel hierfür wäre: http://dev.bukkit.org/bukkit-plugins/sqlibrary/

Google einwenig; Bin mir sicher das du fündig wirst.

 

[Microsamp]

• public static boolean isUserExists(UUID uuid) {

• try {

• PreparedStatement ps = MySQL.getConnection().prepareStatement("SELECT * FROM Nick WHERE UUID = ?");

• ps.setString(1, uuid.toString());

• ResultSet rs = ps.executeQuery();

• rs.next();

• } catch (SQLException e) {

• e.printStackTrace();

• }

• return false;

• }

"rs.next();" bringt dir momentan nicht viel, da du eigtl ein boolean returnen sollst. In diesem Fall wird immer false/0 returnt und somit wird wahrscheinlich delete() nie ausgeführt. Ps: Eine while(rs.next()){return true;} hätte es mehr getan .

 

[McWizzardDev]

Habs schon gelöst

 

[Lui9iGamer]

Liegt es am SQL-Befehl?
Ich bin mir nicht sicher, aber ich glaube das ist "DELETE FROM Nick WHERE UUID=?";?

 

[phenomax]

Also. Erstmal wäre es gut, wenn du noch etwaige Fehlermeldungen dazuschreibst, damit wir nicht rumraten müssen

Ich gehe mal davon aus, dass du dich auf die delete() Methode beziehst.
Zuerst rufst du ja deine isUserExistsMethode auf.

 public static boolean isUserExists(UUID uuid) {
        try {
            PreparedStatement ps = MySQL.getConnection().prepareStatement("SELECT * FROM Nick WHERE UUID = ?");
            ps.setString(1, uuid.toString());
            ResultSet rs = ps.executeQuery();
            rs.next();
        } catch (SQLException e) {
            e.printStackTrace();
        }
        return false;
    }

Das Problem hier ist, dass du immer false returnst und das Ergebnis von rs.next(); ignoriert.
Dementsprechend müsstest du also auch

return rs.next();

einsetzen, wolltest du den Datensatz returnen.
Da du hier jedoch nur abfrage willst, ob ein Datenbankeintrag zur UUID existiert, würde eine einfaches return true reichen, sofern das ResultSet mindestens einen Eintrag hätte.
Richtig wäre also

 public static boolean isUserExists(UUID uuid) {
        try {
            PreparedStatement ps = MySQL.getConnection().prepareStatement("SELECT * FROM Nick WHERE UUID = ?");
            ps.setString(1, uuid.toString());
            ResultSet rs = ps.executeQuery();
            if  (rs.hasNext()) {
             return true;
             }
        } catch (SQLException e) {
            e.printStackTrace();
        }
        return false;
    }

Performancetechnisch wäre es auch besser, wenn du nur einen Wert aus deiner DB selectest, da du hier ja nun keinen Wert weiterverarbeitest.

An dem Rest deiner delete-Methode kann ich so keinen Fehler erkennen - aber bitte reiche noch Errormeldungen und auch sonstige Ausgaben des Plugins nach.

Gruß und schönen Abend!
Max