Upotreba return u funkcijama - programerski stilovi

[Gruja]

Ja UVEK stavljam zagrade u if. Jednostavno, mnogo je manja šansa da u brzini napraviš grešku ovog tipa:

if( $a == 'nesto' ) 
    print(a);
    return FALSE;  // Ovo nije više deo if

[jablan]

Originalno napisao srdjan

^ i ja to radim, ali više praktična stvar: možeš da staviš breakpoint na return

Off Topic: U VS-u može da se stavi breakpoint na nivou karaktera, pojedinačnih bitova unutar karaktera, pa i na elektron koji menja stanje u poluprovodniku.

[srdjan]

Originalno napisao jablan

Off Topic: U VS-u može da se stavi breakpoint na nivou karaktera, pojedinačnih bitova unutar karaktera, pa i na elektron koji menja stanje u poluprovodniku.

Off Topic:
Da znaš da može na nivou nekih karaktera, evo recimo stalo je na "/" :P

[BraMom]

Off Topic:
Pored toga breakpoint može da bude i uslovni, a postoji i import/export...
Debugger-improvements

[_korso_]

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.

[tasmaniski]

^jos samo da si dao ime funkcije: jsonGetUser() - i to je taj prepoznatljiv stil koji mnogi koriste

[robi-bobi]

^ ja sam oduvek pisao funkcije sa donjom crtom ime_funkcije()
meni citljivije, a i blize jeziku u kojem pisem stalno (vecina native PHP funkcija ima taj stil )

i sad ne razumem zasto se fura taj CamelCase ili pak camelCase stil ...

recimo:
dugacko_ime_neke_funkcije ()
neToliKoDugackoIme()

oba se citaju, ali prvo skaniram brze

[salebab]

tasmaniski, možda, da je u pitanju metoda, ovde je u pitanju funkcija :P

_korso_, zanimljiva ideja sa default vrednostima, ali ja bih je izbegavao, deluje mi kao da dolazi do usporenja aplikacije kada 2x dodeljuješ vrednost, prvo default, pa ih onda prepisuješ, samo da bi se izbegao jedan else...

[_korso_]

Originalno napisao salebab

_korso_, zanimljiva ideja sa default vrednostima, ali ja bih je izbegavao, deluje mi kao da dolazi do usporenja aplikacije kada 2x dodeljuješ vrednost, prvo default, pa ih onda prepisuješ, samo da bi se izbegao jedan else...

Hm... Sve zavisi i od situacije kao sto rekoh gore kada se to koristi ili neke varijacije na temu. Isuvise je situacija da bi mogao da generalizujem. Inace kao performance freak veoma drzim to takvih stvari.
A i na danasnjem hw ne bih rekao da je ovo neki performance issue, pogotovo sto je verovatno zahtevnija operacija petlja/uslov nego dodeljivanje vrednosti.
Na stranu koliko sam se bugova nagledao zato sto promenljiva nije inicijalizovana, a uslov nije prosao pa dobijamo Undefined variable...

[Dragi Tata]

Originalno napisao Gruja

Ja UVEK stavljam zagrade u if. Jednostavno, mnogo je manja šansa da u brzini napraviš grešku ovog tipa:

if( $a == 'nesto' ) 
    print(a);
    return FALSE;  // Ovo nije više deo if

Тај аргумент сам чуо сто пута али ми никако не иде у главу: идентација у твом случају просто "вришти" да нешто није у реду - осим ако си предуго програмирао у Python-у