par
Ryle » 11 janv. 2007, 12:09
Tu pourrais faire quelque chose dans ce goût là :
($test) ? true :
(($test) ? true :
(($test) ? true :
(($test) ? true : false)));
C'est à mon sens un peu plus lisible que de tout mettre sur une ligne
Personnellement j'aime pas l'imbrication des ternaires, je préfère donc le if/elseif/else, mais j'aime pas les accolades inutiles, même si certains en ont besoin pour lire leur code, pour moi, une bonne indentation suffit.
Donc entre la version une ligne et la version expand, j'utiliserais une structure intermédiaire :
function get_ip() {
if (isset($_SERVER['HTTP_CLIENT_IP']))
$ip = $_SERVER['HTTP_CLIENT_IP'];
elseif(isset($_SERVER['HTTP_X_FORWARDED_FOR']))
$ip = $_SERVER['HTTP_X_FORWARDED_FOR'];
else
$ip = $_SERVER['REMOTE_ADDR'];
return long2ip(ip2long($ip));
}