Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [MySQL][PHP]Edycja (zmiana) rekordu
Forum PHP.pl > Forum > Przedszkole
szokmen
22.05.2010, 18:32:16
Witam,
napisałem taki kod jaki widać poniżej i mam problem z edycja rekordu. Po prostu po kliknięciu zamień nic się nie dzieje mógłby ktoś spojrzeć. Kod edycji rekordu wziąłem z jakiegoś tutoriala w internecie ale teraz jakiś dziwny wydaje mi się ten kod.
[PHP] pobierz, plaintext 
  1. <html>
  2. <head>
  3. <meta http-equiv="Content-Type" content="text/html; charset=windows-1250" />
  4. <title>Projekt bazy danych</title>
  5. <link rel="stylesheet" type="text/css" href="styl.css" />
  6. </head>
  7. <body>
  8. <div id="h2">
  9. <form action="formularz.php" method="post"> 
  10. <p align="right">
  11. <input type ="hidden" name="wybor" value="dodaj">
  12. <table>
  13. id klienta:<input type ="text" name="id"><br />
  14. imie:<input type ="text" name="imie"><br />
  15. nazwisko:<input type ="text" name="nazwisko"><br />
  16. nazwa firmy:<input type ="text" name="nazwa_firmy"><br />
  17. telefon:<input type ="text" name="telefon"><br />
  18. <input type ="submit" value="dodaj">
  19. </table>
  20. </p>
  21. </form>
  22. </div>
  23.  
  24.  
  25. <?php 
  26. error_reporting(E_ALL); 
  27. ini_set('display_errors','1');
  28.  
  29. mysql_connect("localhost", "xxxx", "yyyy") or die ("Nie można połączyć się z bazą");
  30. mysql_select_db ("firma") or die ("Nie można połączyć się z bazą firma");
  31. echo '<div id="h0">';
  32. echo '<form action= "formularz.php" method="get">';
  33. echo '<input type ="hidden" name="wybor" value="szukaj">';
  34. echo '<input type= "text" name="napis">';
  35. echo '<input type= "submit" value="Szukaj imie">';
  36. echo '</form>';
  37. echo '</div>';
  38.  
  39. echo '<div id="h3">';
  40. echo '<form action= "formularz.php" method="get">';
  41. echo '<input type ="hidden" name="wybor" value="DEL">';
  42. echo '<input type= "hidden" name="id">';
  43. echo '</form>';
  44. echo '</div>';
  45.  
  46. echo '<form action= "formularz.php" method="get">';
  47. echo '<input type ="hidden" name="wybor" value="edit">';
  48. echo '<input type= "hidden" name="id">';
  49. echo '</form>';
  50.  
  51. echo '<form action="formularz.php" method="get">';
  52. echo '<p align="right">';
  53. echo '<input type ="hidden" name="wybor" value="popraw">';
  54. echo '<table>';
  55. echo 'id klienta:<input type ="text" name="id"><br />';
  56. echo 'imie:<input type ="text" name="imie100""><br />';
  57. echo 'nazwisko:<input type ="text" name="nazwisko100"><br />';
  58. echo 'nazwa firmy:<input type ="text" name="nazwa_firmy100" ><br />';
  59. echo 'telefon:<input type ="text" name="telefon100"><br />';
  60. echo '<input type ="submit" value="zmień">';
  61. echo '</table>';
  62. echo '</p>';
  63. echo '</form>';
  64.  
  65.  
  66.  
  67.  
  68.  
  69. if(isset($_POST["wybor"])== 'dodaj')
  70. {
  71. 	$imie1=$_POST['imie'];
  72. 	$nazwisko1=$_POST['nazwisko'];
  73. 	$nazwa1=$_POST['nazwa_firmy'];
  74. 	$telefon1=$_POST['telefon'];
  75.  
  76.  
  77. 	if($imie1 and $nazwisko1 and $nazwa1 and $telefon1)
  78. 	{
  79.  
  80. 	$ins =mysql_query("insert into klienci set imie='$imie1', nazwisko='$nazwisko1', nazwa_firmy='$nazwa1',telefon='$telefon1'");
  81. 	if($ins) echo "Rekord został dodany poprawnie"; 
  82.     else echo "Błąd nie udało się dodać nowego rekordu"; 
  83. 	}
  84. }
  85.  
  86. else if (isset($_GET["wybor"]) AND $_GET["wybor"]=='DEL')
  87. {
  88. //echo $_GET['id'];
  89. //echo $_GET['wybor'];
  90. 	$zapytanie3="DELETE from Klienci where id_klienta=$_GET[Id];";
  91. 	$del = mysql_query($zapytanie3);
  92. if($del)
  93. 	{
  94. 	echo '<h5>rekord usuniety</h5>';
  95.  
  96. 	}
  97. 	else
  98. 	{
  99. 		 echo '<h5>blad</h5>';
  100. 		mysql_query($del) or die('BŁĄD:'.mysql_error());
  101. 	}
  102. }
  103. //EDYCJA
  104. else if (isset($_GET["wybor"]) AND $_GET["wybor"]=='edit')
  105. {
  106. if (isset($_GET["wybor"]) AND $_GET["wybor"]=='popraw')
  107. if($_GET[id] && $_GET[imie100] && $_GET[nazwisko100] && $_GET[nazwa_firmy100] && $_GET[telefon])
  108. {
  109. 	$zap100="update klienci set imie='$_GET[imie100]', nazwisko='$_GET[nazwisko100]', nazwa_firmy='$_GET[nazwa_firmy100]',telefon='$_GET[telefon100]' where id_klienta=$_GET[Id]";
  110. 	$wyk100=mysql_query($zap100);
  111. }
  112. }
  113.  
  114. else if (isset($_GET["napis"]))
  115. {
  116.  
  117. 	$zapytanie2="SELECT *from klienci where imie='$_GET[napis]'";
  118. 	$wyk2=mysql_query($zapytanie2);
  119. //SZUKANIE IMIENIA
  120. if ($_GET["wybor"]== 'szukaj')	
  121. {
  122. if(mysql_num_rows($wyk2) > 0)
  123. {
  124.  
  125. echo ("<h1><b>Znaleziono</b></h1>");
  126. //echo '<div id="h6">';
  127. echo "<table cellpading=5 border=2>";
  128. echo "<tr>";
  129. echo "<td><b>Id_klienta</td></b>";
  130. echo "<td><b>Imie</td></b>";
  131. echo "<td><b>Nazwisko</td></b>";
  132. echo "<td><b>Nazwa firmy</td></b>";
  133. echo "<td><b>Telefon</td></b>";
  134. echo "</tr>\n";
  135. }
  136. else echo ("<h1><b>Nie ma</b></h1>");
  137.  
  138. echo '<br/>';
  139. while($rek=mysql_fetch_array($wyk2))
  140. {
  141.     $id2  =$rek[0];
  142. 	$imie2=$rek[1];
  143. 	$nazwisko2=$rek[2];
  144. 	$nazwa_firmy2=$rek[3];
  145. 	$telefon2=$rek[4];
  146.  
  147. echo "<tr>";
  148. echo "<td><b>$id2</td></b>";
  149. echo "<td><b>$imie2</td></b>";
  150. echo "<td><b>$nazwisko2</td></b>";
  151. echo "<td><b>$nazwa_firmy2</td></b>";
  152. echo "<td><b>$telefon2</td></b>";
  153. echo "</tr>\n";	
  154. //echo '</div>';	
  155. }
  156. echo "</table>";
  157.  
  158. echo '<a href ="formularz.php"> Powrot do formularza</a>';
  159.  
  160. }
  161. }
  162.  
  163. $zapytanie1 = "SELECT *from klienci";
  164. $wyk1=mysql_query($zapytanie1);
  165.  
  166. //echo '<div id="h4">';
  167. echo "<table cellpading=5 border=2>";
  168. echo "<tr>";
  169. echo "<td><b>Id_klienta</td></b>";
  170. echo "<td><b>Imie</td></b>";
  171. echo "<td><b>Nazwisko</td></b>";
  172. echo "<td><b>Nazwa firmy</td></b>";
  173. echo "<td><b>Telefon</td></b>";
  174. echo "</tr>\n";
  175.  
  176. while($rekord=mysql_fetch_array($wyk1))
  177. {
  178. 	$id  =$rekord[0];
  179. 	$imie=$rekord[1];
  180. 	$nazwisko=$rekord[2];
  181. 	$nazwa_firmy=$rekord[3];
  182. 	$telefon=$rekord[4];
  183.  
  184. echo "<tr>";
  185. echo "<td><b>$id</td></b>";
  186. echo "<td><b>$imie</td></b>";
  187. echo "<td><b>$nazwisko</td></b>";
  188. echo "<td><b>$nazwa_firmy</td></b>";
  189. echo "<td><b>$telefon</td></b>";
  190. echo "<td><a href = \"formularz.php?wybor=DEL&Id=$id\">kasuj</a></td>";
  191. echo "<td><a href = \"formularz.php?wybor=edit&Id=$id\">edytuj</a></td>";
  192. echo "</tr>\n";	
  193. echo '</div>';
  194. }
  195. echo "</table>";
  196. //echo '</div>';
  197. ?>
  198.  
  199. </body>
  200. </html> 
