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

    +162

    1. 1
    2. 2
    3. 3
    4. 4
    $res = $db->loadResult();
    if($res); else return false;
    if($res!=$user->id) return false;
    return true;

    vsu, 04 Июня 2010

    Комментарии (43)
  2. PHP / Говнокод #3380

    +166

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    <?php
    session_start();
    // супер-мега защита от ддоса
    $_SESSION['antiddos']==time() ? exit() : $_SESSION['antiddos']=time();
    //конец супер мега защиты
    ....
    ?>

    Найдено в старом проекте, код не мой.
    Но вы знаете, от толпы школьников с кнопками F5 это реально помогает!

    mr.The, 02 Июня 2010

    Комментарии (21)
  3. PHP / Говнокод #3379

    +158

    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
    20. 20
    21. 21
    22. 22
    23. 23
    /**
     * Метод проверяет права доступа на использование этого модуля, возвращает результат проверки.
     * Кроме этого, происходит запись в лог
     *
     * @return boolean
     */
    private function check_access() {
    	Logger::log_exception($_SERVER['REQUEST_URI'].'?'.$_SERVER['QUERY_STRING'], $this->log_file_name);
    	return true;
    }
    
    // [...]
    
    /**
     * Попытка переключить рабочую базу данных
     */
    public function actionChange_db() {
    	if (!$this->check_access()) {
    		echo strval($this->err_failure_access);
    		return;
    	}
    
    // [...]

    Вот такое иногда встречается в коде коллеги, про название метода "log_exception" я промолчу, т.к. это тоже его рук дело, а общий для всех метод Logger::log() ему почему-то не понравился? Я уже просто устал ругаться с ним...

    cr0t, 02 Июня 2010

    Комментарии (19)
  4. PHP / Говнокод #3370

    +159

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    public static function getSizeformat($iNumber){
        $value = "-";
        if($iNumber){
          $iNumber = $iNumber/100000;
          $value = number_format($iNumber, 2, '.', '');
        }
        return $value;
      }

    Кусочек который пришел по наследству, судя по подписям в таблице на фронтэнде в итоге мы должны получить размер файлов в мегабайтах, но как то не получается... :)

    psych, 01 Июня 2010

    Комментарии (23)
  5. PHP / Говнокод #3365

    +175

    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
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    if($hour == 1){
    $hour = "01";
    }
    if($hour == 2){
    $hour = "02";
    }
    if($hour == 3){
    $hour = "03";
    }
    if($hour == 4){
    $hour = "04";
    }
    if($hour == 5){
    $hour = "05";
    }
    if($hour == 6){
    $hour = "06";
    }
    if($hour == 7){
    $hour = "07";
    }
    if($hour == 8){
    $hour = "08";
    }
    if($hour == 9){
    $hour = "09";
    }if($hour == 0){
    $hour = "00";
    }

    Marker689, 01 Июня 2010

    Комментарии (13)
  6. PHP / Говнокод #3363

    +157

    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
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    if ($var == e_UC_MAINADMIN && getperms('0'))
    		{
            	return TRUE;
    		}
    
    		if ($var == e_UC_MEMBER && USER == TRUE)
    		{
    			return TRUE;
    		}
    
    		if ($var == e_UC_GUEST && USER == FALSE) {
    			return TRUE;
    		}
    
    		if ($var == e_UC_PUBLIC) {
    			return TRUE;
    		}
    
    		if ($var == e_UC_NOBODY) {
    			return FALSE;
    		}
    
    		if ($var == e_UC_ADMIN && ADMIN) {
    			return TRUE;
    		}
    		if ($var == e_UC_READONLY) {
    			return TRUE;
    		}

    По ходу, стек мозга автора не вмещает более одного оператора && подряд.
    CMS e107, /class2.php

    telnet, 31 Мая 2010

    Комментарии (11)
  7. PHP / Говнокод #3359

    +163

    1. 1
    <input type="submit" value="<?=lang('Send','Отправить')?>" />

    Говнокод или нет?

    Morsik, 31 Мая 2010

    Комментарии (40)
  8. PHP / Говнокод #3354

    +156

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    function __get($propertyName) {
    ....
          
          if (!isset($propertyName[$this->propertyTable])) {
                var_dump($this->propertyTable);
                throw new Exception("Wrong property name \"$propertyName\"");
          }
         
    .....
        }

    Глубоко в дебрях...

    MaksSlesarenko, 31 Мая 2010

    Комментарии (3)
  9. PHP / Говнокод #3352

    +165

    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
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    function is_bot() {
    	if (defined("IS_BOT")) {
    		return 1;
    	}
    	elseif (defined("IS_NOT_BOT")) {
    		return 0;
    	}
    	elseif (!defined("IS_BOT") and !defined("IS_NOT_BOT")) {
    		if (file_exists(INCLUDE_PATH."includes/core/is_bot.php")) {
    			require_once(INCLUDE_PATH."includes/core/is_bot.php");
    			if (function_exists('is_bot_real')) {
    				return is_bot_real();
    			}
    			else {
    				define("IS_NOT_BOT", true);
    				return 0;
    			}
    		}
    		else {
    			define("IS_NOT_BOT", true);
    			return 0;
    		}
    	}
    	else {
    		define("IS_NOT_BOT", true);
    		return 0;
    	}
    }

    PHPNuke, mainfile.php

    telnet, 30 Мая 2010

    Комментарии (30)
  10. PHP / Говнокод #3349

    +156

    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
    $div = false;
    while (true) {
        if ($div = !$div) {
            // тут действие если четная итерация цикла
        } else {
            // тут если нечетная
        }
    }
    
    ну, или так
    
    for ($i=0, $div = true; $i<$max; $i++, $div = !$div){
        if ($div) {
            // тут действие если четная итерация цикла
        } else {
            // тут если нечетная
        }
    }

    человек один говорит, что такая проверка на четность/нечетность - говно.
    говно ли?

    shmaltorhbooks, 30 Мая 2010

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