Ograniczenie rejestracji - błąd Opcje

[kiepski96]

To kod, który ma za zadanie limitować liczbę rejestracji.
Lecz nie działa, można zarejestrować się ile razy się chce .
Proszę o pomoc .

[PHP] pobierz, plaintext 
<?php
session_start();
if (!isset($_SESSION['username']))
{
   header('Location: ./login.php');
   exit;
}
?>
<?php
session_start();
$error_message = "";
$error_message1 = "";
$action = isset($_POST['action']) ? $_POST['action'] : '';
   $mysql_server = 'localhost';
$mysql_username = 'xxx';
$mysql_password = 'xxx';
$mysql_database = 'xxx';
   $mysql_table = 'nagroda1';
   $db = mysql_connect($mysql_server, $mysql_username, $mysql_password);
         mysql_select_db($mysql_database, $db);
 $sql = "SELECT id FROM ".$mysql_table;
 require_once('libs/recaptchalib.php');
$privatekey = "xxx";
$resp = recaptcha_check_answer ($privatekey,
                                $_SERVER["REMOTE_ADDR"],
                                $_POST["recaptcha_challenge_field"],
                                $_POST["recaptcha_response_field"]);
 
if (!$resp->is_valid) 
{
  $error_message = "Zły kod";
}
$result = mysql_query($sql, $db);
$nagroda1=mysql_num_rows($result); 
 
    if ($_SERVER['REQUEST_METHOD'] == 'POST' && $nagroda1 >= 2 )  
    {
 
    $error_message1 = "Brak nagrody";
    }  
 
 
     $success_page = './sukces.html';
 
   if ($action == 'signup')
   {
      $newusername = $_POST['username'];
      $newemail = $_POST['email'];
      $newfullname = $_POST['fullname'];
 
      if (!ereg("^[A-Za-z0-9_!@$]{1,50}$", $newusername))
      {
         $error_message = 'Popraw dane';
      }
      else
      if (!ereg("^[A-Za-z0-9_!@$.' &]{1,50}$", $newfullname))
      {
         $error_message = 'Popraw dane';
      }
      else
      if (!ereg("^.+@.+\..+$", $newemail))
      {
         $error_message = 'Popraw dane';
      }
      if (empty($error_message))
      {
         $db = mysql_connect($mysql_server, $mysql_username, $mysql_password);
         mysql_select_db($mysql_database, $db);
         $sql = "SELECT username FROM ".$mysql_table." WHERE username = '".$newusername."'";
         $result = mysql_query($sql, $db);
         if ($data = mysql_fetch_array($result))
         {
            $error_message = 'Nick zajęty, spróbuj inny.';
         }
      }
      if (empty($error_message))
      {
         $sql = "INSERT `".$mysql_table."` (`username`,`fullname`, `email`, `active`) VALUES ('$newusername', '$newfullname', '$newemail', 1)";
         $result = mysql_query($sql, $db);
         mysql_close($db);
         $mailto = $newemail;
         $subject = 'Potwierdzenie x';
         $message = 'xxx';
         $message .= "\r\nUsername: ";
         $message .= $newusername;
         $message .= "\r\n";
         $header  = "From: xxx"."\r\n";
         $header .= "Reply-To: xxx"."\r\n";
         $header .= "MIME-Version: 1.0"."\r\n";
         $header .= "Content-Type: text/plain; charset=utf-8"."\r\n";
         $header .= "Content-Transfer-Encoding: 8bit"."\r\n";
         $header .= "X-Mailer: PHP v".phpversion();
         mail($mailto, $subject, $message, $header);
         header('Location: '.$success_page);
         exit;
      }
   }
   ?>
[PHP] pobierz, plaintext 

[matx132]

z tego co widzę to rozumiem że jeśli nagród w tabeli nagroda1 jest więcej niż 1 to ma pisać że brak nagrody

[PHP] pobierz, plaintext 
if ($_SERVER['REQUEST_METHOD'] == 'POST' && $nagroda1 >= 2 )  
[PHP] pobierz, plaintext 

ale nadal nie rozumiem jak chcesz ograniczyć rejestrację ?
sprawdzasz tylko czy username jest zajęty oraz nie wiem po co dwa razy łączysz się z bazą jeśli wcześniejszego nie zamykasz połączenia.

Daj więcej informacji bo ogólnie nie ograniczasz tutaj nic poza tym iż nie może być w bazie taki sam nick

[kiepski96]

Jak będzie więcej niż 2 osoby zarejestrowane to aby wywalało błąd i nie dało się zarejestrować .
Wywalanie błędu już jest ,ale coś z tym zamknięciem rejestracji nie wychodzi.

[matx132]

może tak nie widzę abyś dodawał wpisów do tabeli nagroda1 więc tabela się nie zmienia a dopóki nie ma dwóch lub więcej wpisów skrypt będzie się wykonywał dalej.

i spróbuj zamiast

empty($error_message)

dać

$error_message!=""

może coś pomoże lecz jeśli tabela nagroda1 ma tylko jeden lub mniej wpisów to nic to nie da bo będzie wykonywany nadal skrypt

