1. Список говнокодов пользователя superhacker777

    Всего: 3

  2. PHP / Говнокод #18247

    +142

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    // Check if the subnet begins with $startip and ends before $endip
if (($targetsub_min == $startip) && ip_less_than($targetsub_max, $endip)) {
	break;
}

// Check if the subnet ends at $endip and starts after $startip
if (ip_greater_than($targetsub_min, $startip) && ($targetsub_max == $endip)) {
	break;
}

// Check if the subnet is between $startip and $endip
if (ip_greater_than($targetsub_min, $startip) && ip_less_than($targetsub_max, $endip)) {
	break;
}

    Коль уж старые посты потерлись, принесу это говно снова. Я тогда самое главное забыл. Теперь эти пидорасы не оправдаются ущербностью PHP.

    https://github.com/pfsense/pfsense/blob/master/etc/inc/util.inc#L542-L555

    superhacker777, 27 Мая 2015

    Комментарии (1)

  3. PHP / Говнокод #18191

    +144

    1. 1
    2. 2
    3. 3
    4. 4
    /* Convert IP address to unsigned long int. */
function ip2ulong($ip) {
    return sprintf("%u", ip2long32($ip));
}

    https://github.com/pfsense/pfsense/blob/master/etc/inc/util.inc#L414-L417

    Продолжаем осторожно идти дальше. Это нормально вообще? Да, я видел всякие смехуёчки с магическим приведением типов в PHP и JS, так что наверняка оно где-то правильно конвертится и все работает несмотря ни на что и вопреки. Может, это даже нормальная практика в мире PHP, но мне страшно.

    superhacker777, 17 Мая 2015

    Комментарии (19)

  4. 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)