Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [PHP] Edycja danych
Forum PHP.pl > Forum > Przedszkole
solandsol12
27.09.2008, 14:10:30
Witam,

mam pewien problem,a mianowicie:

stworzyłem panel logowania dla zarejestrowanych użytkowników z możliwością edycji swoich danych. Problem tkwi w tym, że edycja loginu jest bezproblemowa, ale jeżeli zmienię w tym samym czasie Login i Hasło to zmieni się tylko Login, a hasło pozostanie domyślne, natomiast jeżeli wejdę do panelu i zmienię samo hasło, a login pozostawię bez zmian, to hasło zmieni się na nowe. Nie wiem co z tym począć. Proszę o pomoc. Oto kawałek kodu PHP, odpowiedzialny za powyższe czynności:

[PHP] pobierz, plaintext 
  1. <?php
  2. $login_zmien = $_POST[login_zmien];
  3. if (strlen($_POST['login_zmien'])>=5) {
  4. mysql_query("UPDATE `uzytkownicy` SET `login` = '$login_zmien' WHERE `login`= '$login'") or die('Wystąpił błąd!');
  5. session_destroy();
  6. header('Locatio: ../login.php?wyloguj=1');
  7. }
  8.  
  9. else if($_POST[login_zmien]){
  10. echo 'Twój login jest za krótki !';
  11. }
  12.  
  13. if (strlen($_POST['haslo_zmien'])>=5) {
  14. $haslo_zmien = md5($_POST[haslo_zmien]);
  15. mysql_query("UPDATE `uzytkownicy` SET `haslo` = '$haslo_zmien' WHERE `login`= '$login'") or die('Wystąpił błąd!');
  16. session_destroy();
  17. header('Locatio: ../login.php?wyloguj=1');
  18. }
  19.  
  20. else if ($_POST[haslo_zmien]) {
  21. echo 'Wybrane hasło jest za krótkie';
  22. }
  23.  
  24. ob_end_flush();
  25. ?>
[PHP] pobierz, plaintext


Pozdrawiam.
melkorm
27.09.2008, 14:19:40
przyczyną jest to że przy zmianie loginu pierw zmieniasz login przypuśćmy z test na test2 gdzie login = test w drugim kroku zaś zmieniasz hasło z hasło na hasło1 gdzie login = test a jak widzisz powyżej zmieniłeś login i już usera o loginie "test" nie posiadasz w bazie smile.gif tylko usera o loginie test2.

możesz zamienić kolejnością te 2 zapytania i powinno działać bez problemu tongue.gif
solandsol12
27.09.2008, 14:53:09
Zadziałało, lecz mam jeszcze jeden problem, ponieważ dodałem teraz do tego wszystkiego pole e-mail o kodzie:

[PHP] pobierz, plaintext 
  1. <?php
  2. if (strlen($email_zmien)>10) {
  3. $email_zmien = $_POST[email_zmien];
  4. mysql_query("UPDATE `uzytkownicy` SET `haslo` = '$haslo_zmien' WHERE `login`= '$login'") or die('Wyst?pi? b??d!');
  5. echo '<META HTTP-EQUIV="Refresh" CONTENT="1,URL=dane.php">';
  6. }
  7.  
  8. else if ($_POST[email_zmien]) {
  9. echo 'Podany e-mail jest za krótki !<br/>';
  10. }
  11. ?>
[PHP] pobierz, plaintext


Nie wiem dlaczego gdy zmienię e-mail na mniejszy niż 10 znaków to nie pokazuje mi komunikatu o błędzie, tylko po prostu zakańcza sesje, natomiast jeżeli zmienię na e-mail większy niż 10 znaków to go zapisuje.
ArekJ
27.09.2008, 15:07:37
Daj tak:

[PHP] pobierz, plaintext 
  1. <?php
  2. if (strlen($email_zmien)>1) {
  3. $email_zmien = $_POST[email_zmien];
  4. mysql_query("UPDATE `uzytkownicy` SET `haslo` = '$haslo_zmien' WHERE `login`= '$login'") or die('Wyst?pi? b??d!');
  5. echo '<META HTTP-EQUIV="Refresh" CONTENT="1,URL=dane.php">';
  6. }
  7.  
  8. else if ($_POST[email_zmien]) {
  9. echo 'Podany e-mail jest za krótki !<br/>';
  10. }
  11. ?>
