IP SPERRE / Probleme

[Trexx]

Hallo zusammen ich habe hier eine Ip Sperre jedoch finde ich den fehler nicht darin.


hier der Code :

 <?
function getip() {
    // Proxys
    if(getenv("HTTP_X_FORWARDED_FOR"))
        $ip = getenv("HTTP_X_FORWARDED_FOR");
    else
        //Standard
        $ip = getenv("REMOTE_ADDR");
    return $ip;
}

mysql_connect("localhost","web64","passwort") or die(mysql_error());
mysql_select_db("usr_web64_1");

$sperre=time()-180;
$jetzt=time();
$ip = getip();

mysql_query("DELETE FROM ipsperre WHERE time<$sperre") or die(mysql_error());

$pruef=mysql_query("SELECT ip FROM ipsperre WHERE ip='$ip'") or die(mysql_error());
if(@mysql_fetch_array($pruef)) {
    mysql_query("INSERT INTO ipsperre (ip, time) VALUES ('$ip', '$jetzt')") or die(mysql_error());

    // Ausgabe
}
} else {
    echo "Du mußt warten";
}
?>

vielleicht könnt ihr mir eben weiter helfen.

[TobiaZ]

Vielleicht sagst du uns erst mal, was für ein Problem es gibt!!!

[Trexx]

Parse error: parse error in /home/www/web64/html/neu/send2.php on line 27

[TobiaZ]

Dann markiere doch bitte die Zeile.

[TobiaZ]

Und überprüf mal bitte die anzahl deiner { und }

[Trexx]

ALso hier sind die Zeilen von 21 -29

 <? php 
$pruef=mysql_query("SELECT ip FROM ipsperre WHERE ip='$ip'") or die(mysql_error());
if(@mysql_fetch_array($pruef)) {
    mysql_query("INSERT INTO ipsperre (ip, time) VALUES ('$ip', '$jetzt')") or die(mysql_error());

    // Ausgabe
}
}else {
    echo "Du mußt warten";
} 

?>

Und ich habe ein Klammer zuviel nur wenn ich sie wegnehme springt er direkt zur else :-)

[Abraxax]

Original geschrieben von Trexx
Und ich habe ein Klammer zuviel [...]

die aber weg muss, wenn du den fehler beseitigen willst.

Original geschrieben von Trexx
[...] nur wenn ich sie wegnehme springt er direkt zur else :-)

das liegt wohl daran, dass das if() nicht erfüllt ist.

denk doch mal nach, was du eigentlich machst/willst(/kannst?) !

[TobiaZ]

Und ich habe ein Klammer zuviel nur wenn ich sie wegnehme springt er direkt zur else :-)

denk mal drüber nach. vielleicht hast du auch ne if() anweisung zu wenig. Ich weiß es nicht, weil ich nicht nachvollzogen hab, was das script macht.

Aber ich denke das bekommst du jetzt hin.

[marc75]

//denk mal über deine ifanweisung nach, oder überlege mal was mysql_fertch_array() zurückgibt wenn es was findet und wenn nicht.
if(@mysql_fetch_array($pruef)) {
    mysql_query("INSERT INTO ipsperre (ip, time) VALUES ('$ip', '$jetzt')") or die(mysql_error());

    // Ausgabe
}


//eine if mit geschweiften klammern sieht immer so aus:
if()
{}
//und nicht so
if()
{}} 

[TobiaZ]

welch weise worte

[overandover]

altes teil

function getip() {
// Proxys
if(getenv("HTTP_X_FORWARDED_FOR"))
$ip = getenv("HTTP_X_FORWARDED_FOR");
else
//Standard
$ip = getenv("REMOTE_ADDR");
return $ip;
}
}

NEU:

function getip() {
// Proxys
if(getenv("HTTP_X_FORWARDED_FOR"){
$ip = getenv("HTTP_X_FORWARDED_FOR");
}else{
//Standard
$ip = getenv("REMOTE_ADDR");
return $ip;
}

[TobiaZ]

benutz doch PHP-Tags...