Говнокод: по колено в коде.

• Все
• Лучший
• Сток
• Глупый поиск
• Наговнокодить!
• Oтзывы

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

1. PHP (5706)
2. C++ (2739)
3. Куча (2363)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1116)
8. Pascal (648)
9. Python (586)
10. 1C (538)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Go (31)
21. Swift (27)
22. Haskell (26)
23. Kotlin (14)

1. Java / Говнокод #9179

   +65

   1. 1
   2. 2
   3. 3
   4. 4

   public int getSysEmailId() {
           return super.getJdbcTemplate().queryForInt(
                   "SELECT id FROM sys_email");
       }

   Коллега порадовал. Говнонюанс в том, что sys_email имеет структуру (id SERIAL PRIMARY KEY, login text NOT NULL UNIQUE, password text NOT NULL).

   0rt, 20 Января 2012

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

2. C# / Говнокод #9178

   +111

   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)
                   {
                       if (sec > 10)
                       {
                           sec = 0;
                           // Вызор функции
                       }
                       else
                           sec++;
   
                       Thread.Sleep(1000);
                   }

   Боевой код, который допиливаю(

   yorikim, 20 Января 2012

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

3. PHP / Говнокод #9177

   +150

   1. 1
   2. 2

   $id = $_GET['id'];
   $id = (int) intval($id);

   Чтобы уж наверняка :)

   DropWorld, 19 Января 2012

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

4. C# / Говнокод #9176

   +953

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5

   void Update()
   {
   	float anime = (animation.wrapMode = WrapMode.Loop);
   	anime = false;
   }

   Хентай повышенной концентрации.

   alexz, 19 Января 2012

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

5. Java / Говнокод #9175

   +142

   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
   51. 51
   52. 52
   53. 53
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59
   60. 60

   private void run() throws Exception {
   		Scanner s = new Scanner(new File("input.txt"));
   		PrintWriter p = new PrintWriter(new File("output.txt"));
   		
   		int n = s.nextInt();
   		int sum = 0;
   		
   		init();
   		
   		for(int i = 1; i <= n; i++) {
   			sum += find(i);
   		}
   		
   		p.print(sum);
   		
   		s.close();
   		p.close();
   	}
   	
   	private void init() {
   		for(int i = 0; i < 100; i++) {
   			pow_2[i] = (int) Math.pow(2, i + 1);
   		}
   		
   		for(int i = 0; i < 100; i++) {
   			pow_3[i] = (int) Math.pow(3, i + 1);
   		}
   		
   		for(int i = 0; i < 100; i++) {
   			pow_4[i] = (int) Math.pow(4, i + 1);
   		}
   		
   		for(int i = 0; i < 100; i++) {
   			pow_5[i] = (int) Math.pow(5, i + 1);
   		}
   	}
   	
   	private int find(int num) {
   		for(int i = 0; i < 100; i++) {
   			if(pow_2[i] == num) return 2;
   			if(num > pow_2[99]) break;
   		}
   		
   		for(int i = 0; i < 100; i++) {
   			if(pow_3[i] == num) return 3;
   			if(num > pow_3[99]) break;
   		}
   		
   		for(int i = 0; i < 100; i++) {
   			if(pow_4[i] == num) return 4;
   			if(num > pow_4[99]) break;
   		}
   		
   		for(int i = 0; i < 100; i++) {
   			if(pow_5[i] == num) return 5;
   			if(num > pow_5[99]) break;
   		}
   		
   		return 0;
   	}

   Говнолаба... есть идеи как оптимизировать по скорости?

   Nyashka, 19 Января 2012

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