[PHP] pobierz, plaintext
lobopol
22.05.2010, 19:33:50
A niby co się ma zmieniać jak po wybraniu edycji odpala się to:
[PHP] pobierz, plaintext 
  1. else if (isset($_GET["wybor"]) AND $_GET["wybor"]=='edit')
  2. {
  3. if (isset($_GET["wybor"]) AND $_GET["wybor"]=='popraw') //nawet za to nie przejdzie bo jak już tu jest sprawdzenie to $_GET["wybor"]=='edit' przecież
  4. if($_GET[id] && $_GET[imie100] && $_GET[nazwisko100] && $_GET[nazwa_firmy100] && $_GET[telefon])
  5. {
  6. $zap100="update klienci set imie='$_GET[imie100]', nazwisko='$_GET[nazwisko100]', nazwa_firmy='$_GET[nazwa_firmy100]',telefon='$_GET[telefon100]' where id_klienta=$_GET[Id]";
  7. $wyk100=mysql_query($zap100);
  8. }
  9. }
[PHP] pobierz, plaintext

i zatrzymuję się zaraz na pierwszym ifie. Dalej idąc to twój skrypt jest strasznie nielogiczny, dużo rzeczy sobie utrudniasz, wiele razy sprawdzasz czy dana zmienna istnieje np. 
[PHP] pobierz, plaintext 
  1. isset($_GET["wybor"]) AND $_GET["wybor"]=='edit'
[PHP] pobierz, plaintext
wystarczy samo 
[PHP] pobierz, plaintext 
  1. $_GET["wybor"]=='edit'
[PHP] pobierz, plaintext
przecież jak ma ta wartość to musi już istnieć ta zmienna.
thek
23.05.2010, 11:42:18
Za bardzo zamieszałeś w warunkach i teraz to Ci się mści.
Zmienna wystarczy sprawdzić raz czy istnieje i potem sobie tylko w switchu sprawdzisz jaka to jest.
[PHP] pobierz, plaintext 
  1. if( isset( $jakas_zmienna ) ) {
  2.     switch( $jakas_zmienna ) {
  3.         case 'wartosc1':
  4.             //rob coś
  5.             break;
  6.         case 'wartosc2':
  7.             //rob coś
  8.             break;
  9.         default:
  10.             //tak w ramach zabezpieczenia
  11.     }
  12. }
[PHP] pobierz, plaintext
Czy nie prościej? Przebuduj sobie te warunki bo się pogubisz z tymi if-elseif-elseif.
szokmen
23.05.2010, 16:02:10
To już nieco poprawiony kod. Jednak nadal nie działa mi aktualizacja rekordu :-( Prosiłbym o zerknięcie.
[PHP] pobierz, plaintext 
  1. echo '<div id="h0">';
  2. echo '<form action= "przebudowa.php" method="get">';
  3. echo '<input type ="hidden" name="wybor" value="szukaj">';
  4. echo '<input type= "text" name="napis">';
  5. echo '<input type= "submit" value="Szukaj imie">';
  6. echo '</form>';
  7. echo '</div>';
  8.  
  9. echo '<div id="h3">';
  10. echo '<form action= "przebudowa.php" method="get">';
  11. echo '<input type ="hidden" name="wybor" value="DEL">';
  12. echo '<input type= "hidden" name="id">';
  13. echo '</form>';
  14. echo '</div>';
  15.  
  16. echo '<form action= "przebudowa.php" method="get">';
  17. echo '<input type ="hidden" name="wybor" value="edit">';
  18. echo '<input type= "hidden" name="id">';
  19. echo '</form>';
  20.  
  21. echo '<form action= "przebudowa.php" method="get">';
  22. echo '<input type ="hidden" name="wybor" value="popraw">';
  23. echo '<input type= "hidden" name="id">';
  24. echo '</form>';
  25.  
  26.  
  27. if(isset($_POST["wybor"])== 'dodaj')
  28. {
  29. 	$imie1=$_POST['imie'];
  30. 	$nazwisko1=$_POST['nazwisko'];
  31. 	$nazwa1=$_POST['nazwa_firmy'];
  32. 	$telefon1=$_POST['telefon'];
  33.  
  34.  
  35. 	if($imie1 and $nazwisko1 and $nazwa1 and $telefon1)
  36. 	{
  37.  
  38. 	$ins =mysql_query("insert into klienci set         'imie='$imie1',nazwisko='$nazwisko1',nazwa_firmy='$nazwa1',telefon='$telefon1'");
  39. 	if($ins) echo "Rekord został dodany poprawnie"; 
  40.     else echo "Błąd nie udało się dodać nowego rekordu"; 
  41. 	}
  42. }
  43.  
  44. else if (isset($_GET["wybor"])) 
  45. {
  46. 	switch($_GET["wybor"])
  47. 	{
  48. 	case 'DEL':	
  49. 			$zapytanie3="DELETE from Klienci where id_klienta=$_GET[Id];";
  50. 			$del = mysql_query($zapytanie3);
  51. 			if($del)
  52. 			{
  53. 			echo '<h5>rekord usuniety</h5>';
  54.  
  55. 			}
  56. 			else
  57. 			{
  58. 				echo '<h5>blad</h5>';
  59. 				mysql_query($del) or die('BŁĄD:'.mysql_error());
  60. 			} break;
  61.  
  62. 	case 'edit':
  63. 			$zap10="SELECT *from Klienci where id_klienta='$_GET[Id]';";
  64. 			$wyk10=mysql_query($zap10);
  65. 			$reko=mysql_fetch_array($wyk10);
  66.  
  67. 	    	$id10  =$reko[0];
  68. 			$imie10=$reko[1];
  69. 			$nazwisko10=$reko[2];
  70. 			$nazwa_firmy10=$reko[3];
  71. 			$telefon10=$reko[4];
  72.  
  73.  
  74. 			echo '<form action="przebudowa.php" method="get">';
  75. 			echo '<p align="right">';
  76. 			echo '<table>';
  77. 			echo 'id klienta:<input type ="text" name="id" value="'.$id10.'"><br />';
  78. 			echo 'imie:<input type ="text" name="imie20" value="'.$imie10.'"><br />';
  79. 			echo 'nazwisko:<input type ="text" name="nazwisko20" value="'.$nazwisko10.'"><br />';
  80. 			echo 'nazwa firmy:<input type ="text" name="nazwa_firmy20" value="'.$nazwa_firmy10.'"><br />';
  81. 			echo 'telefon:<input type ="text" name="telefon20" value="'.$telefon10.'"><br />';
  82. 			echo '<input type ="submit" value="zmień">';
  83. 			echo '</table>';
  84. 			echo '</p>';
  85. 			echo '</form>';
  86. 			break;	
  87.  
  88. 	case 'popraw':
  89. 			$imie10= $_GET['imie20'];
  90. 			$nazwisko10= $_GET['nazwisko20'];
  91. 			$nazwa_firmy10= $_GET['nazwa_firmy20'];
  92. 			$telefon10=$_GET['telefon20'];
  93.  
  94. $zap11="UPDATE klienci SET imie='$imie10',nazwisko='$nazwisko10',nazwa_firmy='$nazwa_firmy10', telefon='$telefon10' where id_klienta=$_GET[Id]";
  95.  
  96. 			$wyk11=($zap11);
  97. 			break;
  98. 	}
  99.  
  100.  
  101.  
  102. }
  103.  
[PHP] pobierz, plaintext

lobopol
23.05.2010, 17:09:10
No to teraz masz genialny formularz do szukania i kilka poukrywanych formularzy które nic nie robią (nawet ich wysłać nie możesz).
Dalej masz w kodzie mnóstwo głupot np.
[PHP] pobierz, plaintext 
  1. #
  2. if(isset($_POST["wybor"])== 'dodaj')
[PHP] pobierz, plaintext

Jakikolwiek teraz będzie $_POST["wybor"] to if jest spełniony, ale ty jeszcze wysyłasz dane getem. Wybacz kod jest po prostu makabryczny. Przemyśl sobie jeszcze raz cały ten kod oddziel php od html i krok po kroku sobie to zbuduj.
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.