Citat:
Originalno napisao ivanhoe
ok, nije to sto sam izostavio par zagrada oko if bila poenta, neko voli da ih pise, neko ne... pricao sam o zahtevu da se koristi samo jedan return, radi preglednosti koda...
|
Meni je čitljivost uvek pre nečije akademske definicije "lepote" koda.
Trudim se da odmah return-ujem ako to pojednostavljuje ostatak koda. Inače bi imao dodatna ugnježivanja ili administraciju result varijabli. To znači da mogu da mi se dese i jedan i drugi slučaj u istoj funkciji, ali ne mislim da je to loše.
Ne jednom sam koristio goto iz istog razloga... od nedavno stavljam komentar
// kacam rasan
Osim naravno u situacijama kada je očigledno o čemu se radi (npr. @salebab-ov primer). Ja bih ga napisao ovako, prvo proveravam fail situacije i tako ih mentalno "brišem" i manje gledam gore-dole i uparujem zagrade
PHP kôd:
function json_get_user() {
if(empty($_POST['user_id'])) {
$result['status'] = 'fail';
$result['msg'] = 'There is no user id';
}
else {
$user = UserFactory::getById($_POST['user_id']);
if(empty($user)) {
$result['status'] = 'fail';
$result['msg'] = 'User not exists';
} else {
$result['data'] = $user;
$result['status'] = 'ok';
}
}
return json_decode($result);
}