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

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

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

1. PHP (5698)
2. C++ (2733)
3. Куча (2323)
4. JavaScript (2027)
5. C# (1924)
6. Java (1501)
7. Си (1095)
8. Pascal (647)
9. Python (582)
10. 1C (531)
11. SQL (431)
12. ActionScript (292)
13. Objective C (202)
14. bash (196)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (28)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

1. PHP / Говнокод #19637

   +2

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

   // Статическая типизация для PHP < 7.0
   
   set_error_handler(function(){});
   
   function foo(int $bar)
   {
   	var_dump($bar);
   }
   
   foo(1); // Работает и выводит int(1)

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

   KiTE, 16 Марта 2016

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

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

   +3

   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

   /**
        * Validate natural number (0,1,2,3,...)
        *
        * @param string $value number
        * @return string
        */
       public function natural($value)
       {
           $response = '';
           if (!preg_match('/^[0-9]+$/', $value)) {
               $response = $this->lang->get_str('system_invalid_neutral');
           }
           return $response;
       }
   
       /**
        * Validate natural number, but not a zero (1,2,3,...)
        *
        * @param string $value number
        * @return string
        */
       public function natural_no_zero($value)
       {
           $response = '';
           if (!preg_match('/^[0-9]+$/', $value)) {
               $response = $this->lang->get_str('system_invalid_neutral_no_zero');
           }
   
           if ($value == 0) {
               $response = $this->lang->get_str('system_invalid_neutral_no_zero');
           }
           return $response;
       }

   b2012993, 15 Марта 2016

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

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

   +4

   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

   program Project2;
   
   procedure test1;
   var
     arr:array[0..32] of char;
   begin
     fillchar(arr,sizeof(arr),'A');
   end;
   
   procedure test2;
   var
     arr:array[0..32] of char;
   begin
     fillchar(arr,sizeof(arr) div 2,'B');
     writeln(arr);
   end;
   
   begin
     test1;
     test2;
     //BBBBBBBBBBBBBBBBAAAAAAAAAAAAAAA
     readln;
   end.

   http://ideone.com/qJajnb

   fajes_rown, 15 Марта 2016

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

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

   +1

   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

   void g()
   {
       int sum;
       auto rec = [&sum](int i) -> int
       {
           static int (*inner)(int&, int) = [](int& _sum, int i)->int 
           {
               _sum += i;
               return i>0 ? inner(_sum, i-1)*i : 1; 
           };
           return inner(sum, i);
       };
   }

   LispGovno, 15 Марта 2016

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

5. C# / Говнокод #19631

   0

   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

   private AnimatedPanel GetPanelForPage(int page)
   		{
   			switch(page)
   			{
   				case 1:
   					return apnInitial;
   				case 2:
   					if(ShowExtraInfoPage)
   					{
   						DidShowExtraInfo = true;
   						return apnCustomerInfo;
   					}
   					DidShowExtraInfo = false;
   					break;
   				case 3:
   					return apnDeploy;
   				case 4:
   					return apnMetrics;
   				case 5:
   					return apnSecurity;
   				case 6:
   					//if(Static.IsFeatureLicenced((int)Features.PowerManagement, UserMgmt.CustomerGuid))
   					//{
   					//	return apnPowerCosts;
   					//}
   					break;
   				case 7:
   					//if(Static.IsFeatureLicenced((int)Features.PowerManagement, UserMgmt.CustomerGuid))
   					//{
   					//	return apnBaseline;
   					//}
   					break;
   				case 8:
   					//if(Static.IsFeatureLicenced((int)Features.PowerManagement, UserMgmt.CustomerGuid))
   					//{
   					//	return apnPowerPolicy;
   					//}
   					break;
   				case 9:
   					return apnDiscoverTask;
   				case 10:
   					return apnWaitForRep;
   				case 11:
   					return apnClientEndpoints;
   				case 12:
   					return apnWaitForDiscovery;
   				case 13:
   					return apnFinished;
   				default:
   					break;
   			}
   			return null;
   		}

   Перемещаемся по страницам: 1->3->4->5->9

   Zetway, 15 Марта 2016

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

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

   0

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

   QMutableListIterator<AISObjectQML *> it(m_visibleList);
   if (it.hasNext()) {
       AISObjectQML *v = it.next();
       if (v) { // здесь должен быть while, но, признаться, на это уже посрать
           delete v;
       }
   }
   m_visibleList.clear();

   Есть люди, которые придумали концепцию итераторов для реализации generic алгоритмов. А есть люди, которые любят итераторы, но не любят generic алгоритмы. Ведь с ними код не внушает ужаса и рвотных позывов

   Antervis, 15 Марта 2016

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

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

   +3

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

   JSON объект:
   { 
      "Tickets":[100999,100998,100997],
      "Price":[5,5,5],
      "TicketCount":3
   }
   
   Вместо положенных:
   {"Tickets":[ ["100999","5"],["100998","5"],["100997","5"]]}

   Разработчик высылает JSON объект где описывает массив, вместо того чтобы просто выслать массив)))))

   mypaxa, 15 Марта 2016

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

8. C# / Говнокод #19628

   +3

   1. 1
   2. 2

   var listModuleId = dbContext.MODULES.Where(n => n.IFDELETE == 0 && n.DEVICETYPE == 1).Select(m => m.IDMODULE).ToList();
   var ListMeasure = dbContext.MEASURE.Where(n => listModuleId.Contains(n.IDMODULES)).GroupBy(m => m.IDMODULES).ToList();

   Если в MODULES несколько сотен датчиков, то вторая строка разворачивается в sql-запрос на 22 килобайта, а если в MEASURE 13 тысяч записей, то этот запрос выполняется полторы минуты.
   Молчу уже о том, что listModuleId нигде, кроме второй строки, не используется.

   avialaynen, 15 Марта 2016

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

9. C# / Говнокод #19627

   0

   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

   static void Main(string[] args)
   {
   	try
   	{
   		watcher.Path = TargetPath;
   		watcher.Created += Watcher_Created;
   		watcher.Renamed += Watcher_Created;
   		watcher.EnableRaisingEvents = true;
   		logger.Info("Сервис запущен.");
   		System.Threading.Thread.Sleep(System.Threading.Timeout.Infinite);
   	}
   	catch(Exception ex)
   	{
   		logger.Error(ex, "Global error", null);
   		Main(null);
   	}
   }

   хуита, 15 Марта 2016

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

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

    −1

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

    private static async Task<DataTable> GetWaitingBetsHandlingHistory(int customerID, int betID, DateTime date)
           {
               // copied from: CBets.GetWaitingBetsHandlingHistory
               string connectionString = AppConfig.MachineAppSettings("connectionString");
               SqlConnection conn = new SqlConnection(connectionString);
               ...

    Copy-paste-driven development confession

    pro687, 14 Марта 2016

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

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

• 2095
• …
• 1643
• 1642
• 1641
• 1640
• 1639
• 1638
• 1637
• 1636
• 1635
• …
• 1