Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> [MySQL][PHP] Rejestracja
Deotho
post
Post #1





Grupa: Zarejestrowani
Postów: 79
Pomógł: 0
Dołączył: 2.05.2012
Skąd: Hogwart

Ostrzeżenie: (0%)
----- 

[PHP] pobierz, plaintext 
  1. <?php
  2. ob_start();
  3. session_start();
  4. ?>
  5.  
  6. <?php include('templates/header.php'); ?>
  7.  
  8. <?php
  9. include ("config/db.php");
  10.  
  11. //Flirtujemy dane.
  12.      $login = addslashes($_POST['login']);
  13.      $haslo = addslashes($_POST['haslo']);
  14.      $email = addslashes($_POST['email']);
  15.  
  16. //Sprawdzamy czy użytkownik podał poprawne dane.
  17.      if (isset($_POST['submit'])) {
  18. 	     if (preg_match("/^[a-zA-Z0-9_ ]+$/" , $login)) {
  19.              } else {
  20.                  echo "Nazwa użytkownika zawiera niedozwolone znaki!";
  21.              }
  22. 	     if (preg_match("/^[a-zA-Z0-9_ ]+$/" , $haslo)) {
  23.              } else {
  24.                  echo "Hasło zawiera niedozwolone znaki!";
  25.              }
  26. 	     if (!eregi("^[_.0-9a-z-]+@([0-9a-z][0-9a-z-]+.)+[a-z]{2,4}$",$_POST["userEmail"])) {
  27.              } else {
  28.                  echo "Email zawiera niedozwolone znaki!";
  29.              }
  30. 		 if (empty($login)) die ('Pole z nazwą użytkonika nie może być puste!');
  31. 		 if (empty($haslo)) die ('Pole z haslem nie może być puste!');
  32. 		 if (empty($email)) die ('Pole z emailem nie może być puste!');
  33.  
  34. 		 $db = mysql_query("SELECT * FROM users WHERE u_name='$_POST[login]' ");
  35.          if (!$_POST['login'] OR !$_POST['haslo'] OR !$_POST['email']) {
  36.                  print "Nie wypełniono wszystkich pól formularza.";
  37.          if (mysql_num_rows($db)==1) {
  38.                  print "Podana nazwa użytkownika jest już zajęta."; }
  39. 		 else {
  40.            $_POST["haslo"]=SHA1($_POST["haslo"]);
  41.            $actCode=str_shuffle("qwertyuiopasdfghjklzxcvbnm1234567890");
  42.            $headers="MIME-Version: 1.0\r\n";
  43.            $headers.="Content-type: text/html; charset=iso-8859-2\r\n";
  44.            $content="<html><head><title>Aktywacja Konta</title</head>
  45.                 <body>
  46.                 Aby aktywować konto musisz kliknąć poniższy link aktywujący:<br>
  47.                 <a href=\"http://twojastrona.com/register.php?active=".$actCode."\"> <a href="http://twojastrona.com/register.php?active=&quot;.$actCode.&quot;" target="_blank">http://twojastrona.com/register.php?active...;actCode."</a> </a><br></body></html>";
  48.                 mail($_POST["email"], "Link Aktywacyjny", $content, $headers);
  49.            mysql_query("INSERT INTO users (login, haslo, email, active_key) VALUES('$_POST[login]', '$_POST[haslo]', '$_POST[email]', '$actCode')");
  50.  print"Na podany adres emailowy został wysłany klucz aktywacyjny.";
  51.         }
  52. }
  53. elseif($_GET["active"])
  54. {
  55.    mysql_query("UPDATE users SET active=1 WHERE active_key='$_GET[active]' ");
  56.    if(mysql_affected_rows()==1)
  57.         {
  58.                  print"Twoje konto zostało aktywowane.";
  59.          }
  60.          else
  61.          {
  62.                   print"Podany kod aktywacyjny jest nieprawidłowy.";
  63.           }
  64.      }
  65. }
  66. ?>
  67.  
  68. <form action="register.php" method="post">
  69. <input type="hidden" name="submit" value="true">
  70. Nazwa <input type="text" name="login" maxlength="20"><br>
  71. Hasło <input type="password" name="haslo" maxlength="30"><br>
  72. Podaj E-Mail <input type="text" name="email" maxlength="64"><br>
  73. <input type="submit" value="Rejestracja">
  74. </form>
  75.  
  76. <?php include('templates/footer.php'); ?>
