Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [HTML][PHP]Wyniki.
Forum PHP.pl > Forum > Przedszkole
assasin
5.04.2010, 14:32:48
Witam, mam mały problem z wyświetleniem wyniku na stronie:) w php jestem mniej niż początkujący... a więc tak:
Zainstalowałem sobie cms-a WordPress, utworzyłem nową podstrone.. w której chce umieścić rejestracje..
Poniższa rejestracja działa poprawnie, wyświetla dobry wynik bez naruszenia strony:)
[PHP] pobierz, plaintext 
  1.  
  2.  
  3. <?php
  4. /*Config*/
  5.  
  6. $realmd = array(
  7. 'db_host'=> '', //ip of db realm
  8. 'db_username' => '',//realm user
  9. 'db_password' => '',//realm password
  10. 'db_name'=> 'realmd',//realm db name
  11. );
  12.  
  13.  
  14. ///////////////Start script//////////////////
  15.  
  16. /*
  17. Function name: CHECK FOR SYMBOLS
  18. Description: return TRUE if matches. ( True = OK ) ( False = NOT OK)
  19. */
  20. function check_for_symbols($string){
  21. $len=strlen($string);
  22. $alowed_chars="abcdefghijklmnopqrstuvwxyzAŚA¸AYABCDEFGHIJKLMNOPQRSTUVWXYZA†A~A…";
  23. for($i=0;$i<$len;$i++)if(!strstr($alowed_chars,$string[$i]))return TRUE;
  24. return FALSE;
  25.  
  26. }
  27. /*
  28. Function name: OUTPUT USERNAME:PASSWORD AS SHA1 crypt
  29. Description: obious.
  30. */
  31. function sha_password($user,$pass){
  32. $user = strtoupper($user);
  33. $pass = strtoupper($pass);
  34.  
  35. return SHA1($user.':'.$pass);
  36. }
  37.  
  38. if ($_POST['registration']){
  39. /*Connect and Select*/
  40. $realmd_bc_new_connect = mysql_connect($realmd[db_host],$realmd[db_username],$realmd[db_password]);
  41. $selectdb = mysql_select_db($realmd[db_name],$realmd_bc_new_connect);
  42. if (!$realmd_bc_new_connect || !$selectdb){
  43. echo "Could NOT connect to db, please check the config part of the file!";
  44. die;
  45. }
  46.  
  47. /*Checks*/
  48. $username = $_POST['username'];
  49. $password = sha_password($username,$_POST['password']);
  50.  
  51. $qry_check_username = mysql_query("SELECT username FROM `account` WHERE username='$username'");
  52.  
  53. if (check_for_symbols($_POST[password]) == TRUE || check_for_symbols($username) == TRUE || mysql_num_rows($qry_check_username) != 0){
  54. echo "Error with creating account, might already be in use or your username / password has invalid symbols in it.";
  55. }else{
  56. mysql_query("INSERT INTO account (username,sha_pass_hash) VALUES
  57. ('$username','$password')");// Insert into database.
  58. echo "Account created.";
  59. }
  60.  
  61.  
  62. }else{
  63. ///////////////Stop script, Start HTML//////////////////
  64. ?>
  65.  
  66.  
  67. <form action="<?php echo $_SERVER['PHP_SELF'] ?>" method="POST">
  68.  
  69. Username <input type="text" name="username">
  70.  
  71. Password <input type="password" name="password">
  72.  
  73. <input type="submit" name="registration">
  74. </form>
  75.  
  76.  
  77. <?php
  78. // Do not remove this;)
  79. }
  80. ?>
[PHP] pobierz, plaintext


Tylko ja bym chciał umieścić tą rejestracje:

[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. // Configuration.
  4. // Realm database.
  5. $r_db = "realmd";
  6. // IP (and port).
  7. $ip = "127.0.0.1:3306";
  8. // Username.
  9. $user = "trinity";
  10. // Password.
  11. $pass = "trinity";
  12. // Site title.
  13. $title = "Registration Form";
  14. $title2 = "Some Server";
  15. // End config.
  16.  
  17. $page = '<?xml version="1.0" encoding="utf-8" ?>
  18. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
  19. <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
  20. <head>
  21. <title>' . $title . '</title>
  22. </head>
  23. <body style="background-color:black;color:yellow;font-family:verdana;">
  24. <form method="post" action="' . $_SERVER["SCRIPT_NAME"] . '">
  25. <p style="text-align:center;">
  26. <strong>' . $title2 . ' - ' . $title . '</strong>
  27. <br /><br /><br />
  28. Username:
  29. <br /><input name="username" type="text" maxlength="14" /><br />
  30. Password:
  31. <br /><input name="password" type="password" maxlength="12" /><br />
  32. Email:
  33. <br /><input name="email" type="text" maxlength="50" />
  34. <br /><input name="tbc" type="checkbox" checked="checked" /> TBC<br /><br /><br />
  35. <button type="submit">Submit</button>
  36. </p>
  37. </form>
  38. </body>
  39. </html>';
  40.  
  41. function error_s ($text) {
  42. 	echo("<p style=\"background-color:black;color:yellow;font-family:verdana;\">" . $text);
  43. 	echo("<br /><br /><a style=\"color:orange;\" href=\"" . $_SERVER["SCRIPT_NAME"] . "\">Go back...</a></p>");
  44. };
  45.  
  46. $user_chars = "#[^a-zA-Z0-9_\-]#";
  47. $email_chars = "/^[^0-9][A-z0-9_]+([.][A-z0-9_]+)*[@][A-z0-9_]+([.][A-z0-9_]+)*[.][A-z]{2,4}$/";
  48.  
  49. $con = @mysql_connect($ip, $user, $pass);
  50. if (!$con) {
  51. 	error_s("Unable to connect to database: " . mysql_error());
  52. };
  53.  
  54. if (!empty($_POST)) {
  55.         if ((empty($_POST["username"]))||(empty($_POST["password"]))||(empty($_POST["email"]))||(empty($_POST["tbc"])) ) {
  56.                 error_s("You did not enter all the required information.");
  57. 				exit();
  58.         } else {
  59.                 $username = strtoupper($_POST["username"]);
  60.                 $password = strtoupper($_POST["password"]);
  61.                 $email = strtoupper($_POST["email"]);
  62.                 if (strlen($username) < 5) {
  63.                         error_s("Username too short.");
  64.                         exit();
  65.                 };
  66.                 if (strlen($username) > 14) {
  67.                         error_s("Username too long.");
  68.                         exit();
  69.                 };
  70.                 if (strlen($password) < 8) {
  71.                         error_s("Password too short.");
  72.                         exit();
  73.                 };
  74.                 if (strlen($password) > 12) {
  75.                         error_s("Password too long.");
  76.                         exit();
  77.                 };
  78.                 if (strlen($email) < 15) {
  79.                         error_s("Email was too short.");
  80.                         exit();
  81.                 };
  82. 				if (strlen($email) > 50) {
  83.                         error_s("Email was too long.");
  84.                         exit();
  85.                 };
  86.                 if (preg_match($user_chars,$username)) {
  87.                         error_s("Username contained illegal characters.");
  88.                         exit();
  89.                 };
  90.                 if (preg_match($user_chars,$password)) {
  91.                         error_s("Password contained illegal characters.");
  92.                         exit();
  93.                 };
  94.                 if (!preg_match($email_chars,$email)) {
  95.                         error_s("Email was in an incorrect format.");
  96.                         exit();
  97.                 };
  98.                 if ($_POST['tbc'] != "on") {
  99.                         $tbc = "0";
  100.                 } else {
  101.                         $tbc = "1";
  102.                 };
  103.                 $username = mysql_real_escape_string($username);
  104.                 $password = mysql_real_escape_string($password);
  105.                 $email = mysql_real_escape_string($email);
  106.                 $qry = @mysql_query("select username from " . mysql_real_escape_string($r_db) . ".account where username = '" . $username . "'", $con);
  107. 				if (!$qry) {
  108. 					error_s("Error querying database: " . mysql_error());
  109. 				};
  110.                 if ($existing_username = mysql_fetch_assoc($qry)) {
  111.                         foreach ($existing_username as $key => $value) {
  112.                                 $existing_username = $value;
  113.                         };
  114.                 };
  115.                 $existing_username = strtoupper($existing_username);
  116.                 if ($existing_username == strtoupper($_POST['username'])) {
  117.                         error_s("That username is already taken.");
  118.                         exit();
  119.                 };
  120. 				unset($qry);
  121.                 $qry = @mysql_query("select email from " . mysql_real_escape_string($r_db) . ".account where email = '" . $email . "'", $con);
  122. 				if (!$qry) {
  123. 					error_s("Error querying database: " . mysql_error());
  124. 				};
  125.                 if ($existing_email = mysql_fetch_assoc($qry)) {
  126.                         foreach ($existing_email as $key => $value) {
  127.                                 $existing_email = $value;
  128.                         };
  129.                 };
  130.                 if ($existing_email == $_POST['email']) {
  131.                         error_s("That email is already in use.");
  132.                         exit();
  133.                 };
  134. 				unset($qry);
  135.                 $sha_pass_hash = sha1(strtoupper($username) . ":" . strtoupper($password));
  136.                 $register_sql = "insert into " . mysql_real_escape_string($r_db) . ".account (username, sha_pass_hash, email, expansion) values (upper('" . $username . "'),'" . $sha_pass_hash . "','" . $email . "','" . $tbc . "')";
  137.                 $qry = @mysql_query($register_sql, $con);
  138. 				if (!$qry) {
  139. 					error_s("Error creating account: " . mysql_error());
  140. 				};
  141.                 echo("Account successfully created.");
  142. 				exit();
  143.         };
  144. } else {
  145.         echo($page);
  146. };
  147.  
  148. ?>
[PHP] pobierz, plaintext


A w tym wypadku, strona rozsypuje się;/ po wyświetleniu wyniku... Jaka modyfikacja sprawiał by że strona poprawnie by działała.. pozdrawiam.
mortus
5.04.2010, 19:02:16
Zarówno die() jak i exit() zatrzymują wykonywanie skryptu. Nie należy ich zatem stosować notorycznie, a tylko w wyjątkowych przypadkach. Alternatywą jest używanie bloku
[PHP] pobierz, plaintext 
  1. if(warunek) {
  2.     // kod do wykonania
  3. } elseif(inny warunek) {
  4.     // inny kod do wykonania
  5. } else {
  6.     // brak warunków
  7.     // kod do wykonania
  8. }
[PHP] pobierz, plaintext
assasin
5.04.2010, 20:52:17
Ale co to ma się do wyświetlania wyniku?
Zastosowałem tą regułę:

[PHP] pobierz, plaintext 
  1.           if (strlen($username) < 5) {
  2.                         error_s("Username too short.");
  3.  
  4.  
  5.                 } elseif (strlen($username) > 14) {
  6.                         error_s("Username too long.");
  7.  
  8.  
  9.                 } else { (strlen($password) < 8) 
  10.                         error_s("Password too short.");
  11.  
  12.                 }
  13.                 if (strlen($password) > 12) {
  14.                         error_s("Password too long.");
  15.  
  16.                 };
[PHP] pobierz, plaintext


no i
Parse error: syntax error, unexpected T_ELSE in.... w
error_s("Password too short.");
MateuszS
5.04.2010, 20:56:37
Co za konstrukcja... w jakiej wersji PHP takie cuś wprowadzili?

[PHP] pobierz, plaintext 
  1.  } else { (strlen($password) < 8) 
[PHP] pobierz, plaintext
assasin
5.04.2010, 21:17:38
No tak, poprawiłem ale co to ma do wyświetlania się wyniku?
Strona po stworzeniu konta powinna tak wyglądać:

a jest tak:)
mortus
5.04.2010, 21:34:17
Przede wszystkim określ, co według Ciebie jest wynikiem.

Albo opisz dokładniej, co jest nie tak na tym drugim obrazku.
assasin
5.04.2010, 21:50:54
No właśnie.. wynikiem dla mnie jest to co wyświetliło mi się po zakończeniu pracy skryptu:)
Na 2-gim obrazku widać że strona się rozleciała tzn: nie ma menu(sidebar), stopki (footer), i uwaliło trochę głównej strony (single)... coś nie tak z wyświetlaniem wyniku.
mortus
5.04.2010, 22:01:47
Niestety niewiele mogę Ci pomóc mając tylko skrypt rejestracji. Podejrzewam, że ten plik jest wstawiany za pomocą include do pliku, który zawiera layout strony, ale to tylko moje przypuszczenia. Najlepiej wklej tutaj kod pliku odpowiedzialnego za wygląd strony i aktualny kod pliku odpowiedzialnego za rejestrację.
assasin
5.04.2010, 22:19:32
Nie, nie includuje rejestracji do pliku z wyglądem strony:) wszystko jest w jednym:

