MySQL Rand System.

[Walfisch]

Wir haben ein Rang System weclhes über MySQL läuft geschrieben. Dabei wird jedem Spieler ein Rang in einer Tabelle zugewiesen. Beim Join wir dieser Rang abgefragt. Zudem haben wir das ganze so eingestellt, dass wenn der Spieler zum ersten mal joint, er in die Datenbank eingetragen wird. Allerdings wird manchmal beim joinen der Rang des Spielers nicht richtig abgefragt und der Spieler wird ein weiteres mal eingetragen.
Schonmal im Voraus vielen Dank für euere Hilfe!

 

[MichiPlaysYT]

Ich glaube so kann man dir nicht helfen, ein bisschen Code müsste hier noch dabei sein

 

[Walfisch]

Okey, danke.
Hier Die "HasEverJoined Methode:
(Damit prüfen wir ob der Spieler schonmal gejoined ist.)

public static boolean hasEverJoined(Player p) {
        String uuid = p.getUniqueId().toString();
        try {
            ResultSet rs = MySQL.Query("SELECT uuid FROM allplayers WHERE uuid='" + uuid + "'");
            if (rs.next()) {
                return true;
            }

        } catch (Exception localException) {
        }
        return false;
    }

Und hier die "getRang" Methode:
(Damit fragen wir den Rang ab. | Verwenden wir zum Beispiel im PlayerChatListener um den Rang bei einer Nachricht abzufragen.)

public static String getRang(String playeruuid) {

        String s = null;

        ResultSet rs = MySQL.Query("SELECT rang FROM allplayers WHERE uuid='" + playeruuid + "'");

        try {
            java.sql.ResultSetMetaData rsmd = rs.getMetaData();
            int cols;

            cols = rsmd.getColumnCount();

            for (int i = 1; i <= cols; i++)
                rsmd.getColumnLabel(i);

            while (rs.next()) {
                // eine zeile ausgeben
                for (int i = 1; i <= cols; i++)
                    s = rs.getString(i);

            }
        } catch (SQLException e) {
            System.out.println("Fehler!");
        }

        return s;

    }

Und hier ist die "setupSpieler" Methode:

Damit tragen wir den Spieler in die Datenbank ein, wenn der Spieler zum ersten mal joned, bzw wenn die haseverjoined Methode = false ist.

public static void setupSpieler(Player p) {

        String name = p.getName();
        String uuid = p.getUniqueId().toString();

        String s = "spieler";

        if (hasEverJoined(p) == false) {
            MySQL.Update("INSERT INTO allplayers (name,uuid,rang) VALUES ('" + name + "','" + uuid + "','" + s + "')");
        } else {
            MySQL.Update("UPDATE allplayers SET name = '" + name + "' WHERE uuid = '" + uuid + "'");
        }

    }

 

[Baba43]

Also der Code ist allgemein ziemlich schlecht, aber das lassen wir jetzt mal so stehen.

Grundsätzlich solltest du, um Konsistenz zu garantieren, schon der Datenbank mitteilen, dass keine UUID mehrfach in der Tabelle stehen darf. Dann wirft dein Plugin an der Stelle eine Exception, wodurch dein Problem nicht gelöst ist, aber du zumindest keine dauerhaft fehlerhaften Datensätze in der Datenbank hast.

Dann gibt es grundsätzlich die Möglichkeit, der Datenbank mitzuteilen, dass ein Datensatz nur angelegt werden soll, wenn er nicht bereits existiert. Jede Methode hat ihre Nachteile, die dabei berücksichtigt werden sollten.

Um dein eigentliches Problem zu lösen, bräuchte man vermutlich den gesamten Quelltext, denn du schreibst: "Allerdings wird manchmal beim joinen der Rang des Spielers nicht richtig abgefragt und der Spieler wird ein weiteres mal eingetragen.". Wie soll ich das verstehen? Ist damit jetzt die setupSpieler()-Methode gemeint? Wie wäre es für den Anfang damit, dass du Exceptions nicht ins Leere laufen, sondern ausgeben lässt, damit du überhaupt mitbekommst, wenn es zu Fehlern gekommen ist?

Sprechen wir außerdem von einer Serverinstanz oder von einem Netzwerk?

 

[Asgarioth]

Hi,

Exceptions fängt man ja normalerweise nicht ab, um sie zu ignorieren sondern den Fehler zu behandeln... Soweit ich das mit den Schnippseln beurteilen kann liegt die Ursache wahrscheinlich in Der hasEverJoined Methode (soweit nicht im unbekannten Code anderer Unfug passiert). Diese wird nämlich still und leise (da Exceptions zwar gefangen aber nicht behandelt werden) immer FALSE liefern, wenn bei dem DB-Query was schief geht... (Warum da was schief geht ist ein anderes Thema).

In der Folge wird setupSpieler im Fehlerfall bei hasNeverJoined immer einen Insert absetzen. Du musst die Exceptions also entweder sauber verarbeiten, weiterreichen oder eine neue eigene Exceptions werfen. So oder so muss sie verarbeitet und nicht ignoriert werden.

So Long,
~Asgarioth

 

[Cyrox]

http://dev.mysql.com/doc/refman/5.7/en/insert-on-duplicate.html
Lesen, verstehen, code anpassen!