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

    Всего: 4

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

    +159

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    $this->setLayout('empty');
            $this->setTemplate('step3');
    
            if ($this->getBrokerId() == 19 || $this->getBrokerId() == 29)
            {
                $this->setLayout('empty');
                $this->setTemplate('step3');
                $this->brokerId = $this->getBrokerId();
            }

    Нашел в чужом проекте. При этом $this->getBrokerId() не просто возвращает переменную, а производит сравнение строк (то есть вызывать этот метод 3 раза - явный перебор).

    blaster999, 15 Февраля 2011

    Комментарии (2)
  3. JavaScript / Говнокод #3662

    +167

    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
    switch (columnType)
    {
        case 'first':
        {
            className = '.active-tooltip-first';
            break;
        }
    
        case 'second':
        {
            className = '.active-tooltip-second';
            break;
        }
    
        case 'third':
        {
            className = '.active-tooltip-third';
            break;
        }
    }

    Подобных перлов в данном проекте немало. По возможности вычищаю

    blaster999, 08 Июля 2010

    Комментарии (16)
  4. SQL / Говнокод #3374

    −163

    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
    $SQL            =  "SELECT rq.*, c.login, r.name as raceName,                                                                                                                                                                   
        IF(rq.status = 1, '<span style=\'color:green\'>".$this->kernel->msg->get('raceRequest.status.done')."</span>',                                                                          
            IF(rq.status = -1, '<span style=\'color:red\'>".$this->kernel->msg->get('raceRequest.status.cancel')."</span>',                                                                 
                '".$this->kernel->msg->get('raceRequest.status.request')."')) as statusMsg,                                                                                             
        IF(rq.currentLevel = 1, '<span style=\'color:#eeb300\'>".$this->kernel->msg->get('raceRequest.level.gold')."</span>',                                                                   
            IF(rq.currentLevel = 2, '<span style=\'color:#c6cccd\'>".$this->kernel->msg->get('raceRequest.level.silver')."</span>',                                                         
                '<span style=\'color:#d58b72\'>".$this->kernel->msg->get('raceRequest.level.bronzet')."</span>')) as currentLevelMsg,                                                   
        IF(rq.requestLevel = 1, '<span style=\'color:#eeb300\'>".$this->kernel->msg->get('raceRequest.level.gold')."</span>',                                                                   
            IF(rq.requestLevel = 2, '<span style=\'color:#c6cccd\'>".$this->kernel->msg->get('raceRequest.level.silver')."</span>',                                                         
                '<span style=\'color:#d58b72\'>".$this->kernel->msg->get('raceRequest.level.bronzet')."</span>')) as requestLevelMsg                                                    
        FROM raceRequests rq                                                                                                                                                                            
        LEFT JOIN customers c ON c.ID = rq.customerID                                                                                                                                                   
        LEFT JOIN races r ON r.ID = rq.raceID                                                                                                                                                           
        WHERE rq.ID = '%ID'";

    Формирование HTML с помощью SQL

    blaster999, 01 Июня 2010

    Комментарии (32)
  5. JavaScript / Говнокод #3366

    +159

    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
    function HighlightSelectedKaskoPriceCell(currentCell, toggle, useParent)
    {
        if(currentCell)
        {
            if(toggle)
            {
                if(!useParent)
                    $(currentCell).css("background-color", "rgb(247, 144, 52)");
                else
                    $(currentCell).parent().css("background-color", "rgb(247, 144, 52)");
            }
            else
            {
                if(!useParent)
                    $(currentCell).css("background-color", "#FEF3EB");
                else
                    $(currentCell).parent().css("background-color", "#FEF3EB");
    
            }
        }    
    }
    
    function GetSelectedKaskoPriceCellHighlighting(currentCell, useParent)
    {
        if(!useParent)
            return ($(currentCell).css("background-color") == "rgb(247, 144, 52)");
        else
            return ($(currentCell).parent().css("background-color") == "rgb(247, 144, 52)");
    }

    Способ выделить ячейку таблицы и выяснить, была ли она выделена.

    blaster999, 01 Июня 2010

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