[PHP] pobierz, plaintext 
  1. <?php define('WP_USE_THEMES', false); require('./wp-blog-header.php'); get_header(); ?>
  2. <?php get_header(); ?>
  3.  
  4. <div id="container">
  5. <div id="left-div">
  6.     <div id="left-inside">
  7.         <?php if (have_posts()) : ?>
  8.         <?php while (have_posts()) : the_post(); ?>
  9.         <!--Start Post-->
  10.         <span class="single-entry-titles" style="margin-top: 18px;"></span>
  11.         <div class="post-wrapper">
  12.  
  13.  
  14.  
  15.  
  16.  
  17. <!--
  18. ///////////////////////////////////////////////////////////////////////
  19. //                                                                   //
  20. //     Skrypt rejestracyjny kompatybilny z bazą MySQL                //
  21. //     Dostosowany do emulatorów MaNGOS oraz TrinityCore             //
  22. //                                                                   //
  23. //     Wersja 3.2.2a                                                 //
  24. //                                                                   //
  25. //     Kod udostępniony na licencji GNU General Public License v3.0  //
  26. //                                                                   //
  27. //     Autor: Michalo <michalo2882@gmail.com>                        //
  28. //                                                                   //
  29. ///////////////////////////////////////////////////////////////////////
  30. -->
  31.  
  32. <html>
  33.  
  34. <head>
  35.     <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-2" >
  36. </head>
  37.  
  38. <body>
  39.  
  40. <?php
  41.  
  42. // Configuration.
  43. // Realm database.
  44. $r_db = "";
  45. // IP (and port).
  46. $ip = "128124124:3306";
  47. // Username.
  48. $user = "s";
  49. // Password.
  50. $pass = "";
  51. // Site title.
  52. $title = "Registration Form";
  53. $title2 = "Some Server";
  54. // End config.
  55.  
  56. $page = '<?xml version="1.0" encoding="utf-8" ?>
  57. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
  58. <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
  59. <head>
  60. <title>' . $title . '</title>
  61. </head>
  62. <body style="background-color:black;color:yellow;font-family:verdana;">
  63. <form method="post" action="' . $_SERVER["SCRIPT_NAME"] . '">
  64. <p style="text-align:center;">
  65. <strong>' . $title2 . ' - ' . $title . '</strong>
  66. <br /><br /><br />
  67. Username:
  68. <br /><input name="username" type="text" maxlength="14" /><br />
  69. Password:
  70. <br /><input name="password" type="password" maxlength="12" /><br />
  71. Email:
  72. <br /><input name="email" type="text" maxlength="50" />
  73. <br /><input name="tbc" type="checkbox" checked="checked" /> TBC<br /><br /><br />
  74. <button type="submit">Submit</button>
  75. </p>
  76. </form>
  77. </body>
  78. </html>';
  79.  
  80. function error_s ($text) {
  81. 	echo("<p style=\"background-color:black;color:yellow;font-family:verdana;\">" . $text);
  82. 	echo("<br /><br /><a style=\"color:orange;\" href=\"" . $_SERVER["SCRIPT_NAME"] . "\">Go back...</a></p>");
  83. };
  84.  
  85. $user_chars = "#[^a-zA-Z0-9_\-]#";
  86. $email_chars = "/^[^0-9][A-z0-9_]+([.][A-z0-9_]+)*[@][A-z0-9_]+([.][A-z0-9_]+)*[.][A-z]{2,4}$/";
  87.  
  88. $con = @mysql_connect($ip, $user, $pass);
  89. if (!$con) {
  90. 	error_s("Unable to connect to database: " . mysql_error());
  91. };
  92.  
  93. if (!empty($_POST)) {
  94.         if ((empty($_POST["username"]))||(empty($_POST["password"]))||(empty($_POST["email"]))||(empty($_POST["tbc"])) ) {
  95.                 error_s("You did not enter all the required information.");
  96. 				exit();
  97.         } else {
  98.                 $username = strtoupper($_POST["username"]);
  99.                 $password = strtoupper($_POST["password"]);
  100.                 $email = strtoupper($_POST["email"]);
  101.                 if (strlen($username) < 5) {
  102.                         error_s("Username too short.");
  103.  
  104.  
  105.                 } elseif (strlen($username) > 14) {
  106.                         error_s("Username too long.");
  107.  
  108.  
  109.                 } else { 
  110.                         error_s("Password too short.");
  111.  
  112.                 }
  113.                 if (strlen($password) > 12) {
  114.                         error_s("Password too long.");
  115.  
  116.                 };
  117.                 if (strlen($email) < 15) {
  118.                         error_s("Email was too short.");
  119.  
  120.                 };
  121. 				if (strlen($email) > 50) {
  122.                         error_s("Email was too long.");
  123.  
  124.                 };
  125.                 if (preg_match($user_chars,$username)) {
  126.                         error_s("Username contained illegal characters.");
  127.  
  128.                 };
  129.                 if (preg_match($user_chars,$password)) {
  130.                         error_s("Password contained illegal characters.");
  131.  
  132.                 };
  133.                 if (!preg_match($email_chars,$email)) {
  134.                         error_s("Email was in an incorrect format.");
  135.                         exit();
  136.                 };
  137.                 if ($_POST['tbc'] != "on") {
  138.                         $tbc = "0";
  139.                 } else {
  140.                         $tbc = "1";
  141.                 };
  142.                 $username = mysql_real_escape_string($username);
  143.                 $password = mysql_real_escape_string($password);
  144.                 $email = mysql_real_escape_string($email);
  145.                 $qry = @mysql_query("select username from " . mysql_real_escape_string($r_db) . ".account where username = '" . $username . "'", $con);
  146. 				if (!$qry) {
  147. 					error_s("Error querying database: " . mysql_error());
  148. 				};
  149.                 if ($existing_username = mysql_fetch_assoc($qry)) {
  150.                         foreach ($existing_username as $key => $value) {
  151.                                 $existing_username = $value;
  152.                         };
  153.                 };
  154.                 $existing_username = strtoupper($existing_username);
  155.                 if ($existing_username == strtoupper($_POST['username'])) {
  156.                         error_s("That username is already taken.");
  157.                         exit();
  158.                 };
  159. 				unset($qry);
  160.                 $qry = @mysql_query("select email from " . mysql_real_escape_string($r_db) . ".account where email = '" . $email . "'", $con);
  161. 				if (!$qry) {
  162. 					error_s("Error querying database: " . mysql_error());
  163. 				};
  164.                 if ($existing_email = mysql_fetch_assoc($qry)) {
  165.                         foreach ($existing_email as $key => $value) {
  166.                                 $existing_email = $value;
  167.                         };
  168.                 };
  169.                 if ($existing_email == $_POST['email']) {
  170.                         error_s("That email is already in use.");
  171.                         exit();
  172.                 };
  173. 				unset($qry);
  174.                 $sha_pass_hash = sha1(strtoupper($username) . ":" . strtoupper($password));
  175.                 $register_sql = "insert into " . mysql_real_escape_string($r_db) . ".account (username, sha_pass_hash, email, expansion) values (upper('" . $username . "'),'" . $sha_pass_hash . "','" . $email . "','" . $tbc . "')";
  176.                 $qry = @mysql_query($register_sql, $con);
  177. 				if (!$qry) {
  178. 					error_s("Error creating account: " . mysql_error());
  179. 				};
  180.                 echo("Account successfully created.");
  181. 				exit();
  182.         };
  183. } else {
  184.         echo($page);
  185. };
  186.  
  187. ?>
  188.  
  189.  
  190. 			</a></h1>
  191.             <div style="clear: both;"></div>
  192.  
  193.  
  194.  
  195.  
  196.  
  197.         </div>
  198.         <?php endwhile; ?>
  199.         <!--End Post-->
  200.         <?php else : ?>
  201.         <!--If no results are found-->
  202.         <h1>No Results Found</h1>
  203.         <p>The page you requested could not be found. Try refining your search, or use the navigation above to locate the post.</p>
  204.         <!--End if no results are found-->
  205.         <?php endif; ?>
  206.     </div>
  207. </div>
  208. <!--Begin Sidebar-->
  209. <?php get_sidebar(); ?>
  210. <!--End Sidebar-->
  211. <!--Begin Footer-->
  212. <?php get_footer(); ?>
  213. <!--End Footer-->
  214. </body>
  215. </html>
  216. Nie działająca rejestracja:
[PHP] pobierz, plaintext

Cytat
http://www.server.wow-center.pl/register2.php


Działająca:
Cytat
http://www.server.wow-center.pl/register2.php



mortus
5.04.2010, 22:34:33
Nadal używasz exit. Jak już mówiłem exit powoduje zatrzymanie wykonywania skryptu. Zatem kod, który znajduje się pod exit nie zostanie w ogóle wykonany. Tym kodem u Ciebie jest fragment w HTML-u oraz funkcje odpowiedzialne m.in. za wstawienie sidebar i footer (odpowiednio get_sidebar() i get_footer()). Abyś zrozumiał przedstawię Ci przykład:
[PHP] pobierz, plaintext 
  1. <?php
  2. function wydrukuj() {
  3.     echo 'breakpint';
  4. }
  5. wydrukuj();
  6. exit();
  7. wydrukuj();
  8. ?>
  9. <p>A tutaj jakiś akapit</p>
[PHP] pobierz, plaintext
W powyższym przykładzie funkcja wydrukuj wykona się tylko raz w linii 5. Funkcja w linii 7 nie zostanie wykonana. Podobnie kod w linii 9 nie zostanie wyświetlony. Twoja strona wywali się wszędzie tam, gdzie spełnione będą określone warunki, a po nich występuje exit.
assasin
5.04.2010, 22:44:58
Heh, bardzo ale to bardzo pomogłeśsmile.gif... tylko jest mały problem:) gdy usunę exit() to skrypt i tak wykona swoje działanie czyli np: zarejestruje pomimo tego że nie zostały spełnione warunki, jaką funkcją mam zastąpić exit??
Pozdrawiam.
mortus
5.04.2010, 22:56:45
Nie zastąpisz funkcji exit. Możesz to zrobić tak:
[PHP] pobierz, plaintext 
  1. if (!empty($_POST)) {
  2.         // dodajemy zmienną, która mówi, czy formularz został prawidłowo wypełniony
  3.         // na początku przyjmujemy, że tak
  4.         $isValid = true;
  5.         if ((empty($_POST["username"]))||(empty($_POST["password"]))||(empty($_POST["email"]))||(empty($_POST["tbc"])) ) {
  6.                 error_s("You did not enter all the required information.");
  7.                 // jeśli jednak któreś pole jest puste, to formularz nie został prawidłowo wypełniony
  8.                 isValid = false;
  9.         } else {
  10.                 $username = strtoupper($_POST["username"]);
  11.                 $password = strtoupper($_POST["password"]);
  12.                 $email = strtoupper($_POST["email"]);
  13.                 if (strlen($username) < 5) {
  14.                         error_s("Username too short.");
  15.                         // formularz nie został dobrze wypełniony również wtedy, gdy nazwa użytkownika jest zbyt krótka
  16.                         isValid = false;
  17.                 } elseif (strlen($username) > 14) {
  18.                         error_s("Username too long.");
  19.  
  20.                         // lub zbyt długa
  21.                         isValid = false;
  22.  
  23.                 }
  24.                 // itd,
  25.                 // ...
  26.                 if(isValid) {
  27.                     $sha_pass_hash = sha1(strtoupper($username) . ":" . strtoupper($password));
  28.                     $register_sql = "insert into " . mysql_real_escape_string($r_db) . ".account (username, sha_pass_hash, email, expansion) values (upper('" . $username . "'),'" . $sha_pass_hash . "','" . $email . "','" . $tbc . "')";
  29.                     $qry = @mysql_query($register_sql, $con);
  30. 				if (!$qry) {
  31. 					error_s("Error creating account: " . mysql_error());
  32. 				};
  33.                 echo("Account successfully created.");
  34.                 } else {
  35.                        echo $page;
  36.                 }
  37.          };
  38. } else {
  39.         echo($page);
  40. };
  41. ...