[PHP] pobierz, plaintext


Po kliknięciu na Rejestracja wszystko jest ok, ale nie tworzy konta w bazie.
Dodatkowo prosiłbym o ocenę kodu - to mój pierwszy.

Ten post edytował Deotho 13.05.2012, 16:54:24
Go to the top of the page
+Quote Post
Lwik
post
Post #2





Grupa: Zarejestrowani
Postów: 39
Pomógł: 8
Dołączył: 11.05.2012

Ostrzeżenie: (0%)
-----

Przejrzałem wybiórczo bo napaćkane strasznie, mnie osobiściem mieszanie php z HTML'em strasznie denerwuje..
przy zapytaniu insertującym 
[PHP] pobierz, plaintext 
  1. mysql_query(Tresc zapytania) or die(mysql_error());
[PHP] pobierz, plaintext

Przeanalizuj to co Ci zwróci błąd. 
[PHP] pobierz, plaintext 
  1. <?php
  2. ob_start();
  3. session_start();
  4. ?>
  5.  
  6. <?php include('templates/header.php'); ?>
  7.  
  8. <?php
  9. include ("config/db.php");
[PHP] pobierz, plaintext

Po co tak?
[PHP] pobierz, plaintext 
  1. <?php
  2. ob_start();
  3. session_start();
  4. include_once('templates/header.php');
  5. include_once("config/db.php");
[PHP] pobierz, plaintext


PS: Na start możesz poczytać o MYSQLi i PDO.

Ten post edytował Lwik 13.05.2012, 17:05:08
Go to the top of the page
+Quote Post
Niktoś
post
Post #3





Grupa: Zarejestrowani
Postów: 1 195
Pomógł: 109
Dołączył: 3.11.2011

Ostrzeżenie: (10%)
X----

Cytat
$_POST["haslo"]=SHA1($_POST["haslo"]);
-To jest błędne, gdyż nie nadpiszesz zmiennej z tablicy $_POST, ani jej nie zmodyfikujesz.
Powinno być:
$haslo=SHA1($_POST["haslo"]); -czyli powinieneś utworzyć zmienną i przypisać do niej jakąś wartość $_POST.

Ten post edytował Niktoś 13.05.2012, 17:06:11
Go to the top of the page
+Quote Post
Kokoszz
post
Post #4





Grupa: Zarejestrowani
Postów: 7
Pomógł: 0
Dołączył: 28.03.2012

Ostrzeżenie: (0%)
-----

Cytat
Przejrzałem wybiórczo bo napaćkane strasznie, mnie osobiściem mieszanie php z HTML'em strasznie denerwuje..

a że wtrącę się w temat offtopickiem lekkim - jak to wg Ciebie powinno wyglądać? smile.gif

W ten sposób?
[PHP] pobierz, plaintext 
  1. <html>
  2. //kod strony
  3. </html>
  4. <?php
  5. // kod php
  6. ?>
[PHP] pobierz, plaintext


czy raczej, osobno plik html i osobno plik php?
Go to the top of the page
+Quote Post
Deotho
post
Post #5





Grupa: Zarejestrowani
Postów: 79
Pomógł: 0
Dołączył: 2.05.2012
Skąd: Hogwart

Ostrzeżenie: (0%)
-----

Cytat
Notice: Undefined index: userEmail in D:\Program Files\WebServ\httpd\register.php on line 24
Notice: Undefined index: active in D:\Program Files\WebServ\httpd\register.php on line 52


To wyskakuje po dodaniu kodu:
[PHP] pobierz, plaintext 
  1. error_reporting(E_ALL); 
  2. ini_set('display_errors','1');
[PHP] pobierz, plaintext


Wiem, że funkcja email nie będzie działała na localhost, ale konto i tak powinno tworzyć.

