1. Лучший говнокод

    В номинации:
    За время:
  2. PHP / Говнокод #12103

    +65

    1. 1
    $query = $this->db->query("SELECT DISTINCT *, pd.name AS name, p.image, m.name AS manufacturer, (SELECT price FROM " . DB_PREFIX . "product_discount pd2 WHERE pd2.product_id = p.product_id AND pd2.customer_group_id = '" . (int)$customer_group_id . "' AND pd2.quantity = '1' AND ((pd2.date_start = '0000-00-00' OR pd2.date_start < NOW()) AND (pd2.date_end = '0000-00-00' OR pd2.date_end > NOW())) ORDER BY pd2.priority ASC, pd2.price ASC LIMIT 1) AS discount, (SELECT price FROM " . DB_PREFIX . "product_special ps WHERE ps.product_id = p.product_id AND ps.customer_group_id = '" . (int)$customer_group_id . "' AND ((ps.date_start = '0000-00-00' OR ps.date_start < NOW()) AND (ps.date_end = '0000-00-00' OR ps.date_end > NOW())) ORDER BY ps.priority ASC, ps.price ASC LIMIT 1) AS special, (SELECT points FROM " . DB_PREFIX . "product_reward pr WHERE pr.product_id = p.product_id AND customer_group_id = '" . (int)$customer_group_id . "') AS reward, (SELECT ss.name FROM " . DB_PREFIX . "stock_status ss WHERE ss.stock_status_id = p.stock_status_id AND ss.language_id = '" . (int)$this->config->get('config_language_id') . "') AS stock_status, (SELECT wcd.unit FROM " . DB_PREFIX . "weight_class_description wcd WHERE p.weight_class_id = wcd.weight_class_id AND wcd.language_id = '" . (int)$this->config->get('config_language_id') . "') AS weight_class, (SELECT lcd.unit FROM " . DB_PREFIX . "length_class_description lcd WHERE p.length_class_id = lcd.length_class_id AND lcd.language_id = '" . (int)$this->config->get('config_language_id') . "') AS length_class, (SELECT AVG(rating) AS total FROM " . DB_PREFIX . "review r1 WHERE r1.product_id = p.product_id AND r1.status = '1' GROUP BY r1.product_id) AS rating, (SELECT COUNT(*) AS total FROM " . DB_PREFIX . "review r2 WHERE r2.product_id = p.product_id AND r2.status = '1' GROUP BY r2.product_id) AS reviews, p.sort_order FROM " . DB_PREFIX . "product p LEFT JOIN " . DB_PREFIX . "product_description pd ON (p.product_id = pd.product_id) LEFT JOIN " . DB_PREFIX . "product_to_store p2s ON (p.product_id = p2s.product_id) LEFT JOIN " . DB_PREFIX . "manufacturer m ON (p.manufacturer_id = m.manufacturer_id) WHERE p.product_id = '" . (int)$product_id . "' AND pd.language_id = '" . (int)$this->config->get('config_language_id') . "' AND p.status = '1' AND p.date_available <= NOW() AND p2s.store_id = '" . (int)$this->config->get('config_store_id') . "'");

    Opencart, catalog/model/catalog/product.php

    Они что издеваются? А я-то думаю что у меня ложит mysql сервер на колени...

    fooser, 12 Ноября 2012

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

    +151

    1. 1
    var ext = /.+[.](jpg|JPG|gif|GIF|bmp|BMP|png|PNG|jpeg|JPEG)$/;

    Stallman, 20 Октября 2012

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

    +140

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    var actions = new List<Action>();
    foreach (var i in Enumerable.Range(1, 3))
    {
        actions.Add(() => Console.WriteLine(i));
    }
    
    foreach (var action in actions)
    {
        action();
    }

    Внезапно:
    http://ideone.com/RaiHr
    http://ideone.com/EMQBA

    LispGovno, 17 Октября 2012

    Комментарии (12)
  5. Куча / Говнокод #11844

    +134

    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
    31. 31
    .events-bx-calendar .calendar-wrapper .calendar .days #day_1,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_2,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_3,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_4,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_5,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_6,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_7,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_8,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_9,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_10,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_11,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_12,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_13,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_14,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_15,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_16,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_17,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_18,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_19,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_20,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_21,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_22,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_23,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_24,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_25,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_26,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_27,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_28,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_29,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_30,
    .events-bx-calendar .calendar-wrapper .calendar .days #day_31 { padding: 5px 3px; }

    Un1oR, 27 Сентября 2012

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

    −203

    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
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    // Тут начинается сильнейший программинг - формирование GET запроса на основе уже существующего, курите.
    
      private function getFilterURLParams($filter = 0, $option_id, $value_id, $variable = '') {
        // При изменении этих параметров, нужно будет поменять соответсвенно их в других файлах. Менять их не советую.
    
        $sep_par = ';'; // разделитель пар опций -> значений: opt1=val1,val2,val3;opt2=val1,val2,val3 ...
        $sep_opt = '='; // разделитель внутри пары опция -> значения: opt1=val1,val2,val3 ...
        $sep_val = ','; // разделитель для параметров опции: val1,val2,val3 ...
    
          if ($filter) {
    
          $matches = explode($sep_par, $filter);
    
          $options = array();
          $values = array();
          $parts = array();
    
         foreach ($matches as $option) {
            $data = explode($sep_opt, $option);
            $parts[] = $option;
            $options[] = $data[0];
            $values[] = explode($sep_val, $data[1]);
          }
    
          if (in_array($option_id, $options)) { // если эта опция уже есть в запросе, то мы не добавляем её
    
            $key = array_keys($options, $option_id); // вычисляем ключ массива для дальнейшей работы с именно этой опцией
    
            if (in_array($value_id, $values[$key[0]])) { // если это значение уже есть в запросе
              if (count($values[$key[0]]) == 1) { // и если оно единственное
                if (count($matches) == 1) { // еще и с единственной опцией, то удаляем из запроса весь фильтр
                  $out = '';
                } else { // если опция не одна, удаляем только эту опцию с её параметром
                  $out = '&' . $variable . '=' . str_replace((array_search($parts[$key[0]], $parts) ? $sep_par . $parts[$key[0]] : $parts[$key[0]] . $sep_par), '', $filter);
                }
              } else { // если значений несколько, удаляем это значение, оставляя другие с опцией
                $out = '&' . $variable . '=' . str_replace($parts[$key[0]], $options[$key[0]] . $sep_opt . str_replace((array_search($value_id, $values[$key[0]]) ? $sep_val . $value_id : $value_id . $sep_val), '', implode($sep_val, $values[$key[0]])), $filter);
              }
            } else { // если значения нет в запросе, то добавляем его к значениям этой опции
              $out = '&' . $variable . '=' . str_replace($parts[$key[0]], $options[$key[0]] . $sep_opt . $value_id, $filter);
            }
          } else { // если этой опции нет в запросе
            $out = '&' . $variable . '=' . $filter . $sep_par . $option_id . $sep_opt . $value_id;
          }
        } else { // если в запросе вообще нет переменной filter
          $out = '&' . $variable . '=' . $option_id . $sep_opt . $value_id;
        }
    
        return $out; // фух.
      }

    я после этого долго и нервно курил. чувствовал себя отбросом, а не программистом

    Snickers, 25 Сентября 2012

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

    +168

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    function confirmDialog(sMessage) {
        if (confirm(sMessage)) {
            return true;
        } else {
            return false;
        }
    }

    Lokich, 19 Сентября 2012

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

    +140

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    private void SendFile(string root, string url, DateTime dateTimeBegin)
    {
    	if ( ... isSend ... )
    	{
    		...MoveToArchive();
    		...Dispose();
    	}
    	else
    	{
    		SendFile(root, url, dateTimeBegin);
    	}
    }

    Ну тут всё ясно...
    Если файлики отсылаются нормально, то всё работает хорошо.
    Зато если файлики не отсылаются, то начинается злая рекурсия, которая постепенно сжирает всю память :).

    ddv_demon, 13 Сентября 2012

    Комментарии (12)
  9. Си / Говнокод #11710

    +116

    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
    #define KRATNOST6_STEPE32_(VAL) \ 
      (                            \ 
      (\ 
      (VAL/32*32)+(((VAL/32)^1)*(\ 
        (VAL/16*16)+(((VAL/16)^1)*(\ 
          (VAL/8*8)+(((VAL/8)^1)*(\ 
            (VAL/4*4)+(((VAL/4)^1)*(\ 
              (VAL/2*2)\ 
            ))\ 
          ))\ 
        ))\ 
      ))\ 
      )\ 
      +\ 
      ((\ 
      (VAL/32)+(((VAL/32)^1)*(\ 
        (VAL/16)+(((VAL/16)^1)*(\ 
          (VAL/8)+(((VAL/8)^1)*(\ 
            (VAL/4)+(((VAL/4)^1)*(\ 
              (VAL/2)\ 
            ))\ 
          ))\ 
        ))\ 
      ))\ 
      )^1)*32\ 
      ) 
    #define KRATNOST6_STEPE32(MAX_STEPEN6,VAL) KRATNOST6_STEPE32_(((int)VAL&((1<<(MAX_STEPEN6+1))-1)))

    Вычисление ближайшей кратности к степ 2 в препроцессоре, такую байду сорудил. (не знаю правильно ли выразился)
    Тут максимальная 32. Если есть выше то вот тут выставляется 32 всёравно.

    Проще не варик?

    TarasB, 05 Сентября 2012

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

    +62

    1. 1
    2. 2
    3. 3
    4. 4
    $date = substr($row[date], 0, 16);
    $dat1 = explode(" ", $date);
    $dat = explode("-", $dat1[0]);
    $date = "$dat[2]/$dat[1]/$dat[0]";

    Изящно

    kindofbear, 24 Августа 2012

    Комментарии (12)
  11. Куча / Говнокод #11633

    +115

    1. 1
    2. 2
    3. 3
    4. 4
    -- Настало время запостить самое главное гавно Хаскеля:
    -- http://www.johndcook.com/blog/2010/05/18/pure-functions-have-side-effects/
    -- У чистых функций всегда есть side-effects, которые выражаются в пожирании памяти и CPU. 
    -- Очевидно конечно, но почему об этом не орут на первой странице каждой книги по функциональному программированию?

    Я проверил, код компилируется:
    http://ideone.com/sozuV

    HaskellGovno, 23 Августа 2012

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