[PHP] pobierz, plaintext
solandsol12
27.09.2008, 15:12:49
Dalej to samo, zauważyłem jeszcze jedną rzecz, a mianowicie gdy wpiszę w haśle jedną literke, np. s to zapisuje mi ją w bazie danych jako hasło.
melkorm
27.09.2008, 15:29:37 
[PHP] pobierz, plaintext 
  1. <?php
  2. if (strlen($email_zmien)>10) {
  3. $email_zmien = $_POST[email_zmien];
  4. ?>
[PHP] pobierz, plaintext


Pierw przypisujesz zmienną później ją sparwdzasz nie na odwrót - zmień to tongue.gif
solandsol12
27.09.2008, 15:31:20
Niestety nic to nie dało.
melkorm
27.09.2008, 15:42:13
Oka ... dlaczego updejtujesz pole hasło jeżeli zmieniasz e-mail ?

[PHP] pobierz, plaintext 
  1. <?php
  2. $email_zmien = $_POST['email_zmien'];
  3. if (strlen($email_zmien) > 10) {
  4. mysql_query("UPDATE `uzytkownicy` SET `haslo` = '$haslo_zmien' WHERE `login`= '$login'") or die('Wyst?pi? b??d!'); // zmien to zapytanie na poprawne!
  5. echo '<META HTTP-EQUIV="Refresh" CONTENT="1,URL=dane.php">';
  6. }
  7.  
  8. else if ($_POST['email_zmien']) {
  9. echo 'Podany e-mail jest za krótki !<br/>';
  10. }
  11. ?>
[PHP] pobierz, plaintext
solandsol12
27.09.2008, 15:50:28
Niestety dalej nie pokazuje komunikatu , że e-mail jest za krótki.
Lion_87
27.09.2008, 15:54:21
Cytat
else if ($_POST['email_zmien']) {
echo 'Podany e-mail jest za krótki !<br/>';
}


a niemozesz tak

[PHP] pobierz, plaintext 
  1. <?php
  2. if (strlen($email_zmien) < 10) {
  3. echo 'Podany e-mail jest za krótki !<br/>';
  4. }
  5. ?>
[PHP] pobierz, plaintext
Petermechanic
27.09.2008, 15:55:07
<?php
[PHP] pobierz, plaintext 
  1. <?php
  2. $email_zmien = $_POST['email_zmien'];
  3. if (strlen($email_zmien) > 10) {
  4. mysql_query("UPDATE `uzytkownicy` SET `haslo` = '$haslo_zmien' WHERE `login`= '$login'") or die('Wyst?pi? b??d!'); // zmien to zapytanie na poprawne!
  5. echo '<META HTTP-EQUIV="Refresh" CONTENT="1,URL=dane.php">';
  6. }
  7.  
  8. else{
  9. echo 'Podany e-mail jest za krótki !<br/>';
  10. }
  11. ?>
[PHP] pobierz, plaintext
solandsol12
27.09.2008, 15:56:40
Dalej to samo.
melkorm
27.09.2008, 15:57:11
daj przed warunkiem :

[PHP] pobierz, plaintext 
  1. <?php
  2. var_dump(strlen($_POST['email_zmien']));
  3. ?>
[PHP] pobierz, plaintext


I pokaż nam co wyświetliło :]
solandsol12
27.09.2008, 16:01:43
Wyświetliło : int(0) .

Pozdrawiam.
melkorm
27.09.2008, 16:06:28
Jeżeli coś wpisujesz i wyświetla int(0) to wtedy nie zlicza, pokaż formularz od e-mail'a.
solandsol12
27.09.2008, 16:08:43
Nie mam jak sprawdzić ile po wysłaniu formularza będzie mieć znaków dane pole, ponieważ jak wysyłam to zakańcza mi sesję i wylogowuje, a nie pokazuje błędu o nieprawidłowej ilości znaków w polu email
melkorm
27.09.2008, 16:17:02
var_dump blokuje bodajże zmiane headera ale nie wiem czy w tym przypadku tongue.gif