Całość koduje po edycji:
[PHP] pobierz, plaintext 
  1. <?php
  2. error_reporting(E_ALL); 
  3. ini_set('display_errors','1');
  4. ob_start();
  5. session_start();
  6. include_once('templates/header.php');
  7. include_once("config/db.php");
  8.  
  9. //Flirtujemy dane.
  10.      $login = addslashes($_POST['login']);
  11.      $haslo = md5($_POST['haslo']);
  12.      $email = addslashes($_POST['email']);
  13.  
  14. //Sprawdzamy czy użytkownik podał poprawne dane.
  15.      if (isset($_POST['submit'])) {
  16. 	     if (preg_match("/^[a-zA-Z0-9_ ]+$/" , $login)) {
  17.              } else {
  18.                  echo "Nazwa użytkownika zawiera niedozwolone znaki!";
  19.              }
  20. 	     if (preg_match("/^[a-zA-Z0-9_ ]+$/" , $haslo)) {
  21.              } else {
  22.                  echo "Hasło zawiera niedozwolone znaki!";
  23.              }
  24. 	     if (!eregi("^[_.0-9a-z-]+@([0-9a-z][0-9a-z-]+.)+[a-z]{2,4}$",$_POST["userEmail"])) {
  25.              } else {
  26.                  echo "Email zawiera niedozwolone znaki!";
  27.              }
  28. 		 if (empty($login)) die ('Pole z nazwą użytkonika nie może być puste!');
  29. 		 if (empty($haslo)) die ('Pole z haslem nie może być puste!');
  30. 		 if (empty($email)) die ('Pole z emailem nie może być puste!');
  31.  
  32. 		 $db = mysql_query("SELECT * FROM users WHERE u_name='$_POST[login]' ");
  33.          if (!$_POST['login'] OR !$_POST['haslo'] OR !$_POST['email']) {
  34.                  print "Nie wypełniono wszystkich pól formularza.";
  35.          if (mysql_num_rows($db)==1) {
  36.                  print "Podana nazwa użytkownika jest już zajęta."; }
  37. 		 else {
  38.            $_POST["haslo"]=md5($_POST["haslo"]);
  39.            $actCode=str_shuffle("qwertyuiopasdfghjklzxcvbnm1234567890");
  40.            $headers="MIME-Version: 1.0\r\n";
  41.            $headers.="Content-type: text/html; charset=iso-8859-2\r\n";
  42.            $content="<html><head><title>Aktywacja Konta</title</head>
  43.                 <body>
  44.                 Aby aktywować konto musisz kliknąć poniższy link aktywujący:<br>
  45.                 <a href=\"http://twojastrona.com/register.php?active=".$actCode."\"> <a href="http://twojastrona.com/register.php?active=&quot;.$actCode.&quot;" target="_blank">http://twojastrona.com/register.php?active...;actCode."</a> </a><br></body></html>";
  46.                 mail($_POST["email"], "Link Aktywacyjny", $content, $headers);
  47.            mysql_query("INSERT INTO users (login, haslo, email, active_key) VALUES('$_POST[login]', '$_POST[haslo]', '$_POST[email]', '$actCode')");
  48. 		   mysql_query("INSERT INTO users (login, haslo, email, active_key) VALUES('$_POST[login]', '$_POST[haslo]', '$_POST[email]', '$actCode')") or die('BŁĄD:'.mysql_error());
  49.  print"Na podany adres emailowy został wysłany klucz aktywacyjny.";
  50.         }
  51. }
  52. elseif($_GET["active"])
  53. {
  54.    mysql_query("UPDATE users SET active=1 WHERE active_key='$_GET[active]' ");
  55.    if(mysql_affected_rows()==1)
  56.         {
  57.                  print"Twoje konto zostało aktywowane.";
  58.          }
  59.          else
  60.          {
  61.                   print"Podany kod aktywacyjny jest nieprawidłowy.";
  62.           }
  63.      }
  64. }
  65. ?>
  66.  
  67. <form action="register.php" method="post">
  68. <input type="hidden" name="submit" value="true">
  69. Nazwa <input type="text" name="login" maxlength="20"><br>
  70. Hasło <input type="password" name="haslo" maxlength="30"><br>
  71. Podaj E-Mail <input type="text" name="email" maxlength="64"><br>
  72. <input type="submit" value="Rejestracja">
  73. </form>
  74.  
  75. <?php include_once('templates/footer.php'); ?>