6. C++ / Говнокод #9173

   +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

   QSqlQueryModel model,model1,model2;
   ...
   model2.setQuery("SELECT * FROM cardInfoChange");
   while(model2.canFetchMore())
       model2.fetchMore();
   ...
    for(int i = 0; i < model2.rowCount(); i++)
           {
               QString t =  model2.record(i).value(1).toString().toUtf8()+
                       ";"+model2.record(i).value(2).toString().toUtf8()+";"+model2.record(i).value(3).toString().toUtf8()+
                       ";"+model2.record(i).value(4).toString().toUtf8()+
                       ";"+model2.record(i).value(5).toString().toUtf8()+
                       ";"+model2.record(i).value(6).toString().toUtf8()+
                       ";"+model2.record(i).value(7).toString().toUtf8()+
                       ";"+model2.record(i).value(8).toString().toUtf8()+
                       ";"+model2.record(i).value(9).toString().toUtf8()+
                       ";"+model2.record(i).value(10).toString().toUtf8()+
                       ";"+model2.record(i).value(11).toString().toUtf8()+
                       ";"+model2.record(i).value(12).toString().toUtf8()+
                       ";"+model2.record(i).value(13).toString().toUtf8()+
                       ";"+model2.record(i).value(14).toString().toUtf8()+
                       ";"+model2.record(i).value(15).toString().toUtf8()+
                       ";\n";
               f.write( t.toAscii());
           }

   Продолжаем стрелять по воробьям из пушки.
   
   Особое внимание на строки 4 и 5 так как QSqlQueryModel предназначена для вывода отображения в QTableView она не загружает все данные сразу а делает это постепенно.
   
   Ну и напоследок шедевральный способ записи полученных данных в файл.

   ir4y, 19 Января 2012

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

7. C# / Говнокод #9172

   +106

   1. 01
   2. 02
   3. 03
   4. 04
   5. 05
   6. 06
   7. 07
   8. 08
   9. 09
   10. 10
   11. 11

   public void CheckMemoStyle(bool isChecked)
   {
       if (isChecked)
       {
           memoStyleLayoutCntrl.Selected = true;
       }
       else
       {
           memoStyleLayoutCntrl.Selected = false;
       }
   }

   memoStyleLayoutCntrl.Selected = isChecked; уже не модно?

   alpha2050, 19 Января 2012

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

8. PHP / Говнокод #9171

   +146

   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
   51. 51
   52. 52
   53. 53
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59
   60. 60
   61. 61
   62. 62

   class BbCod{
   		
   		function BbCod(){
   				
   		} 
   		
   	 	static function BbB($text){
   			
   			if (is_array($text)) {
   				$text = '<b>'.$text[1].'</b>';
   			}
   			
   			return preg_replace_callback("~\[b]((?:[^[]|\[(?!/?b])|(?R))+)\[/b]~is", "BbCod::BbB" , $text); 
   		}
   		
   		
   		static function BbI($text){
   			
   			if (is_array($text)) {
   				$text = '<i>'.$text[1].'</i>';
   			}
   			
   			return preg_replace_callback("~\[i]((?:[^[]|\[(?!/?i])|(?R))+)\[/i]~is", "BbCod::BbI" , $text); 
   		}
   		
   		
   
   
   // ...
   
   		function Bb($text, $param = array()){
   			if(!isset($param['b']) || $param['b'] !== false) $text = $this->BbB($text);				
   			if(!isset($param['i']) || $param['i'] !== false) $text = $this->BbI($text);	
   			if(!isset($param['u']) || $param['u'] !== false) $text = $this->BbU($text);
   			if(!isset($param['ol']) || $param['ol'] !== false) $text = $this->BbOl($text);
   			if(!isset($param['ul']) || $param['ul'] !== false) $text = $this->BbUl($text);
   			if(!isset($param['li']) || $param['li'] !== false) $text = $this->BbLi($text);
   			if(!isset($param['quote']) || $param['quote'] !== false) $text = $this->BbQuote($text);
   			if(!isset($param['left']) || $param['left'] !== false) $text = $this->BbLeft($text);
   			if(!isset($param['right']) || $param['right'] !== false) $text = $this->BbRight($text);
   			if(!isset($param['center']) || $param['center'] !== false) $text = $this->BbCenter($text);
   			if(!isset($param['justify']) || $param['justify'] !== false) $text = $this->BbJustify($text);
   			if(!isset($param['size']) || $param['size'] !== false) $text = $this->BbSize($text);
   			if(!isset($param['color']) || $param['color'] !== false) $text = $this->BbColor($text);
   			if(!isset($param['fright']) || $param['fright'] !== false) $text = $this->BbFRight($text);
   			if(!isset($param['fleft']) || $param['fleft'] !== false) $text = $this->BbFLeft($text);
   			if(!isset($param['table']) || $param['table'] !== false) $text = $this->BbTable($text);
   			if(!isset($param['td']) || $param['td'] !== false) $text = $this->BbTd($text);
   			if(!isset($param['th']) || $param['th'] !== false) $text = $this->BbTh($text);
   			if(!isset($param['tr']) || $param['tr'] !== false) $text = $this->BbTr($text);
   			if(!isset($param['br']) || $param['br'] !== false) $text = $this->BbBr($text);
   			if(!isset($param['hr']) || $param['hr'] !== false) $text = $this->BbHr($text);
   			if(!isset($param['youtube']) || $param['youtube'] !== false) $text = $this->BbYoutube($text);
   			if(!isset($param['img']) || $param['img'] !== false) $text = $this->BbIms($text);
   			if(!isset($param['src']) || $param['src'] !== false) $text = $this->BbSrc($text);
   			if(!isset($param['mail']) || $param['mail'] !== false) $text = $this->BbMail($text);
   			
   			return $text;	
   		}
   		
   }
   		// end class

   Писал в само начале осваивания ООП в php. И причем такое ощущения что бухой писал)

   vir-mir, 19 Января 2012

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

