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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2373)
4. JavaScript (2031)
5. C# (1929)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (588)
10. 1C (539)
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. Haskell (30)
22. Swift (27)
23. Kotlin (14)

1. SQL / Говнокод #16099

   −169

   1. 1

   SELECT Status_ID FROM User,UserStatus,Status WHERE User_ID=UserStatus_UserID AND UserStatus_StatusID=Status_ID GROUP BY Status_ID

   dafuq, 03 Июня 2014

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

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

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

   //checks if the string is a hex stream e.g. "31 32 33 6A F8"
           private bool _IsHexStream(string sValue)
           {
               sValue = sValue.Trim();
   
               
               if (sValue.Length < 2)
               {
                   return false;
               }
   
               for (int i = 0; i < sValue.Length; i++)
               {
                   if(_IsHexChar(Convert.ToChar(sValue.Substring(i,1))) == false)
                   {
                       return false;
                   }
               }
   
               //every third char must be a space, only possible in case of two bytes
               if (sValue.Length > 3)
               {
                   for (int i = 2; i < sValue.Length; i += 3)
                   {
                       string sBuffer = sValue.Substring(i, 1);
   
                       if (sBuffer.Equals(" ") == false)
                       {
                           return false;
                       }
                   }
               }
   
               //string is a hex stream 
               return true;
           }

   blackhearted, 02 Июня 2014

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

