Problem mit CREATE PROCEDURE

**onemorenerd** · 17.10.2009, 18:30

Anderes Symptom, aber vielleicht der gleiche Bug: MySQL Bugs: #3157: Crash if stored procedure contains IF EXISTS

Zieh das SELECT doch mal vor das IF, speichere den Wert in einer Variablen und benutze diese in der IF-Bedingung. Vielleicht klappts dann schon.

**Flip7** · 17.10.2009, 18:44

Okay interessant, danke!
Ich bin nun hier, jedoch funktioniert es trotzdem nicht :-( ...
Gleicher Fehler

Code:

CREATE PROCEDURE click( address INT, partyId INT )
BEGIN
DECLARE c INT;

SELECT count(*) FROM clicklock WHERE ip = address LIMIT 1 INTO c;
IF c = 1 THEN -- wenn nicht gesperrt
UPDATE party SET clicks = clicks + 1 WHERE id = partyId LIMIT 1; -- feld "clicks" in tabelle "party" inkrementieren
INSERT INTO clicklock VALUES( address ); -- Ip sperren ( das zweite Feld hat automatisch CURRENT_TIMESTAMP )
END IF;

END;

**onemorenerd** · 17.10.2009, 19:42

Ah du musst den Delimiter ändern, z.B. in // un dann

Code:

CREATE PROCEDURE click( address INT, partyId INT )
BEGIN

IF EXISTS(SELECT 1 FROM clicklock WHERE ip = address) THEN 
    UPDATE party SET clicks = clicks + 1 WHERE id = partyId LIMIT 1; 
    INSERT INTO clicklock VALUES( address ); 
END IF;

END//

PhpMyAdmin hat im Tab SQL unter der Textarea ein Feld für den Delimiter.

**Flip7** · 17.10.2009, 21:08

großer Forschritt :-D
es funktioniert schon mal, aber exists mag er immer noch nicht, bei der Ausführung mit "CALL". Er erzählt mir irgendwas von "count field nix exists" oder ähnlich.
Die nachfolgende Prozedur wird geparst und ausgeführt, allerdings funktioniert der insert in die tabelle nicht. Wenn ich einen Eintrag in "clicklock" mache meckert der Server allerdings, wenn ich mit der Prozedur den Insert erzwinge. "Primärschlüssel existiert schon", also funktioniert der insert schon, aber gibt es einen insert mit delay oder ähnlich?

Code:

CREATE PROCEDURE click( address INT, partyId INT )
BEGIN
DECLARE c INT;

SELECT count(*) FROM clicklock WHERE ip = address LIMIT 1 INTO c;
IF c = 1 THEN
BEGIN
INSERT INTO clicklock VALUES( address, NOW( ) );
UPDATE party SET clicks = clicks + 1 WHERE id = partyId LIMIT 1;
END;
END IF;

END //

**onemorenerd** · 17.10.2009, 22:01

Dein Problem ist wohl, dass mehrere Clicks von der selben IP in einer Sekunde stattfinden können. Entweder verhinderst du das oder änderst deine Schlüsseldefinition oder benutzt Timestamps mit Mikrosekunden.

**AmicaNoctis** · 18.10.2009, 00:28

Hallo,

also wenn ich nicht total falsch liege, stimmt einfach deine Bedingung nicht. Ich hab es so verstanden:

In der Tabelle clicklock stehen die gesperrten IP-Adressen als Primärschlüssel drin. Wenn du also die Prozedur click mit einer IP-Adresse aufrufst, die in der Tabelle enthalten ist, soll gar nichts passieren. Wenn die aktuelle IP-Adresse nicht enthalten ist, dann soll sie eingetragen werden und in der party-Tabelle das Feld clicks inkrementiert werden. Ist das soweit richtig?

Wenn ja, musst du die Bedingung auf IF c = 0 umschreiben, denn c = 1 ist ja erfüllt, wenn die IP-Adresse bereits gesperrt ist. Ein weiteres INSERT führt dann natürlich zu einer Fehlermeldung, weil die Adresse ja schon drin steht.

Gruß,

Amica

Problem mit CREATE PROCEDURE