9. PHP / Говнокод #9170

   +155

   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

   <?php
   function variation($name, $cache = true)
   {
       $hach = md5($name);
       //Проверяем кэш, если его нет - делаем запрос к API(JSON).
       if (!file_exists('cache/variation/' . $hach . '.cache')) {
           $c = file_get_contents('http://export.yandex.ru/inflect.xml?name=' . urlencode($name) . '&format=json');
           //Записываем кэш
           if ($cache === true) {
               file_put_contents('cache/variation/' . $hach . '.cache', $c);
           }
       }
       //Если кэш есть - берем оттуда.
       else {
           $c = file_get_contents('cache/variation/' . $hach . '.cache');
       }
       //Возвращаем массив(Функция работает только с UTF-8)
       return json_decode($c, true);
   }
   $arr = variation('Москва');
   /*
   //Переводим в cp1251(JSON).
   $arr = variation('Москва');
   //Для всего массива
   function _iconv($val)
   {
   return iconv('utf-8', 'cp1251', $val);
   }
   $arr = array_map('_iconv', $arr);
   //Или для отдельного элемента
   $arr[1] = iconv('utf-8', 'cp1251', $arr[1]);
   */
   echo 'Оригинал: ', $arr['original'], '<br/>';
   echo 'Именительный: ', $arr[1], '<br/>';
   echo 'Родительный: ', $arr[2], '<br/>';
   echo 'Дательный: ', $arr[3], '<br/>';
   echo 'Винительный: ', $arr[4], '<br/>';
   echo 'Творительный: ', $arr[5], '<br/>';
   echo 'Предложный: ', $arr[6], '<br/>';

   http://moicoding.ru/php-poleznosti/8-Yandeks-sklonyator

   Unknown, 19 Января 2012

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

10. C# / Говнокод #9169

    +114

    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

    public void Dispose()
            {
                Dispose(true);  
                GC.SuppressFinalize(this);
            }
    
            protected virtual void Dispose(bool disposing)
            {
                _channelRedrawManager.StopProcessing();
            }
    
            ~ChannelControlViewModel()
            {
                Dispose(false);
            }

    Деструкти правильно..

    Lifter, 19 Января 2012

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

• ← влево
• вправо →

• 2105
• …
• 884
• 883
• 882
• 881
• 880
• 879
• 878
• 877
• 876
• …
• 1