[PHP] pobierz, plaintext

assasin
5.04.2010, 22:58:08
Zdemotywowałeś mnie..
mortus
5.04.2010, 23:03:40
Cytat(assasin @ 5.04.2010, 23:58:08 ) *
Zdemotywowałeś mnie..
Dlaczego? Co to za problem. Po sprawdzeniu, czy formularz nie jest pusty deklarujesz zmienną isValid i ustawiasz jej wartość na true. Później zastępujesz wszystkie exit-y kodem isValid=false;, a na końcu, przed wysłaniem zapytania do bazy danych dokładasz jeden warunek. Naprawdę niewiele roboty.
assasin
6.04.2010, 09:30:25
Wywala mi że: isValid = false;

Parse error: syntax error, unexpected '=' in
pedro84
6.04.2010, 09:32:23
Kod
   isValid = false;


Eee, co to jest?
mortus
6.04.2010, 09:34:01
Zgubiłem prawie wszędzie znak $, ale o tej godzinie chyba mogłem ;P

Cytat(pedro84 @ 6.04.2010, 10:32:23 ) *
Kod
   isValid = false;


Eee, co to jest?

Instrukcja php bez "zgubionego" znaku $. Powinno być 
[PHP] pobierz, plaintext 
  1. $isValid = false;
[PHP] pobierz, plaintext
assasin
6.04.2010, 09:39:28
No właśnie też tak podejrzewałem, przecież zadeklarowaliśmy $isValid..

Hmm a tu:

[PHP] pobierz, plaintext 
  1. }
  2.                 $existing_username = strtoupper($existing_username);
  3.                 if ($existing_username == strtoupper($_POST['username'])) {
  4.                         error_s("That username is already taken.");
  5.                         $isValid = false;
  6.                 }
  7. 				unset($qry);
  8.                 $qry = @mysql_query("select email from " . mysql_real_escape_string($r_db) . ".account where email = '" . $email . "'", $con);
  9. 				} elseif (!$qry) {
  10. 					error_s("Error querying database: " . mysql_error());
  11. 					$isValid = false;
  12. 				}
[PHP] pobierz, plaintext


W 9 linijce jest błąd.
mortus
6.04.2010, 09:42:23
Bo tam nie ma być } elseif, tylko po prostu if z tym samym warunkiem.
assasin
6.04.2010, 09:50:42
No i mam nadzieje ostatni problem w linijce 4.. Parse error: syntax error, unexpected '}, próbowałem wiele kombinacji winksmiley.jpg żadna nie poskutkowała.
Cytat
} else {
echo $page;
}
};
} else {
echo($page);
};
?>
mortus
6.04.2010, 09:52:51
Wklej cały kod. Zamykasz zbyt wiele nawiasów, niż otworzyłeś. Ale bez kodu nie powiem Ci, w którym miejscu.
assasin
6.04.2010, 10:00:19
<?php

// Configuration.
// Realm database.
$r_db = "realmd_tbc";
// IP (and port).
$ip = ":3306";
// Username.
$user = "";
// Password.
$pass = "";
// Site title.
$title = "Registration Form";
$title2 = "Some Server";
// End config.

$page = '<?xml version="1.0" encoding="utf-8" ?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
<head>
<title>' . $title . '</title>
</head>
<body style="background-color:black;color:yellow;font-family:verdana;">
<form method="post" action="' . $_SERVER["SCRIPT_NAME"] . '">
<p style="text-align:center;">
<strong>' . $title2 . ' - ' . $title . '</strong>
<br /><br /><br />
Username:
<br /><input name="username" type="text" maxlength="14" /><br />
Password:
<br /><input name="password" type="password" maxlength="12" /><br />
Email:
<br /><input name="email" type="text" maxlength="50" />
<br /><input name="tbc" type="checkbox" checked="checked" /> TBC<br /><br /><br />
<button type="submit">Submit</button>
</p>
</form>
</body>
</html>';

function error_s ($text) {
echo("<p style=\"background-color:black;color:yellow;font-family:verdana;\">" . $text);
echo("<br /><br /><a style=\"color:orange;\" href=\"" . $_SERVER["SCRIPT_NAME"] . "\">Go back...</a></p>");
};

$user_chars = "#[^a-zA-Z0-9_\-]#";
$email_chars = "/^[^0-9][A-z0-9_]+([.][A-z0-9_]+)*[@][A-z0-9_]+([.][A-z0-9_]+)*[.][A-z]{2,4}$/";

$con = @mysql_connect($ip, $user, $pass);
if (!$con) {
error_s("Unable to connect to database: " . mysql_error());
};