No ale wyświetliło Tobie int(0) .. więc jak dasz w pole post'a od maila coś to ci pokaże ile znaków wpisałeś ... .

Pokaż ten formularz! tongue.gif
solandsol12
27.09.2008, 16:19:45
Oto formularz:

[HTML] pobierz, plaintext 
  1. <form action="dane.php" method="post">
  2. Login: <input type="text" name="login_zmien" value="'.$_SESSION['login'].'"><br/><br/>
  3. Hasło: <input type="password" name="haslo_zmien"><br/><br/>
  4. E-mail: <input type="text" name="email_zmien" value="'.$email_wyswietl.'"><br/><br/>
  5. <input type="submit" value="zmień">
  6. </form>
[HTML] pobierz, plaintext
melkorm
27.09.2008, 16:42:16 
[PHP] pobierz, plaintext 
  1. <?php
  2. $email_zmien = $_POST['email_zmien'];
  3. if (strlen($email_zmien)>10) {
  4. mysql_query("UPDATE `uzytkownicy` SET `haslo` = '$haslo_zmien' WHERE `login`= '$login'") or die('Wyst?pi? b??d!');
  5. echo '<META HTTP-EQUIV="Refresh" CONTENT="1,URL=dane.php">';
  6. }
  7.  
  8. else if ($_POST['email_zmien']) {
  9. echo 'Podany e-mail jest za krótki !<br/>';
  10. }
  11. ?>
[PHP] pobierz, plaintext 

[HTML] pobierz, plaintext 
  1. <form action="" method="post">
  2. <input type="text" name="email_zmien" value="aaaaaaaa">
  3. <input type="submit">
  4. </form>
[HTML] pobierz, plaintext


Wrzuciłem na localhost dokładnie tak jak powyżej i zadziałało - wyświetla błąd iż e-mail jest zakrótki.
solandsol12
27.09.2008, 16:44:47
Jak usunąłem te dwie linijki:

[HTML] pobierz, plaintext 
  1. Login: <input type="text" name="login_zmien" value="'.$_SESSION['login'].'"><br/><br/>
  2. Hasło: <input type="password" name="haslo_zmien"><br/><br/>
[HTML] pobierz, plaintext


z formularza to działa, hmmm
melkorm
27.09.2008, 16:48:34
z tego co widze po session wyświetlasz tego forma z echo -.-? zrób to normalnei w HTML'u ... i daj w value poprostu:
[PHP] pobierz, plaintext 
  1. <?php echo $_SESSION['login'] ?>
[PHP] pobierz, plaintext


bo jak wkleiłem te dwa inputy do siebie to działało ;]
solandsol12
27.09.2008, 16:52:29
Czy mógłbym podać cały kod, bo nie wiem, coś źle mi wychodzi.
melkorm
27.09.2008, 16:53:19
możesz chyba że jest strasznie długi to wrzuć na phpfi.com ;p
solandsol12
27.09.2008, 17:00:06
Albo może lepiej: podaj jak możesz poprawiony kod, bo już sam nie wiem co jest źle.
melkorm
27.09.2008, 17:04:48 
[PHP] pobierz, plaintext 
  1. <?php
  2. $email_zmien = $_POST['email_zmien'];
  3. if (strlen($email_zmien)>10) {
  4. mysql_query("UPDATE `uzytkownicy` SET `haslo` = '$haslo_zmien' WHERE `login`= '$login'") or die('Wyst?pi? b??d!');
  5. echo '<META HTTP-EQUIV="Refresh" CONTENT="1,URL=dane.php">';
  6. }
  7.  
  8. else if ($_POST['email_zmien']) {
  9. echo 'Podany e-mail jest za krótki !<br/>';
  10. }
  11. ?>
  12.  
  13. <form action="dane.php" method="post">
  14. Logi: <input type="text" name="login_zmien" value="<?php echo $_SESSION['login'] ?>"><br/><br/>
  15. Hasło: <input type="password" name="haslo_zmien"><br/><br/>
  16. Mail: <input type="text" name="email_zmien" value="<?php echo $_SESSION['mail'] ?>">
  17. <input type="submit">
  18. </form>
