Code einer Funktion eleganter umschreiben

[slaughter89]

Hi,
ich bin Anfänger in Sachen PHP und bin gerade dabei ein Script zu schreiben

Ich wollte euch fragen, ob und wie ich folgende Funktion eleganter schreiben kann:

Hier gehe ich davon aus, dass der Query nur ein Ergebnis liefert:

function get_user($ss) {
$user_query = dbquery("SELECT user_name FROM ".DB_PREFIX."users WHERE user_id = '".$ss."'");
while($res = mysql_fetch_assoc($user_query))
{
$res_user[] = $res['user_name'];
}
return $res_user[0];
}

 

[Gumbo]

Mein Vorschlag:

function getUserNameById( $id )
{
	$result = dbquery("SELECT user_name FROM ".DB_PREFIX."users WHERE user_id = '".(int)$id."'");
	if( !mysql_num_rows($result) ) {
		return false;
	}
	return mysql_result($result, 0);
}

Der Funktionsname ist nun auch etwas informativer.

 

[Hallasas]

Hallo,
ich denke mal, dass "dies" etwas "eleganter" ist, aber "übersichtsmäßig" muss man sich gewöhnen...

<?php

$config['mysql']['username'] = ''; # Mysql username
$config['mysql']['password'] = ''; # Mysql password
$config['mysql']['r_server'] = ''; # Mysql server
$config['mysql']['database'] = ''; # Mysql database
$config['mysql']['u_prefix'] = ''; # Mysql prefix


function get_user($ss)
{
	$config['mysql']['serv_con'] =
		mysql_connect(
			$config['mysql']['r_server'],
			$config['mysql']['username'],
			$config['mysql']['password']
		);

	mysql_select_db(
		$config['mysql']['database'],
		$config['mysql']['serv_con']
	);

	$config['mysql']['user_que'] = 
		mysql_query("SELECT user_name FROM "
					.$config['mysql']['u_prefix'].
					" users WHERE user_id = '".$ss."'");
	
	while($res = mysql_fetch_assoc($config['mysql']['user_que']))
	{
		$res_user[] = $res['username'];
	}
	return $res_user[0];
}


?>

Ich hoffe, ich konnte helfen und das sich keine Fehler reingeschlichen haben

 

[daddz]

Das wird so nicht funktionieren, außer du definierst die "config"-Variable als "global".

 

[slaughter89]

Vielen Dank, ich habe mich für die Lösung von Gumbo entschieden