Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP][MySQL] - błąd w zapytaniu po pojawieniu się zmiennej $_POST
Forum PHP.pl > Forum > Przedszkole
elemeledutki
13.06.2017, 09:44:33
Cześć.

Jestem zbyt świeżutki żeby znaleźć się "aż" w Przedszkolu, ale niższej kategorii nie znalazłem, więc piszę tutaj.

Napisałem (posługując się głównie przykładami z netu) bardzo prosty skrypt, który wyświetla kilka informacji z bardzo prostej bazy danych (dwie tabele: users (id, username, password, active) i karty (id, kartanr, data, danie, cena, firma)
Po zalogowaniu skrypt wyświetla odpowiednie dane (działa)

dodanie bardzo prostego formularza:
[HTML] pobierz, plaintext 
  1. <form method='POST'>
  2. 		  <input type='text' name='newpass' placeholder='Nowe haslo' required>
  3. 		  <button type='submit'>ZMIEN HASLO</button>
  4. 	</form>
[HTML] pobierz, plaintext


zapytanie (które działało "przed chwilą") zwraca błąd (sprawdziłem, to to samo zapytanie i POWINNO działać)

Różnica, którą widzę jest taka, że w momencie pierwszego (poprawnego) zapytania istnieją zmienne $_POST['username'] i $_POST['password'], przy ponownym wykonaniu nie ma ich, jest tylko $_POST['newpass'] (którą chcę użyć do możliwości zmiany hasła). Tyle, że dane używane w zapytaniu przechowywane są w $_SESSION a ta zmiennna się NIE ZMIENIA.

łatwiej byłoby zobaczyć na działającym przykładzie: link: karty.delihouse.pl login: 686 hasło: haslo . Po zalogowaniu wszystko działa jak powinno, po kliknięciu guzika "ZMIEN HASLO" zapytanie wywraca się, mimo, że (mi się wydaje) jest poprawne...

Pomożecie?

CAŁY KOD:
[PHP] pobierz, plaintext 
  1. <?php  //Start the Session
  2. session_start();
  3. // require('connect.php');
  4. //3. If the form is submitted or not.
  5. 	//3.1 If the form is submitted
  6. 	if (isset($_POST['username']) and isset($_POST['password'])){
  7. 		//3.1.1 Assigning posted values to variables.
  8. 		$username = $_POST['username'];
  9. 		$password = $_POST['password'];
  10. 		//3.1.2 Checking the values are existing in the database or not
  11.  
  12.  
  13. //dane dostępowe		
  14.  
  15. 		$query = "SELECT * FROM `users` WHERE username='$username' and password='$password'";
  16.  
  17. 		     if (mysql_connect($serwer, $login, $haslo) and mysql_select_db($baza)) { 
  18.     // zapytanie do bazy danych 
  19.  
  20.  
  21. 	//	echo "<style>
  22.  
  23. 	//	</style>
  24. 	//	";
  25.  
  26.  
  27.  
  28. 	} 
  29.     else echo "Nie mogę połączyć się z bazą danych!"; 
  30.  
  31.  
  32.  
  33.  
  34. 		$result = mysql_query($query) or die("Błąd w zapytaniu! 0"); 
  35. 		$count = mysql_num_rows($result);
  36. 		//3.1.2 If the posted values are equal to the database values, then session will be created for the user.
  37. 		if ($count == 1){
  38. 		$_SESSION['username'] = $username;
  39.  
  40. 		$query = mysql_query("SELECT firma FROM `karty` WHERE kartanr='$username' LIMIT 1") or die("Błąd w zapytaniu! 4");
  41. 		$nazwafirmy = mysql_fetch_array($query);
  42. 		$_SESSION['firma'] = $nazwafirmy[0];
  43.  
  44. 		}else{
  45.  
  46. 		//3.1.3 If the login credentials doesn't match, he will be shown with an error message.
  47. 		$fmsg = "NIEPRAWIDLOWE DANE LUB BRAK KARTY.";
  48. 		}
  49. 		}
  50.  
  51. 		//3.1.4 if the user is logged in Greets the user with message
  52. 		if (isset($_SESSION['username'])){
  53. 		$username = $_SESSION['username'];
  54.  
  55. 		$day   = date('d'); // dzień
  56. 		$month = date('m'); // miesiąc
  57. 		$year  = date('Y'); // rok
  58.  
  59. 		if ($_SESSION['firma'] == 'NETCOMPANY') {
  60. 			$data_in = date("Y-m-d", mktime (0,0,0,$month-1,19,$year));
  61. 			$data_out = date("Y-m-d", mktime (0,0,0,$month,20,$year));
  62. 		} 
  63. 		else {
  64. 			$data_in = date("Y-m-d", mktime (0,0,0,$month,1,$year));
  65. 			$data_out = date("Y-m-d", mktime (0,0,0,$month,date("t"),$year));
  66. 		}
  67.  
  68.  
  69. 		 		echo "<!DOCTYPE html>
  70. 	<head>
  71. 	<meta http-equiv=content-type content=text/html; charset=iso-8859-2>
  72. 	<link rel=stylesheet href=styles.css >
  73. 	<title>Page Title</title>
  74. 	<meta name=description content=Write some words to describe your html page>
  75. 	</head>
  76. 	<body>
  77. 	<div class=blended_grid>
  78. 	<div class=pageHeader>
  79. 	Wersja MOCNO robocza. Dane na razie uaktualniane sa recznie poprzez export z naszego programu sprzedazowego do excela i stamtad do bazy. MOZE sie zdarzyc tak, ze zapomne, ale staral sie bede aktualizoweac kazdego dnia po jego zakonczeniu.<br />
  80. 	W kazdym razie w tej plikacji moga pojawic sie bledy (glownie ludzkie), w razie watpliwosci prosze o mail.<br />
  81. 	<small> V 0.1.6 (dodane rozliczenie po okresie wybranym dla danej firmy)</small>
  82. 	";
  83. 		//*
  84. 	echo "cena: " . $rek[1];
  85.  
  86. 	echo "<pre>";
  87. 	print_r($_POST);
  88. 	unset($_POST['newpass']);
  89. 	print_r($_POST);
  90. 	print_r($_SESSION);
  91. 	echo "</pre>";
  92. 	//*/			
  93.  
  94. 	echo "
  95. 	</div>
  96. 	<div class=pageLeftMenu>KARTA NR:<br />
  97.  
  98. 	<b>
  99. 	<font color='#FAA' size='7'>" . $username . "</font>
  100. 	<br />
  101. 	<font color='#AA5050' size='5'>" . $_SESSION['firma'] . "</font>
  102. 	</b>
  103.  
  104. 	<br />Data od:<br /><b>". $data_in . "</b><br />Data do:<br /><b>". $data_out . "</b>
  105. 	<small><p>Na razie automatycznie wybierany jest okres aktualny (wlasciwy dla danej firmy).<br />Pozniej postaram sie dopisac mozliwosc wyboru wlasnego zakresu dat.</p></small>
  106. 	<br /><strong><a href='logout.php'><br />LOGOUT</a></strong><br />
  107. 	<br /><strong><a href='logout.php'><br />ZMIEN HASLO</a></strong><br /><small>Na razie nie dziala</small>
  108.  
  109. 	<form method='POST'>
  110. 		  <input type='text' name='newpass' placeholder='Nowe haslo' required>
  111. 		  <button type='submit'>ZMIEN HASLO</button>
  112. 	</form>
  113.  
  114. 	</div>
  115. 	<div class=pageContent>
  116. 	";
  117.  
  118.  
  119. 	echo "
  120. 		<div class=divTable>
  121. 		<div class=divTableBody>
  122. 		"; 
  123.  
  124. 		$wynik = mysql_query("SELECT data FROM karty WHERE kartanr='$username' AND data>='$data_in' AND data<'$data_out' GROUP BY data") or die("BLAD w zapytaniu 1: <pre>SELECT data FROM karty WHERE kartanr='$username' AND data>='$data_in' AND data<'$data_out' GROUP BY data</pre>"); 
  125. 		while($rek1 = mysql_fetch_array($wynik)) { 
  126. 			echo "<br /><b>";
  127. 			echo $rek1[0];
  128. 			echo "</b>";
  129. 			$dania = mysql_query("SELECT danie, cena FROM karty WHERE data='$rek1[0]' AND kartanr='$username'") or die("Błąd w zapytaniu 2: <pre>SELECT danie, cena FROM karty WHERE data='$rek1[0]' AND kartanr='$username'</pre>"); 
  130. 			while($rek = mysql_fetch_array($dania)) { 
  131. 				echo "<div class=divTableRow><div class=divTableCell>".$rek[0]."</div><div class=divTableCell>".$rek[1]."</div></div>"; 
  132. 				/*echo "<pre>";
  133. 				print_r ($rek);
  134. 				echo "</pre>";*/
  135. 			} 
  136. 		}
  137. 	echo "
  138. 	</div>
  139. 	</div>
  140. 	";
  141.  
  142. 	echo "
  143. 	<br /><font color='#CCC'><p>Niestety nie umiem jeszcze tego podsumowac (cos poczytam o zamianie stringa na liczbe, pewnie o to chodzi), na razie trzeba uzyc kalkulatora albo oszacowac w pamieci...</p></font><pre>
  144. 	";
  145.  
  146. 	echo "
  147. 	</pre></div>
  148. 	<div class=pageFooter>";
  149. 	echo "
  150. 	</div>
  151. 	</div>
  152. 	</boy>
  153. 	</html>
  154. 		";
  155.  
  156.  
  157.  
  158.  
  159.  
  160. 		}else{
  161. 	//3.2 When the user visits the page first time, simple login form will be displayed.
  162. 	?>
  163. 	<html>
  164. 	<head>
  165. 		<title>Nie mam pomyslu na title</title>
  166.  
  167. 	</head>
  168. 	<body>
  169.  
  170. 	<div class="container">
  171. 		  <form class="form-signin" method="POST">
  172. 		  <?php if(isset($fmsg)){ ?><div class="alert alert-danger" role="alert"> <?php echo $fmsg; ?> </div><?php } ?>
  173. 			<h2 class="form-signin-heading">Please Login</h2>
  174. 			<div class="input-group">
  175. 		  <span class="input-group-addon" id="basic-addon1">@</span>
  176. 		  <input type="text" name="username" class="form-control" placeholder="Username" required>
  177. 		</div>
  178. 			<label for="inputPassword" class="sr-only">Password</label>
  179. 			<input type="password" name="password" id="inputPassword" class="form-control" placeholder="Password" required>
  180. 			<button class="btn btn-lg btn-primary btn-block" type="submit">Login</button>
  181.  
  182. 		  </form>
  183. 	</div>
  184.  
  185. 	</body>
  186.  
  187. 	</html>
  188. 	<?php } ?>
[PHP] pobierz, plaintext
nospor
13.06.2017, 10:02:09
Zamiast wyswietlac nic nie mowiace:

or die("Blablabla");

wyswietlaj blad zapytania

or die(mysql_error());
elemeledutki
13.06.2017, 10:30:20
Kod
Access denied for user 'srv17998'@'localhost' (using password: NIE)


czyli wiem, gdzie szukać...

PODZIĘKOWAŁ.
nospor
13.06.2017, 10:39:50
ps: skoro dopiero zaczynasz zabawe, przerzuc sie na PDO albo chociaz na mysqli_. mysql_ jest juz przestarzale i wylecialo z nowych wersji php
elemeledutki
13.06.2017, 15:08:44
właściwie to nie zaczynam - po prostu muszę zrobić taki jeden malutki projekcik i tyle. A że kiedyś (kilka lat temu) cośtam cośtam próbowałem pisać w php/mysql to stare przyzwyczajenia zostały...
Dzięki wielkie za pomoc - właściwie za zwrócenie uwagi (myślałem że wypisywanie które zapytanie jest wadliwe jest dobrym pomysłem - okazało się, że lepszym jest sprawdzenie DLACZEGO jest wadliwe:)) Człowiek się całe życie uczy..