[PHP] pobierz, plaintext


Jeżeli to nei zadziała to daj cały kod .... . winksmiley.jpg
solandsol12
27.09.2008, 17:08:56 
[PHP] pobierz, plaintext 
  1. <?php
  2. ob_start();
  3. session_start();
  4.  
  5. include("../config.php");
  6.  
  7. // zmiana loginu
  8.  
  9. $login_zmien = $_POST[login_zmien];
  10. if (strlen($_POST['login_zmien'])>=5) {
  11. mysql_query("UPDATE `uzytkownicy` SET `login` = '$login_zmien' WHERE `login`= '$login'") or die('Wystąpił błąd!');
  12. session_destroy();
  13. header('Locatio: ../login.php?wyloguj=1');
  14. }
  15.  
  16. else if($_POST[login_zmien]){
  17. echo 'Twój login jest za krótki !';
  18. }
  19.  
  20. // zmiana hasła
  21.  
  22. if (strlen($_POST['haslo_zmien'])>1) {
  23. $haslo_zmien = md5($_POST['haslo_zmien']);
  24. mysql_query("UPDATE `uzytkownicy` SET `haslo` = '$haslo_zmien' WHERE `login`= '$login_zmien'") or die('Wystąpił błąd!');
  25. session_destroy();
  26. header('Locatio: ../login.php?wyloguj=1');
  27. }
  28.  
  29. else if ($_POST['haslo_zmien']) {
  30. echo 'Wybrane hasło jest za krótkie';
  31. }
  32.  
  33. // zmiana emaila
  34.  
  35. $email_zmien = $_POST['email_zmien'];
  36. if (strlen($email_zmien)>10) {
  37. mysql_query("UPDATE `uzytkownicy` SET `haslo` = '$haslo_zmien' WHERE `login`= '$login'") or die('Wyst?pi? b??d!');
  38. echo '<META HTTP-EQUIV="Refresh" CONTENT="1,URL=dane.php">';
  39. }
  40.  
  41. else if ($_POST['email_zmien']) {
  42. echo 'Podany e-mail jest za krótki !<br/>';
  43. }
  44.  
  45. if ($_SESSION['login']) {
  46. ?>
  47.  
  48. Twoje dan: <br/><br/>
  49.  
  50. <form action="dane.php" method="post">
  51. Log: <input type="text" name="login_zmien" value="<?php echo $_SESSION['login'] ?>"><br/><br/>
  52. Hasło: <input type="password" name="haslo_zmien"><br/><br/>
  53. Mail: <input type="text" name="email_zmien">
  54. <input type="submit">
  55. </form>
  56. </form>
  57.  
  58. <br/><br/>
  59. <a href="glowna.php">wróć</a>
  60. <?php
  61. }
  62.  
  63. else {
  64. echo 'Nie jestes zalogowany !';
  65. }
  66.  
  67. ob_end_flush(); 
  68. ?>
[PHP] pobierz, plaintext
melkorm
27.09.2008, 17:21:03
ponieważ zawsze robisz update loginu .... i zawsze następuje zniszczenie sessji i wylogowanie ... .
solandsol12
27.09.2008, 17:23:32
A jak to zmienić?
melkorm
27.09.2008, 17:29:29
Dane z sessji możesz porównać z danymi wpisywanymi przes usera czyli np :

warunek :
[PHP] pobierz, plaintext 
  1. <?php
  2. (strlen($_POST['login_zmien'])>=5)
  3. ?>
[PHP] pobierz, plaintext

zmienić na:
[PHP] pobierz, plaintext 
  1. <?php
  2. (strlen($_POST['login_zmien'])>=5 && $_POST['login_zmien'] != $_SESSION['login'])
  3. ?>
[PHP] pobierz, plaintext


oczywiście to wiele nei da bo przy zmianie loginu i maila zmien isię tylko login .. ale łatwo w Twoim przypadku to zmienić:

[PHP] pobierz, plaintext 
  1. <?php
  2. $zmiana = false;
  3. $login_zmien = $_POST['login_zmien'];
  4. if ((strlen($_POST['login_zmien'])>=5 && $_POST['login_zmien'] != $_SESSION['login'])) {
  5. mysql_query("UPDATE `uzytkownicy` SET `login` = '$login_zmien' WHERE `login`= '$login'") or die('Wystąpił błąd!');
  6. $zmiana = true;
  7. }
  8. else if($_POST['login_zmien']){
  9. echo 'Twój login jest za krótki !';
  10. }
  11.  
  12. // zmiana hasła
  13.  
  14. if (strlen($_POST['haslo_zmien'])>1) {
  15. $haslo_zmien = md5($_POST['haslo_zmien']);
  16. mysql_query("UPDATE `uzytkownicy` SET `haslo` = '$haslo_zmien' WHERE `login`= '$login_zmien'") or die('Wystąpił błąd!');
  17. $zmiana = true;
  18. }
  19. else if ($_POST['haslo_zmien']) {
  20. echo 'Wybrane hasło jest za krótkie';
  21. }
  22.  
  23. // zmiana emaila
  24.  
  25. $email_zmien = $_POST['email_zmien'];
  26. if (strlen($email_zmien)>10) {
  27. mysql_query("UPDATE `uzytkownicy` SET `haslo` = '$haslo_zmien' WHERE `login`= '$login'") or die('Wyst?pi? b??d!');
  28. echo '<META HTTP-EQUIV="Refresh" CONTENT="1,URL=dane.php">';
  29. }
  30. else if ($_POST['email_zmien']) {
  31. echo 'Podany e-mail jest za krótki !<br/>';
  32. }
  33.  
  34. if($zmiana) {
  35.     session_destroy();
  36. header('Location &nbsp: ../login.php?wyloguj=1');
  37. }
  38. ?>
[PHP] pobierz, plaintext
solandsol12
27.09.2008, 17:44:20
Niestety teraz gdy chcę zmienić sam e-mail to w bazie się on nie zmienia, a hasło wtedy zmienia się na pustkę w bazie i wyskakuje napis: Twój login jest za krótki !

Oto kod:

[PHP] pobierz, plaintext 
  1. <?php
  2. ob_start();
  3. session_start();
  4.  
  5. include("../config.php");
  6.  
  7. // zmiana loginu
  8. $zmiana = false;
  9. $login_zmien = $_POST['login_zmien'];
  10. if ((strlen($_POST['login_zmien'])>=5 && $_POST['login_zmien'] != $_SESSION['login'])) {
  11. mysql_query("UPDATE `uzytkownicy` SET `login` = '$login_zmien' WHERE `login`= '$login'") or die('Wystąpił błąd!');
  12. $zmiana = true;
  13. }
  14. else if($_POST['login_zmien']){
  15. echo 'Twój login jest za krótki !';
  16. }
  17.  
  18. // zmiana hasła
  19.  
  20. if (strlen($_POST['haslo_zmien'])>1) {
  21. $haslo_zmien = md5($_POST['haslo_zmien']);
  22. mysql_query("UPDATE `uzytkownicy` SET `haslo` = '$haslo_zmien' WHERE `login`= '$login_zmien'") or die('Wystąpił błąd!');
  23. $zmiana = true;
  24. }
  25. else if ($_POST['haslo_zmien']) {
  26. echo 'Wybrane hasło jest za krótkie';
  27. }
  28.  
  29. // zmiana emaila
  30.  
  31. $email_zmien = $_POST['email_zmien'];
  32. if (strlen($email_zmien)>10) {
  33. mysql_query("UPDATE `uzytkownicy` SET `email` = '$email_zmien' WHERE `login`= '$login_zmien'") or die('Wyst?pi? b??d!');
  34. echo '<META HTTP-EQUIV="Refresh" CONTENT="1,URL=dane.php">';
  35. }
  36. else if ($_POST['email_zmien']) {
  37. echo 'Podany e-mail jest za krótki !<br/>';
  38. }
  39.  
  40. if($zmiana) {
  41. session_destroy();
  42. header('Location &nbsp: ../login.php?wyloguj=1');
  43. }
  44.  
  45. if ($_SESSION['login']) {
  46. ?>
  47.  
  48. Twoje dan: <br/><br/>
  49.  
  50. <form action="dane.php" method="post">
  51. Log: <input type="text" name="login_zmien" value="<?php echo $_SESSION['login'] ?>"><br/><br/>
  52. Hasło: <input type="password" name="haslo_zmien"><br/><br/>
  53. Mail: <input type="text" name="email_zmien">
  54. <input type="submit">
  55. </form>
  56. </form>
  57.  
  58. <br/><br/>
  59. <a href="glowna.php">wróć</a>
  60. <?php
  61. }
  62.  
  63. else {
  64. echo 'Nie jestes zalogowany !';
  65. }
  66.  
  67. ob_end_flush();
