MudGuard: linkcounter

Beitrag lesen

Hi,

// Die per GET übergebene URL einlesen und datenbanksicher machen
$url = addslashes($_GET['url']);

hier wäre eher die escape-Funktion der genutzten Datenbank, in diesem Fall also mysql_real_escape_string angebracht.

// Eintrag fuer die übergebene URL um 1 erhöhen.
        $query = "UPDATE $table SET count = count + 1 WHERE url = '$url'";
        @mysql_query($query);
       // Überprüfung, ob noch kein Eintrag für URL vorhanden ist
        if (mysql_affected_rows() == 0) {
            $sql_insert = "INSERT INTO $table (url, count) VALUES ('$url', '1')";
            @mysql_query($sql_insert);
        }

Das schreit ziemlich laut nach
INSERT ON DUPLICATE KEY UPDATE

cu,
Andreas

--
Warum nennt sich Andreas hier MudGuard?
O o ostern ...
Fachfragen per Mail sind frech, werden ignoriert. Das Forum existiert.