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

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

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

1. PHP (5710)
2. C++ (2742)
3. Куча (2391)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1120)
8. Pascal (648)
9. Python (591)
10. 1C (540)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (43)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

1. C++ / Говнокод #27009

   0

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

   #include <iostream>
   using namespace std;
    
   struct Foo {char a; int b; char c;};
   struct Bar {char a; char b; int c;};
    
    
   int main() {
   	cout << sizeof(Foo) << endl;
   	cout << sizeof(Bar) << endl;
   }

   https://ideone.com/XKWey3
   
   Какой бароп )))

   3_dar, 08 Октября 2020

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

2. C++ / Говнокод #27001

   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

   Node* reverse(Node* head) 
   {
   	Node *end = head, *current = head;
   	while (end->next != nullptr) {
   		end = end->next;
   	}
   	
   	Node *initial_end = end, *temp = nullptr, *temp_2 = nullptr;
   	
   	end->next = current;
   	temp = current;
   	current = current->next;
   	temp->next = nullptr;
   	
   	while (current != initial_end) {
   		
   		temp_2 = initial_end->next;
   		initial_end->next = current;
   		temp = current;
   		current = current->next;
   		temp->next = temp_2;
   	}
   	
   	return initial_end;
   }

   Я где-то прочитал, что на собесе нужно написать переворот односвязного списка за 5 минут... Спустя 2 дня получилось это.

   oaoaoammm, 04 Октября 2020

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

