Problem ze skryptem. Sesje. Opcje

[bleze]

Skrypt pownien nie wykonywać tego co jest w warunku ($message && flood($_SESSION['last_session_request'])==FALSE && $znaki<=500 && $znaki !== 0 && $wulgar==FALSE) jeśli skrypt zostanie wykonany więcej razy niż 1 w czasie 30 sekund (zabezpieczenie przed floodowaniem). Niestety tak się nie dzieje, nie wiem już co może być nie tak. Za każdym razem jak testuje w odstępie czasu 6-7 sekund to wywala mi komunikat "Komentarz dodano!" i komentarz zostaje dodany.

[PHP] pobierz, plaintext 
<?
@session_start();
function  flood() {
if($_SESSION['last_session_request'] > time() - 30){
 
  echo "<div style=\"color:red\">Aby dodać następny komentarz musisz odczekać 30 sekund!</div>";
  exit;
}
$_SESSION['last_session_request'] = time();
}
 
$id = $_GET[id];
$message=$_POST['message']; 
$name=$_POST['name'];
$message = htmlspecialchars($message, ENT_QUOTES);
$name = htmlspecialchars($name, ENT_QUOTES);
$message = str_replace("r\n", " ", $message);
$name = str_replace("r\n", " ", $name);
$message = str_replace("
", " ", $message);
$name = str_replace("
", " ", $name);
$message = str_replace("|", "&sl&", $message);
$name = str_replace("|", "&sl&", $name);
if ($name == FALSE) {$name="Anonim";}
 
include "../profilter.php";
 
$message = iconv("utf-8","ISO-8859-2",$message);
$name = iconv("utf-8","ISO-8859-2",$name);
 
$znaki = strlen($message);
 
if ($message && flood($_SESSION['last_session_request'])==FALSE && $znaki<=500 && $znaki !== 0 && $wulgar==FALSE)  
{ 
    $date = date("Y-m-d H:i");  
    $message = "$name|$date|$message\n";  
    #$fp = fopen ("comment/$id.comment", "a");  
    $fp = fopen ("comment/" . $id . ".comment", "a"); 
    fwrite ($fp, $message);  
    fclose ($fp); 
echo "<div style='color:green'>Komentarz dodano!</div>"; 
}
else {
 
    if ($znaki>=500) {echo "<div style='color:red'>Twój komentarz musi mieć mniej niż 500 znaków!</div>";}
 
    else if ($znaki==0) { echo "<div style='color:red'>Musisz wpisać jakiś tekst!</div>";}
 
    echo "<div style='color:red'>$wulgar</div>";
 
    echo "<FORM id=\"form\" method=\"post\" action=\"comments.php?id=".$id."\">  
<b>Imię/nick:</b><BR><INPUT name=\"name\" type=\"text\" size=\"25\" maxlength=\"15\"><BR>  
<DIV id=\"iloscZnakow\"><b>Twój komentarz:</b> (wpisanych znaków 0<script>document.write(parseInt(document.forms[0].opis.value.length))</SCRIPT>/500)</DIV>
<TEXTAREA onFocus=\"iloscZnakow(this)\" onBlur=\"skoncz(this)\" style=\"overflow:auto\" name=\"message\" rows=\"5\" cols=\"45\">$message</TEXTAREA>
</FORM><button onclick=\"SendRequestForm()\">Dodaj komentarz</button>";
}
?>
[PHP] pobierz, plaintext 

[legorek]

Na pierwszy rzut oka problem tkwi w:

[PHP] pobierz, plaintext 
<?php
if($_SESSION['last_session_request'] > time() - 30){
 
 echo "<div style=\"color:red\">Aby dodać następny komentarz musisz odczekać 30 sekund!</div>";
 exit;
}
?>
[PHP] pobierz, plaintext 

Zamień exit; na return true;

Ten post edytował legorek 20.01.2009, 15:19:10

[bleze]

próbowałem już tak, niestety nic to nie dało. gdzie indziej jeszcze może tkwić problem?

[erix]

[PHP] pobierz, plaintext 
<?php
if($_SESSION['last_session_request'] > time() - 30){
 
 echo "<div style=\"color:red\">Aby dodać następny komentarz musisz odczekać 30 sekund!</div>";
 exit;
}
$_SESSION['last_session_request'] = time();
?>
[PHP] pobierz, plaintext 

Za każdym żądaniem resetujesz czas, stąd problem.

[bleze]

już sobie poradziłem. Problem był związany z tworzeniem/utrzymywaniem sesji, dodałem do wszystkich stron z których wywołuję ten skrypt - @session_start(); i po problemie.
Natomiast zastanawia mnie jeszcze jedno. Dlaczego skoro ten warunek o którym pisałem w pierwszym poście nie jest spełniony przez flood($_SESSION['last_session_request'])==FALSE to nie wyświetla się ten formularz:

[PHP] pobierz, plaintext 
<?php
echo "<FORM id=\"form\" method=\"post\" action=\"comments.php?id=".$id."\">  
<b>Imię/nick:</b><BR><INPUT name=\"name\" type=\"text\" size=\"25\" maxlength=\"15\"><BR>  
<DIV id=\"iloscZnakow\"><b>Twój komentarz:</b> (wpisanych znaków 0<script>document.write(parseInt(document.forms[0].opis.value.length))</SCRIPT>/500)</DIV>
<TEXTAREA onFocus=\"iloscZnakow(this)\" onBlur=\"skoncz(this)\" style=\"overflow:auto\" name=\"message\" rows=\"5\" cols=\"45\">$message</TEXTAREA>
</FORM><button onclick=\"SendRequestForm()\">Dodaj komentarz</button>";
?>
[PHP] pobierz, plaintext 

Jeżeli zaś ten warunek nie spełnia się przez pozostałe np. $znaki<=500 czy $message==FALSE to formularz zostaje wyświetlony.

Ten post edytował bleze 20.01.2009, 17:21:13