[PHP] pobierz, plaintext


Nie wyskakuje error, a co do mieszania HTML i PHP to HTML będzie potem w oddzielnym pliku.
Go to the top of the page
+Quote Post
Lwik
post
Post #6





Grupa: Zarejestrowani
Postów: 39
Pomógł: 8
Dołączył: 11.05.2012

Ostrzeżenie: (0%)
-----

Cytat(Kokoszz @ 13.05.2012, 18:10:36 ) *
a że wtrącę się w temat offtopickiem lekkim - jak to wg Ciebie powinno wyglądać? smile.gif

W ten sposób?
[PHP] pobierz, plaintext 
  1. <html>
  2. //kod strony
  3. </html>
  4. <?php
  5. // kod php
  6. ?>
[PHP] pobierz, plaintext


czy raczej, osobno plik html i osobno plik php?

raczej opcja druga, ew jakiś system template'ów, gdzie będziemy wyświetlać php'owe szczątki. : )
[PHP] pobierz, plaintext 
  1. $login = addslashes($_POST['login']);
  2. $haslo = md5($_POST['haslo']);
  3. $email = addslashes($_POST['email']);
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
  1. mysql_query("INSERT INTO users (login, haslo, email, active_key) VALUES('$_POST[login]', '$_POST[haslo]', '$_POST[email]', '$actCode')") or die('BŁĄD:'.mysql_error());
[PHP] pobierz, plaintext

albo $_POST albo zmienna.

Ten post edytował Lwik 13.05.2012, 17:21:02
Go to the top of the page
+Quote Post
Niktoś
post
Post #7





Grupa: Zarejestrowani
Postów: 1 195
Pomógł: 109
Dołączył: 3.11.2011

Ostrzeżenie: (10%)
X----

Struktura html ogólnie źle zaprojektowana.Przy planowanym przerabianiu tego skryptu musisz zwrócić uwagę jaką strukturę html będzie Ci generował skrypt.Na razie to us ciebie wygląda tak:
[HTML] pobierz, plaintext 
  1. <html>
  2. <head>
  3. </head>
  4. <body>
  5. </body>
  6. </html>
  7. <form>
  8. </form>
[HTML] pobierz, plaintext

no i przy okazji zauważyłem dobry błąd u ciebie:

Cytat
$content="<html><head><title>Aktywacja Konta</title</head>
powinno być:

[HTML] pobierz, plaintext 
  1. $content="<html><head><title>Aktywacja Konta</title></head>
[HTML] pobierz, plaintext



Można robić to w ten sposób jak Ty to robisz, czyli mieszać html z PHP, aby dynamicznie(w locie) generować tagi w zależności od potrzeb aplikacji ,ale trzeba robić to z głową i zgodnie z normami html.

Ten post edytował Niktoś 13.05.2012, 17:25:27
Go to the top of the page
+Quote Post
Deotho
post
Post #8





Grupa: Zarejestrowani
Postów: 79
Pomógł: 0
Dołączył: 2.05.2012
Skąd: Hogwart

Ostrzeżenie: (0%)
-----