3. C++ / Говнокод #26988

   0

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

   class std::unordered_map<class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> >,
   enum REG,struct std::hash<class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > >,
   struct std::equal_to<class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > >,
   class std::allocator<struct std::pair<class std::basic_string<char,struct std::char_traits<char>,
   class std::allocator<char> > const ,enum REG> > > registers" 
   (?registers@@3V?$unordered_map@V?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@W4REG@@U?$hash@V?$
   basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@@2@U?$equal_to@V?$basic_string@DU?$char_traits@D@std@@V?
   $allocator@D@2@@std@@@2@V?$allocator@U?$pair@$$CBV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@W4
   REG@@@std@@@2@@std@@A) уже определен в decoder.obj	PVC-16	C:\Users\Люда\source\repos\PVC-16\PVC-16\interrupt.obj	1

   Похлопаем visual c++ за понятный лог.

   digitalEugene, 30 Сентября 2020

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

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

   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
   54. 54
   55. 55
   56. 56
   57. 57
   58. 58
   59. 59
   60. 60
   61. 61
   62. 62
   63. 63
   64. 64
   65. 65

   #include <cstdlib>
   #include <chrono>
   #include <iostream>
   #include <thread>
   
   int p = 0;
   int *q = nullptr;
   
   void g()
   {
       using namespace std::chrono_literals;
   
       std::cout << "g()" << std::endl;
   
       std::cout << "g(): p = 1" << std::endl;
       p = 1;
   
       std::this_thread::sleep_for(1s);
   
       
       if (q != nullptr) {
           std::cout << "g(): *q = 1" << std::endl;
           *q = 1;
       } else {
           std::cout << "g(): q == nullptr" << std::endl;
       }
   }
   
   void f()
   {
       using namespace std::chrono_literals;
   
       std::cout << "f()" << std::endl;
   
       if (p == 0) {
           std::cout << "f(): first loop start" << std::endl;
           while (p == 0) { }  // Потенциально конечный
           std::cout << "f(): first loop end" << std::endl;
       }
   
       int i = 0;
       q = &i;
       std::cout << "f(): second loop start" << std::endl;
       while (i == 0) { }  // Потенциально конечный, хотя в условии только автоматическая пельменная
       std::cout << "f(): second loop end" << std::endl;
   }
   
   int main()
   {
       using namespace std::chrono_literals;
   
       std::cout << "f() thread start" << std::endl;
       auto thr1 = std::thread(f);
       thr1.detach();
       std::this_thread::sleep_for(1s);
   
       std::cout << "g() thread start" << std::endl;
       auto thr2 = std::thread(g);
       thr2.detach();
       std::this_thread::sleep_for(2s);
   
       std::cout << "Done" << std::endl;
       
       std::_Exit(EXIT_SUCCESS);
   }

   Ожидание:
   

   f() thread start
   
   f()
   
   f(): first loop start
   
   g() thread start
   
   g()
   
   g(): p = 1
   
   f(): first loop end
   
   f(): second loop start
   
   g(): *q = 1
   
   f(): second loop end
   
   Done

   gost, 28 Сентября 2020

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

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

   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
   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
   88. 88
   89. 89
   90. 90
   91. 91
   92. 92
   93. 93
   94. 94
   95. 95
   96. 96
   97. 97
   98. 98

   #include <iostream>
   #include <ctime>
   #include <cstdlib>
   #include <conio.h>
   const int x_size(20), y_size(10); int x_pos(x_size/2+1); int y_pos(y_size/2+1);
   enum border_types{lineNL, single, singleNL};
   enum directions{UpLeft=1, UpRight, DownLeft, DownRight}dir;
   void draw_border(enum border_types borders) {
   	do{
   		if(borders == single || borders == singleNL) break;
   		for(int i=0; i<x_size+1; i++)
   	  	  putchar('#');
   	}while(false);
   	putchar('#');
   	if(borders == singleNL || borders == lineNL) std::cout << '\n';}
   void display_update() {
   	system("cls");
   	draw_border(lineNL);
   	for(int i=1; i<=y_size; i++)
   	{
   		draw_border(single);
   		for(int j=1; j<=x_size; j++)
   		{
   			if(j == x_pos && i == y_pos)
   			{
   				putchar('x');
   				continue;
   			}
   			putchar(32);
   		}
   		draw_border(singleNL);;
   	}
   	draw_border(lineNL);
   	std::cout << "X: " << x_pos << "\tY: " << y_pos;}
   void logic() {
   	switch(x_pos)
   	{
   		case 1:
   			if(dir == UpLeft) dir = UpRight;
   			if(dir == DownLeft) dir = DownRight;
   			break;
   		case x_size:
   			if(dir == UpRight) dir = UpLeft;
   			if(dir == DownRight) dir = DownLeft;
   	}
   	switch(y_pos)
   	{
   		case 1:
   			if(dir == UpLeft) dir = DownLeft;
   			if(dir == UpRight) dir = DownRight;
   			break;
   		case y_size:
   			if(dir == DownLeft) dir = UpLeft;
   			if(dir == DownRight) dir = UpRight;
   	}}
   void move() {
   	switch(dir)
   	{
   		case UpLeft:
   			x_pos--;
   			y_pos--;
   			break;
   		case UpRight:
   			x_pos++;
   			y_pos--;
   			break;
   		case DownLeft:
   			x_pos--;
   			y_pos++;
   			break;
   		case DownRight:
   			x_pos++;
   			y_pos++;
   	}}
   int main() {
   	srand(time(0));
   	rand();
   	switch(rand()%4+1)
   	{
   		case UpLeft:
   			dir = UpLeft;
   			break;
   		case UpRight:
   			dir = UpRight;
   			break;
   		case DownLeft:
   			dir = DownLeft;
   			break;
   		case DownRight:
   			dir = DownRight;
   	}
   	while(!kbhit())
   	{
   		display_update();
   		logic();
   		move();
   	}
   	return 0;}

   Сорян, пришлось уплотнить фигурные скобки, чтобы код уместился в 100 строк.

   BelCodeMonkey, 18 Сентября 2020

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

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

   0

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

   class UnitedFigure : public Figure {
       Figure &f1;
       Figure &f2;
   
   public:
       UnitedFigure (Figure &_f1, Figure &_f2) : f1(_f1), f2(_f2) {}
   
       double distance_to(const Point &p) const override {
           return std::min(f1.distance_to(p), f2.distance_to(p));
       }
   }

   Завезли ссылочные поля класса, это в каком стандарте?
   Даже тестил когда то такое, наверное на C++03 и получал ошибку компилятора.
   Я и не знал, что добавили такую прекрасную возможность выстрелить себе в ногу.

   YpaHeLI_, 16 Сентября 2020

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

