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

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

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

1. PHP (5712)
2. C++ (2746)
3. Куча (2409)
4. JavaScript (2035)
5. C# (1930)
6. Java (1503)
7. Си (1122)
8. Pascal (648)
9. Python (592)
10. 1C (541)
11. SQL (433)
12. ActionScript (292)
13. Objective C (202)
14. bash (201)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (46)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   В номинации:

   • Лучший рейтинг
   • Самый комментируемый

   За время:

   • за сегодня
   • за неделю
   • за месяц
   • за всё время

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

   +157

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

   foreach($nodes as $xm){
       $arr[count($arr)] = new StdClass();
       $arr[count($arr)-1]->id = $xm->title;
       $arr[count($arr)-1]->nid = $xm->id;
   }

   Lowezar, 10 Июня 2011

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

3. ActionScript / Говнокод #6906

   −109

   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

   var q=0.15
   var w=false
   stroka.alpha=0
   knopka.addEventListener(MouseEvent.MOUSE_OVER,qwe)
   function qwe(e:MouseEvent){
   w=true
   }
   knopka.addEventListener(MouseEvent.MOUSE_OUT,qwe2)
   function qwe2(e:MouseEvent){
   w=false
   }
   stage.addEventListener(Event.ENTER_FRAME,qw)
   function qw(e:Event):void{
   if(w==true){
    stroka.alpha+=q
   }
   if(w==false){
    stroka.alpha-=q
   }
   if(stroka.alpha>=1){
    stroka.alpha=1
   }
   if(stroka.alpha<=0){
    stroka.alpha=0
   }
   }

   Поделка все-того же любителя ENTER_FRAME'а. На этот раз попытка сделать плавное появление/исчезновение объекта при наведении на него мышкой.

   KirAmp, 09 Июня 2011

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

4. Java / Говнокод #6900

   +84

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

   if(obj instanceof TreeDictEntry) {
            TreeDictEntry tde = (TreeDictEntry) obj;
            Integer index = hm_IdDocs_idVectors.get(tde.getId());
            if(index == null) {
               Exception ex =
                  new Exception("Epic Faile в хэш мапе элементов не найден индекс в векторе");
               ex.printStackTrace();
            }
            vecElements.removeElementAt(index);
            return;
         }

   к этому я даже название придумал: "Эпичный говнокод"

   maxt, 08 Июня 2011

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

5. PHP / Говнокод #6858

   +147

   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

   <?php
   $this->phptal->id=$id;
   $this->phptal->returnid=$returnid;
   
   if(isset($params['catalogue']))
   {
   	if(isset($params['category']))
   	{
   		if(isset($params['news']))
   		{
   			$nid=(int)$params['news'];
   			require_once(cms_join_path($this->GetModulePath(), 'classes','dao', 'EventNewsDAO.php'));
   			$nldao=new EventNewsDAO($this);
   			$this->phptal->newsline=$nldao->get($nid);
   			echo $this->ProcessTalTemplate('admin/admin_newsline.tal.tpl');
   		}
   		else
   		{
   			$cid=(int)$params['catalogue'];
   			require_once(cms_join_path($this->GetModulePath(), 'classes','dao', 'EventCatalogueDAO.php'));
   			$cdao=new EventCatalogueDAO($this);
   			$this->phptal->catalogue=$cdao->get($cid);
   			require_once(cms_join_path($this->GetModulePath(), 'classes','dao', 'EventCategoryDAO.php'));
   			$ccdao=new EventCategoryDAO($this);
   			$this->phptal->categories=$ccdao->all($cid);
   
   			$ccid=(int)$params['category'];
   			$this->phptal->category=$ccdao->get($ccid);
   			require_once(cms_join_path($this->GetModulePath(), 'classes','dao', 'EventNewsDAO.php'));
   			$ndao=new EventNewsDAO($this);
   			$this->phptal->news=$ndao->all($ccid);
   			echo $this->ProcessTalTemplate('admin/admin_news.tal.tpl');
   		}
   	}
   	else
   	{
   		$cid=(int)$params['catalogue'];
   		require_once(cms_join_path($this->GetModulePath(), 'classes','dao', 'EventCatalogueDAO.php'));
   		$cdao=new EventCatalogueDAO($this);
   		$this->phptal->catalogue=$cdao->get($cid);
   		require_once(cms_join_path($this->GetModulePath(), 'classes','dao', 'EventCategoryDAO.php'));
   		$ccdao=new EventCategoryDAO($this);
   		$this->phptal->categories=$ccdao->all($cid);
   		echo $this->ProcessTalTemplate('admin/admin_categories.tal.tpl');
   	}
   }
   else
   {
   	require_once(cms_join_path($this->GetModulePath(), 'classes','dao', 'EventCatalogueDAO.php'));
   	$cdao=new EventCatalogueDAO($this);
   	$this->phptal->catalogues=$cdao->all();
   	echo $this->ProcessTalTemplate('admin/admin_catalogues.tal.tpl');
   }

   я молодец. наговнокодил как школьник.
   ветвистое ветвление + жуткая копипаста.
   буду это рефакторить, когда в целом закончу базовую функциональность.

   Lure Of Chaos, 04 Июня 2011

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

6. PHP / Говнокод #6810

   +181

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7
   8. 8
   9. 9

   if ( 
   	isset(parent::$db['title'])	&&	// если существует такое поле в данной таблице
   	!parent::$db['title']		&&	// и оно пустое
   	isset($inDB['title'])		&&	// и таки необходимо записать в базу это поле
   	!$inDB['title']			&&	// но записывать нечего (но надо!)
   	isset(parent::$db['header'])		// зато присутствует то поле, из которого можно получить данные
   ) {
   	$inDB['title'] = strip_tags(parent::$db['header']);
   }

   Ниточки Ариадны...

   istem, 01 Июня 2011

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

7. Java / Говнокод #6770

   +78

   1. 1
   2. 2
   3. 3
   4. 4
   5. 5
   6. 6
   7. 7
   8. 8
   9. 9

   public static DBFactory getInstance() throws SQLException
   	{
   		synchronized (DBFactory.class)
   		{
   			        if (_dbinstance == null)
   				_dbinstance = new DBFactory();
   		}
   		return _dbinstance;
   	}

   Серийный и беспощадный доступ к инстансу ДБ-Фэктори -_-
   Причем, данный код фурычит в многопоточной среде, где порой по 20-30 потоков одновременно обращаются к Фэктори, чтобы взять коннект с пула.

   VirtualVoid, 27 Мая 2011

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

8. Python / Говнокод #6733

   −179

   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
   63. 63
   64. 64
   65. 65
   66. 66
   67. 67
   68. 68
   69. 69
   70. 70
   71. 71

   a=0
   b=0
   c=0
   d=0
   ...
   z=0
    
   A=0
   B=0
   C=0
   D=0
   ...
   Z=0
   ile=int(input())
   while ile>0:
       pom=0
       ile-=1
       slowo=input()
       dlugosc=len(slowo)
       while dlugosc>0:
           dlugosc-=1
          
           if slowo[pom]=='a':
               a=a+1;
           if slowo[pom]=='b':
               b=b+1
           if slowo[pom]=='c':
               c=c+1
           if slowo[pom]=='d':
               d=d+1
           ...
           if slowo[pom]=='z':
                z=z+1
          
           if slowo[pom]=='A':
               A=A+1
           if slowo[pom]=='B':
               B=B+1
           if slowo[pom]=='C':
               C=C+1
           if slowo[pom]=='D':
               D=D+1
           ...
           if slowo[pom]=='Z':
               Z=Z+1
                  
           pom+=1
      
   if(a>0):
       print('a', a)
   if(b>0):
       print('b', b)
   if(c>0):
       print('c', c)
   if(d>0):
       print('d', d)
   ...
   if(z>0):
       print('z', z)
    
   if(A>0):
       print('A', A)
   if(B>0):
       print('B', B)
   if(C>0):
       print('C', C)
   if(D>0):
       print('D', D)
   ...
   if(Z>0):
       print('Z', Z)

   Чел походу не знает о существовании массивов и циклов.
   ... заменено, уж больно код длинный.
   Типичный пример китайского кода.
   Оригинал - http://ideone.com/kkZ51,8lrxt
   
   З.Ы. Да, в питоне не массивы, а списки, но так понятнее

   pabloid, 23 Мая 2011

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

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

   +162

   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 ($size) { 
      case 'small':
       if ($crop_thumbnail == 3 || $crop_thumbnail == 5 || $crop_thumbnail == 6 || $crop_thumbnail == 7 ) {
        $crop = 1;
       }
      break;
      
      case 'medium':
       if ($crop_thumbnail == 2 || $crop_thumbnail == 4 || $crop_thumbnail == 5 || $crop_thumbnail == 7 ) {
        $crop = 1;
       }
      break;
      
      case 'large':
      Default:
       if ($crop_thumbnail == 1 || $crop_thumbnail == 4 || $crop_thumbnail == 6 || $crop_thumbnail == 7 ) {
        $crop = 1;
       }
      break;
   }

   Phocagallery module, Joomla!

   somebody, 20 Мая 2011

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

10. bash / Говнокод #6711

    −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

    # Fallback static file handler, now with XSendfile support.
        if not if(test -f $local_path) {
            if(! ~ $#xsendfile 0) {
                if(! ~ $#xsendfile_alternate 0)
                    XSendfileHeader='X-Accel-Redirect'
                if not
                    XSendfileHeader='X-Sendfile'
                echo $XSendfileHeader': '`{pwd}^'/'$local_path
            }
            if not
                static_file $local_path
        }
        if not if(~ $req_path /pub/* && test -f .$req_path) {
            if(! ~ $#xsendfile 0) {
                if(! ~ $#xsendfile_alternate 0)
                    XSendfileHeader='X-Accel-Redirect'
                if not
                    XSendfileHeader='X-Sendfile'
                echo $XSendfileHeader': '`{pwd}^'/'$req_path
            }
            if not
                static_file .$req_path
        }

    dukzcry, 19 Мая 2011

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

11. JavaScript / Говнокод #6701

    +175

    1. 1
    2. 2

    // The blank option to make sure the DropDownList keeps its width
    var BlankOption = new Option('             ', 0);

    Eugene, 18 Мая 2011

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

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

• 1979
• …
• 838
• 837
• 836
• 835
• 834
• 833
• 832
• 831
• 830
• …
• 1