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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2341)
4. JavaScript (2030)
5. C# (1930)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

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

   В номинации:

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

   За время:

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

2. Java / Говнокод #8255

   +78

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

   // inside some method
   final DateFormat dateFormat = i18n.getDateFormat();
   synchronized (dateFormat) {
       formatedViolationDate = (violationDate != null) ?
       dateFormat.format(violationDate) : "";
   }

   i18n.getDateFormat() возвращает статический объект DateFormat, который может использоваться несколькими потоками. В руки бы накласть тому, кто это писал.
   Решение: getDateFormat() возвращает строку формата, объект формата создаем при каждом вызове.

   roman-kashitsyn, 20 Октября 2011

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

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

   +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

   function validemail($email){
   
    if (!ereg("^[^@]{1,64}@[^@]{1,255}$", $email))
   
    return false;
   
   
   
    $email_array = explode("@", $email);
   
    $local_array = explode(".", $email_array[0]);
   
    for ($i = 0; $i < sizeof($local_array); $i++) {
   
     if (!ereg("^(([A-Za-z0-9!#$%&'*+/=?^_`{|}~-][A-Za-z0-9!#$%&'*+/=?^_`{|}~\.-]{0,63})|(\"[^(\\|\")]{0,62}\"))$", $local_array[$i]))
   
     return false;
   
    }

   jQuery, 08 Сентября 2011

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

4. C++ / Говнокод #7338

   +170

   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

   if(SCREEN_WIDTH==800)
   			{
   				fm1 = 62;
   				fm2 = 61;
   				top = 86;
   			}
   
   			if(SCREEN_WIDTH==1024){
   
   				    fm1 = 27;
   					fm2 = 26;
   					top = 78;
   			}
   			if(SCREEN_WIDTH==1680||SCREEN_WIDTH==1920){
   
   				       fm1 = 72;
   			           fm2 = 73;
   					   top = 81;
   			}
   			if(SCREEN_WIDTH==1280 || SCREEN_WIDTH == 2560){
   
   				       fm1 = 76;
   					   fm2 = 75;
   					   top = 77;
   			}
   				
   			if(SCREEN_WIDTH == 1366 || SCREEN_WIDTH == 1440){
   			        fm1=  67;
   			        fm2=  66;
   					top = 79;
   			}

   Это капля в море гавнокода одного из проектов на который меня посадили.
   Чуть менее 1000 багов открыто только по GUI.
   Все это дело рук одной девушки))

   BlaZe, 25 Июля 2011

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

5. C++ / Говнокод #7020

   +188

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

   typedef enum
     {
       FFALSE = 0,
       TTRUE,
       MMAYBE
     } Truth_t;

   ну почти квантовое программирование.
   
   ЗЫ да, это из С++ программы.

   Dummy00001, 21 Июня 2011

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

6. Куча / Говнокод #6335

   +136

   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

   WebKill
   29.11.2007, 11:02
   Я - WebKill
   
   WebKill - это мой псевдоним - начинающего хакера по имени Миша. Мне 15 лет, увлекаюсь компами, кодингом, хаком, и всем что с этим связано. Люблю писать вирусы, которые сбрасывают комп ламера.
   
   Как WebKill знает комп?
   html 95%
   Pasлal 45%
   JawaSkript 21%
   php 18%
   C\C++ 10%
   VBS 15%
   Хакерская догадка 45%
   Компьютерный сленг 45%
   Безопасность хакера 31%
   Хак теория 12%
   Хак софт 5%
   perl 2%
   Асамблер 2%
   Практика 1%
   Всреднем 21%
   Здесь 100% - полное владение языком, умение написать с его помощю прогу любой сложности
   1% - Прога типа HelloWord
   5% - Знание ЯП с помощю которого можно написаь простенькую заподлужку, например удаление системных файлов, создание по диску файлов с разными словами
   
   
   Я, как истинный хакер могу помочь вам хакать. Если вы неумеете, либо ваша хакерская догадка стремится к нулю - Я помогу вам!
   
   Также мне нужны начинающие хакеры (начинающие в моём понимании - это хакер знающий html, JavaScript, паскаль, C\C++, php + теория сетей, желание и мозги. Всё это есть у меня, если ты чего-то не знаешь, не расстраивайся)
   Если ты знаешь комп примерно как я, умеешь сбрасывать комп, то присоединяйся к моей хак группе - webkillgroup. Мы занимаемся совмесным изучением компа.
   
   Свои вопросы можно оставлять в этой теме.

   Деанонимизация ВебКилла. Взято отсюда: http://forum.asechka.ru/archive/index.php/t-100897.html

   Govnocoder#0xFF, 13 Апреля 2011

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

7. JavaScript / Говнокод #4996

   +174

   1. 1
   2. 2
   3. 3

   <script>
      location.href=location.href;
   </script>

   Этот код работает - он обновляет страницу, встречал не раз.
   window.location.reload() все-таки гораздо красивее...

   elCreator, 20 Декабря 2010

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

8. Java / Говнокод #4956

   +70

   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

   public class HttpServer {
   
       public static void main(String[] args) throws Throwable {
           ServerSocket ss = new ServerSocket(8080);
           while (true) {
               Socket s = ss.accept();
               System.err.println("Client accepted");
               new Thread(new SocketProcessor(s)).start();
           }
       }
   
       private static class SocketProcessor implements Runnable {
   
           private Socket s;
           private InputStream is;
           private OutputStream os;
   
           private SocketProcessor(Socket s) throws Throwable {
               this.s = s;
               this.is = s.getInputStream();
               this.os = s.getOutputStream();
           }
   
           public void run() {
               try {
                   readInputHeaders();
                   writeResponse("<html><body><h1>Hello from Habrahabr</h1></body></html>");
               } catch (Throwable t) {
                   /*do nothing*/
               } finally {
                   try {
                       s.close();
                   } catch (Throwable t) {
                       /*do nothing*/
                   }
               }
               System.err.println("Client processing finished");
           }
   
           private void writeResponse(String s) throws Throwable {
               String response = "HTTP/1.1 200 OK\r\n" +
                       "Server: YarServer/2009-09-09\r\n" +
                       "Content-Type: text/html\r\n" +
                       "Content-Length: " + s.length() + "\r\n" +
                       "Connection: close\r\n\r\n";
               String result = response + s;
               os.write(result.getBytes());
               os.flush();
           }
   
           private void readInputHeaders() throws Throwable {
               BufferedReader br = new BufferedReader(new InputStreamReader(is));
               while(true) {
                   String s = br.readLine();
                   if(s == null || s.trim().length() == 0) {
                       break;
                   }
               }
           }
       }
   }

   Это весь код вебсервера.
   К слову сказать, это от сюда:
   http://habrahabr.ru/blogs/java/69136/

   Говногость, 17 Декабря 2010

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

9. Куча / Говнокод #4645

   +135

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

   <p id="entrance">
               Нашли или ...
   </p>
   
   <p id="entrance" style="background: #fff; padding: 8px; -webkit-border-radius: 8px; -moz-border-radius: 8px; border-radius: 8px; -webkit-box-shadow: 0 0 5px #aaa">
   	Дорогие пользователи
   </p>

   Решил я написать скрипт для GreaseMonkey, который убирает сообщение "Дорогие пользователи, "
   И вот что обнаружил: в коде есть два одинаковых айдишника id="entrance".
   
   Используется стиль p#entrance.
   
   Если надо использовать стили в нескольких элементах, используй классы.
   
   А разные айдишники оставь для джаваскрипта,
   а то не выбрать определенный элемент с помощью document.getElementById.
   
   В крайнем случае используй атрибут name. Для него есть метод document.getElementsByName

   Oleg_quadro, 16 Ноября 2010

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

10. Perl / Говнокод #4589

    −119

    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

    sub mainMenu {
    	if ($action eq "addtab" && $iamadmin) { require "$sourcedir/AdvancedTabs.pl"; &AddNewTab; }
    	elsif ($action eq "edittab" && $iamadmin) { require "$sourcedir/AdvancedTabs.pl"; &EditTab; }
    	elsif ($action ne "") {
    		if ($action eq "search2") {
    			$tmpaction = "search";
    		} elsif ($action eq "favorites" || $action eq "shownotify" || $action eq "im" || $action eq "imdraft" || $action eq "imoutbox" || $action eq "imstorage" || $action eq "imsend" || $action eq "imsend2" || $action eq "imshow" || $action eq "profileCheck" || $action eq "myviewprofile" || $action eq "myprofile" || $action eq "myprofileContacts" || $action eq "myprofileOptions" || $action eq "myprofileBuddy" || $action eq "myprofileIM" || $action eq "myprofileAdmin" || $action eq "myusersrecentposts") {
    			$tmpaction = "mycenter";
    		} elsif ($action eq "messagepagetext" || $action eq "messagepagedrop" || $action eq "threadpagetext" || $action eq "threadpagedrop" || $action eq "post" || $action eq "notify" || $action eq "boardnotify" || $action eq "sendtopic" || $action eq "modify") {
    			$tmpaction = "home";
    		} elsif ($action eq "guestpm2") {
    			$tmpaction = "guestpm";
    		} else { $tmpaction = $action; }
    
    	} else {
    		$tmpaction = "home";
    	}
    
    	$tab{'home'} = qq~<span |><a href="$scripturl" title = "$img_txt{'103'}" style="padding: 3px 0 4px 0;">$tabfill$img_txt{'103'}$tabfill</a></span>~;
    	$tab{'help'} = qq~<span |><a href="$scripturl?action=help" title = "$img_txt{'119'}" style="padding: 3px 0 4px 0; cursor:help;">$tabfill$img_txt{'119'}$tabfill</a></span>~;
    	if ($maxsearchdisplay > -1) {
    		$tab{'search'} = qq~<span |><a href="$scripturl?action=search" title = "$img_txt{'182'}" style="padding: 3px 0 4px 0;">$tabfill$img_txt{'182'}$tabfill</a></span>~;
    	}
    	if (!$ML_Allowed || ($ML_Allowed == 1 && !$iamguest) || ($ML_Allowed == 2 && $staff) || ($ML_Allowed == 3 && ($iamadmin || $iamgmod))) {
    		$tab{'ml'} = qq~<span |><a href="$scripturl?action=ml" title = "$img_txt{'331'}" style="padding: 3px 0 4px 0;">$tabfill$img_txt{'331'}$tabfill</a></span>~;
    	}
    	if ($iamadmin) {
    		$tab{'admin'} = qq~<span |><a href="$boardurl/AdminIndex.$yyaext" title = "$img_txt{'2'}" style="padding: 3px 0 4px 0;">$tabfill$img_txt{'2'}$tabfill</a></span>~;
    	}
    	if ($iamgmod) {
    		if (-e "$vardir/gmodsettings.txt") { require "$vardir/gmodsettings.txt"; }
    		if ($allow_gmod_admin) {
    			$tab{'admin'} = qq~<span |><a href="$boardurl/AdminIndex.$yyaext" title = "$img_txt{'2'}" style="padding: 3px 0 4px 0;">$tabfill$img_txt{'2'}$tabfill</a></span>~;
    		}
    	}
    	if ($sessionvalid == 0 && !$iamguest) {
    		my $sesredir;
    		unless (!$testenv || $action eq "revalidatesession" || $action eq "revalidatesession2") {
    			$sesredir = $testenv;
    			$sesredir =~ s/\=/\~/g;
    			$sesredir =~ s/;/x3B/g;
    			$sesredir = qq~;sesredir=$sesredir~;
    		}
    		$tab{'revalidatesession'} = qq~<span |><a href="$scripturl?action=revalidatesession$sesredir" title = "$img_txt{'34a'}" style="padding: 3px 0 4px 0;">$tabfill$img_txt{'34a'}$tabfill</a></span>~;
    	}
    
    // далее мало что меняется в стиле...

    Формируем меню...

    Arigato, 11 Ноября 2010

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

11. Java / Говнокод #4587

    +145

    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

    /**
         * Returns an array of all the objects currently registered
         * as <code><em>Foo</em>Listener</code>s
         * upon this <code>Component</code>.
         * <code><em>Foo</em>Listener</code>s are registered using the
         * <code>add<em>Foo</em>Listener</code> method.
         *
         * <p>
         * You can specify the <code>listenerType</code> argument
         * with a class literal, such as
         * <code><em>Foo</em>Listener.class</code>.
         * For example, you can query a
         * <code>Component</code> <code>c</code>
         * for its mouse listeners with the following code:
         *
         * <pre>MouseListener[] mls = (MouseListener[])(c.getListeners(MouseListener.class));</pre>
         *
         * If no such listeners exist, this method returns an empty array.
         *
         * @param listenerType the type of listeners requested; this parameter
         *          should specify an interface that descends from
         *          <code>java.util.EventListener</code>
         * @return an array of all objects registered as
         *          <code><em>Foo</em>Listener</code>s on this component,
         *          or an empty array if no such listeners have been added
         * @exception ClassCastException if <code>listenerType</code>
         *          doesn't specify a class or interface that implements
         *          <code>java.util.EventListener</code>
         *
         * @see #getComponentListeners
         * @see #getFocusListeners
         * @see #getHierarchyListeners
         * @see #getHierarchyBoundsListeners
         * @see #getKeyListeners
         * @see #getMouseListeners
         * @see #getMouseMotionListeners
         * @see #getMouseWheelListeners
         * @see #getInputMethodListeners
         * @see #getPropertyChangeListeners
         *
         * @since 1.3
         */
        public <T extends EventListener> T[] getListeners(Class<T> listenerType) { 
            EventListener l = null; 
            if  (listenerType == ComponentListener.class) { 
                l = componentListener;
            } else if (listenerType == FocusListener.class) {
                l = focusListener;
            } else if (listenerType == HierarchyListener.class) {
                l = hierarchyListener;
            } else if (listenerType == HierarchyBoundsListener.class) {
                l = hierarchyBoundsListener;
            } else if (listenerType == KeyListener.class) {
                l = keyListener;
            } else if (listenerType == MouseListener.class) {
                l = mouseListener;
            } else if (listenerType == MouseMotionListener.class) {
                l = mouseMotionListener; 
            } else if (listenerType == MouseWheelListener.class) {
                l = mouseWheelListener; 
            } else if (listenerType == InputMethodListener.class) {
                l = inputMethodListener; 
            } else if (listenerType == PropertyChangeListener.class) {
                return (T[])getPropertyChangeListeners();
            }
            return AWTEventMulticaster.getListeners(l, listenerType);
        }

    как вы думаете, что это? внутренности Java
    java.awt.Component

    Lure Of Chaos, 11 Ноября 2010

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

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

• 1963
• …
• 1672
• 1671
• 1670
• 1669
• 1668
• 1667
• 1666
• 1665
• 1664
• …
• 1