Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [php]Proszę o poradę !
Forum PHP.pl > Forum > Przedszkole
hostcom
3.03.2008, 16:38:20
witam
właśnie zaczynam się bawić w php i mysql proszę o porady i o naniesienie ewentualnych zmian 
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. require_once("backend/functions.php");
  4. dbconn(false);
  5. loggedinorreturn();
  6. if ($submit == "1") {
  7. $set = array();
  8. $id = addslashes($_POST['id']);
  9. $updateset[] = "id = '$id'";
  10. $url_dotpay = $_POST['url_dotpay'];
  11. $updateset[] = "url_dotpay = '$url_dotpay'";
  12. $url= addslashes($_POST['url']);
  13. $updateset[] = "url = '$url'";
  14. $dot_urlc = addslashes($_POST['urlc']);
  15. $updateset[] = "urlc = '$urlc'";
  16. $tytul = addslashes($_POST['tytul']);
  17. $updateset[] = "tytul = '$tytul'";
  18. $sms = addslashes($_POST['sms']);
  19. $updateset[] = "sms = '$sms'";
  20. $blokuj = addslashes($_POST['blokuj']);
  21. $updateset[] = "blokuj = '$blokuj'";
  22. mysql_query("UPDATE dotpay_config SET " . implode(",", $updateset) . "");
  23. $edited=1;
  24. header("Location: dotpay-config-edit.php?save=ok");
  25. die; 
  26. }
  27. $config_1 = mysql_query("SELECT * FROM dotpay_config");
  28. $config_2 = mysql_query("SELECT * FROM dotpay_config");
  29. $config_3 = mysql_query("SELECT * FROM dotpay_config");
  30. $config_4 = mysql_query("SELECT * FROM dotpay_config");
  31. $config = mysql_query("SELECT * FROM dotpay_config");
  32. stdhead("User CP");
  33. begin_frame("Konfiguracja Modułu DOTPAY.pl (INFO)");
  34. if ($save == ok)
  35. bark2("Aktualizacja OK", "Zmiany zostały wprowadzone!");
  36. print("<table>");
  37. print("<tr><td class=alt3 align=center><font size=1 face=Verdana color=white> </td><td class=alt3 align=center><font size=1 face=Verdana color=white><b>OPIS </b></td></tr>");
  38. print("<tr><td class=table_col1 align=right><b>DotPay.pl ID </td><td class=table_col1 align=left>ID użytkownika serwisu Dotpay(ID sprzedawcy) </td></tr>");
  39. print("<tr><td class=table_col2 align=right><b>URL DotPay.pl: </td><td class=table_col2 align=left>URL serwisu DotPay, gdzie przysyłane są zmoenne z formularza metodą POST </td></tr>");
  40. print("<tr><td class=table_col1 align=right><b>URL powrotny </td><td class=table_col1 align=left>URL strony, gdzie po dokonaniu tranzakcji klient zostanie przeniesiony.<b> Pamiętaj,</b> aby URL zawsze prowadził do pliku <b>(donate-dotpay-confirn.php)</b></td></tr>");
  41. print("<tr><td class=table_col2 align=right><b>URLC powrotny </td><td class=table_col2 align=left>URLC dziła na tej samej zasadzie co URL. Z tym, iż informacje o tranzakcjach 
    są przesyłane <b>niejawnie</b> do serwisu</td></tr>"    );
  42. print("<tr><td class=table_col1 align=right><b>Tytuł</td><td class=table_col1 align=left>Domyślny dla każdej tranzakcji </td></tr>");
  43. print("<tr><td class=table_col2 align=right><b> Tryb SMS Premium</td><td class=table_col2 align=left>Jeżeli jest włączony, działa system SMS przy rejestracji nowego użytkownika</td></tr>");
  44. print("<tr><td class=table_col1 align=right><b> Blokada kanału</td><td class=table_col1 align=left>Zablokowanie wybranego kanału podanego w parametrze kanal. Parametr ten pozwa
    la na ograniczenie możliwości wyboru określonego kanału płatniczego zdefiniowaneg
    o na stronach Sprzedawcy. </td></tr>"    );
  45. print("</table>");
  46. end_frame();
  47. begin_frame("Konfiguracja Modułu DOTPAY.pl</b>");
  48. if ($save == ok)
  49. bark2("Aktualizacja OK", "Zmiany zostały wprowadzone!");
  50. print("<center><table>");
  51. print("<form method=post action=dotpay-config-edit.php>");
  52. print("<input type=hidden name=submit value=1>");
  53. 	while($id = mysql_fetch_array($config))
  54. print("<tr><td class=table_col1 align=right><b>DotPay.pl ID:</b></<td><td class=table_col2><input type=text size=41 name=id value=".$id['id']."></td><td class=table_col1><center><b>".$id['id']."</td></tr>") .
  55. 	 ("<tr><td class=table_col1 align=right><b>URL DotPay.pl:</b></<td><td class=table_col2><input type=text size=41 name=url_dotpay value=".$id['url_dotpay']."></td><td class=table_col1><center><b>".$id['url_dotpay']."</td></tr>") .
  56. 	 ("<tr><td class=table_col1 align=right><b>URL powrotny:</b></<td><td class=table_col2><input type=text size=41 name=url value=".$id['url']."></td><td class=table_col1><b><center>".$id['url']."</td></tr>") .
  57. 	 ("<tr><td class=table_col1 align=right><b>URLC powrotny:</b></<td><td class=table_col2><input type=text size=41 name=urlc value=".$id['urlc']."></td><td class=table_col1><b><center>".$id['urlc']."</td></tr>") .
  58. 	 ("<tr><td class=table_col1 align=right><b>Tytuł:</b></<td><td><textarea type=text cols=30 rows=5 name=tytul value=>" . $id['tytul'] . "</textarea></td><td class=table_col1><center><b>" . $id['tytul'] . "</td></tr>");
  59.  echo("<tr><td class=table_col1><b>Tryb SMS Premium:</td><td class=table_col2><table width=100%><tr><td class=table_col1><center>");
  60. 	  while($g_nowa = mysql_fetch_array($config_3))
  61. 	{	
  62. 	++$num;
  63. unset($selected);
  64. 	if ($g_nowa['sms'] == 'tak')
  65. 	{ $selected[1] = 'selected';  $blokuj1='checked id=tak';	}
  66.    else if ($g_nowa['sms'] == 'nie')
  67. 	{ $selected[3] = 'selected';$blokuj='checked id=nie';}
  68.  echo("<input type=radio name=sms value=tak " . $blokuj1 . "><b><font color=green>WŁĄCZONY </font> </td><td class=table_col1><center>") .
  69. 	 ("<input type=radio name=sms value=nie " . $blokuj . "><font color=red><b>WYŁĄCZONY</font></td></tr></table></td>");
  70. 			}
  71. 			 while($t_nowa = mysql_fetch_array($config_2))
  72. 	{	
  73. 	++$num;
  74. unset($selected);
  75. 	if ($t_nowa['sms'] == 'tak')
  76. 	{ $selected[1] = 'selected';  $name='<font color=green><center><b>WŁĄCZONA</font></b></center>';	}
  77.    else if ($t_nowa['sms'] == 'nie')
  78. 	{ $selected[3] = 'selected';$name='<font color=red><center><b>WYŁĄCZONY</font></b></center>';}
  79.  echo("<td class=table_col1>" . $name . "</td></tr>");
  80. 			 }
  81. 			  while($s_nowa = mysql_fetch_array($config_1))
  82. 	{	
  83. 	++$num;
  84. unset($selected);
  85. 	if ($s_nowa['blokuj'] == '1')
  86. 	{ $selected[0] = 'selected';  $neme='<font color=green><center><b>WŁĄCZONA</font></b></center>';	}
  87.    else if ($s_nowa['blokuj'] == '0')
  88. 	{ $selected[2] = 'selected';$neme='<font color=red><center><b>WYŁĄCZONY</font></b></center>';}
  89. echo ("<tr><td class=table_col1><b>Blokada kanału:</td><td class=table_col2><table width=100%><tr><td class=table_col1><center>");
  90.  }
  91.  while($h_nowa = mysql_fetch_array($config_4))
  92. 	{	
  93. 	++$num;
  94. unset($selected);
  95. 	if ($h_nowa['blokuj'] == '1')
  96. 	{ $selected[1] = 'selected';  $blokuj3='checked id=tak';	}
  97.    else if ($h_nowa['blokuj'] == '0')
  98. 	{ $selected[3] = 'selected';$blokuj4='checked id=nie';}
  99. echo("<input type=radio name=blokuj value=1 " . $blokuj3 . "><font color=green><b>WŁĄCZONY</font></td><td class=table_col1> <center><input type=radio name=blokuj value=0 " . $blokuj4 . "><font color=red><b>WYŁĄCZONY</font></td></tr></table></td><td class=table_col1>" . $neme . "</td></tr>");
  100.  
  101.  echo("<tr><td class=table_col1></td><td class=table_col2><center><input type=submit value=Zmień name=b1></td><td></td><td><tr>");
  102.    }
  103. 	 print("</table>");
  104. end_frame();
  105. stdfoot();
  106. ?>
[PHP] pobierz, plaintext

z góry dziekuje
Xniver
3.03.2008, 16:57:06
1. Kod jest strasznie nie czytelny.(może to wina edytora, polecam darmowego eclipse + PDT)
2. Używaj apostrofów(') zamiast cudzysłowów(")
3. Używasz tutaj register_globals? 
[PHP] pobierz, plaintext 
  1. <?php
  2. if ($submit == "1") {
  3. ?>
[PHP] pobierz, plaintext

4. Zamiast używać mysql_* użyj PDO, Creole, albo jakiegoś ORMa(np. Propel)
5. Nie twórz ,aż tylu tymczasowych zmiennych, np. zamiast
[PHP] pobierz, plaintext 
  1. <?php
  2. $sms = addslashes($_POST['sms']);
  3. $updateset[] = 'sms = "' . $sms; . '"';
  4. ?>
[PHP] pobierz, plaintext

używaj
[PHP] pobierz, plaintext 
  1. <?php
  2. $updateset[] = 'sms = "' . addslashes($_POST['sms']) . '"';
  3. ?>
[PHP] pobierz, plaintext

6. Sprawdzaj czy $_POST['id'] jest liczbą(np. funkcja is_int()) albo obrazu dokonaj konwersji (np. intval())
7. Zainteresuj się programowaniem obiektowym(OOP)
8. Nieprawidłowy kod HTML(zapomniałeś o cudzysłowach)
Cysiaczek
3.03.2008, 17:03:08
Kolego, napisz co jest dla Ciebie niejasne, z czym masz problem etc. Jak kod działa, to reszta przyjdzie z czasem.

Przenoszę na Przedszkole
webdice
3.03.2008, 17:39:07
Co to:

[PHP] pobierz, plaintext 
  1. <?php
  2. $config_1 = mysql_query("SELECT * FROM dotpay_config");
  3. $config_2 = mysql_query("SELECT * FROM dotpay_config");
  4. $config_3 = mysql_query("SELECT * FROM dotpay_config");
  5. $config_4 = mysql_query("SELECT * FROM dotpay_config");
  6. $config = mysql_query("SELECT * FROM dotpay_config");
  7. ?>
[PHP] pobierz, plaintext


niby jest? Po co 5 takich zapytań do bazy? Tworzysz jedno i operujesz na jednej zmiennej.
hostcom
3.03.2008, 22:15:34
tak chcialem zrobi ale od wiesza 65 już nie nie pobiera danych z bazy wiec musiale zrobić osobne

nie wiem gdzie lezy błąd
lukasz_test
25.10.2011, 18:32:15
Wprowadzaj wcięcia w kodzie.
Daiquiri
25.10.2011, 18:34:46
lukasz_test, data!
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.