1. PHP / Говнокод #18190

    +144

    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, 17 Мая 2015

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

    • ava inkanus-gray 17.05.2015 22:09 # +1

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

    • ava Stallman 18.05.2015 00:40 # 0

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

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

    • ava Lure Of Chaos 18.05.2015 09:03 # +2 

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

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