7. C++ / Говнокод #26953

   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

   uint64_t stored_msg_id = _container_msg_id.get(ctrl_msg.sequence); // Получаем msg_id из мапы для связки сообщений
   if (stored_msg_id)
       proto_fill(ctrl_msg, stored_msg_id); // если он там был то новому сообщению даем этот msg_id
   else
       proto_fill(ctrl_msg);  // Иначе формируем новый msg_id
   
   // Отсылаем сообщение 
   ...
   
   // Если msg_id не был в _container_msg_id то произойдет попытка вставки msg_id полученного через proto_fill.
   if (!stored_msg_id && !_container_msg_id.insert(ctrl_msg.sequence, ctrl_msg.msg_id))
   {
       DEBUG(L, 0, "[%p] Can't store request's control message id (%llu) in bunch map" \
                   ", response's msg_id will differ", this, msg.msg_id);
   }

   С точки зрения читабельности кода, в последнем ветвлении говнокод?

   OlegUP, 15 Сентября 2020

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

8. C++ / Говнокод #26947

   +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
   14. 14
   15. 15

   catch (Exception &exception)
   	{
   		Application->ShowException(&exception);
   	}
   	catch (...)
   	{
   		try
   		{
   			throw Exception("");
   		}
   		catch (Exception &exception)
   		{
   			Application->ShowException(&exception);
   		}
   	}

   https://github.com/greatis/Anti-WebMiner/blob/master/AntiWebMiner.cpp#L23

   MAKAKA, 13 Сентября 2020

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

9. C++ / Говнокод #26946

   +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
   34. 34
   35. 35

   foreach ($files as $n => $f) {
       $new_content = $common.$namespace_begin.$f.$namespace_end;
   
       $std_methods = array();
       preg_match_all('/std::[a-z_0-9]*/', $new_content, $std_methods);
       $std_methods = array_unique($std_methods[0]);
   
       $needed_std_headers = array();
       $type_headers = array(
           'std::move' => '',
           'std::vector' => '',
           'std::string' => '',
   // [...]
           'std::unordered_set' => 'unordered_set');
       foreach ($type_headers as $type => $header) {
           if (in_array($type, $std_methods)) {
               $std_methods = array_diff($std_methods, array($type));
               if ($header && !in_array($header, $needed_std_headers)) {
                   $needed_std_headers[] = $header;
               }
           }
       }
   
       if (!$std_methods) { // know all needed std headers
           $new_content = preg_replace_callback(
               '/#include <([a-z_]*)>/',
               function ($matches) use ($needed_std_headers) {
                   if (in_array($matches[1], $needed_std_headers)) {
                       return $matches[0];
                   }
                   return '';
               },
               $new_content
           );
       }

   Тут вот в https://govnokod.ru/20049 CHayT предлагал использовать «PHP» в качестве препроцессора для «C» — так вот в «Телеграме» совет оценили и решили взять на вооружение.
   https://github.com/tdlib/td/blob/master/SplitSource.php

   gost, 13 Сентября 2020

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

10. C++ / Говнокод #26943

    +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
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19

    #include <vector>
    #include <cwchar>
    #include <algorithm>
    #include <iostream>
     
    int main()
    {
        std::vector<const wchar_t*> leaders{L"Ленин", L"Сталин", L"Маленков",
            L"Хрущёв", L"Брежнев", L"Андропов", L"Черненко", L"Горбачёв"};
     
        std::sort(leaders.begin(), leaders.end(), [](auto strA, auto strB) {
            return std::wcscmp(strA, strB) < 0;
        });
     
        std::setlocale(LC_ALL, "en_US.utf8");
        std::wcout.imbue(std::locale("en_US.utf8"));
        for (auto leader : leaders)
            std::wcout << leader << '\n';
    }

    Отсюда:
    https://en.cppreference.com/w/cpp/string/wide/wcscmp

    Naf-Naf, 11 Сентября 2020

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

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

• 275
• …
• 258
• 257
• 256
• 255
• 254
• 253
• 252
• 251
• 250
• …
• 1