Dobra poprawki:
[PHP] pobierz, plaintext 
  1. <?php
  2. error_reporting(E_ALL); 
  3. ini_set('display_errors','1');
  4. ob_start();
  5. session_start();
  6. include_once('templates/header.php');
  7. include_once("config/db.php");
  8.  
  9. //Flirtujemy dane.
  10.      $login = addslashes($_POST['login']);
  11.      $haslo = md5($_POST['haslo']);
  12.      $email = addslashes($_POST['email']);
  13.  
  14. //Sprawdzamy czy użytkownik podał poprawne dane.
  15.      if (isset($_POST['submit'])) {
  16. 	     if (preg_match("/^[a-zA-Z0-9_ ]+$/" , $login)) {
  17.              } else {
  18.                  echo "Nazwa użytkownika zawiera niedozwolone znaki!";
  19.              }
  20. 	     if (preg_match("/^[a-zA-Z0-9_ ]+$/" , $haslo)) {
  21.              } else {
  22.                  echo "Hasło zawiera niedozwolone znaki!";
  23.              }
  24. 	     if (!eregi("^[_.0-9a-z-]+@([0-9a-z][0-9a-z-]+.)+[a-z]{2,4}$",$_POST["email"])) {
  25.              } else {
  26.                  echo "Email zawiera niedozwolone znaki!";
  27.              }
  28. 		 if (empty($login)) die ('Pole z nazwą użytkonika nie może być puste!');
  29. 		 if (empty($haslo)) die ('Pole z haslem nie może być puste!');
  30. 		 if (empty($email)) die ('Pole z emailem nie może być puste!');
  31.  
  32. 		 $db = mysql_query("SELECT * FROM users WHERE u_name='$_POST[login]' ");
  33.          if (!$_POST['login'] OR !$_POST['haslo'] OR !$_POST['email']) {
  34.                  print "Nie wypełniono wszystkich pól formularza.";
  35.          if (mysql_num_rows($db)==1) {
  36.                  print "Podana nazwa użytkownika jest już zajęta."; }
  37. 		 else {
  38.            $_POST["haslo"]=md5($_POST["haslo"]);
  39.            $actCode=str_shuffle("qwertyuiopasdfghjklzxcvbnm1234567890");
  40.            $headers="MIME-Version: 1.0\r\n";
  41.            $headers.="Content-type: text/html; charset=iso-8859-2\r\n";
  42.            $content="<html><head><title>Aktywacja Konta</title></head>
  43.                 <body>
  44.                 Aby aktywować konto musisz kliknąć poniższy link aktywujący:<br>
  45.                 <a href=\"http://twojastrona.com/register.php?active=".$actCode."\"> <a href="http://twojastrona.com/register.php?active=&quot;.$actCode.&quot;" target="_blank">http://twojastrona.com/register.php?active...;actCode."</a> </a><br></body></html>";
  46.                 mail($_POST["email"], "Link Aktywacyjny", $content, $headers);
  47.            mysql_query("INSERT INTO users (login, haslo, email, active_key) VALUES('$login', '$haslo', '$email', '$actCode')");
  48. 		   mysql_query("INSERT INTO users (login, haslo, email, active_key) VALUES('$login', '$haslo', '$email', '$actCode')") or die('BŁĄD:'.mysql_error());
  49.  print"Na podany adres emailowy został wysłany klucz aktywacyjny.";
  50.         }
  51. } else 
  52. if($_GET["active"]) {
  53.    mysql_query("UPDATE users SET active=1 WHERE active_key='$_GET[active]' ");
  54.    if (mysql_affected_rows()==1) {
  55.                  print"Twoje konto zostało aktywowane.";
  56.          } else {
  57.                   print"Podany kod aktywacyjny jest nieprawidłowy.";
  58.           }
  59.      }
  60. }
  61. ?>
  62.  
  63. <form action="register.php" method="post">
  64. <input type="hidden" name="submit" value="true">
  65. Nazwa <input type="text" name="login" maxlength="20"><br>
  66. Hasło <input type="password" name="haslo" maxlength="30"><br>
  67. Podaj E-Mail <input type="text" name="email" maxlength="64"><br>
  68. <input type="submit" value="Rejestracja">
  69. </form>
  70.  
  71. <?php include_once('templates/footer.php'); ?>
[PHP] pobierz, plaintext


Z błędem numer jeden się uporałem po prostu źle wpisana zmienna = brak sprawdzenia emailu.
Teraz sprawdza email, ale gdy wpiszę test@interia.pl wyskakuje, że znaki nieprawidłowe, a @ musi być.
Co do mysql_error poprawiłem na zmienne wszędzie i nadal nie wyskakuje nic.
Go to the top of the page
+Quote Post
Niktoś
post
Post #9





Grupa: Zarejestrowani
Postów: 1 195
Pomógł: 109
Dołączył: 3.11.2011

Ostrzeżenie: (10%)
X----

<a href=\"http://twojastrona.com/register.php?active=".$actCode."\"> <a href="http://twojastrona.com/register.php?active=&quot;.$actCode.&quot;" target="_blank">http://twojastrona.com/register.php?active...;actCode."</a> </a><br></body></html>";

Zagnieżdżanie a href w a href?Doprawdy nie mam pojęcia czy taki odnośnik zadziała w ogóle.Pomijając usuń te tagi co pogrubione i dodaj je do liniki:

[PHP] pobierz, plaintext 
  1. <?php include_once('templates/footer.php'); 
  2.  $content.="</body></html>";
  3. ?>
[PHP] pobierz, plaintext

zamykając tym samym strukturę html.

Ten post edytował Niktoś 13.05.2012, 17:45:02
Go to the top of the page
+Quote Post
Deotho
post
Post #10





Grupa: Zarejestrowani
Postów: 79
Pomógł: 0
Dołączył: 2.05.2012
Skąd: Hogwart

Ostrzeżenie: (0%)
-----

Nic to nie dało, a tylko wyskoczył błąd z content. 
[PHP] pobierz, plaintext 
  1. <?php include_once('templates/footer.php'); 
  2. $content.="</body></html>";
  3. ?>
[PHP] pobierz, plaintext


Notice: Undefined index: active in D:\Program Files\WebServ\httpd\register.php on line 52
Notice: Undefined variable: content in D:\Program Files\WebServ\httpd\register.php on line 72

[PHP] pobierz, plaintext 
  1. <?php
  2. error_reporting(E_ALL); 
  3. ini_set('display_errors','1');
  4. ob_start();
  5. session_start();
  6. include_once('templates/header.php');
  7. include_once("config/db.php");
  8.  
  9. //Flirtujemy dane.
  10.      $login = addslashes($_POST['login']);
  11.      $haslo = md5($_POST['haslo']);
  12.      $email = addslashes($_POST['email']);
  13.  
  14. //Sprawdzamy czy użytkownik podał poprawne dane.
  15.      if (isset($_POST['submit'])) {
  16. 	     if (preg_match("/^[a-zA-Z0-9_ ]+$/" , $login)) {
  17.              } else {
  18.                  echo "Nazwa użytkownika zawiera niedozwolone znaki!";
  19.              }
  20. 	     if (preg_match("/^[a-zA-Z0-9_ ]+$/" , $haslo)) {
  21.              } else {
  22.                  echo "Hasło zawiera niedozwolone znaki!";
  23.              }
  24. 	     if (!eregi("^[_.0-9a-z-]+@([0-9a-z][0-9a-z-]+.)+[a-z]{2,4}$",$_POST["email"])) {
  25.              } else {
  26.                  echo "Email zawiera niedozwolone znaki!";
  27.              }
  28. 		 if (empty($login)) die ('Pole z nazwą użytkonika nie może być puste!');
  29. 		 if (empty($haslo)) die ('Pole z haslem nie może być puste!');
  30. 		 if (empty($email)) die ('Pole z emailem nie może być puste!');
  31.  
  32. 		 $db = mysql_query("SELECT * FROM users WHERE u_name='$_POST[login]' ");
  33.          if (!$_POST['login'] OR !$_POST['haslo'] OR !$_POST['email']) {
  34.                  print "Nie wypełniono wszystkich pól formularza.";
  35.          if (mysql_num_rows($db)==1) {
  36.                  print "Podana nazwa użytkownika jest już zajęta."; }
  37. 		 else {
  38.            $_POST["haslo"]=md5($_POST["haslo"]);
  39.            $actCode=str_shuffle("qwertyuiopasdfghjklzxcvbnm1234567890");
  40.            $headers="MIME-Version: 1.0\r\n";
  41.            $headers.="Content-type: text/html; charset=iso-8859-2\r\n";
  42.            $content="<html><head><title>Aktywacja Konta</title></head>
  43.                 <body>
  44.                 Aby aktywować konto musisz kliknąć poniższy link aktywujący:<br>
  45.                 <a href=\"http://twojastrona.com/register.php?active=".$actCode."\"> <a href="http://twojastrona.com/register.php?active=&quot;.$actCode.&quot;" target="_blank">http://twojastrona.com/register.php?active...;actCode."</a> </a>";
  46.                 mail($_POST["email"], "Link Aktywacyjny", $content, $headers);
  47.            mysql_query("INSERT INTO users (login, haslo, email, active_key) VALUES('$login', '$haslo', '$email', '$actCode')");
  48. 		   mysql_query("INSERT INTO users (login, haslo, email, active_key) VALUES('$login', '$haslo', '$email', '$actCode')") or die('BŁĄD:'.mysql_error());
  49.  print"Na podany adres emailowy został wysłany klucz aktywacyjny.";
  50.         }
  51. } else 
  52. if($_GET["active"]) {
  53.    mysql_query("UPDATE users SET active=1 WHERE active_key='$_GET[active]' ");
  54.    if (mysql_affected_rows()==1) {
  55.                  print"Twoje konto zostało aktywowane.";
  56.          } else {
  57.                   print"Podany kod aktywacyjny jest nieprawidłowy.";
  58.           }
  59.      }
  60. }
  61. ?>
  62.  
  63. <form action="register.php" method="post">
  64. <input type="hidden" name="submit" value="true">
  65. Nazwa <input type="text" name="login" maxlength="20"><br>
  66. Hasło <input type="password" name="haslo" maxlength="30"><br>
  67. Podaj E-Mail <input type="text" name="email" maxlength="64"><br>
  68. <input type="submit" value="Rejestracja">
  69. </form>
  70.  
  71. <?php include_once('templates/footer.php'); 
  72. $content.="</body></html>";
  73. ?>
