Autor | Zpráva | ||
---|---|---|---|
Ben Profil * |
#1 · Zasláno: 17. 9. 2010, 09:54:15
Hledám optimálnější řešení než toto
if($_POST['ok']) { require "config.php"; mysql_query($sql) or die(print "Chyba. Kontaktujte manažera!"); } Nějaké nápady? |
||
Budulinek Profil |
#2 · Zasláno: 17. 9. 2010, 10:31:18
Někde jsem zaslech, že na hledání je snad dobrý google :)
Ale vážně. Z tohoto dotazu nejsem moc chytrý. Co konkrétně se ti nelíbí? |
||
Ben Profil * |
#3 · Zasláno: 17. 9. 2010, 12:11:52
if($_POST['ok']) { //vloží soubor na připojení k DB require "config.php"; //provede SQL dotaz mysql_query($sql) //pokud se neprovede dotaz, ukončí s hláškou or die(print "Chyba. Kontaktujte manažera!"); } Jen mě zajímá názor ostatních, zkušenějších, zdali by bylo vhodné ještě něco ověřovat, či přidat/ubrat/změnit. |
||
Ben Profil * |
#4 · Zasláno: 17. 9. 2010, 13:35:53
Takže nakonec takhle:
if($_POST['ok']) { require "config.php"; if (mysql_query($sql)){echo "Děkujeme za Váš názor!";} else echo "Chyba. Kontaktujte manažera!"; } else { //formulář } |
||
Keeehi Profil |
#5 · Zasláno: 17. 9. 2010, 15:12:41
raději to mám takto:
if(!empty($_POST['ok'])){ |
||
Alphard Profil |
#6 · Zasláno: 17. 9. 2010, 16:16:27
Ben:
require v podmínce se mi nezdá jako šťastné řešení. Co když budete mít za sebou dva podobné kódy? Dáte všude require_once()? Config se obvykle načítá na začátku (protože v něm nemusí být jen připojení k databázi). Jestli se nechcete zbytečně připojovat, lze použít lazy přístup. Pokud jde o ošetření chyb, tak by se hodil testovací mód, ve které se budou vypisovat mysql_error() a sestavené dotazy. Všechno nemusíte vytvářet sám, ale můžete zkusit třeba dibi. Usnadní vám i sestavování dotazů, ošetření proti sql injection apod. |
||
Časová prodleva: 3 dny
|
|||
Ben Profil * |
#7 · Zasláno: 20. 9. 2010, 21:21:20
Keeehi:
> raději to mám takto: > > if(!empty($_POST['ok'])){ A je nějaký rozdíl mezi řešením mím a tímto? Alphard: „můžete zkusit třeba dibi“ O nějakém Frameworku uvažuju už dlouho. Jdu do toho ;) Děkuji |
||
Časová prodleva: 14 let
|
0