Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [PHP] rejestracja
gigzorr
post 6.02.2010, 22:34:51
Post #1





Grupa: Zarejestrowani
Postów: 652
Pomógł: 47
Dołączył: 6.02.2010
Skąd: Radom

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

[PHP] pobierz, plaintext 
  1. <?php
  2. //data
  3. $fullname = strip_tags($_POST['fullname']);
  4. $username = strip_tags($_POST['username']);
  5. $password = strip_tags($_POST['password']);
  6. $repeatpassword = strip_tags($_POST['repeatpassword']);
  7. $date = date("Y-m-d");
  8.  
  9. if ($submit)
  10. {
  11.   //check for existance
  12.   if ($fullname&&$username&&$password&&$repeatpassword)
  13.   {
  14.     // encrypt password
  15.     $password = md5($password);
  16.     $repeatpassword = md5($repeatpassword);
  17.  
  18.     if ($password==$repeatpassword)
  19.     {
  20.       //check char lenght of username and fullname
  21.       if (strlen($username)>25||strlen($fullname)>25)
  22.       {
  23.         echo "username albo fullname jest za dlugie";
  24.       }
  25.       else
  26.       {
  27.  
  28.             //check password lenght
  29.             if(strlen($password)>25||strlen($password)<6)
  30.             {
  31.                 echo "Haslo musi byc pomiedzy 6 a 25 znakami!";
  32.             }
  33.             else
  34.             {
  35.                  //register info
  36.  
  37.                 echo "Success!!";
  38.  
  39.  
  40.             }
  41.  
  42.  
  43.       }
  44.  
  45.  
  46.  
  47.  
  48.     }
  49.     else
  50.         echo "Twoje haslo sie nie zgadza";
  51.  
  52.  
  53.   }
  54.   else
  55.       echo "<b>Prosze uzupelnic wszystkie pola!";
  56.  
  57.  
  58. }
  59.  
  60.  
  61. ?>
  62.  
  63. <html>
  64. <p>
  65. <form action='register.php'>
  66.        <table>
  67.             <tr>
  68.                 <td>
  69.                   Your full name:
  70.                   </td>
  71.                  <td>
  72.                   <input type='text' name='fullname'>
  73.                   </td>
  74.             </tr>
  75.     <td>
  76.        <tr>
  77.                 <td>
  78.                   Choose a username:
  79.                   </td>
  80.                  <td>
  81.                   <input type='text' name='fullname'>
  82.                   </td>
  83.             </tr>
  84.     <td>
  85.                   Choose a password:
  86.                   </td>
  87.                  <td>
  88.                   <input type='password' name='password'>
  89.                   </td>
  90.              </tr>
  91.             <br>
  92.     <td>
  93.                   Repeat your password:
  94.                   </td>
  95.                  <td>
  96.                   <input type='password' name='repeatpassword'>
  97.                   </td>
  98.             </tr>
  99.  
  100.  
  101. </table>
  102. <p>
  103. <input type='submit' name='submit' value="Register">
  104. <a href="index.php">Back</a>
  105. </form>
  106. </html>
[PHP] pobierz, plaintext


Błąd jest w tym, że jak wpisze wszystkie dane(lub byle co) poprawnie to dalej pokazuje, że "Prosze uzupelnić wszystkie pola" , i tak w kółko .Dodam , że kod jest nie skończony , ale np jak nic nie wpisze albo za dlugie username itp. to powinno błąd pokazywać.

Ten post edytował gigzorr 6.02.2010, 22:36:46
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi
altruista2
post 9.02.2010, 02:09:54
Post #2





Grupa: Zarejestrowani
Postów: 127
Pomógł: 32
Dołączył: 8.02.2010

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

1. Możesz zacząć od prostej "optymalizacji" zmieniająć
[PHP] pobierz, plaintext 
  1. $password = md5($password);
  2. $repeatpassword = md5($repeatpassword);
  3. if ($password==$repeatpassword) {
[PHP] pobierz, plaintext

na
[PHP] pobierz, plaintext 
  1. if ($password==$repeatpassword) {
  2. $password = md5($password);
[PHP] pobierz, plaintext


Ale to nie jest ważne smile.gif

Jeśli chodzi o:
Cytat
Mam jeszcze jeden problem, otóż na tych samych danych czyli loginie haśle/name , mogę się rejestrować do woli , nie mam zielonego pojęcia jak to naprawić?


Jeśli działasz na bazie danych to:
[PHP] pobierz, plaintext 
  1. if(@mysql_num_rows(mysql_query("SELECT ID FROM Tabela_z_uzytkownikami WHERE Nazwa_uzytkownika='".$username."'")) > 0) {
  2.       // nie można się zarejestrować, logi już występuje.
  3. }
[PHP] pobierz, plaintext


Dodam że:
a) Wypada mieć index na "Nazwa_uzytkownika" (np. varchar(32))
cool.gif strip_tags chroni cię tylko przed atakiem XSS ze strony przeglądarki. Lektura "do poduszki":
http://pl.wikipedia.org/wiki/Cross-site_scripting
Temat: SQL Injection Insertion
http://segfaultlabs.com/files/pdf/bezpiecz...-zagrozenia.pdf


--------------------
Jeśli Ci pomogłem kliknij pomógł. W ten sposób temat zaświeci się na żółto i użytkownicy którzy pomagają nie będą musieli niepotrzebnie klikać. Dziękuję.

"Pomaganie"
Go to the top of the page
+Quote Post

Posty w temacie
- gigzorr   [PHP] rejestracja   6.02.2010, 22:34:51
- - fifi209   Zacznijmy od faktu, że sprawdzasz potem "dług...   6.02.2010, 22:40:09
- - darko   Brakuje definicji zmiennej $submit to raz. Dw...   6.02.2010, 22:43:17
|- - gigzorr   Cytat(darko @ 6.02.2010, 22:43:17 ) B...   7.02.2010, 16:28:49
- - sniffer32   ..linia 9   7.02.2010, 16:31:32
- - darko   [PHP] pobierz, plaintext $submit = $_POST['sub...   7.02.2010, 16:34:56
|- - gigzorr   Cytat(darko @ 7.02.2010, 16:34:56 ) [...   7.02.2010, 16:58:51
- - darko   Przeczytaj kilka postów wyżej na temat hasła i dłu...   7.02.2010, 17:03:02
|- - gigzorr   Cytat(darko @ 7.02.2010, 17:03:02 ) P...   7.02.2010, 19:18:12
- - darko   Cytat(gigzorr @ 6.02.2010, 22:34:51 )...   8.02.2010, 04:19:23
|- - gigzorr   Cytat(darko @ 8.02.2010, 04:19:23 ) O...   8.02.2010, 09:13:33
- - Armstrong   Wiesz do czego służy die() Dobrze przeanalizuj lin...   8.02.2010, 11:33:59
- - gigzorr   Jestem idiota. W bazie danych brakowało name/daty....   8.02.2010, 12:02:23
- - altruista2   1. Możesz zacząć od prostej "optymalizacji...   9.02.2010, 02:09:54

« Następny starszy · PHP · 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: Przełącz na: Standardowy · Przełącz na: Linearny+ · Drzewo

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

RSS Wersja Lo-Fi Aktualny czas: 14.08.2025 - 19:01
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn