Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [MySQL][PHP]Nie przetwarza wprowadzonych danych
Forum PHP.pl > Forum > Przedszkole
abel1
27.07.2016, 07:26:35
Witam
W zasadzie tak jak wpisałem w temacie
Uczę się PHP i w ramach nauki pisze sobie system logowania
Mam problem ze zmianą hasła , mianowicie niby wszystko działa bo hasło zmienia i zapisuje go w bazie
Ale dodałem do rejestracji pytanie zabezpieczające i do skryptu przywracania próbuje tez to dodać . jednak nie działa on tak jak bym tego chciał
Pierwsza próba która zakończyła się sukcesem (zmieniło hasło) stała się moją zmora
Teraz nieważne do jakiego konta bym nie chciał zmienić hasło to nie przyjmuje pytania i odpowiedzi zabezpieczającej zapisanej w bazie
Zmienia hasło jedynie z pytaniem i odpowiedzią jakiej użyłem pierwszy raz
Proszę o pomoc w rozwiązaniu tego problemu , poniżej zamieszczam kod 
[PHP] pobierz, plaintext 
  1. <?php
  2. session_start();
  3.  
  4. ini_set( 'display_errors', 'On' ); 
  5. error_reporting( E_ALL );
  6.  
  7.     if(isset($_POST['wyslane'])){
  8.  
  9.                 $wszystko_OK=true;      
  10.                 $user = $_POST['login'];
  11.                 $pytanie = $_POST['pytanie'];
  12.                 $odpowiedz = $_POST['odpowiedz'];                
  13.                 $haslo1 = $_POST['haslo1'];
  14.                 $haslo2 = $_POST['haslo2'];
  15.                 if((mb_strlen($haslo1)<8) || (mb_strlen($haslo1)>20))
  16.         {
  17.             $wszystko_OK=false;
  18.             $_SESSION['e_haslo']="Hasło musi posiadać od 8 do 20 znaków!";
  19.         }
  20.         if($haslo1!=$haslo2)
  21.             {
  22.             $wszystko_OK=false;
  23.             $_SESSION['e_haslo']="Hasła nie zgadzają się!";
  24.         }
  25.  
  26.         $haslo_hash = password_hash($haslo1, PASSWORD_DEFAULT);
  27.  
  28.  
  29.                     require_once "connect.php";
  30.             mysqli_report(MYSQLI_REPORT_STRICT);
  31.  
  32.             try
  33.             {
  34.  
  35.                 $polaczenie= new mysqli($host,$db_user,$db_password,$db_name);
  36.                     if ($polaczenie->connect_errno!=0)
  37.                     {
  38.                         throw new Exception(mysqli_connect_errno());
  39.                     }
  40.                     else
  41.                     {
  42.                     $zapytanie = $polaczenie->query("SELECT * FROM uzytkownicy WHERE pytanie='$pytanie' AND odpowiedz='$odpowiedz'");
  43.  
  44. 						if(!$zapytanie) throw new Exception($polaczenie->error);
  45. 						else{
  46. 								$ile_pytanie=$zapytanie->num_rows;
  47. 								if($ile_pytanie!=$pytanie)
  48. 								{
  49. 								$wszystko_OK=false;
  50. 								$_SESSION['e_pytanie']="Pytania nie zgadzają się!";	
  51. 								}
  52. 								$ile_odpowiedz=$zapytanie->num_rows;
  53. 								if($ile_odpowiedz!=$odpowiedz)
  54. 								{
  55. 								$wszystko_OK=false;
  56. 								$_SESSION['e_odpowiedz']="Odpowiedzi nie zgadzają się!";
  57. 								}
  58. 							}
  59. 					$zapytanie->close();
  60.                         if ($wszystko_OK==true)
  61.                             {
  62.  
  63.                                 $zmiana = "UPDATE uzytkownicy SET pass = '$haslo_hash' WHERE user='$user'";
  64.                                 if ($polaczenie->query($zmiana) === true){
  65.                                     echo '<h3> hasło zmienione , możesz się zalogować</h3> ';
  66.                                 }
  67.                                 else{
  68.                                         throw new Exception($polaczenie->error);
  69.                                         }
  70.                              }
  71.  
  72.                         $polaczenie->close();
  73.                     }
  74.     }
  75.             catch(Exception $e)
  76.             {
  77.                 echo'<br />info deweloperskie: '.$e;
  78.             }
  79. }
  80.  
  81.  
  82. ?>
  83.  
  84. <!DOCTYPE HTML>
  85. <html lang="pl">
  86. <head>
  87.     <meta charset="utf-8" />
  88.     <meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1" />
  89.     <title>Reset hasła </title>
  90. </head>   
  91. <body>
  92. <form method="post">
  93.  
  94.         Nazwa użytkownika: <br /> <input type="text" name="login" /><br />
  95.         Twoje hasło: <br /> <input type="text" name="haslo1" /><br />
  96.                  <?php
  97. 	            if (isset($_SESSION['e_haslo']))
  98. 	            {
  99. 	                echo '<div class="error">'.$_SESSION['e_haslo'].'</div>';
  100. 	                unset($_SESSION['e_haslo']);
  101. 	            }
  102. 	        ?>
  103.         <input type="hidden" name="wyslane" value="TRUE" />
  104.         Powtórz hasło: <br /> <input type="text" name="haslo2" /><br />
  105.         Pytanie zabezpieczające: <br /> <input type="text" name="pytanie" /><br />
  106.         <?php
  107. 	            if (isset($_SESSION['e_pytanie']))
  108. 	            {
  109. 	                echo '<div class="error">'.$_SESSION['e_pytanie'].'</div>';
  110. 	                unset($_SESSION['e_pytanie']);
  111. 	            }
  112. 	        ?>
  113.         Odpowiedż: <br /> <input type="text" name="odpowiedz" /><br />
  114.         <?php
  115. 	            if (isset($_SESSION['e_odpowiedz']))
  116. 	            {
  117. 	                echo '<div class="error">'.$_SESSION['e_odpowiedz'].'</div>';
  118. 	                unset($_SESSION['e_odpowiedz']);
  119. 	            }
  120. 	        ?>
  121.         <br />
  122.         <input type="submit" value="Zmień hasło" />
  123.  
  124.  
  125.     </form>
  126.  
  127.  
  128. </body>
  129. </html>
[PHP] pobierz, plaintext

Będę wdzięczny za wszelkie podpowiedzi , wskazanie błędu .Podpowiedź jak go rozwiązać
Szymas
27.07.2016, 12:57:42
Myślę, że problem pojawia się tutaj:
[SQL] pobierz, plaintext 
  1. $zapytanie = $polaczenie->query("SELECT * FROM uzytkownicy WHERE pytanie='$pytanie' AND odpowiedz='$odpowiedz'");
[SQL] pobierz, plaintext


Czy nie powinieneś pobrać pytania danego użytkownika? W tym momencie pobierasz użytkowników, którzy mają ustawione takie pytanie i odpowiedź. Co w przypadku gdy kilka osób będzie miało takie same wartości? smile.gif
abel1
27.07.2016, 20:16:59
Miałeś na myśli żeby w zapytaniu wstawić użytkownika który prosi o zmianę hasła ?

w formularzu jest zmienne 
[PHP] pobierz, plaintext 
  1.   $user = $_POST['login'];
[PHP] pobierz, plaintext

W zapytaniu jak zmieniam na 
[PHP] pobierz, plaintext 
  1. $zapytanie = $polaczenie->query("SELECT user='$user' FROM uzytkownicy WHERE pytanie='$pytanie' AND odpowiedz='$odpowiedz'");
[PHP] pobierz, plaintext

nie przynosi to efektu , nadal wyświetla że pytanie i odpowiedz są nieprawidłowe

Naprawdę nikt nie wie co powoduje ten problem ?
Gdzie w tym kodzie mam skopane ?
viking
27.07.2016, 20:43:08
Teraz masz skopane zapytanie. Sprawdzanie usera powinno być w WHERE.
Poczytaj o bindowaniu parametrów bo masz możliwe sql injection
abel1
28.07.2016, 06:07:47
Jak dam usera do WHERE to wywala się błędem w zapytaniu więc user musi pozostać gdzie jest
Bindowanie , no tak jak będe miał zamiar tego gdziesz urzywać to dodam ,ale to musi najpierw działac
Ja sie ucze PHP a to robie jako ćwiczenie
Kurcze i nie moge przebrnac przez ten problem już 3 dzień
viking
28.07.2016, 06:22:39
W jaki sposób dodajesz do where?
abel1
28.07.2016, 14:42:20 
[PHP] pobierz, plaintext 
  1. $zapytanie = $polaczenie->query("SELECT pytanie='$pytanie' AND odpowiedz='$odpowiedz' FROM uzytkownicy WHERE user='$user'");
[PHP] pobierz, plaintext