[PHP] pobierz, plaintext


Ten post edytował Deotho 13.05.2012, 17:50:18
Go to the top of the page
+Quote Post
Lwik
post
Post #11





Grupa: Zarejestrowani
Postów: 39
Pomógł: 8
Dołączył: 11.05.2012

Ostrzeżenie: (0%)
-----

Powiem tak.. raz używasz jednej zmiennej raz drugiej.. IFy masz cholernie dziwnie ułożone.
[PHP] pobierz, plaintext 
  1. $db = mysql_query("SELECT * FROM users WHERE u_name='$_POST[login]' ");
[PHP] pobierz, plaintext

u_name?
później
[PHP] pobierz, plaintext 
  1. mysql_query("INSERT INTO users (login, haslo, email, active_key) VALUES('$login', '$haslo', '$email', '$actCode')") or die('BŁĄD:'.mysql_error());
[PHP] pobierz, plaintext


Tak szybko w edytorze forumowym:
[PHP] pobierz, plaintext 
  1. if (isset($_POST['login'])) {
  2. $login = addslashes($_POST['login']);
  3. $haslo = md5($_POST['haslo']);
  4. $email = addslashes($_POST['email']);
  5. if (preg_match("/^[a-zA-Z0-9_ ]+$/" , $login)) {
  6. } else {
  7. echo "Nazwa użytkownika zawiera niedozwolone znaki!";
  8. }
  9. if (preg_match("/^[a-zA-Z0-9_ ]+$/" , $haslo)) {
  10. } else {
  11. echo "Hasło zawiera niedozwolone znaki!";
  12. }
  13.  
  14. if (empty($login)) die ('Pole z nazwą użytkonika nie może być puste!');
  15. if (empty($haslo)) die ('Pole z haslem nie może być puste!');
  16. if (empty($email)) die ('Pole z emailem nie może być puste!');
  17.  
  18. $db = mysql_query("SELECT * FROM users WHERE login=' ".$login."' ") or die(mysql_error());
  19. if (!$_POST['login'] || !$_POST['haslo'] || !$_POST['email']) die ("Nie wypełniono wszystkich pól formularza.");
  20. if (mysql_num_rows($db)==1) {
  21. print "Podana nazwa użytkownika jest już zajęta."; }
  22. else {
  23. $actCode=str_shuffle("qwertyuiopasdfghjklzxcvbnm1234567890");
  24.  
  25. mysql_query("INSERT INTO users (`login`, `haslo`, `email`, `active_key`) VALUES('$login', '$haslo', '$email', '$actCode')") or die('BŁĄD:'.mysql_error());
  26. echo"Na podany adres emailowy został wysłany klucz aktywacyjny.";
  27. }
  28.  
  29. }
  30.  
  31. ?>
  32.  
  33. <form action="index.php" method="post">
  34. Nazwa <input type="text" name="login" maxlength="20"><br>
  35. Hasło <input type="password" name="haslo" maxlength="30"><br>
  36. Podaj E-Mail <input type="text" name="email" maxlength="64"><br>
  37. <input type="submit" value="Rejestracja">
  38. </form>
  39.  
