1. SQL / Говнокод #1878

    −863.7

    1. 1
    2. 2
    3. 3
    Нашёл в проекте
    SELECT DISTINCT sale_date, SUM(sale_price), buyer FROM ... 
    GROUP BY sale_date, buyer;

    DISTINCT здесь лишний

    Oleg_quadro, 21 Сентября 2009

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

    +157.3

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    function SetStyleForFilterOver(this2) {
        this2.style.border = "1px solid #ea0000";
        this2.style.color = "white";
        this2.style.background = "#404040";
    }
    function SetStyleForFilterOut(this2) {
        this2.style.border = "1px solid #404040";
        this2.style.color = "#ea0000";
        this2.style.color = "#ea0000";
        this2.style.background = "#eeeeee";
    }

    Тс-с-с-с... Только не говорите автору про css и селектор ":hover" :о)

    Ordos, 21 Сентября 2009

    Комментарии (18)
  3. 1C / Говнокод #1876

    −92.1

    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
    Запрос.Текст = 
    "ВЫБРАТЬ
    |	СостояниеРаботниковОрганизаций.Регистратор,
    |	СостояниеРаботниковОрганизаций.ФизЛицо,
    |	СостояниеРаботниковОрганизаций.Организация,
    |	СостояниеРаботниковОрганизаций.Приказ,
    |	СостояниеРаботниковОрганизаций.Состояние,
    |	СостояниеРаботниковОрганизаций.Период
    |ИЗ
    |	(ВЫБРАТЬ
    |		СостояниеРаботниковОрганизаций.Регистратор КАК ДокументРегистратор
    |	ИЗ
    |		РегистрСведений.СостояниеРаботниковОрганизаций КАК СостояниеРаботниковОрганизаций
    |	ГДЕ
    |		СостояниеРаботниковОрганизаций.Период МЕЖДУ &ДатаНачала И &ДатаКонца
    |		И СостояниеРаботниковОрганизаций.Организация = &Организация
    |		И СостояниеРаботниковОрганизаций.ФизЛицо = &ФизЛицо
    |		И СостояниеРаботниковОрганизаций.Приказ = &Приказ) КАК ДокРегистратор
    |		ВНУТРЕННЕЕ СОЕДИНЕНИЕ РегистрСведений.СостояниеРаботниковОрганизаций КАК СостояниеРаботниковОрганизаций
    |		ПО ДокРегистратор.ДокументРегистратор = СостояниеРаботниковОрганизаций.Регистратор";
    .

    Просто выбрать движения из регистра - это не круто.

    Kopchuga, 21 Сентября 2009

    Комментарии (5)
  4. C# / Говнокод #1875

    +136.4

    1. 1
    2. 2
    3. 3
    4. 4
    // Работаю asp.net программистом.
    // Показал главному программисту, как с помощью System.Web.Routing можно легко делать человекопонятные урлы.
    // Итог: Теперь у сайта урлы вида http://domain/filterDisks/42/0/14/1.
    // Вот сижу и думаю, что значат все эти циферки :'о(

    Ordos, 21 Сентября 2009

    Комментарии (8)
  5. C++ / Говнокод #1874

    +55.9

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    while (true)
      {
        m_CurrentDevice += delta;
    
        if (m_CurrentDevice < 0)
          m_CurrentDevice = devices[DEV_PLATE]-1;
    
        if (m_CurrentDevice > devices[DEV_PLATE]-1)
          m_CurrentDevice = 0;
    
        break;
      }

    Это конечно написано не специально - в результат целой кучи итераций и переделок. но результата это не отменяет

    TObject, 21 Сентября 2009

    Комментарии (6)
  6. JavaScript / Говнокод #1873

    +158.1

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    case 'down':
    	if (parseInt(pageTop) + parseInt(pageHeight) - parseInt(elementTop) - parseInt(elementHeight) - parseInt(elementBorderTop) - parseInt(elementBorderBottom) >= parseInt(stepOfMove)) {
    		var topParam = parseInt(elementTop) - 0 + parseInt(stepOfMove) + 'px';
    		elementToMove.style.top = topParam;
    		this.updateElementsParams(id,'top',topParam);
    	} else {
    		stepOfMoveElement.value = parseInt(pageTop) - 0 + parseInt(pageHeight) - parseInt(elementTop) - parseInt(elementHeight) - parseInt(elementBorderTop) - parseInt(elementBorderBottom);
    	}
    break;

    и так миллион тыщ раз...

    Sadie, 21 Сентября 2009

    Комментарии (1)
  7. C++ / Говнокод #1872

    +60.6

    1. 1
    2. 2
    3. 3
    static char *szClassName = new char[14];
    static char *szCurrentDirectory = new char[MAX_BUFFER];
    static char *szNewFolder = new char[MAX_BUFFER];

    глобальные указатели рулят, delete нигде не вызывается

    shomeser, 20 Сентября 2009

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

    +158.3

    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
    30. 30
    <?php
    
    function secure2() {
        secure();
    }
    
    function secure() {
        secure3();
    }
    
    function secure3() {
        if (!check_session()) {
            $cookie = try_cookie();
            tiny_login($cookie['login'], $cookie['pass'], 1);
            if (check_session()) {
                $ret = true;
            } else {
                $ret = false;
            }
        } else {
            $ret = true;
        }
    
        if (!$ret) {
            $_SESSION['loginerror'] = 'Ошибка авторизации';
            header('location:(ссылка)');
        }
    }
    
    ?>

    В догонку к http://govnokod.ru/1820 по просьбе в комментах
    Реальный код из реального проекта. Что самое интересное — используется secure2
    Как оно работает для меня — загадка.

    Shock, 19 Сентября 2009

    Комментарии (7)
  9. JavaScript / Говнокод #1870

    +163.8

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    //Я ТИПА ЗДЕСЬ СДЕЛАЛ НА ГАЛЕРЕЕ 4 КАРТИНКИ
                   if(this.id=='gallery') {
                        this.previewCount = 4;
                   } else {
                        this.previewCount = 4;
                   }

    Недавно код пересматривали. Авторы долго отнекивались

    hagnar, 19 Сентября 2009

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

    +152.8

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    Перл #1:
     ...
     setCityId(getCityId()); 
     ...
    
    Перл #2:
    function performOrgSearch(){
      // ...
      cityId = getCityId(); // функция getCityId вычищает всё из некого input type=text

    yvu, 18 Сентября 2009

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