Mijn script heeft een opbouw zoals:


<?

// id uit database.
$id_query = "SELECT user_id FROM dbm_login WHERE user_name = '".$login_name."'";
$id_result = mysql_query($id_query,$db);
$user_id = mysql_fetch_assoc($id_result);

// Registration string uit database.
$reg_str = "SELECT sess_verify FROM dbm_login WHERE user_name = '".$login_name."'";
$string_result = mysql_query($reg_str,$db);
$session_string = mysql_fetch_assoc($string_result); ?>

-------

Simpele vraag: Is dit een nette manier van phpen ?
ik heb een heel rijtje zoals dit en vraag me af of ik het kwa querys niet dubbel-op doe.
* word verandert *
het ziet er opzich wel duidelijk uit
zolang je het zelf maar duidelijk kan lezen is het goed he ;)
Ik denk niet dat dit werkt:

if(isset($config_inc,$themes_inc,$functions_inc,$banned_inc))
blijkbaar wel... alsik een van die variablen verander in iets wat niet bestaat geeft hij netjes mijn error weer..

Maar hoe zou ik dat anders moeten doen dan?
SanThe schreef op 10.11.2005 11:23
Ik denk niet dat dit werkt:

if(isset($config_inc,$themes_inc,$functions_inc,$banned_inc))


Dat dacht ik ook in eerse instantie, maar het mag!
isset () mag meer dan één variabele meegegeven worden. Handig om te weten!

PHP Manual: isset
En dat geld voor alle php versies vanaf 3.0? :D

Hebben jullie verder nog opmerkingen?

iets wat onveilig is , netter kan etc etc ...

Doel = zo goed moglijk te scripten namelijk :P

Reageren