Autor Zpráva
Ben
Profil *
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
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 *

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 *
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
raději to mám takto:
if(!empty($_POST['ok'])){
Alphard
Profil
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.
Ben
Profil *
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

Vaše odpověď

Mohlo by se hodit


Prosím používejte diakritiku a interpunkci.

Ochrana proti spamu. Napište prosím číslo dvě-sta čtyřicet-sedm:

0