Co do samego mysqli - (to chyba bardziej obiektowe podejście do mysql) - czytałem - i gdybym miał zrobić coś jeszcze - pewnie poszedłbym w tym kierunku - ale na razie zgodnie z zasadą "nie zepsute - nie naprawiać" - póki działa zostaje jak jest:)

Szczególnie, że jeszcze kilka pierdółek mam do dopisania (ale to już poza sql).

jeszcze raz dzięki.
patwoj98
13.06.2017, 20:45:30
Cytat(elemeledutki @ 13.06.2017, 16:08:44 ) *
właściwie to nie zaczynam - po prostu muszę zrobić taki jeden malutki projekcik i tyle. A że kiedyś (kilka lat temu) cośtam cośtam próbowałem pisać w php/mysql to stare przyzwyczajenia zostały...
Dzięki wielkie za pomoc - właściwie za zwrócenie uwagi (myślałem że wypisywanie które zapytanie jest wadliwe jest dobrym pomysłem - okazało się, że lepszym jest sprawdzenie DLACZEGO jest wadliwe:)) Człowiek się całe życie uczy..

Co do samego mysqli - (to chyba bardziej obiektowe podejście do mysql) - czytałem - i gdybym miał zrobić coś jeszcze - pewnie poszedłbym w tym kierunku - ale na razie zgodnie z zasadą "nie zepsute - nie naprawiać" - póki działa zostaje jak jest:)

Szczególnie, że jeszcze kilka pierdółek mam do dopisania (ale to już poza sql).

jeszcze raz dzięki.


Jeśli mogę się jeszcze odnieść - mysqli owszem obiektowe, ALE możesz również zrobić ze zwykłego mysql, mysqli. Wystarczy, że dodając tę jedną samogłoskę, popatrzysz w dokumentacji i ewentualnie dodasz zmienną z połączeniem bazy danych jako pierwszy argument. Tak nie wiele, a będziesz miał pewność, że zadziała na nowszych wersjach PHP.
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.