// dodajemy zmienną, która mówi, czy formularz został prawidłowo wypełniony
// na początku przyjmujemy, że tak
$isValid = true;
if ((empty($_POST["username"]))||(empty($_POST["password"]))||(empty($_POST["email"]))||(empty($_POST["tbc"])) ) {
error_s("You did not enter all the required information.");
// jeśli jednak któreś pole jest puste, to formularz nie został prawidłowo wypełniony
$isValid = false;
} else {
$username = strtoupper($_POST["username"]);
$password = strtoupper($_POST["password"]);
$email = strtoupper($_POST["email"]);
if (strlen($username) < 5) {
error_s("Username too short.");
// formularz nie został dobrze wypełniony również wtedy, gdy nazwa użytkownika jest zbyt krótka
$isValid = false;
} elseif (strlen($username) > 14) {
error_s("Username too long.");

// lub zbyt długa
$isValid = false;

}

if (strlen($password) < 8) {
error_s("Password too short.");
$isValid = false;

} elseif (strlen($password) > 14) {
error_s("Password too long.");
$isValid = false;

}
if (strlen($email) < 15) {
error_s("Email was too short.");
$isValid = false;

} elseif (strlen($email) > 50) {
error_s("Email was too long.");
$isValid = false;
}

if (preg_match($user_chars,$username)) {
error_s("Username contained illegal characters.");
$isValid = false;

} elseif (preg_match($user_chars,$password)) {
error_s("Password contained illegal characters.");
$isValid = false;

}
if (!preg_match($email_chars,$email)) {
error_s("Email was in an incorrect format.");
$isValid = false;


};
if ($_POST['tbc'] != "on") {
$tbc = "0";
} else {
$tbc = "1";
};
$username = mysql_real_escape_string($username);
$password = mysql_real_escape_string($password);
$email = mysql_real_escape_string($email);
$qry = @mysql_query("select username from " . mysql_real_escape_string($r_db) . ".account where username = '" . $username . "'", $con);
if (!$qry) {
error_s("Error querying database: " . mysql_error());
$isValid = false;


} elseif ($existing_username = mysql_fetch_assoc($qry)) {
foreach ($existing_username as $key => $value) {
$existing_username = $value;
}
}
$existing_username = strtoupper($existing_username);
if ($existing_username == strtoupper($_POST['username'])) {
error_s("That username is already taken.");
$isValid = false;
}
unset($qry);
$qry = @mysql_query("select email from " . mysql_real_escape_string($r_db) . ".account where email = '" . $email . "'", $con);
} if (!$qry) {
error_s("Error querying database: " . mysql_error());
$isValid = false;
}
if ($existing_email = mysql_fetch_assoc($qry)) {
foreach ($existing_email as $key => $value) {
$existing_email = $value;
}

} if ($existing_email == $_POST['email']) {
error_s("That email is already in use.");
$isValid = false;

}
if(isValid) {
$sha_pass_hash = sha1(strtoupper($username) . ":" . strtoupper($password));
$register_sql = "insert into " . mysql_real_escape_string($r_db) . ".account (username, sha_pass_hash, email, expansion) values (upper('" . $username . "'),'" . $sha_pass_hash . "','" . $email . "','" . $tbc . "')";
$qry = @mysql_query($register_sql, $con);
if (!$qry) {
error_s("Error creating account: " . mysql_error());
$isValid = false;
};
echo("Account successfully created.");
} else {
echo $page;
};

} else {
echo($page);
};
?>
mortus
6.04.2010, 10:09:19
Usunąłeś linię if(!empty($_POST)) { przed
Kod
// dodajemy zmienną, która mówi, czy formularz został prawidłowo wypełniony
// na początku przyjmujemy, że tak


Poprawiony kod:
[PHP] pobierz, plaintext 
  1. <?php 
  2. ini_set('display_errors', 1);
  3. error_reporting(E_ALL);
  4.  
  5. // Configuration.
  6. // Realm database.
  7. $r_db = "realmd_tbc";
  8. // IP (and port).
  9. $ip = ":3306";
  10. // Username.
  11. $user = "";
  12. // Password.
  13. $pass = "";
  14. // Site title.
  15. $title = "Registration Form";
  16. $title2 = "Some Server";
  17. // End config.
  18.  
  19. function error_s($text) {
  20.     echo("<p style=\"background-color:black;color:yellow;font-family:verdana;\">".$text);
  21.     echo("<br /><br /><a style=\"color:orange;\" href=\"".$_SERVER["SCRIPT_NAME"]."\">Go back...</a></p>");
  22. }
  23.  
  24. $page = '<?xml version="1.0" encoding="utf-8" ?>
  25. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
  26. <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
  27. <head>
  28. <title>' . $title . '</title>
  29. </head>
  30. <body style="background-color:black;color:yellow;font-family:verdana;">
  31. <form method="post" action="' . $_SERVER["SCRIPT_NAME"] . '">
  32. <p style="text-align:center;">
  33. <strong>' . $title2 . ' - ' . $title . '</strong>
  34. <br /><br /><br />
  35. Username:
  36. <br /><input name="username" type="text" maxlength="14" /><br />
  37. Password:
  38. <br /><input name="password" type="password" maxlength="12" /><br />
  39. Email:
  40. <br /><input name="email" type="text" maxlength="50" />
  41. <br /><input name="tbc" type="checkbox" checked="checked" /> TBC<br /><br /><br />
  42. <button type="submit">Submit</button>
  43. </p>
  44. </form>
  45. </body>
  46. </html>';
  47.  
  48. $user_chars = "#[^a-zA-Z0-9_\-]#";
  49. $email_chars = "/^[^0-9][A-z0-9_]+([.][A-z0-9_]+)*[@][A-z0-9_]+([.][A-z0-9_]+)*[.][A-z]{2,4}$/";
  50.  
  51. $con = @mysql_connect($ip, $user, $pass);
  52. if (!$con) {
  53.     error_s("Unable to connect to database: ".mysql_error());
  54. }
  55.  
  56. if (! empty($_POST)) {
  57.     // dodajemy zmienną, która mówi, czy formularz został prawidłowo wypełniony
  58.     // na początku przyjmujemy, że tak
  59.     $isValid = true;
  60.     if (( empty($_POST["username"])) || ( empty($_POST["password"])) || ( empty($_POST["email"])) || ( empty($_POST["tbc"]))) {
  61.         error_s("You did not enter all the required information.");
  62.         // jeśli jednak któreś pole jest puste, to formularz nie został prawidłowo wypełniony
  63.         $isValid = false;
  64.     } else {
  65.         $username = strtoupper($_POST["username"]);
  66.         $password = strtoupper($_POST["password"]);
  67.         $email = strtoupper($_POST["email"]);
  68.         if (strlen($username) < 5) {
  69.             error_s("Username too short.");
  70.             // formularz nie został dobrze wypełniony również wtedy, gdy nazwa użytkownika jest zbyt krótka
  71.             $isValid = false;
  72.         } elseif (strlen($username) > 14) {
  73.             error_s("Username too long.");
  74.  
  75.             // lub zbyt długa
  76.             $isValid = false;
  77.  
  78.         }
  79.  
  80.         if (strlen($password) < 8) {
  81.             error_s("Password too short.");
  82.             $isValid = false;
  83.  
  84.         } elseif (strlen($password) > 14) {
  85.             error_s("Password too long.");
  86.             $isValid = false;
  87.  
  88.         }
  89.         if (strlen($email) < 15) {
  90.             error_s("Email was too short.");
  91.             $isValid = false;
  92.  
  93.         } elseif (strlen($email) > 50) {
  94.             error_s("Email was too long.");
  95.             $isValid = false;
  96.         }
  97.  
  98.         if (preg_match($user_chars, $username)) {
  99.             error_s("Username contained illegal characters.");
  100.             $isValid = false;
  101.  
  102.         } elseif (preg_match($user_chars, $password)) {
  103.             error_s("Password contained illegal characters.");
  104.             $isValid = false;
  105.  
  106.         }
  107.         if (!preg_match($email_chars, $email)) {
  108.             error_s("Email was in an incorrect format.");
  109.             $isValid = false;
  110.  
  111.  
  112.         }
  113.         if ($_POST['tbc'] != "on") {
  114.             $tbc = "0";
  115.         } else {
  116.             $tbc = "1";
  117.         }
  118.         $username = mysql_real_escape_string($username);
  119.         $password = mysql_real_escape_string($password);
  120.         $email = mysql_real_escape_string($email);
  121.         $qry = @mysql_query("select username from ".mysql_real_escape_string($r_db).".account where username = '".$username."'", $con);
  122.         if (!$qry) {
  123.             error_s("Error querying database: ".mysql_error());
  124.             $isValid = false;
  125.  
  126.  
  127.         } elseif ($existing_username = mysql_fetch_assoc($qry)) {
  128.             foreach ($existing_username as $key=>$value) {
  129.                 $existing_username = $value;
  130.             }
  131.         }
  132.         $existing_username = strtoupper($existing_username);
  133.         if ($existing_username == strtoupper($_POST['username'])) {
  134.             error_s("That username is already taken.");
  135.             $isValid = false;
  136.         }
  137.         unset($qry);
  138.         $qry = @mysql_query("select email from ".mysql_real_escape_string($r_db).".account where email = '".$email."'", $con);
  139.     }
  140.     if (!$qry) {
  141.         error_s("Error querying database: ".mysql_error());
  142.         $isValid = false;
  143.     }
  144.     if ($existing_email = mysql_fetch_assoc($qry)) {
  145.         foreach ($existing_email as $key=>$value) {
  146.             $existing_email = $value;
  147.         }
  148.  
  149.     }
  150.     if ($existing_email == $_POST['email']) {
  151.         error_s("That email is already in use.");
  152.         $isValid = false;
  153.  
  154.     }
  155.     if ($isValid) {
  156.         $sha_pass_hash = sha1(strtoupper($username).":".strtoupper($password));
  157.         $register_sql = "insert into ".mysql_real_escape_string($r_db).".account (username, sha_pass_hash, email, expansion) values (upper('".$username."'),'".$sha_pass_hash."','".$email."','".$tbc."')";
  158.         $qry = @mysql_query($register_sql, $con);
  159.         if (!$qry) {
  160.             error_s("Error creating account: ".mysql_error());
  161.             $isValid = false;
  162.         }
  163.         echo("Account successfully created.");
  164.     } else {
  165.         echo $page;
  166.     }
  167.  
  168. } else {
  169.     echo($page);
  170. }
  171. ?>
[PHP] pobierz, plaintext
assasin
6.04.2010, 10:13:55
Nadal gdzieś jest błąd, ponieważ rejestruje pomimo nie spełnionych warunków...
Taki jest tego wynik:
Cytat
Username too short.

Go back...

Password too short.

Go back...

That username is already taken.

Go back...

Error creating account: Duplicate entry 'ASD' for key 2

Go back...
Account successfully created.
mortus
6.04.2010, 10:28:01
Efekt końcowy:
[PHP] pobierz, plaintext 
  1. <?php 
  2. ini_set('display_errors', 1);
  3. error_reporting(E_ALL);
  4.  
  5. // Configuration.
  6. // Realm database.
  7. $r_db = "realmd_tbc";
  8. // IP (and port).
  9. $ip = ":3306";
  10. // Username.
  11. $user = "";
  12. // Password.
  13. $pass = "";
  14. // Site title.
  15. $title = "Registration Form";
  16. $title2 = "Some Server";
  17. // End config.
  18.  
  19. function error_s($text) {
  20.     echo("<p style=\"background-color:black;color:yellow;font-family:verdana;\">".$text);
  21.     echo("<br /><br /><a style=\"color:orange;\" href=\"".$_SERVER["SCRIPT_NAME"]."\">Go back...</a></p>");
  22. }
  23.  
  24. $page = '<?xml version="1.0" encoding="utf-8" ?>
  25. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
  26. <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
  27.     <head>
  28.         <title>'.$title.'</title>
  29.     </head>
  30.     <body style="background-color:black;color:yellow;font-family:verdana;">
  31.         <form method="post" action="'.$_SERVER["SCRIPT_NAME"].'">
  32.             <p style="text-align:center;">
  33.                 <strong>'.$title2.' - '.$title.'</strong>
  34.                 <br /><br /><br />
  35.                 Username:
  36.                 <br /><input name="username" type="text" maxlength="14" /><br />
  37.                 Password:
  38.                 <br /><input name="password" type="password" maxlength="12" /><br />
  39.                 Email:
  40.                 <br /><input name="email" type="text" maxlength="50" />
  41.                 <br /><input name="tbc" type="checkbox" checked="checked" /> TBC<br /><br /><br />
  42.                 <button type="submit">Submit</button>
  43.             </p>
  44.         </form>
  45.     </body>
  46. </html>';
  47.  
  48. $user_chars = "#[^a-zA-Z0-9_\-]#";
  49. $email_chars = "/^[^0-9][A-z0-9_]+([.][A-z0-9_]+)*[@][A-z0-9_]+([.][A-z0-9_]+)*[.][A-z]{2,4}$/";
  50.  
  51. $con = @mysql_connect($ip, $user, $pass);
  52. if (!$con) {
  53.     error_s("Unable to connect to database: ".mysql_error());
  54. }
  55.  
  56. if (! empty($_POST)) {
  57.     // dodajemy zmienną, która mówi, czy formularz został prawidłowo wypełniony
  58.     // na początku przyjmujemy, że tak
  59.     $isValid = true;
  60.     if (( empty($_POST["username"])) || ( empty($_POST["password"])) || ( empty($_POST["email"])) || ( empty($_POST["tbc"]))) {
  61.         error_s("You did not enter all the required information.");
  62.         // jeśli jednak któreś pole jest puste, to formularz nie został prawidłowo wypełniony
  63.         $isValid = false;
  64.     } else {
  65.         $username = strtoupper($_POST["username"]);
  66.         $password = strtoupper($_POST["password"]);
  67.         $email = strtoupper($_POST["email"]);
  68.         if (strlen($username) < 5) {
  69.             error_s("Username too short.");
  70.             // formularz nie został dobrze wypełniony również wtedy, gdy nazwa użytkownika jest zbyt krótka
  71.             $isValid = false;
  72.         } elseif (strlen($username) > 14) {
  73.             error_s("Username too long.");
  74.             // lub zbyt długa
  75.             $isValid = false;
  76.         }
  77.         if (strlen($password) < 8) {
  78.             error_s("Password too short.");
  79.             $isValid = false;
  80.         } elseif (strlen($password) > 14) {
  81.             error_s("Password too long.");
  82.             $isValid = false;
  83.         }
  84.         if (strlen($email) < 15) {
  85.             error_s("Email was too short.");
  86.             $isValid = false;
  87.         } elseif (strlen($email) > 50) {
  88.             error_s("Email was too long.");
  89.             $isValid = false;
  90.         }
  91.         if (preg_match($user_chars, $username)) {
  92.             error_s("Username contained illegal characters.");
  93.             $isValid = false;
  94.         } elseif (preg_match($user_chars, $password)) {
  95.             error_s("Password contained illegal characters.");
  96.             $isValid = false;
  97.         }
  98.         if (!preg_match($email_chars, $email)) {
  99.             error_s("Email was in an incorrect format.");
  100.             $isValid = false;
  101.         }
  102.         if ($_POST['tbc'] != "on") {
  103.             $tbc = "0";
  104.         } else {
  105.             $tbc = "1";
  106.         }
  107.         $username = mysql_real_escape_string($username);
  108.         $password = mysql_real_escape_string($password);
  109.         $email = mysql_real_escape_string($email);
  110.         $qry = @mysql_query("select username from ".mysql_real_escape_string($r_db).".account where username = '".$username."'", $con);
  111.         if (!$qry) {
  112.             error_s("Error querying database: ".mysql_error());
  113.             $isValid = false;
  114.         } elseif ($existing_username = mysql_fetch_assoc($qry)) {
  115.             foreach ($existing_username as $key=>$value) {
  116.                 $existing_username = $value;
  117.             }
  118.         }
  119.         $existing_username = strtoupper($existing_username);
  120.         if ($existing_username == strtoupper($_POST['username'])) {
  121.             error_s("That username is already taken.");
  122.             $isValid = false;
  123.         }
  124.         unset($qry);
  125.         $qry = @mysql_query("select email from ".mysql_real_escape_string($r_db).".account where email = '".$email."'", $con);
  126.         if (!$qry) {
  127.             error_s("Error querying database: ".mysql_error());
  128.             $isValid = false;
  129.         }
  130.         if ($existing_email = mysql_fetch_assoc($qry)) {
  131.             foreach ($existing_email as $key=>$value) {
  132.                 $existing_email = $value;
  133.             }
  134.         }
  135.         if ($existing_email == $_POST['email']) {
  136.             error_s("That email is already in use.");
  137.             $isValid = false;
  138.         }
  139.     }
  140.     if ($isValid) {
  141.         $sha_pass_hash = sha1(strtoupper($username).":".strtoupper($password));
  142.         $register_sql = "insert into ".mysql_real_escape_string($r_db).".account (username, sha_pass_hash, email, expansion) values (upper('".$username."'),'".$sha_pass_hash."','".$email."','".$tbc."')";
  143.         $qry = @mysql_query($register_sql, $con);
  144.         if (!$qry) {
  145.             error_s("Error creating account: ".mysql_error());
  146.             $isValid = false;
  147.         }
  148.         echo("Account successfully created.");
  149.     } else {
  150.         echo $page;
  151.     }
  152.  
  153. } else {
  154.     echo($page);
  155. }
  156. ?>
[PHP] pobierz, plaintext
Przeglądnij i sprawdź różnice. Najważniejszą jest to, że było if(isValid) { w linii 140, a teraz jest if($isValid){. Poza tym sprawdzenie, czy email już istnieje w bazie danych przeniosłem do właściwego bloku.
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.