// 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.
Dat is nogal dubbelop ja :)
Je kunt met 1 query gewoon meerdere kolommen selecteren, dus dat stukje code kan gewoon korter. Daarnaast doe je niet aan foutcontrole en gebruik je de optionele connection-identifier van MySQL niet, deze laatste is niet verplicht, maar wel aan te raden om te gebruiken.
Je zou zo kunne doen:
Hierbij haal je alleen user_id en sess_verify eruit
<?
"SELECT user_id, sess_verify FROM dbm_login WHERE user_name = '".$login_name."'";
?>
En zo:
<?
"SELECT * FROM dbm_login WHERE user_name = '".$login_name."'";
?>
Hierbij haal je alle records eruit waar usernaam $login_name is.
Leer je snel genoeg, als je grotere scripts gaat maken zit je je helemaal rot te zoeken naar een foutje, omdat je zoveel dubbel hebt, en dan ga je vanzelf netter scripten.
Ja in he begin is dat wel moeilijk ja maar dat leer je vanzelf, en als je 1 x hetzelfde typt in 1 script dan klopt er iets niet dan moet het anders kunnen ;)