[PHP] pobierz, plaintext
melkorm
27.09.2008, 17:56:56 
[PHP] pobierz, plaintext 
  1. <?php
  2. ob_start();
  3. session_start();
  4.  
  5. include("../config.php");
  6. $zmiana = false;
  7. $login = $_SESSION['login'];
  8. // zmiana hasła
  9. if (strlen($_POST['haslo_zmien'])>1) {
  10.     $haslo_zmien = md5($_POST['haslo_zmien']);
  11.     mysql_query("UPDATE `uzytkownicy` SET `haslo` = '$haslo_zmien' WHERE `login`= '$login'") or die('Wystąpił błąd!');
  12.     $zmiana = true;
  13. }
  14. elseif($_POST['haslo_zmien']) {
  15.     echo 'Wybrane hasło jest za krótkie';
  16. }
  17.  
  18. // zmiana emaila
  19.  
  20. $email_zmien = $_POST['email_zmien'];
  21. if (strlen($email_zmien)>10) {
  22. mysql_query("UPDATE `uzytkownicy` SET `email` = '$email_zmien' WHERE `login`= '$login'") or die('Wyst?pi? b??d!');
  23.     echo '<META HTTP-EQUIV="Refresh" CONTENT="1,URL=dane.php">';
  24. }
  25. else if ($_POST['email_zmien']) {
  26.     echo 'Podany e-mail jest za krótki !<br/>';
  27. }
  28.  
  29. // zmiana loginu
  30.  
  31. $login_zmien = $_POST['login_zmien'];
  32. if ($_POST['login_zmien'] != $_SESSION['login']) {
  33.     if(strlen($_POST['login_zmien'])>=5) {
  34.         mysql_query("UPDATE `uzytkownicy` SET `login` = '$login_zmien' WHERE `login`= '$login'") or die('Wystąpił błąd!');
  35.         $zmiana = true;
  36.     }
  37.     else {
  38.         echo 'Twój login jest za krótki !';
  39.     }
  40. }
  41.  
  42. if($zmiana) {
  43.     session_destroy();
  44.     header('Location &nbsp: ../login.php?wyloguj=1');
  45. }
  46.  
  47. if ($_SESSION['login']) {
  48. ?>
  49.  
  50. Twoje da: <br/><br/>
  51.  
  52. <form action="dane.php" method="post">
  53. Lo: <input type="text" name="login_zmien" value="<?php echo $_SESSION['login'] ?>"><br/><br/>
  54. Hasło: <input type="password" name="haslo_zmien"><br/><br/>
  55. Mail: <input type="text" name="email_zmien">
  56. <input type="submit">
  57. </form>
  58. </form>
  59.  
  60. <br/><br/>
  61. <a href="glowna.php">wróć</a>
  62. <?php
  63. }
  64.  
  65. else {
  66. echo 'Nie jestes zalogowany !';
  67. }
  68.  
  69. ob_end_flush();
  70. ?>
[PHP] pobierz, plaintext


Powinno działać ;p
solandsol12
27.09.2008, 18:20:45
Działa smile.gif, wielkie dzięki !
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.