Pogledajte određenu poruku
Staro 19. 01. 2012.   #8
srdjan
xyz
Grand Master
 
Datum učlanjenja: 25.10.2006
Poruke: 893
Hvala: 87
346 "Hvala" u 163 poruka
srdjan ima spektakularnu aurusrdjan ima spektakularnu aurusrdjan ima spektakularnu aurusrdjan ima spektakularnu auru
Default

Citat:
Originalno napisao ivanhoe Pogledajte poruku
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);


Poslednja izmena od srdjan : 19. 01. 2012. u 11:17.
srdjan je offline   Odgovorite uz citat
"Hvala" srdjan za poruku: