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

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

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

1. PHP (5712)
2. C++ (2745)
3. Куча (2405)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1121)
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. PHP / Говнокод #12624

   +154

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

   if(!empty($_POST['action']) && !empty($_POST['product_id']))
   {
   	$product_id=$_POST['product_id']+0;
           etc...
   }

   Наткнулся вот!)

   Impossible, 21 Февраля 2013

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

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

   +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
   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
   72. 72
   73. 73
   74. 74
   75. 75
   76. 76
   77. 77
   78. 78
   79. 79
   80. 80
   81. 81
   82. 82
   83. 83
   84. 84
   85. 85
   86. 86
   87. 87

   function poisk_potomok_item ($id_otca, $glubina, $i_array)
                 {
                  global /*$style,*/ $kol_n, $sm, $fire;
   
                  if ($glubina==$max_vlog) return 1;
   
                  switch ($glubina)
                       {
                        case "1":
                         {
                          $base_style="menu2_level3";
                          break;
                         }
                        case "2":
                         {
                          $base_style="menu2_level4";
                          break;
                         }
                        case "3":
                         {
                          $base_style="menu2_level5";
                          break;
                         }
                        case "4":
                         {
                          $base_style="menu2_level6";
                          break;
                         }
                        case "5":
                         {
                          $base_style="menu2_level7";
                          break;
                         }
                        case "6":
                         {
                          $base_style="menu2_level8";
                          break;
                         }
                        case "7":
                         {
                          $base_style="menu2_level8";
                          break;
                         }
                       }
                  for ($j=1; $j<=$kol_n; $j++)
                     {
                      $mas5=explode(" ~ ", $i_array[$j]);
                      if ($mas5[1]==$id_otca and $mas5[2]=="1")
                        {
                         $style_pot="<table cellspacing=\"0\"><tr><td class=\"$base_style\"><a href=\"?r=item&gs=".$mas5[0]."\" class=\"menu2_2\">".$mas5[4]."</a></td></tr></table>";
   
                         for($kkkk=count($sm);$kkkk > 0; $kkkk--)
                            {
                             if ($mas5[0]==$sm[$kkkk])
                               {
                                if ($mas5[0]==$sm[1])
                                  {
                                   $style_pot="<table cellspacing=\"0\"><tr><td class=\"$base_style\"><span class=\"menu2_2a\">".$mas5[4]."</span></td></tr></table>";
                                   
                                    $new_gl=$glubina+1;
   
                                    $gi=poisk_potomok_item ($mas5[0],$new_gl, $i_array);
                                   
                                    if ($gi != "") {$style_pot.=$fire.$gi.$fire;}
                                  }
                                else
                                  {
                                 //  $style_pot="<table cellspacing=\"0\"><tr><td class=\"$base_style\"><a href=\"?r=item&gs=".$mas5[0]."\" class=\"menu2_2\">".$mas5[4]."</a></td></tr></table>";
                                   $style_pot="<table cellspacing=\"0\"><tr><td class=\"$base_style\"><span class=\"menu2_2a\">".$mas5[4]."</span></td></tr></table>";
   
                                   $new_gl=$glubina+1;
   
                                    $gi=poisk_potomok_item ($mas5[0],$new_gl, $i_array);
   
                                    if ($gi != "") {$style_pot.=$fire.$gi.$fire;}
   
                                  }
                               }
                            }
   
                         $style.=$style_pot;
   
                        }
                 }
   
                return $style;
                 }

   В одной из ведущих веб-студий Екатеринбурга, директор 5 лет назад был кодером)))
   вот одно их его творений..
   
   Обратите внимание на идентификаторы, такие как poisk_potomok_item, $id_otca, $glubina, $i_array.
   Вообще, как человек, директор - норм чувак, но кажется мне придется пересмотреть свои взгляды...
   
   Если человек называет функцию так: "poisk_potomok_item" - ему явно жить труднее остальных людей)))))

   arbora, 21 Февраля 2013

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

