Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Problem z formularzem php i mysql
Forum PHP.pl > Forum > PHP
stusu
17.04.2010, 17:40:16
Witam... mam problem. Zrobiłem stronę ze skryptem rejestracji i logowania (korzystając z dostępnych na forach skryptów, bo dopiero zaczynam zabawę z php) chcę dodać podstronę na której każdy użytkownik zobaczy swoje dane w formularzu i będzie mógł je zmienić (po za loginem i nickiem)
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. if($_SESSION['logged']==false)
  4. {
  5. echo 'Nie masz dostępu do tej częœci witryny.'; 
  6. }
  7. else
  8. {
  9.     $login = $_SESSION['login'];
  10.     $nick = $_SESSION['nick'];
  11.  
  12.         echo 'Zmień swoje dane:';
  13.         // tworzymy prosty formularz
  14.         echo <br /><br /><form action="index.php" method="POST">
  15.         Login: <br />
  16.         <input type="text" name="login" value="$login" disabled="disabled"><br />
  17.         Nick: <br />
  18.         <input type="text" name="nick" value="$nick" disabled="disabled"><br />
  19.  
  20. }
  21. ?>
[PHP] pobierz, plaintext


To jest tylko kawałek kodu... co mam jak mam zmienić kod żeby w formularzu na stronie wyświetlały mi się nick i login zamiast $nick i $login (wiem że coś skopałem, ale nie wiem co)
tehaha
17.04.2010, 17:45:38
przede wszystkim zapomniałeś dać na samym początku session_start(), więc skrypt nie odczytuje danych z tablicy $_SESSION
MateuszS
17.04.2010, 17:45:45 
[PHP] pobierz, plaintext 
  1.         echo '<br /><br /><form action="index.php" method="POST">
  2.         Login: <br />
  3.         <input type="text" name="login" value="'.$login.'" disabled="disabled"><br />
  4.         Nick: <br />
  5.         <input type="text" name="nick" value="'.$nick.'" disabled="disabled"><br />';
  6.  
[PHP] pobierz, plaintext

Spróbuj tak.
stusu
17.04.2010, 18:20:04
tehaha pisałem że jest to fragment kodu sesje startowałem wcześniej ;p....
MateuszS dzięki działa (taki mały błąd :/)

No dobrze w formularze wskakuje mi login i nick zalogowanego użytkownika, ale za każdym razem gdy próbuje zapisać zmiany otrzymuje komunikat że muszę uzupełnić wszystkie pola a są uzupełnione... po za tym co mam wpisać zamiast "$query = "INSERT INTO `users` (`pass`, `email`) VALUES ('$pass', '$email')";" żeby wprowadzić zmiany do bazy danych?

[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. if($_SESSION['logged']==false)
  4. {
  5. echo 'Nie masz dostępu do tej części witryny.'; 
  6. }
  7. else
  8. {
  9. 	$login = $_SESSION['login'];
  10. 	$nick = $_SESSION['nick'];
  11. 	$pass = $_SESSION['nick'];
  12. 	$email = $_SESSION['email'];
  13.  
  14. 		echo 'Zmień swoje dane:';
  15. 		// tworzymy prosty formularz
  16. 		echo '<br /><br /><form action="useredit.php" method="POST">
  17. 		Login: <br />
  18. 		<input type="text" name="login" value="'.$login.'" disabled="disabled"><br />
  19. 		Nick: <br />
  20. 		<input type="text" name="nick" value="'.$nick.'" disabled="disabled"><br />
  21. 		Hasło: <br />
  22. 		<input type="password" name="pass" value="'.$pass.'"><br />
  23. 		Powtórz hasło: <br />
  24. 		<input type="password" name="pass" value="'.$pass.'"><br />
  25. 		Email: <br />
  26. 		<input type="text" name="email" value="'.$email.'"><br />
  27. 		<input type="submit" name="zapisz" value="Zapisz">
  28. 		</form>';
  29.  
  30. 		if(isset($_POST['zapisz']))
  31. 		{
  32.  
  33. 			$pass = $_POST['pass'];
  34. 			$vpass = $_POST['vpass'];
  35. 			$email = $_POST['email'];
  36.  
  37. 			if(empty($pass) || empty($vpass) || empty($email)) echo 'Wpisz wszystkie pola!';
  38. 			else
  39. 			{
  40. 				// filtrujemy dane
  41. 				$pass = trim(strip_tags(mysql_real_escape_string(HTMLSpecialChars($pass))));
  42. 				$email = trim(strip_tags(mysql_real_escape_string(HTMLSpecialChars($email))));
  43.  
  44.  
  45. 				$resultmail = mysql_query("SELECT * FROM users WHERE email='$email'");
  46.  
  47. 				if(mysql_num_rows($resultmail)!=0) echo 'Już istnieje konto z takim emailem!';	
  48. 				else
  49. 				{
  50. 					if($pass != $vpass) echo 'Hasła się nie zgadzają!';
  51. 					else
  52. 					{
  53. 						// kodujemy hasło
  54. 						$pass = md5($pass);
  55. 						// tworzymy zapytanie
  56. 						$query = "INSERT INTO `users` (`pass`, `email`) VALUES ('$pass', '$email')";
  57.  
  58. 						// jeśli zapytanie wykona się poprawnie to zostanie wyświetlony stosowny komunikat
  59. 						if(mysql_query($query)) echo 'Dane zostały zmienione!';
  60. 					}
  61. 				}
  62. 			}
  63. 		}
  64. }
  65. ?>
[PHP] pobierz, plaintext
MateuszS
17.04.2010, 18:26:14
Nigdzie nie widzę u ciebie pola (inputa) vpass, tylko dwa pass. Żeby wyedytować musisz użyć UPDATE 

[PHP] pobierz, plaintext 
  1. UPDATE tabela SET wartosc=bla AND wartosc2=sada //itd
  2. WHERE id=idusera
  3.  
  4. //taki pseudokod
[PHP] pobierz, plaintext
stusu
17.04.2010, 19:06:40
W tej chwili wygląda to tak:
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. if($_SESSION['logged']==false)
  4. {
  5. echo 'Nie masz dostępu do tej częœci witryny.'; 
  6. }
  7. else
  8. {
  9. 	$login = $_SESSION['login'];
  10. 	$nick = $_SESSION['nick'];
  11. 	$pass = $_SESSION['nick'];
  12. 	$email = $_SESSION['email'];
  13. 	$id = $_SESSION['id'];
  14.  
  15. 		echo 'Zmień swoje dane:';
  16. 		// tworzymy prosty formularz
  17. 		echo '<br /><br /><form action="useredit.php" method="POST">
  18. 		Login: <br />
  19. 		<input type="text" name="login" value="'.$login.'" disabled="disabled"><br />
  20. 		Nick: <br />
  21. 		<input type="text" name="nick" value="'.$nick.'" disabled="disabled"><br />
  22. 		Hasło: <br />
  23. 		<input type="password" name="newpass" value="'.$pass.'"><br />
  24. 		Powtórz hasło: <br />
  25. 		<input type="password" name="newvpass" value="'.$pass.'"><br />
  26. 		Email: <br />
  27. 		<input type="text" name="newemail" value="'.$email.'"><br />
  28. 		<input type="submit" name="zapisz" value="Zapisz">
  29. 		</form>';
  30.  
  31. 		if(isset($_POST['zapisz']))
  32. 		{
  33.  
  34. 			$newpass = $_POST['newpass'];
  35. 			$newvpass = $_POST['newvpass'];
  36. 			$newemail = $_POST['newemail'];
  37.  
  38. 			if(empty($newpass) || empty($newvpass) || empty($newemail)) echo 'Wpisz wszystkie pola!';
  39. 			else
  40. 			{
  41. 				// filtrujemy dane
  42. 				$newpass = trim(strip_tags(mysql_real_escape_string(HTMLSpecialChars($newpass))));
  43. 				$newemail = trim(strip_tags(mysql_real_escape_string(HTMLSpecialChars($newemail))));
  44.  
  45.  
  46. 				$resultmail = mysql_query("SELECT * FROM users WHERE email='$newemail'");
  47.  
  48. 				if(mysql_num_rows($resultmail)!=0) echo 'Już istnieje konto z takim emailem!';	
  49. 				else
  50. 				{
  51. 					if($newpass != $newvpass) echo 'Hasła się nie zgadzają!';
  52. 					else
  53. 					{
  54. 						// kodujemy hasło
  55. 						$newpass = md5($newpass);
  56. 						// tworzymy zapytanie
  57. 						$query = "UPDATE users SET pass='$newpass' email='$newemail' WHERE id='$id'";
  58.  
  59. 						// jeœli zapytanie wykona się poprawnie to zostanie wyœwietlony stosowny komunikat
  60. 						if(mysql_query($query)) echo 'Dane zostały zmienione!';
  61. 					}
  62. 				}
  63. 			}
  64. 		}
  65. }
  66. ?>
[PHP] pobierz, plaintext


Ale dalej nie działa... za każdym razem wyświetla że hasła się nie zgadzają (co jest nie możliwe)
MateuszS
17.04.2010, 19:13:00
1. Zapomniałeś o AND w zapytaniu (zobacz na moj przyklad)
2. Problem leży tu

[PHP] pobierz, plaintext 
  1. $newpass = trim(strip_tags(mysql_real_escape_string(HTMLSpecialChars($newpass))));
  2. //jedna zmienna przefiltrowales drugiej juz nie, ustaw te filtracje pozniej
[PHP] pobierz, plaintext

stusu
17.04.2010, 19:27:14
Dodałem AND i 
[PHP] pobierz, plaintext 
  1. $newvpass = trim(strip_tags(mysql_real_escape_string(HTMLSpecialChars($newvpass))));
[PHP] pobierz, plaintext

i teraz nie wyświetla się żaden błąd, a mimo to zmiany nie zapisują się w bazie.
MateuszS
17.04.2010, 19:31:37
no to zastosuj samo

[PHP] pobierz, plaintext 
  1. $newvpass=trim(strip_tags($newvpass));
  2. $newpass=trim(strip_tags($newpass));
[PHP] pobierz, plaintext
stusu
17.04.2010, 19:37:28
No niestety dalej nic...
MateuszS
17.04.2010, 19:40:49
Pokaż jak twój kod teraz wygląda bo się pogubiłem.
stusu
17.04.2010, 19:42:49 
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. if($_SESSION['logged']==false)
  4. {
  5. echo 'Nie masz dostępu do tej częœci witryny.'; 
  6. }
  7. else
  8. {
  9. 	$login = $_SESSION['login'];
  10. 	$nick = $_SESSION['nick'];
  11. 	$pass = $_SESSION['nick'];
  12. 	$email = $_SESSION['email'];
  13. 	$id = $_SESSION['id'];
  14.  
  15. 		echo 'Zmień swoje dane:';
  16. 		// tworzymy prosty formularz
  17. 		echo '<br /><br /><form action="useredit.php" method="POST">
  18. 		Login: <br />
  19. 		<input type="text" name="login" value="'.$login.'" disabled="disabled"><br />
  20. 		Nick: <br />
  21. 		<input type="text" name="nick" value="'.$nick.'" disabled="disabled"><br />
  22. 		Hasło: <br />
  23. 		<input type="password" name="newpass" value="'.$pass.'"><br />
  24. 		Powtórz hasło: <br />
  25. 		<input type="password" name="newvpass" value="'.$pass.'"><br />
  26. 		Email: <br />
  27. 		<input type="text" name="newemail" value="'.$email.'"><br />
  28. 		<input type="submit" name="zapisz" value="Zapisz">
  29. 		</form>';
  30.  
  31. 		if(isset($_POST['zapisz']))
  32. 		{
  33.  
  34. 			$newpass = $_POST['newpass'];
  35. 			$newvpass = $_POST['newvpass'];
  36. 			$newemail = $_POST['newemail'];
  37.  
  38. 			if(empty($newpass) || empty($newvpass) || empty($newemail)) echo 'Wpisz wszystkie pola!';
  39. 			else
  40. 			{
  41. 				// filtrujemy dane
  42. 				$newvpass=trim(strip_tags($newvpass));
  43. 				$newpass=trim(strip_tags($newpass));
  44. 				$newemail = trim(strip_tags($newemail));
  45.  
  46.  
  47. 				$resultmail = mysql_query("SELECT * FROM users WHERE email='$newemail'");
  48.  
  49. 				if(mysql_num_rows($resultmail)!=0) echo 'Już istnieje konto z takim emailem!';	
  50. 				else
  51. 				{
  52. 					if($newpass != $newvpass) echo 'Hasła się nie zgadzają!';
  53. 					else
  54. 					{
  55. 						// kodujemy hasło
  56. 						$newpass = md5($newpass);
  57. 						// tworzymy zapytanie
  58. 						$query = "UPDATE users SET pass='$newpass' AND email='$newemail' WHERE id='$id'";
  59.  
  60. 						// jeœli zapytanie wykona się poprawnie to zostanie wyœwietlony stosowny komunikat
  61. 						if(mysql_query($query)) echo 'Dane zostały zmienione!';
  62. 					}
  63. 				}
  64. 			}
  65. 		}
  66. }
  67. ?>
[PHP] pobierz, plaintext


Po zmianie danych powinien się wyświetlić komunikat, a się nie wyświetla i nic się nie dzieje.
MateuszS
17.04.2010, 19:52:23 
[PHP] pobierz, plaintext 
  1.  
  2. $query = "UPDATE users SET pass='$newpass' AND email='$newemail' WHERE id='$id'";
  3. $s = mysql_query($query) or die(mysql_error());
  4. if($s) echo 'Dane zostały zmienione!';
[PHP] pobierz, plaintext

Teraz powinno wywalić Ci błąd w przypadku błędu w zapytaniu. Może też zmienna $id nie ma wartości?
stusu
17.04.2010, 19:58:05
Access denied for user 'www-data'@'localhost' (using password: NO)
hmm łączenie z bazą jest w porządku, bo bez problemu się loguję... nie rozumiem o co mu chodzi:p
MateuszS
17.04.2010, 20:00:29
Pewnie nie ustawiłeś połączenia lub podałeś złe dane ;P

[PHP] pobierz, plaintext 
  1. $connect = mysql_connect('localhost','username','password') or die(mysql_error());
  2. $db = mysql_select_db('nazwabazy', $connect) or die(mysql_error());
[PHP] pobierz, plaintext
stusu
17.04.2010, 20:03:28
Dane są na 100% ok... logowanie działa...
MateuszS
17.04.2010, 20:04:58
Skoro wywala błąd to nie są OK ;P Pokaż jak łączysz się z bazą i czy includujesz plik połączenia.
stusu
17.04.2010, 20:06:38 
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. 	// rozpoczęcie buforowania (jest to potrzebne by nie mieć błędów typu headers already sent)
  4. 	ob_start();
  5.  
  6. 	// start sesji
  7. 	session_start();
  8.  
  9. 	// połšczenie z mysql
  10. 	$connect = mysql_connect('mysql.cba.pl', '***', '***') or die('Nieudane polaczenie z baza danych...');
  11.  
  12. 	// wybór bazy danych
  13. 	$db = mysql_select_db('***_cba_pl')
  14. 	or die('Nie udalo sie wybrac bazy danych...');
  15.  
  16. ?>
  17.  
  18. <html>
  19.  
  20. <head>
  21. </head>
  22.  
  23. <body>
  24.  
  25. 	<center>
  26. <div id="top">
  27. <div id="NAGLOWEK"><img src="banner.jpg" alt="banner" width="600" height="130" align="center" /></div>
  28. <div id="MENU">
  29. menu
  30.  
  31. </div>
  32. <div id="INFORMACJE">
  33.  
  34. 	<?php
  35.  
  36. 		include("logowanie.php");
  37.  
  38. 	?>
  39.  
  40. </div>
  41. <div id="TRESC">
  42.  
  43. <?php
  44.  
  45. if($_SESSION['logged']==false)
  46. {
  47. echo 'Nie masz dostępu do tej częœci witryny.'; 
  48. }
  49. else
  50. {
  51. 	$login = $_SESSION['login'];
  52. 	$nick = $_SESSION['nick'];
  53. 	$pass = $_SESSION['nick'];
  54. 	$email = $_SESSION['email'];
  55. 	$id = $_SESSION['id'];
  56.  
  57. 		echo 'Zmień swoje dane:';
  58. 		// tworzymy prosty formularz
  59. 		echo '<br /><br /><form action="useredit.php" method="POST">
  60. 		Login: <br />
  61. 		<input type="text" name="login" value="'.$login.'" disabled="disabled"><br />
  62. 		Nick: <br />
  63. 		<input type="text" name="nick" value="'.$nick.'" disabled="disabled"><br />
  64. 		Hasło: <br />
  65. 		<input type="password" name="newpass" value="'.$pass.'"><br />
  66. 		Powtórz hasło: <br />
  67. 		<input type="password" name="newvpass" value="'.$pass.'"><br />
  68. 		Email: <br />
  69. 		<input type="text" name="newemail" value="'.$email.'"><br />
  70. 		<input type="submit" name="zapisz" value="Zapisz">
  71. 		</form>';
  72.  
  73. 		if(isset($_POST['zapisz']))
  74. 		{
  75.  
  76. 			$newpass = $_POST['newpass'];
  77. 			$newvpass = $_POST['newvpass'];
  78. 			$newemail = $_POST['newemail'];
  79.  
  80. 			if(empty($newpass) || empty($newvpass) || empty($newemail)) echo 'Wpisz wszystkie pola!';
  81. 			else
  82. 			{
  83. 				// filtrujemy dane
  84. 				$newvpass=trim(strip_tags($newvpass));
  85. 				$newpass=trim(strip_tags($newpass));
  86. 				$newemail = trim(strip_tags($newemail));
  87.  
  88.  
  89. 				$resultmail = mysql_query("SELECT * FROM users WHERE email='$newemail'");
  90.  
  91. 				if(mysql_num_rows($resultmail)!=0) echo 'Już istnieje konto z takim emailem!';	
  92. 				else
  93. 				{
  94. 					if($newpass != $newvpass) echo 'Hasła się nie zgadzają!';
  95. 					else
  96. 					{
  97. 						// kodujemy hasło
  98. 						$newpass = md5($newpass);
  99. 						// tworzymy zapytanie
  100. 						$query = "UPDATE users SET pass='$newpass' AND email='$newemail' WHERE id='$id'";
  101.  
  102. 						$s = mysql_query($query) or die(mysql_error());
  103. if($s) echo 'Dane zostały zmienione!';
  104. 					}
  105. 				}
  106. 			}
  107. 		}
  108. }
  109. ?>
  110. </div>
  111.  
  112.  
  113. <div id="STOPKA">Stopka serwisu
  114. </div>
  115. </div>
  116. </center>
  117.  
  118. </body>
[PHP] pobierz, plaintext
MateuszS
17.04.2010, 20:08:11
Zamiast mysql.cba.pl daj localhost, może zadziała i gwiazdkuj dane do bazy w poście bo ktoś ci kawał zrobi ;P
stusu
17.04.2010, 20:12:21
tak od razu wyskakuje mi informacja: "Nieudane polaczenie z baza danych..."

Nie widziałeś gdzieś może całego skryptu który by pozwalał na edycję danych o użytkowniku (bo tracę powoli cierpliwość)
jezoo
18.04.2010, 12:48:34
a spróbuj zastosować PDO, np:
[PHP] pobierz, plaintext 
  1. $dsn = "mysql:dbname=".$DB_NAME."; host=".$DB_HOST;
  2.         try{
  3.             $db = new PDO($dsn,$DB_USER,$DB_PASSWD);
  4.            $this->db->beginTransaction();
  5.         }catch(PDOException $e){
  6.             echo $e->getMessage();
  7.         }
  8. // ja stosuje do INSERT,DELETE,UPDATE
  9. $db->exec("TWOJE ZAPYTANIE");
  10. try {
  11.        $db->commit();
  12.        echo "OK";
  13. }catch(PDOException $e){
  14.     echo $e->getMessage();
  15. }
[PHP] pobierz, plaintext

i taka mała rada, postaraj się oddzielić HTML od PHP bo w tedy kod jest czytelniejszy i szybciej błąd znajdziesz.
druga sprawa, serwis CBA może mieć msqli_connect a nie mysql_connect czyli zamiast mysql daj mysqli.
stusu
18.04.2010, 19:15:51
Przy msqli_connect w ogóle nie łączy się z bazą... a ten kod który mi napisałeś jest dla mnie nie zrozumiały (dopiero zaczynam zabawę z php), czy mógłbyś mi wyjaśnić łopatologicznie co odpowiiada za co itd.
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.