Niestety pisze mi że błedne pytanie i odpowiedź
A w bazie taki format zapytania działa
Pyton_000
28.07.2016, 14:44:50
var_dump("SELECT pytanie='$pytanie' AND odpowiedz='$odpowiedz' FROM uzytkownicy WHERE user='$user'");
viking
28.07.2016, 14:54:58
I naprawdę nie rzuca ci się w oczy że twoje zapytanie nijak ma się do składni select?
http://dev.mysql.com/doc/refman/5.7/en/select.html
Pyton_000
28.07.2016, 15:03:00
Viking jakbyś odpalił to byś zobaczył że zadziała biggrin.gif
Ale co fakt to fakt że jest bezsensowne smile.gif
viking
28.07.2016, 15:05:53
Pewnie zwróci 1 ale takie błądzenie nie ma sensu.
abel1
28.07.2016, 15:12:55
var_dump jest błedne
Fatal error: Call to undefined method mysqli::var_dump()

Ja sie dopiero ucze , prosze o wyrozumiałość
Pyton_000
28.07.2016, 15:14:08
Ty masz to odpalić przed zapytaniem a nie wywoływać z mysqli...

Poza tym to że się uczysz nie zwalnia Cię z używania manuala...
abel1
28.07.2016, 17:28:48
var_dump jak zapewne było do przewidzenia wyświetlił to co wpisuje w formularzu

string(85) "SELECT pytanie='*******' AND odpowiedz='*********' FROM uzytkownicy WHERE user='*******'"
Więc z formularza jest to przesyłane
Rysh
28.07.2016, 17:32:59
To teraz porównaj sobie Twoje zapytanie z np tym:
[SQL] pobierz, plaintext 
  1. SELECT col_name FROM tbl_name WHERE col_name > 0;
[SQL] pobierz, plaintext

I koniecznie do przeczytania:
http://dev.mysql.com/doc/refman/5.7/en/select.html
abel1
28.07.2016, 19:52:42
Próbowałem na różne sposoby zadawać pytanie do MySQL ale coś mi to nie wychodzi
Ciekawe jet natomiast to że jak podam pytanie i odpowiedź które użyłem na samym początku (kilka dni temu) jak wydawało mi sie że to działa hasło zmienia i nie wywala się błędem
Trochę zagmatwałem chyba wyjaśnię.
Kilka dni temu ja zaczęło mi to działać użyłem pytania dajmy na to "pytaniezabezpieczajace" odpowiedż " odpowiedznatopytanie"
To pytanie i odpowiedź była przypisana do użytkownika którego ręcznie skasowałem z bazy danych
Teraz tylko i wyłącznie z tą kombinacja ( pytanie - odpowiedź ) działa
Wpisywanie innych (zapisanych w bazie MySQL ) pytań i odpowiedzi nie działa ,zawsze pisze że są blednę
viking
28.07.2016, 20:10:11
W pierwszym poście miałeś chociaż poprawne zapytanie do bazy. Sensu co prawda to rozwiązanie nie ma ale za to składnia była prawidłowa. Moja rada. Na początek, jeśli jeszcze nie masz, zainstaluj lokalnie mysql, jakiś program do zarządzania typu heidisql, stwórz sobie strukturę testową i popróbuj napisać różne zapytania. Zobacz jakie są efekty w edytorze i dopiero je przenieś do PHP. I CZYTAJ dokumentację bo widać wyraźnie że tego nie robisz.
Dopiero jak to ogarniesz poczytaj o zasadach normalizacji, kluczach obcych, złączeniach JOIN. Potem wybierz użytkownika po loginie i dopiero sprawdzaj jego pytanie. Pewnie masz jakieś dane zapisane w sesji stąd za drugim razem działa inaczej.
Z innych uwag. Nie zapisuj komunikatów w sesji tylko np. jakiejś tablicy. Nie podstawiaj danych z formularza bezpośrednio do zapytania - binduj parametry. MYSQLI_REPORT_STRICT ustawia rzucanie wyjątków więc bez sensu jest if -> Exception, warunek $ile_pytanie!=$pytanie: jaki sens ma porównanie pytania z ilością zwróconych wierszy?
abel1
29.07.2016, 08:01:17
Tak jak napisałeś tak tez zrobie . Wezmę się jeszcze raz od początku za naukę .
Ale nie daje mi to spokoju , jak się to dzieje że nie przyjmuje tego co wpisuje w formularzu ?
Rozumiem że sesja może trwać i dlatego to co wpisze może nie brać pod uwagę.Ale na innej przeglądarce nie ma tej sesji więc nie powinno to mieć wpływu
viking
29.07.2016, 11:56:09
Już pisałem ze ten if nie ma żadnego sensu. Porównujesz 2 różne dane.
abel1
29.07.2016, 15:55:55
Dziękuje takiej podpowiedzi właśnie oczekiwałem.
Teraz wszystko działa jak powinno
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.