Citat:
Originalno napisao srdjan
PHP kôd:
function json_get_user() {
$user = UserFactory::getById($_POST['user_id']);
if(empty($user)) {
$result['status'] = 'fail';
$result['msg'] = 'User not exists';
} else {
$result['data'] = $user;
$result['status'] = 'ok';
}
}
|
Svako ima svoje navike i svoje tumacenje sta je citljivo sta ne.
Verovatno zato i mnogi ozbiljni fw imaju neki coding guides (kako pisati uslove, imenovati promenjive ...)bilo da su Javascript, PHP, Ruby, kako bi naveli timove da pisu kod koji svako moze da cita, a ne da svako pise kako mu je volja.
Ja u PHP-u cesto(zavisi od nekih faktora) definisem defaultne vrednosti promenjive, bilo da je niz ili int (mada opet radim i kao sto je gornji deo koda - tesko je izolovati i generalizovati, a nekada opet kada ima mnogo uslova imam i vise returna).
Tako da bi ovo napisao verovatno i ovako:
PHP kôd:
function json_get_user() {
$user = UserFactory::getById($_POST['user_id']);
$result = array('status' => 'fail', 'data' => 'User not exists');
if(!empty($user)) {
$result['data'] = $user;
$result['status'] = 'ok';
}
}
Eliminisem jedan else, vidim gde je definisana promenjiva i ako ne prodje uslov imace *uvek* ove izdvojene vrednosti.
Mozda je to nekom manje citljivo, mozda vise, ali tim u kome radim je navikao da radi sa ovakvim nekim stilom pisanja.