3. Куча / Говнокод #12622

   +124

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

   import Control.Applicative
   import Data.List
   isSimple n | n > 0 = not $ or $ map (\n1 -> n `rem` n1 == 0) [2..n-1]
   isNaturalAndSimple n = n > 0 && isSimple n
   generateSource b e = map (flip replicate $ [b..e]) [2..]
   splitAll = groupBy $ \_ _->False
   splitedNumberRangesToNumberSequence l = foldl (liftA2 (++)) (head l) (tail l)
   toNumberSequences = map $ splitedNumberRangesToNumberSequence . map splitAll
   onlySimple = map $ filter $ isNaturalAndSimple . sum
   main = print $ take 4 $ onlySimple $ toNumberSequences $ generateSource 2 3

   HaskellGovno, 20 Февраля 2013

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

4. Куча / Говнокод #12621

   +118

   1. 1
   2. 2

   fib = 0 : 1 : zipWith (+) fib (tail fib)
   main = print $ take 700 fib

   HaskellGovno, 20 Февраля 2013

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

5. Куча / Говнокод #12620

   +124

   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

   import Control.Monad
   import Control.Arrow
   import Data.List
   
   solve' :: [String] -> [[String]]
   solve' = nub . filter (
               and . uncurry (
                   zipWith (
                       (.head) . (==) . last
                   )
               ) . (id &&& tail)
            ) . uncurry ($) . (
               last . (((
                   map (
                       last . fst &&& uncurry (++) . (init . fst &&& snd)
                   ) . tail . uncurry (zipWith (,)) . (inits &&& tails)
               ) >=> (uncurry map) . 
                   ((:) *** solve')
               ):
               ) . (uncurry takeWhile) . (
                       const . null &&& const [const [[]]]
                   ) &&& id
            )
   
   main = print $ solve' ["123","321","123"]

   HaskellGovno, 20 Февраля 2013

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

6. Objective C / Говнокод #12618

   −106

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

   - (void)shutdownSoundManager {
   	@synchronized(self) {
   		if(sharedSoundManager != nil) {
   			[self dealloc];
   		}
   	}
   }

   Убийца синглетонов, блджд

   clockworkman, 20 Февраля 2013

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

7. SQL / Говнокод #12617

   −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
   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

   /*здесь было много кода*/
   
   DECLARE @Strings TABLE /* in this temporary table we keep all strings, even the names of the elements, since they are 'escaped' in a different way, and may contain, unescaped, brackets denoting objects or lists. These are replaced in the JSON string by tokens representing the string */
       (
        String_ID INT IDENTITY(1, 1),
        StringValue NVARCHAR(MAX)
       )
     SELECT--initialise the characters to convert hex to ascii
       @characters='0123456789abcdefghijklmnopqrstuvwxyz',
     /* firstly we process all strings. This is done because [{} and ] aren't escaped in strings, which complicates an iterative parse. */
       @parent_ID=0;
     WHILE 1=1 --forever until there is nothing more to do
       BEGIN
         SELECT
           @start=PATINDEX('%[^a-zA-Z]["]%', @json collate SQL_Latin1_General_CP850_Bin);--next delimited string
         IF @start=0 BREAK --no more so drop through the WHILE loop
         IF SUBSTRING(@json, @start+1, 1)='"'
           BEGIN --Delimited Name
             SET @start=@Start+1;
             SET @end=PATINDEX('%[^\]["]%', RIGHT(@json, LEN(@json+'|')-@start) collate SQL_Latin1_General_CP850_Bin);
           END
         IF @end=0 --no end delimiter to last string
           BREAK --no more
         SELECT @token=SUBSTRING(@json, @start+1, @end-1)
         --now put in the escaped control characters
         SELECT @token=REPLACE(@token, FROMString, TOString)
         FROM
           (SELECT
             '\"' AS FromString, '"' AS ToString
            UNION ALL SELECT '\\', '\'
            UNION ALL SELECT '\/', '/'
            UNION ALL SELECT '\b', CHAR(08)
            UNION ALL SELECT '\f', CHAR(12)
            UNION ALL SELECT '\n', CHAR(10)
            UNION ALL SELECT '\r', CHAR(13)
            UNION ALL SELECT '\t', CHAR(09)
           ) substitutions
         SELECT @result=0, @escape=1
     --Begin to take out any hex escape codes
         WHILE @escape>0
           BEGIN
             SELECT @index=0,
             --find the next hex escape sequence
             @escape=PATINDEX('%\x[0-9a-f][0-9a-f][0-9a-f][0-9a-f]%', @token collate SQL_Latin1_General_CP850_Bin)
             IF @escape>0 --if there is one
               BEGIN
                 WHILE @index<4 --there are always four digits to a \x sequence  
                   BEGIN
                     SELECT --determine its value
                       @result=@result+POWER(16, @index)
                       *(CHARINDEX(SUBSTRING(@token, @escape+2+3-@index, 1),
                                   @characters)-1), @index=@index+1 ;
           
                   END
                   -- and replace the hex sequence by its unicode value
                 SELECT @token=STUFF(@token, @escape, 6, NCHAR(@result))
               END
           END
         --now store the string away
         INSERT INTO @Strings (StringValue) SELECT @token
         -- and replace the string with a token
         SELECT @JSON=STUFF(@json, @start, @end+1,
                       '@string'+CONVERT(NVARCHAR(5), @@identity))
       END

   парсинг json-строки sql скриптом. полная версия тут : https://www.simple-talk.com/sql/t-sql-programming/consuming-json-strings-in-sql-server/

   mozg_raka, 20 Февраля 2013

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

8. Objective C / Говнокод #12616

   −98

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

   dispatch_async(dispatch_get_global_queue(0, 0), ^{
               [NSThread sleepForTimeInterval:0.35];
               [[NSThread mainThread] performBlock:^{
                   [titleLabel setHidden:NO];
               }];
           });

   torip3ng, 20 Февраля 2013

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

9. Java / Говнокод #12615

   +67

   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

   public final class SomeActivity extends Activity {
          @Override
   	protected Dialog onCreateDialog(int id) {
   		Dialog dialog = null;
   		if (id == DialogGenerator.SETTINGS_DIALOG) {
   			dialog = mDialogGenerator.createSettingsDialog();
   		}  {
   			dialog = super.onCreateDialog(id);
   		}
   		return dialog;
   	}
   
   	@Override
   	protected void onPrepareDialog(int id, Dialog dialog) {
   		super.onPrepareDialog(id, dialog);
   		if (id == DialogGenerator.SETTINGS_DIALOG) {
   			mDialogGenerator.prepareSettingsDialog((AlertDialog) dialog, someBoolValue, someObjectValue);
   		} 
   	}
   }
   
   public final class DialogGenerator {
   	public Dialog createSettingsDialog() {
   		int dialogId = SETTINGS_DIALOG;
   		int titleId = R.string.settingsTitle;
   		String[] itemsArray = getStringArray(R.array.settings);
   		ThreeTypeOptionsAdapter adapter = new ThreeTypeOptionsAdapter(mControllerAsActivity, itemsArray,
   				mCheckableOptions, mTwoTextOptions);
   		AlertDialogCallback dialogCallback = new AlertDialogCallback(mControllerAsDialogHost, dialogId);
   		AlertDialog.Builder dialogBuilder = new AlertDialog.Builder(mControllerAsActivity);
   		dialogBuilder.setTitle(titleId);
   		// костыль
   		dialogBuilder.setSingleChoiceItems(itemsArray, 0, null);
   		// конец костыля
   		dialogBuilder.setAdapter(adapter, dialogCallback);
   		AlertDialog dialog = dialogBuilder.create();
   		setMainListenersOnDialog(dialog);
   		dialog.getListView().setChoiceMode(ListView.CHOICE_MODE_MULTIPLE);
   		return dialog;
   	}
   	
   	public void prepareSettingsDialog(AlertDialog dialog, boolean someBoolValue, String someStringValue) {
   		//костыль
   		ListView dialogList = dialog.getListView();
   		dialogList.clearChoices();
   		dialogList.setItemChecked(POSITION_FOR_BOOL, someBoolValue);
   		//конец костыля
   		ThreeTypeOptionsAdapter adapter = (ThreeTypeOptionsAdapter) dialogList.getAdapter();
   		adapter.setAdditionalValue(POSITION_FOR_STRING, someStringValue);	
   	}
   }

   Задача: показать диалог, внутри которого есть 3 типа ячеек: выделяемые с чекбоксом справа, невыделяемые с дополнительной надписью справа, обычные невыделяемые.
   Примерно так:
   Use GPS checkbox
   Selected country Russia
   Launch some activity
   
   В комментах к методам я описал костыль:
   Здравствуйте, дорогие друзья. Сегодня мы с вами поговорим об уникальном виде животных, которых открыли только в конце XX века. Это, дорогие друзья, Ява-обезьяны. Давайте дружно откроем сырцы Андроид-СДК, а именно - класс com.android.internal.app.AlertController .AlertParams - и метод createListView(AlertController). Видите, одна обезьяна решила, что нам будет удобней, если диалог автоматом задисмиссится, когда мы щёлкнем по элементу списка в CHOICE_MODE_NONE, и OnClickListener будет не null? А теперь давайте перейдём в android.app.AlertDialog.Builder. Видите, там другая обезьяна решила, что для multi-choice списка нам не потребуется хитроумный адаптер? Поэтому, дорогие друзья, когда мы поставили кастомный адаптер (setAdapter()) и сопроводили его слушателем, то какой бы мы choice mode не ставили после AlertDialog.Builder.create(), диалог дисмиссился. Поэтому, дорогие друзья, пришлось пойти на хитрость и вначале установить single choice (чтобы флаг mIsSingleChoice стал true), а потом уже ставить адаптер.

   QuickNick, 20 Февраля 2013

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

10. Куча / Говнокод #12614

    +133

    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

    drwxrwxrwx  1 wvxvw None       0 02-17 16:59 .svn
      -rw-rw-rw-  1 wvxvw None   22082 02-04 11:03 header.asp
      -rw-rw-rw-  1 wvxvw None   14331 02-04 11:02 header.utf.asp
      -rw-rw-rw-  1 wvxvw None   10333 02-04 11:01 header2.asp
      -rw-rw-rw-  1 wvxvw None   10368 02-04 11:02 header2_test.asp
      -rw-rw-rw-  1 wvxvw None   11825 02-04 11:02 header3.asp
      -rw-rw-rw-  1 wvxvw None   11734 02-04 11:03 header3_liran.asp
      -rw-rw-rw-  1 wvxvw None   11238 02-04 11:01 header3_old.asp
      -rw-rw-rw-  1 wvxvw None    7403 02-04 11:02 header3_ssl.asp
      -rw-rw-rw-  1 wvxvw None    9045 02-04 11:01 header3del.asp
      -rw-rw-rw-  1 wvxvw None   39451 02-04 11:02 header4.asp
      -rw-rw-rw-  1 wvxvw None   40224 02-04 11:02 header4_new.asp
      -rw-rw-rw-  1 wvxvw None   13348 02-04 11:02 header4rus.asp
      -rw-rw-rw-  1 wvxvw None     230 02-04 11:02 headers.asp
      -rw-rw-rw-  1 wvxvw None     208 02-06 09:37 headers.php

    Наследство.

    wvxvw, 19 Февраля 2013

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

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

• 2116
• …
• 1138
• 1137
• 1136
• 1135
• 1134
• 1133
• 1132
• 1131
• 1130
• …
• 1