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

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

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

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

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

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

   <?php
   	require_once("inc/mysql.php");
   	sleep(3); //Для ajax запроса, потом удалить
   
   	// Проверяем, что к нам идёт Ajax запрос
   	if (isset($_SERVER['HTTP_X_REQUESTED_WITH']) && !empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest') {
   		$city1 = $_POST['city1'];
   	}else{
   		exit(); // Заканчиваем работу скрипта, если это не ajax запрос
   	}
   
   	if (isset($city1)) {
   		$query = "SELECT id_city, id_region, id_country FROM cities WHERE city_name_ru LIKE '$city1'";
   		$result = mysqli_query($link, $query);// or trigger_error(mysql_error($link)." ".$query);
   		$row = mysqli_fetch_array($result);
   		$country = $row['id_country'];
   		//echo $country;
   		//echo '<br>';
   		$region = $row['id_region'];
   		//echo $region;
   		//echo '<br>';
   		$query1 = "SELECT country_name_ru FROM countries WHERE id_country = '$country'";
   		$result1 = mysqli_query($link, $query1);// or trigger_error(mysql_error($link)." ".$query);
   		$row1 = mysqli_fetch_array($result1);
   		echo '<p class=\'country\'>Страна '.$row1['0'].'</p>';
   		//echo '<br>';
   		$query2 = "SELECT region_name_ru FROM regions WHERE id_region = '$region'";
   		$result2 = mysqli_query($link, $query2);// or trigger_error(mysql_error($link)." ".$query);
   		$row2 = mysqli_fetch_array($result2);
   		echo '<p class=\'reqion\'>Область '.$row2['0'].'</p>';
   	}
   ?>

   Прокомментируйте, пожалуйста, как улучшить этот говнокод.

   eprivalov1, 25 Апреля 2014

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

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

   +130

   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

   using System;
   using System.Collections.Generic;
   using System.Linq;
   using System.Text;
   using System.Text.RegularExpressions;
   using System.IO;
   
   namespace SimpleLex
   {
       class Lexical
       {
           //Сюда передается путь к файлу конфигурации
           public string Path = "";
           //Определяем ключевые слова 
           string regular = "Color|Size|Name";
           public void lexical()
           {
               //Создаем массив строк для дальнейшего заполнения
               string[] conf = new string[1024];
               //Переменная счетчик
               int i = 0;
               //Заполняем массив из файла 
               //Path мы присваиваем значение при создании
               //объекта нашего класса.
               conf = File.ReadAllLines(Path);
               //Запускаем цикл чтения 
               while (conf.Length - 1 >= i)
               {
               // Создаем новый объект класса Regex
               // и передаем ему в качестве конструктора
               // cписок ключевых слов
               Regex RegularExp = new Regex(regular);
               //Начинаем поиск совпадений в текущей строке
               Match match = RegularExp.Match(conf[i]);
               while (match.Success)
               {
                   //Ожидаем наличия совпадений
                   break;
                   //Если нашли то выпрыгиваем 
               }
   
               switch (match.Value) // Смотрим что же мы обнаружили и вызываем соответствующий метод
               {
                   case "Color":
                       //вызываем метод Color_
                       //и передаем ему текущую строку для разбора
                       //Предварительно удалив все пробелы с двух сторон
                       //если они были метод Trim()
                       Color_(conf[i].Trim());
                       i++;
                       break;
                   case "Size":
                       size_(conf[i].Trim());
                       i++;
                       break;
                   default:
                       //все другое пропускаем
                       i++;
                       break;
               }
               }
           }
           void Color_(string str)
           {
               int i = 0;
               while (str.Substring(i,1)!= "=")
               {
                   //Ищем разделитель в данном случае "="
                   i++;
               }
               //Находим и извлекаем нужную нам информацию
               string value_ = str.Substring(i + 1, str.Length - (i+1)).Trim();
               //Теперь все зависит от вашей фантазии хотите 
               //Создайте класс который будет устанавливать цвет шрифта
               //И.т.д
   
               Console.WriteLine(value_);
               Console.ReadKey();
           }
           void size_(string str)
           {
               /*
                * В этом методе я хочу показать как быть если вы используете
                * в своем файле для каждой команды разные разделители
                * в теории конечно можно все это засунуть в один метод
                * Но я советую в дальнейшем если вы будите применять
                * именно такую структуру передавать в метод тип разделителя
                * и соответственно искать именно его другое дело если вы имеете
                * различный формат входных данных как у нас параметр size имеет
                * вид shize X=100,Y=500
                */
               //Есть прекрасный метод 
               int i = 0;
               while (str.Substring(i,1) != " ")
               {
                   i++;
               }
               str = str.Substring(i, str.Length - i).Trim();
               string[] commandMas = new string[3];
               commandMas = str.Split(',');

   (GetPrivateProfileString? Не, не слышал)
   Лютый песдеzz с сайта vbbook.ru

   brutushafens, 24 Апреля 2014

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

3. Си / Говнокод #15843

   +127

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

   #define PIXEL_PARTS_NUM 3
   
   typedef struct _pixel {
           char parts[PIXEL_PARTS_NUM];
   } pixel;

   evg_ever, 24 Апреля 2014

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

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

   +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

   import elements
   import functions
   import system
   error = 'Ошибок нет'
   add = input ('Введите элемент, который вы хотите добавить в таблицу')
   table = element.table
   addToTable : add
   if addTo.table : Successful {
    put 'Ошибок нет' }
   else 
   put error
   system.exit

   На случай, если друг подумает, что ничего не работает..
   Писалось на языке без названия, не придумал ещё :)

   Mobac, 24 Апреля 2014

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

5. PHP / Говнокод #15841

   +154

   1. 1
   2. 2
   3. 3

   foreach(array_keys(get_defined_vars())as$__v)$__refs[$__v]=&$$__v;
   do_action("ws_plugin__s2member_before_paypal_api_response", get_defined_vars());
   unset /* Unset defined __refs, __v. */($__refs, $__v);

   И опять s2member для wordpress

   antongorodezkiy, 24 Апреля 2014

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

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

   +143

   1. 1

   CFGDBFACADOEBFAAAACOABAOBBGAAODBFADCCCFDOGDACCOGAAAAAACOBDBGAAOCBGDEBDFDADGCBEDAABCOBEECBDAOECFCCFCCEOBDOAABDFOAOBDBOBGOBDBBEECBBAOOBADBADAEGAOCBEFAABCFDBGOAFGBGEBBDBBGBBGBEACOOACAAABCBDADGACAAABDOOAAFCBBDBODCBDFDCBEDAAACEAGACFDCADDFOBCCBOBDGEDEFBAAAABDAACDBDGODGBCBBDCOAEGAOBAEFBOEDGCCGEDBOCFDBEBBEFAOCACGBFEAAEBGFDOBCAADACAAECGEAABDBFACGBGECEGCDADBFACGABOCGGDFEABOCOGCFGCOAAAAAOEAAABGECCODAD

   "Змейка" в машинных кодах на системе счисления wct. WctEditor - http://yadi.sk/d/i9Me1IQ7MbHCD.

   Mobac, 24 Апреля 2014

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

7. C# / Говнокод #15839

   +138

   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

   private static Dictionary<RoleEnum, string> Roles = new Dictionary<RoleEnum, string>
           {
               {RoleEnum.TeamMember, "Team Member"},
               {RoleEnum.ProjectManager, "Project Manager"},
               {RoleEnum.ProgramManager, "Program Manager"},
               {RoleEnum.PortfolioManager, "Portfolio Manager"},
               {RoleEnum.Executive, "Executive"},
               {RoleEnum.Undefined, "Undefined"}
           };
   
           public static RoleEnum ParseRole(string role)
           {
               RoleEnum result = RoleEnum.Undefined;
               Roles
                   .Where(_ => _.Value == role)
                   .ToList()
                   .ForEach(_ => result = _.Key);
   
               return result;
           }

   Странное использование дикшинари, очень странное, в обратную сторону можна сказать

   boades, 24 Апреля 2014

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

8. Куча / Говнокод #15838

   +126

   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

   program FeyverkFirework;
   
   {$R+}
   
   uses
     Crt, Graph;
   
   var
     GraphDriver, GraphMode, ErrorCode : Integer;
     Ch : Char;
     I, J, K, XStart, YStart : Integer;
     XX,YY,CC : Array[1..1000] of Integer;
     FX,FY,DX,DY : Array[1..70] of Real;
     R, T, X, Y : Real;
   
   procedure Explode (C : Integer);
   
   begin
     T := -Pi;
     I := 0;
     Repeat
       I := I + 1;
       DX[I] := Sin(T)*5;
       DY[I] := Cos(T)*5;
       FX[I] := X;
       FY[I] := Y;
       T := T + 0.15;
     Until T > Pi;
   
     K := 0;
     Repeat
       For J := 1 to I do
         begin
           If C = 0 then
             PutPixel (Round(FX[J]),Round(FY[J]),0)
           else
             PutPixel (Round(FX[J]),Round(FY[J]),Random(GetMaxColor+1));
           FX[J] := FX[J] + DX[J];
           FY[J] := FY[J] - DY[J];
           DY[J] := DY[J] - 0.2;
         end;
       K := K + 1;
       Delay(10); { This was not in the original. }
     Until Keypressed or (K > 50);
   end;
   
   procedure ShootFireWork;
   
   begin
     { Delay (1000); }
     Randomize;
     T := Random / 2 - 0.25 + Pi/2;
     X := XStart;
     Y := YStart;
     R := 20;
     I := 0;
     Repeat
       Inc (I);
       XX[I] := Round(X);
       YY[I] := Round(Y);
       CC[I] := GetPixel (XX[I],YY[I]);
       If I > 1 then
         If (XX[I] = XX[I-1]) and (YY[I] = YY[I-1]) then CC[I] := CC[I-1];
       PutPixel (XX[I],YY[I],Random(GetMaxColor+1));
       If I > 5 then
         PutPixel (XX[I-5],YY[I-5],CC[I-5]);
       X := X + Cos(T)*R;
       Y := Y - Sin(T)*R;
       If T > Pi/2 then
         T := T + 0.02
       else
         T := T - 0.02;
       R := R * 0.93;
       Delay (20);
     Until KeyPressed or (T < 0) or (T > Pi);
   
     For J := I-5 to I do
   
        PutPixel (XX[J],YY[J],CC[J]);
   {    ch := readkey; }
     Explode (1);
     Explode (0);
   end;
   
   begin
     GraphDriver := Detect;
     InitGraph (GraphDriver, GraphMode, '..');
     ErrorCode := GraphResult;
     If ErrorCode <> grOk then
       begin
         Writeln ('Graphics Error: ',GraphErrorMsg(ErrorCode));
         Halt;
       end;
     SetColor (GetMaxColor);
     XStart := GetMaxX div 2;
     YStart := GetMaxY - 20;
   
     Repeat
       ShootFireWork;
     Until Keypressed;

   Ch := ReadKey;
   ClearDevice;
   CloseGraph;
   end.
   
   Работает же! Красиво получается :)
   Выполнить код можно здесь: http://pma.clan.su/index.html.
   Алгоритм выполнения кода. Жмём E. Вводим код. Нажимаем в левом нижнем углу CoPas editor. Нажимаем R. Радуемся! :)

   Mobac, 24 Апреля 2014

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

9. Pascal / Говнокод #15837

   +83

   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

   program FeyverkFirework;
   
   {$R+}
   
   uses
     Crt, Graph;
   
   var
     GraphDriver, GraphMode, ErrorCode : Integer;
     Ch : Char;
     I, J, K, XStart, YStart : Integer;
     XX,YY,CC : Array[1..1000] of Integer;
     FX,FY,DX,DY : Array[1..70] of Real;
     R, T, X, Y : Real;
   
   procedure Explode (C : Integer);
   
   begin
     T := -Pi;
     I := 0;
     Repeat
       I := I + 1;
       DX[I] := Sin(T)*5;
       DY[I] := Cos(T)*5;
       FX[I] := X;
       FY[I] := Y;
       T := T + 0.15;
     Until T > Pi;
   
     K := 0;
     Repeat
       For J := 1 to I do
         begin
           If C = 0 then
             PutPixel (Round(FX[J]),Round(FY[J]),0)
           else
             PutPixel (Round(FX[J]),Round(FY[J]),Random(GetMaxColor+1));
           FX[J] := FX[J] + DX[J];
           FY[J] := FY[J] - DY[J];
           DY[J] := DY[J] - 0.2;
         end;
       K := K + 1;
       Delay(10); { This was not in the original. }
     Until Keypressed or (K > 50);
   end;
   
   procedure ShootFireWork;
   
   begin
     { Delay (1000); }
     Randomize;
     T := Random / 2 - 0.25 + Pi/2;
     X := XStart;
     Y := YStart;
     R := 20;
     I := 0;
     Repeat
       Inc (I);
       XX[I] := Round(X);
       YY[I] := Round(Y);
       CC[I] := GetPixel (XX[I],YY[I]);
       If I > 1 then
         If (XX[I] = XX[I-1]) and (YY[I] = YY[I-1]) then CC[I] := CC[I-1];
       PutPixel (XX[I],YY[I],Random(GetMaxColor+1));
       If I > 5 then
         PutPixel (XX[I-5],YY[I-5],CC[I-5]);
       X := X + Cos(T)*R;
       Y := Y - Sin(T)*R;
       If T > Pi/2 then
         T := T + 0.02
       else
         T := T - 0.02;
       R := R * 0.93;
       Delay (20);
     Until KeyPressed or (T < 0) or (T > Pi);
   
     For J := I-5 to I do
   
        PutPixel (XX[J],YY[J],CC[J]);
   {    ch := readkey; }
     Explode (1);
     Explode (0);
   end;
   
   begin
     GraphDriver := Detect;
     InitGraph (GraphDriver, GraphMode, '..');
     ErrorCode := GraphResult;
     If ErrorCode <> grOk then
       begin
         Writeln ('Graphics Error: ',GraphErrorMsg(ErrorCode));
         Halt;
       end;
     SetColor (GetMaxColor);
     XStart := GetMaxX div 2;
     YStart := GetMaxY - 20;
   
     Repeat
       ShootFireWork;
     Until Keypressed;

   Работает же! Красиво получается :)
   Выполнить код можно здесь: http://pma.clan.su/index.html.
   Алгоритм выполнения кода. Жмём E. Вводим код. Нажимаем в левом нижнем углу CoPas editor. Нажимаем R. Радуемся! :)

   Mobac, 24 Апреля 2014

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

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

    +137

    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

    Index and length must refer to a location within the string.
    Parameter name: length
    Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.
    
    Exception Details: System.ArgumentOutOfRangeException: Index and length must refer to a location within the string.
    Parameter name: length
    
    Source Error:
    
    
    Line 1239:    // СЕГОДНЯ И ВЧЕРА
    Line 1240:    string agent = Request.UserAgent;
    Line 1241:    string sid = DateTime.Now.ToShortDateString().Substring(0, 10).Replace(".", "").Replace("/", "").Replace("-", "");
    Line 1242:    string sid2 = DateTime.Now.AddDays(-1).ToShortDateString().Substring(0, 10).Replace(".", "").Replace("/", "").Replace("-", "");
    Line 1243:    //

    http://tltgorod.ru/news/theme-5/news-21502
    
    Хуяк-хуяк и в продакшн.

    bakagaijin, 24 Апреля 2014

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

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

• 2107
• …
• 1341
• 1340
• 1339
• 1338
• 1337
• 1336
• 1335
• 1334
• 1333
• …
• 1