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

    +140

    1. 1
    http://phpocean.com/tutorials/back-end/create-your-first-desktop-application-with-php-and-php-desktop/4

    верстка самого сайта, phpdesktop, zooboole - у меня жестко бортануло

    Fike, 20 Мая 2015

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

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

    +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
    15. 15
    foreach ($id as $k) {
		//Помним, что меньший id всегда пишется первым
		if ($firm_id < $k)
			{ $a = 1; $b = 2; }
		else
			{ $a = 2; $b = 1; }
			if ($_REQUEST['submit'] == 1) {
                                //  ...
			}
			if ($_REQUEST['submit'] > 1) {
				sql_query("UPDATE partners SET ".$type.$a.$b." = 1, firm{$a}_date = {$NOW} WHERE firm{$a} = $firm_id AND firm{$b} = $k");
                                //  ...
			}
	}
}

    А Вы знаете что будет если -1 отправить?)

    dimka3210, 19 Мая 2015

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

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

    +163

    1. 1
    $age = floor( (strtotime(date('Ymd')) - strtotime($date)) / 31556926);

    31556926 / 365 / 24 / 60 / 60 ~ 1

    crook, 18 Мая 2015

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

  4. PHP / Говнокод #18192

    +145

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (count($stores) == 1) {
    foreach ($stores as $store) {
        $storeId = $store->id;
    }
}

    jbot, 18 Мая 2015

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

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

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

  7. PHP / Говнокод #18186

    +143

    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
    15. 15
    16. 16
    if (is_array($search_ct_properties->getEngines()))
		{
			$where_engines = array();
			foreach ($search_ct_properties->getEngines() as $engine)
			{
				if ($engine == 5)
				{
					$where_engines[] = 'ct_modifications.fuel_id != 6';
				}
				else
				{
					$where_engines[] = 'ct_modifications.fuel_id = 6';
				}
			}
			$where[] = '(' . implode(' OR ', $where_engines) . ')';
		}

    если 5, то это точно не 6!

    smail01, 16 Мая 2015

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

  8. PHP / Говнокод #18178

    +141

    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
    15. 15
    16. 16
    $page = strtoupper($_SERVER['REQUEST_URI']);
    if (strpos($page, "%D0%A2%D0%BE%D0%BF10")){
      echo "<li class=\"active\"><a  href=\"/%D0%A2%D0%BE%D0%BF10\">Топ-10</a></li>";
    } else {
      echo "<li><a href=\"/%D0%A2%D0%BE%D0%BF10\">Топ-10</a></li>";
    }
    if (strpos($page, "%D0%A2%D1%80%D0%B0%D1%84%D0%B8%D0%BA")){
      echo "<li class=\"active\"><a  href=\"/%D0%A2%D1%80%D0%B0%D1%84%D0%B8%D0%BA\">Трафик</a></li>";
    } else {
      echo "<li><a href=\"/%D0%A2%D1%80%D0%B0%D1%84%D0%B8%D0%BA\">Трафик</a></li>";
    } 
    if (strpos($page, "%D0%9B%D0%B8%D0%B4%D0%B5%D1%80")) {
      echo "<li class=\"active\"><a  href=\"/%D0%9B%D0%B8%D0%B4%D0%B5%D1%80\">Лидер</a></li>";
    } else {
      echo "<li><a href=\"/%D0%9B%D0%B8%D0%B4%D0%B5%D1%80\">Лидер</a></li>";
    }

    ...а кроме того, в данном случае strtoupper бесполезен

    olvin, 15 Мая 2015

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

  9. PHP / Говнокод #18176

    +145

    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
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    <?
for ($i = 1; $i <= 100; $i++) {
    $t1 = $i / 3;
    $t2 = $i / 5;
    if (preg_match("/\./", "$t1") == false && preg_match("/\./", "$t2") == false) {
        echo "FizzBuzz<br>";
    } else {
        if (preg_match("/\./", "$t1") == false) {
            echo "Fizz<br>";
        } else {
            if (preg_match("/\./", "$t2") == false) {
                echo "Buzz<br>";
            } else {
                echo "$i<br>";
            }
        }
    }
}
?>

    ZF...

    Yshk, 15 Мая 2015

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

  10. PHP / Говнокод #18172

    +141

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    $phsms = '';

for($p=0;$p<strlen($phonesms);$p++)
    if(intval($phonesms[$p]) >= 0) $phsms = $phsms.intval($phonesms[$p]);

preg_match('/.*(9[0-9]{2})([0-9]{7})/', $phsms, $match2);

$phsms = $match2[1].$match2[2];

    Валидируем телефоны.

    aego09, 15 Мая 2015

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