3. C# / Говнокод #16096

   +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
   32. 32
   33. 33
   34. 34

   TestScriptResult Test_method( ... )
   {
   	TestScriptResult result = new TestScriptResult();
   	object obj = null;
   	///...
   	obj = foo.Set(...);
   	if (obj.GetType() == typeof(Exception))
   	{
   		result.SetResult(TestScriptResult.eTestResult.FAIL_SET_REQUEST, ((Exception)obj).Message);
   		_LogTestMethodEnd(result);
   		return result;
   	}
   	else
   	{
   	}
   }
   
   public Object Set(...)
   {
   	//...
   	if(CheckForErrors(res) == true)
   	{
   		//create error description
   		string sErrorDescription = string.Format("Error during ...."));
   		//create exception object and return this
   		Exception ex = new Exception(sErrorDescription);
   		//error logging
   		m_logger.Error(sErrorDescription);
   		return ex;
   	}
   	else
   	{
   	}
   }

   Ну нах так жыть, котаны???

   blackhearted, 02 Июня 2014

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

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

   +8

   1. 1
   2. 2

   #define FOR_INCLUSION (iostream) (studio.h) (stdlib.h) (boost/phoenix.hpp) (my_drugs_library.h)
   #include <include_and_disable_some_warnings>

   laMer007, 01 Июня 2014

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

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

   +96

   1. 1
   2. 2

   var m = map[int]string { 1: "one", 2: "two", 3: "three", }
   if val, ok := m[3]; ok { fmt.Printf("Yes! %s\n",val) }	else { fmt.Println("No!") }

   Почему в Go первое присваивание надо писать через =
   А то что в if через :=
   http://ideone.com/cPf2cw
   http://ideone.com/fork/cPf2cw
   

   3.14159265, 01 Июня 2014

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

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

   +150

   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

   <?php
   $subjects = array (
     0 => 
       array (
         'title' => 'ОС Unix',
         'teacher' => 'Вася',
         'mark' => '3'),
     1 => 
       array (
         'title' => 'Компьютерные сети',
         'teacher' => 'Петя',
         'mark' =>'4'),
     2 => 
       array (
         'title' =>  'Алгоритмы',
         'teacher' =>  'Коля',
         'mark' => '4'),
   );?>
   <table>
   <?  foreach ($subjects as $key=>$subject) : 
   $number = $key+1;
   ?>
               <tr>
                   <td><?echo $number;?></td>
   				<td><?echo $subject['title'];?></td>
                   <td><?echo $subject['teacher'];?></td>
                   <td><?echo $subject['mark'];?></td>
               </tr>
   <? endforeach; ?>
   </table>
   <? var_dump($subject)?>

   wiedzal, 31 Мая 2014

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

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

   +73

   1. 001
   2. 002
   3. 003
   4. 004
   5. 005
   6. 006
   7. 007
   8. 008
   9. 009
   10. 010
   11. 011
   12. 012
   13. 013
   14. 014
   15. 015
   16. 016
   17. 017
   18. 018
   19. 019
   20. 020
   21. 021
   22. 022
   23. 023
   24. 024
   25. 025
   26. 026
   27. 027
   28. 028
   29. 029
   30. 030
   31. 031
   32. 032
   33. 033
   34. 034
   35. 035
   36. 036
   37. 037
   38. 038
   39. 039
   40. 040
   41. 041
   42. 042
   43. 043
   44. 044
   45. 045
   46. 046
   47. 047
   48. 048
   49. 049
   50. 050
   51. 051
   52. 052
   53. 053
   54. 054
   55. 055
   56. 056
   57. 057
   58. 058
   59. 059
   60. 060
   61. 061
   62. 062
   63. 063
   64. 064
   65. 065
   66. 066
   67. 067
   68. 068
   69. 069
   70. 070
   71. 071
   72. 072
   73. 073
   74. 074
   75. 075
   76. 076
   77. 077
   78. 078
   79. 079
   80. 080
   81. 081
   82. 082
   83. 083
   84. 084
   85. 085
   86. 086
   87. 087
   88. 088
   89. 089
   90. 090
   91. 091
   92. 092
   93. 093
   94. 094
   95. 095
   96. 096
   97. 097
   98. 098
   99. 099
   100. 100

   int a = 1;
       int b = 2;
       int c = 2;
       String d = " ";
       System.out.print(a+d);
       System.out.print(b+d);
       System.out.print(b+a+d);
       System.out.print(4+d);
       System.out.print(5+d);
       System.out.print(6+d);
       System.out.print(7+d);
       System.out.print(8+d);
       System.out.print(9+d);
       System.out.println(10+d);
       System.out.print(2+d);
       System.out.print(4+d);
       System.out.print(6+d);
       System.out.print(8+d);
       System.out.print(10+d);
       System.out.print(12+d);
       System.out.print(14+d);
       System.out.print(16+d);
       System.out.print(18+d);
       System.out.println(20+d);
       System.out.print(3+d);
       System.out.print(6+d);
       System.out.print(9+d);
       System.out.print(12+d);
       System.out.print(15+d);
       System.out.print(18+d);
       System.out.print(21+d);
       System.out.print(24+d);
       System.out.print(27+d);
       System.out.println(30+d);
       System.out.print(4+d);
       System.out.print(8+d);
       System.out.print(12+d);
       System.out.print(16+d);
       System.out.print(20+d);
       System.out.print(24+d);
       System.out.print(28+d);
       System.out.print(32+d);
       System.out.print(36+d);
       System.out.println(40+d);
       System.out.print(5+d);
       System.out.print(10+d);
       System.out.print(15+d);
       System.out.print(20+d);
       System.out.print(25);
       System.out.print(30+d);
       System.out.print(35+d);
       System.out.print(40+d);
       System.out.print(45+d);
       System.out.println(50+d);
       System.out.print(6+d);
       System.out.print(12+d);
       System.out.print(18+d);
       System.out.print(24+d);
       System.out.print(30+d);
       System.out.print(36+d);
       System.out.print(42+d);
       System.out.print(48+d);
       System.out.print(54+d);
       System.out.println(60+d);
       System.out.print(7+d);
       System.out.print(14+d);
       System.out.print(21+d);
       System.out.print(28+d);
       System.out.print(35+d);
       System.out.print(42+d);
       System.out.print(49+d);
       System.out.print(56+d);
       System.out.print(63+d);
       System.out.println(70+d);
       System.out.print(8+d);
       System.out.print(16+d);
       System.out.print(24+d);
       System.out.print(32+d);
       System.out.print(40+d);
       System.out.print(48+d);
       System.out.print(56+d);
       System.out.print(64+d);
       System.out.print(72+d);
       System.out.println(80+d);
       System.out.print(9+d);
       System.out.print(18+d);
       System.out.print(27+d);
       System.out.print(36+d);
       System.out.print(45+d);
       System.out.print(54+d);
       System.out.print(63+d);
       System.out.print(72+d);
       System.out.print(81+d);
       System.out.println(90+d);
       System.out.print(10+d);
       System.out.print(20+d);
       System.out.print(30+d);
       System.out.print(40+d);
       System.out.print(50+d);
       System.out.print(60+d);

   Пытался таблицу умножения сделать в детстве...

   thematver, 31 Мая 2014

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

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

   +157

   1. 1
   2. 2
   3. 3

   if (strlen($arResult["ERROR_MESSAGE"]) <= 0){
   // любая ахинея
   }

   2014 год. Битрикс все так же терпеливо ожидает патент на строки отрицательной длины.
   Полагаю, такой икспрешн используется не только в компоненте корзины, учитывая страсть авторов к копипасте.

   velosipedistorg, 30 Мая 2014

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

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

   +154

   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

   if ($_SERVER["REQUEST_METHOD"] == "POST" && strlen($_REQUEST["backButton"]) > 0 && ($arParams["DELIVERY_NO_SESSION"] == "N" || check_bitrix_sessid()))
   {
   	if($arResult["POST"]["CurrentStep"] == 6 && $arResult["SKIP_FORTH_STEP"] == "Y")
   		$arResult["CurrentStepTmp"] = 3;
   
   	if($arResult["POST"]["CurrentStepTmp"] <= 5 && $arResult["SKIP_THIRD_STEP"] == "Y")
   		$arResult["CurrentStepTmp"] = 2;
   
   	if($arResult["POST"]["CurrentStepTmp"] <= 3 && $arResult["SKIP_SECOND_STEP"] == "Y")
   		$arResult["CurrentStepTmp"] = 1;
   
   	if(IntVal($arResult["CurrentStepTmp"])>0)
   		$arResult["CurrentStep"] = $arResult["CurrentStepTmp"];
   	else
   		$arResult["CurrentStep"] = $arResult["CurrentStep"] - 2;
   	$arResult["BACK"] = "Y";
   }
   if ($arResult["CurrentStep"] <= 0)
   	$arResult["CurrentStep"] = 1;
   $arResult["ERROR_MESSAGE"] = "";

   Компонент страницы заказа в магазине Битрикса, 14я версия.
   Тут вам и отсутствие констант для осмысленных шагов, и отсутствие конфигурирования этих шагов, как они i18n-ятся, вообще не представляю.
   КАК ЭТО ВСЁ РАБОТАЕТ??!!!
   
   ЗЫ мало того, тут же в компонент вкопипащен (или переговнокоден заново, не сравнивал) процесс залогинивания/регистрации нового юзера, елси он не залогинен. Не смог перекопипастить эту копипасту лишь потому, что она содержит 310 строк. Неслабый DRY.

   velosipedistorg, 30 Мая 2014

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