[PHP] pobierz, plaintext


@niktos, a może domyka to w pliku footer'a? Nigdy nic nie wiadomo, więc ja byłbym daleki od takich poprawek, skoncentrujmy się na błędzie który znalazł.

Ten post edytował Lwik 13.05.2012, 17:51:11
Go to the top of the page
+Quote Post
Deotho
post
Post #12





Grupa: Zarejestrowani
Postów: 79
Pomógł: 0
Dołączył: 2.05.2012
Skąd: Hogwart

Ostrzeżenie: (0%)
-----

Faktycznie Lawik nie zauwazyłem tego.

Wszystko już działa, ale duplikuje mi wpisy w bazie jak wpisze takie same dane kilka razy.
Można jakoś zrobić, aby nie można było na ten sam login oraz email się zarejestrować?

Ten post edytował Deotho 13.05.2012, 18:02:16
Go to the top of the page
+Quote Post
Niktoś
post
Post #13





Grupa: Zarejestrowani
Postów: 1 195
Pomógł: 109
Dołączył: 3.11.2011

Ostrzeżenie: (10%)
X----

Ok ,drobna poprawka ,powinno działać.
[PHP] pobierz, plaintext 
  1. <?php 
  2.  
  3. include_once('templates/footer.php');
  4. if (isset($content)){
  5. $content.="</body></html>";
  6. }
  7.  ?>
[PHP] pobierz, plaintext


Ten post edytował Niktoś 13.05.2012, 18:09:51
Go to the top of the page
+Quote Post
Lwik
post
Post #14





Grupa: Zarejestrowani
Postów: 39
Pomógł: 8
Dołączył: 11.05.2012

Ostrzeżenie: (0%)
-----

Odpaliłem sobie z ciekawości to na localhoście tworząc przykładową bazę:
[SQL] pobierz, plaintext 
  1. CREATE TABLE IF NOT EXISTS `users` (
  2.   `id` int(11) NOT NULL AUTO_INCREMENT,
  3.   `login` varchar(20) NOT NULL,
  4.   `haslo` varchar(64) NOT NULL,
  5.   `email` varchar(25) NOT NULL,
  6.   `active_key` varchar(50) NOT NULL,
  7.   PRIMARY KEY (`id`)
  8. ) ENGINE=InnoDB  DEFAULT CHARSET=latin1 AUTO_INCREMENT=2 ;
[SQL] pobierz, plaintext

działa poprawnie.
Co do unikalności rekordów
[PHP] pobierz, plaintext 
  1. $db = mysql_query("SELECT * FROM users WHERE login=' ".$login."' or email = '".$email."' ") or die(mysql_error());
[PHP] pobierz, plaintext

i jakiś klucz w mysql.
Go to the top of the page
+Quote Post
Deotho
post
Post #15





Grupa: Zarejestrowani
Postów: 79
Pomógł: 0
Dołączył: 2.05.2012
Skąd: Hogwart

Ostrzeżenie: (0%)
-----

Wszystko działa jak należy. Dziękuje panowie.

Ten post edytował Deotho 13.05.2012, 18:07:34
Go to the top of the page
+Quote Post
« Następny starszy · Przedszkole · Następny nowszy »
 

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Standardowy · Przełącz na: Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Aktualny czas: 20.08.2025 - 13:31
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn