Vinzenz Mai: Bessere Möglichkeit das Abzufragen?

Beitrag lesen

Hallo,

Hi, ich überprüfe mit folgender Funktion, ob ein User schon etwas gepostet hat, und verhindere so Spamming:

das bezweifle ich.

public function CompareIP() {
    $IP = $_SERVER['REMOTE_ADDR'];
    $IP = ip2long($IP);

warum nicht einfach wie "E wie einfach"?
Warum wandelst Du die IP-Adresse um?
Warum speicherst Du sie nicht in einfacher, lesbarer Punkt-Notation?

$DB_IP = mysql_query("SELECT IP,Date FROM test WHERE IP = '$IP'");
    if(mysql_affected_rows() == 1) {

Warum verwendest Du hier mysql_affected_rows()?

Warum verwendest Du nicht eine Abfrage, die schlicht und einfach eine einzige Zeile mit einer Spalte zurückliefert, die den gewünschten boolschen Wert enthält? Das wäre doch viel einfacher. MySQL hat alle notwendigen Funktionen eingebaut. Nutze sie.

Warum fehlt die Fehlerbehandlung?

Fragende Grüße

Vinzenz