10. Java / Говнокод #16085

    +72

    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

    public class PLock {
        private Map<Thread, Integer> readLocks = new HashMap<Thread, Integer>();
        private Thread writeLock = null;
        private int writeLockCount = 0;
        
        public synchronized void getReadLock() {
            Thread currentThread = Thread.currentThread();
            long startTimeMillis = System.currentTimeMillis();
            boolean gotStuck = false;
            while (canClaimReadLock(currentThread) == false) {
                gotStuck = true;
                try {
                    wait();
                } catch (InterruptedException ex) {
                    Log.warn("Interrupted while attempting to get read lock.", ex);
                }
            }
            report(gotStuck, startTimeMillis, "read");
            if (readLocks.containsKey(currentThread)) {
                readLocks.put(currentThread, 1 + readLocks.get(currentThread));
            } else {
                readLocks.put(currentThread, 1);
            }
        }
        
        ...
        
        public synchronized void relinquishReadLock() {
            Thread currentThread = Thread.currentThread();
            if (readLocks.containsKey(currentThread) == false) {
                throw new RuntimeException("Cannot relinquish read lock on thread " + currentThread + " because it does not hold a lock.");
            }
            int newLockCount = readLocks.get(currentThread) - 1;
            if (newLockCount == 0) {
                readLocks.remove(currentThread);
                notifyAll();  // IMPORTANT: allow other threads to wake up and check if they can get locks now.
            } else {
                readLocks.put(currentThread, newLockCount);
            }
        }
        
        public synchronized void getWriteLock() {
            //Log.warn("getWriteLock() in thread " + Thread.currentThread());
            //dumpLocks();
            Thread currentThread = Thread.currentThread();
            long startTimeMillis = System.currentTimeMillis();
            boolean gotStuck = false;
            while (canClaimWriteLock(currentThread) == false) {
                gotStuck = true;
                try {
                    wait();
                } catch (InterruptedException ex) {
                    Log.warn("Interrupted while attempting to get write lock.", ex);
                }
            }
            report(gotStuck, startTimeMillis, "write");
            writeLock = currentThread;
            writeLockCount++;
        }
        
        ...
        
        public synchronized void relinquishWriteLock() {
            //Log.warn("relinquishWriteLock() in thread " + Thread.currentThread());
            Thread currentThread = Thread.currentThread();
            if (writeLock != currentThread) {
                throw new RuntimeException("Cannot relinquish write lock on thread " + currentThread + " because it does not hold the lock.");
            }
            if (writeLockCount <= 0) {
                throw new RuntimeException("Tried to relinquish write lock on thread " + currentThread + " while write lock count is " + writeLockCount);
            }
            writeLockCount--;
            if (writeLockCount == 0) {
                writeLock = null;
                notifyAll();  // IMPORTANT: allow other threads to wake up and check if they can get locks now.
            }
        }
        
        ...
    }

    А у вас уже есть свой теплый ламповый ReentrantReadWriteLock?
    Нет? Тогда https://github.com/orph/jujitsu/blob/master/src/e/ptextarea/PLock.java идет к вам...

    kostoprav, 30 Мая 2014

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

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

• 2107
• …
• 1361
• 1360
• 1359
• 1358
• 1357
• 1356
• 1355
• 1354
• 1353
• …
• 1