"PHP" / Говнокод #18190 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
if ($ipaddr == $ip_reverse) {
    return true;
} else {
    return false;
}

https://github.com/pfsense/pfsense/blob/master/etc/inc/util.inc#L625-L629

Давно хотел спросить: это как-нибудь здравым смыслом оправдывается? Часто встречаю похожее, когда чужой код смотрю. Может, это профи делают для… читаемости какой-то, я не знаю. Или я ищу хорошее там, где его нет?

Запостил: superhacker777 superhacker777, (Updated )

Комментарии (4) RSS

  • ava inkanus-gray # Ссылка на оригинал 0

    Не будет этому оправдания. Автора гнёт страшный недуг: боязнь булевых типов.
    Ответить

  • ava Stallman # Ссылка на оригинал 0

    Скажи спасибо, что автор не сделал так, пхп же:
    return ($ipaddr == $ip_reverse) === false ? false : true;

    >Может, это профи делают для… читаемости какой-то
    Один знакомый джуниор оправдывал такие высеры именно читаемостью.
    Ответить

  • ava Lure Of Chaos # Ссылка на оригинал 0 

    if ($ipaddr == $ip_reverse) {
    // здесь мог бы быть Ваш код
    return true;
} else {
    // здесь мог бы быть Ваш код
    return false;
}
    Ответить

Добавить комментарий

Из-за тебя ушел bormand, guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