[kiepski96]

Jak zamiast :

empty($error_message)

daję

$error_message!=""

To wywala mi :

Parse error: syntax error, unexpected T_VARIABLE, expecting '(' in /home/restock/public_html/nagroda1.php on line 65

[greycoffey]

Pomyślcie, empty() używa się aby sprawdzić czy zmienna jest pusta lub nie istnieje - w tym wypadku nie istnieje.

[matx132]

@up zmienna istnieje jest deklarowana na początku

$error_message='';

co do if nie powinno być błędu jak jest

if (empty($error_message))

a zamiast tego dajesz

if ($error_message!='')

[kiepski96]

to teraz nawet jak nic nie uzupełnię to mi zalicza

[matx132]

to zrób inaczej
usuń
$error_message = ""
na początku skryptu i daj if(isset($error_message)) - sprawdza czy zmienna istnieje

[kiepski96]

Nadal można zarejestrować się parę razy,więcej niż dwa

[matx132]

a jak usuniesz w if $_SERVER['REQUEST_METHOD'] == 'POST' &&
aby zostało samo if($nagroda1>1) .....

jeśli będzie nadal działać wydrukuj sobie $nagroda1 lub w mysql_num_rows daj or die(mysql_error())

[kiepski96]

Ale po co wydrukować błąd , jak błędu nie ma ?
PS. Nie działa

Może się jeszcze ktoś wypowie ?

Matx,czy mógłbyś edytować kod ten co dałem w pierwszym poście ? Bo może coś źle robię.

Ten post edytował kiepski96 6.04.2011, 20:05:27

[Daiquiri]

Pamiętajcie, że forum to nie parser. Jeszcze jeden parse error i będę zmuszona zamknąć temat.

[matx132]

ja bym to tak zrobił:

[PHP] pobierz, plaintext 
<?php
session_start();
if (!isset($_SESSION['username']))
{
   header('Location: ./login.php');
   exit;
}
?>
<?php
session_start();
$error_message = "";
$error_message1 = "";
$action = isset($_POST['action']) ? $_POST['action'] : '';
   $mysql_server = 'localhost';
$mysql_username = 'xxx';
$mysql_password = 'xxx';
$mysql_database = 'xxx';
   $mysql_table = 'nagroda1';
   $db = mysql_connect($mysql_server, $mysql_username, $mysql_password);
         mysql_select_db($mysql_database, $db);
 $sql = "SELECT username FROM $mysql_table";
 require_once('libs/recaptchalib.php');
$privatekey = "xxx";
$resp = recaptcha_check_answer ($privatekey,
                                $_SERVER["REMOTE_ADDR"],
                                $_POST["recaptcha_challenge_field"],
                                $_POST["recaptcha_response_field"]);
 
if (!$resp->is_valid) 
{
  $error_message = "Zły kod";
}
$result = mysql_query($sql, $db);
$nagroda1=mysql_num_rows($result) or die(mysql_error()); 
 
    if ($nagroda1 > 1 )  
    {
 
    $error_message1 = "Brak nagrody";
    }  
else
 echo $nagroda1."<br/><br/>";
 
 
     $success_page = './sukces.html';
 
   if ($action == 'signup')
   {
      $newusername = $_POST['username'];
      $newemail = $_POST['email'];
      $newfullname = $_POST['fullname'];
 
      if (!ereg("^[A-Za-z0-9_!@$]{1,50}$", $newusername))
      {
         $error_message = 'Popraw dane';
      }
      else
      if (!ereg("^[A-Za-z0-9_!@$.' &]{1,50}$", $newfullname))
      {
         $error_message = 'Popraw dane';
      }
      else
      if (!ereg("^.+@.+\..+$", $newemail))
      {
         $error_message = 'Popraw dane';
      }
      if (empty($error_message))
      {
         $db = mysql_connect($mysql_server, $mysql_username, $mysql_password);
         mysql_select_db($mysql_database, $db);
         $sql = "SELECT username FROM ".$mysql_table." WHERE username = '".$newusername."'";
         $result = mysql_query($sql, $db);
         if ($data = mysql_fetch_array($result))
         {
            $error_message = 'Nick zajęty, spróbuj inny.';
         }
      }
      if (empty($error_message))
      {
         $sql = "INSERT `".$mysql_table."` (`username`,`fullname`, `email`, `active`) VALUES ('$newusername', '$newfullname', '$newemail', 1)";
         $result = mysql_query($sql, $db);
         mysql_close($db);
         $mailto = $newemail;
         $subject = 'Potwierdzenie x';
         $message = 'xxx';
         $message .= "\r\nUsername: ";
         $message .= $newusername;
         $message .= "\r\n";
         $header  = "From: xxx"."\r\n";
         $header .= "Reply-To: xxx"."\r\n";
         $header .= "MIME-Version: 1.0"."\r\n";
         $header .= "Content-Type: text/plain; charset=utf-8"."\r\n";
         $header .= "Content-Transfer-Encoding: 8bit"."\r\n";
         $header .= "X-Mailer: PHP v".phpversion();
         mail($mailto, $subject, $message, $header);
         header('Location: '.$success_page);
         exit;
      }
   }
   ?>
[PHP] pobierz, plaintext 

wtedy jeśli $nagroda1 nie jest większa od 1 to wydrukuje ją lub jeśli mysql zglosi bląd to go wydrukuje (jeśli nie ma die mysql_error... to php się nie zatrzymuje

[kiepski96]

Sprawdziłem i... nie działa :<

[matx132]

ha jestem ślepy dopiero teraz zauważyłem heh
to musi już dzialac:D

[PHP] pobierz, plaintext 
<?php
session_start();
if (!isset($_SESSION['username']))
{
   header('Location: ./login.php');
   exit;
}
?>
<?php
session_start();
$action = isset($_POST['action']) ? $_POST['action'] : '';
   $mysql_server = 'localhost';
$mysql_username = 'xxx';
$mysql_password = 'xxx';
$mysql_database = 'xxx';
   $mysql_table = 'nagroda1';
   $db = mysql_connect($mysql_server, $mysql_username, $mysql_password);
         mysql_select_db($mysql_database, $db);
 $sql = "SELECT username FROM $mysql_table";
 require_once('libs/recaptchalib.php');
$privatekey = "xxx";
$resp = recaptcha_check_answer ($privatekey,
                                $_SERVER["REMOTE_ADDR"],
                                $_POST["recaptcha_challenge_field"],
                                $_POST["recaptcha_response_field"]);
 
if (!$resp->is_valid) 
{
  $error_message = "Zły kod";
}
$result = mysql_query($sql, $db);
$nagroda1=mysql_num_rows($result) or die(mysql_error()); 
 
    if ($nagroda1 > 1 )  
    {
 
    $error_message1 = "Brak nagrody";
    }  
else
 echo $nagroda1."<br/><br/>";
 
 
     $success_page = './sukces.html';
 
   if ($action == 'signup')
   {
      $newusername = $_POST['username'];
      $newemail = $_POST['email'];
      $newfullname = $_POST['fullname'];
 
      if (!ereg("^[A-Za-z0-9_!@$]{1,50}$", $newusername))
      {
         $error_message = 'Popraw dane';
      }
      else
      if (!ereg("^[A-Za-z0-9_!@$.' &]{1,50}$", $newfullname))
      {
         $error_message = 'Popraw dane';
      }
      else
      if (!ereg("^.+@.+\..+$", $newemail))
      {
         $error_message = 'Popraw dane';
      }
      if (empty($error_message))
      {
         $db = mysql_connect($mysql_server, $mysql_username, $mysql_password);
         mysql_select_db($mysql_database, $db);
         $sql = "SELECT username FROM ".$mysql_table." WHERE username = '".$newusername."'";
         $result = mysql_query($sql, $db);
         if ($data = mysql_fetch_array($result))
         {
            $error_message = 'Nick zajęty, spróbuj inny.';
         }
      }
      if (isset($error_message)||isset($error_message1))
      {
         $sql = "INSERT `".$mysql_table."` (`username`,`fullname`, `email`, `active`) VALUES ('$newusername', '$newfullname', '$newemail', 1)";
         $result = mysql_query($sql, $db);
         mysql_close($db);
         $mailto = $newemail;
         $subject = 'Potwierdzenie x';
         $message = 'xxx';
         $message .= "\r\nUsername: ";
         $message .= $newusername;
         $message .= "\r\n";
         $header  = "From: xxx"."\r\n";
         $header .= "Reply-To: xxx"."\r\n";
         $header .= "MIME-Version: 1.0"."\r\n";
         $header .= "Content-Type: text/plain; charset=utf-8"."\r\n";
         $header .= "Content-Transfer-Encoding: 8bit"."\r\n";
         $header .= "X-Mailer: PHP v".phpversion();
         mail($mailto, $subject, $message, $header);
         header('Location: '.$success_page);
         exit;
      }
   }
   ?>
[PHP] pobierz, plaintext 

byłem ślepy i nie zauważyłem że zmienna to $error_message1
a Ty nigdzie jej nie sprawdzasz dlatego dołożyłem sprawdzanie czy istnieje taka zmienna

[kiepski96]

Coś dziwnego.
Uzupełniłem kod i wrzuciłem na serwer.
Raz zamówiłem nagrodę.
Lecz teraz nie chce mi się wyświetlić strona,po prostu jest biało.
Tak ma być ?

[matx132]

według kodu tak.
jeśli chcesz żeby coś się wyświetliło musisz dać w ostatni if else { .... }

[kiepski96]

Nie działa,
ale błąd może stoi w tabeli,bo w niej nie ma żadnego counta i nic .
Tabela jest ok ?

CREATE TABLE `nagroda1` (
`id` INT NOT NULL AUTO_INCREMENT PRIMARY KEY ,
`username` VARCHAR( 50 ) NOT NULL ,
`password` VARCHAR( 50 ) NOT NULL ,
`fullname` VARCHAR( 50 ) NOT NULL ,
`email` VARCHAR( 100 ) NOT NULL ,
`active` INT NOT NULL
)