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)