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

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

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

1. PHP (5708)
2. C++ (2739)
3. Куча (2380)
4. JavaScript (2032)
5. C# (1929)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (591)
10. 1C (539)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (146)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (34)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   −118

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

   IF @stage = 9
   BEGIN
   	UPDATE dbo.Company SET [Site] = @site_uri, [EMail] = @email 
   		FROM dbo.DirtyApartments dirty
   		WHERE [Name] = dirty.Компания_Clean
   END

   Довелось столкнуться с проектом, в котором в СУБД mssql в половине таблиц поля написаны по-русски, а так-же, по русско-английски.

   nolka4, 19 Мая 2012

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

2. Assembler / Говнокод #10322

   +265

   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

   00410803  PUSH OFFSET 004FC0C8                     ASCII "AnVir Task Manager Pro"
   0041189D  PUSH OFFSET 004FC0A4                     ASCII "data Task Manager Pro"
   0041A074  PUSH OFFSET 00500120                     ASCII "%sAnVir Task Manager Pro"
   0042F9CE  PUSH OFFSET 005012A4                     ASCII "AnVir Task Manager Pro %s %s!"
   00410803  PUSH OFFSET 006AC6E8                     ASCII "AnVir Task Manager Pro"
   004479E9  PUSH EDI                                 ASCII "AnVir Task Manager Pro"
   0044FE01  PUSH OFFSET 00502C34                     ASCII "AnVir Task Manager Pro "
   00463DC9  PUSH OFFSET 00503708                     ASCII "AnVir Task Manager Pro folder selection"
   004606BE  MOV DWORD PTR SS:[EBP-130],OFFSET 005035>ASCII "AnVir Task Manager Pro file selection"
   00490CE9  PUSH OFFSET 00507E5C                     ASCII "AnVir Task Manager Pro. "
   00495441  PUSH OFFSET 00508450                     ASCII "AnVir Task Manager Pro, 6.7.1"
   00497858  PUSH OFFSET 0050866C                     ASCII "Logfile of AnVir Task Manager Pro v6.7.1 "
   004A3DE6  PUSH OFFSET 00509B54                     ASCII "AnVir Task Manager"

   Вывод ольки по поиску всех строк. (лишнее порезал)
   Надо напомнить сотню раз пользователю имя программы, а то вдруг он забудет что он запустил!
   Немного удивляет такой маразм.

   ReckO, 19 Мая 2012

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

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

   +92

   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

   Console.Write("Введите координату Xm: ");
               double Xm = Convert.ToDouble(Console.ReadLine());
               Console.Write("Введите координату Ym: ");
               double Ym = Convert.ToDouble(Console.ReadLine());
   
               Console.Write("Введите координату Xa: ");
               double Xa = Convert.ToDouble(Console.ReadLine());
               Console.Write("Введите координату Ya: ");
               double Ya = Convert.ToDouble(Console.ReadLine());
   
               Console.Write("Введите координату Xb: ");
               double Xb = Convert.ToDouble(Console.ReadLine());
               Console.Write("Введите координату Yb: ");
               double Yb = Convert.ToDouble(Console.ReadLine());
   
               Console.Write("Введите координату Xc: ");
               double Xc = Convert.ToDouble(Console.ReadLine());
               Console.Write("Введите координату Yc: ");
               double Yc = Convert.ToDouble(Console.ReadLine());
   
               if (
                   ((((Xb - Xa) * Ym + (Xa * Yb - Xb * Ya) - (Yb - Ya) * Xm <= 0) && ((Xb - Xa) * Yc + (Xa * Yb - Xb * Ya) - (Yb - Ya) * Xc <= 0))
                   || (((Xb - Xa) * Ym + (Xa * Yb - Xb * Ya) - (Yb - Ya) * Xm >= 0) && ((Xb - Xa) * Yc + (Xa * Yb - Xb * Ya) - (Yb - Ya) * Xc >= 0)))
   
                   && ((((Xc - Xa) * Ym + (Xa * Yc - Xc * Ya) - (Yc - Ya) * Xm <= 0) && ((Xc - Xa) * Yb + (Xa * Yc - Xc * Ya) - (Yc - Ya) * Xb <= 0))
                   || (((Xc - Xa) * Ym + (Xa * Yc - Xc * Ya) - (Yc - Ya) * Xm >= 0) && ((Xc - Xa) * Yb + (Xa * Yc - Xc * Ya) - (Yc - Ya) * Xb >= 0)))
                   
                   && ((((Xc - Xb) * Ym + (Xb * Yc - Xc * Yb) - (Yc - Yb) * Xm <= 0) && ((Xc - Xb) * Ya + (Xb * Yc - Xc * Yb) - (Yc - Yb) * Xa <= 0))
                   || (((Xc - Xb) * Ym + (Xb * Yc - Xc * Yb) - (Yc - Yb) * Xm >= 0) && ((Xc - Xb) * Ya + (Xb * Yc - Xc * Yb) - (Yc - Yb) * Xa >= 0)))
                   ) Console.WriteLine("Точка ({0}, {1}) входит в треугольник", Xm, Ym);
               else Console.WriteLine("Точка ({0}, {1}) не входит в треугольник", Xm, Ym);
               Console.ReadKey();

   Проверка входит ли точка в треугольник.

   maksim_ovcharik, 19 Мая 2012

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

4. Objective C / Говнокод #10320

   −103

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

   - (void)updateTableHeaderOfOrangeColor:(BOOL)reallyOrange andTitle:(NSString *)title
   {
       table.tableHeaderView = reallyOrange ? orangeTableHeaderView : marineTableHeaderView;
       (reallyOrange ? orangeTableHeaderLabel : marineTableHeaderLabel).text = title;
   
       UIImage *tileImage = [UIImage imageNamed:(reallyOrange ? @"tapeWideTile.png" : @"tapeWideTileBlue.png")];
       table.backgroundColor = [UIColor colorWithPatternImage:tileImage];
   }

   причем оранжевый на самом деле красный в интерфейсе

   Venomous, 19 Мая 2012

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

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

   +98

   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

   public class XmlParserClass
       {
           private XmlDocument xmlDoc;
           public static List<Attribute> listOfAttributes;
           public static Dictionary<int, AttributeValue> listOfAttributeValues;
           public static Dictionary<int, File> listOfFiles;        
           public static List<Product> listOfProducts;
           public List<FileCategory> listOfFileCategories;
           public List<Distributor> listOfDistributors;
           public static bool cryptedFiles = false;
   
           private string xmlfileLocation; 
           private string XmlfileLocation  
           {
               get 
               { 
                   if (xmlfileLocation == null)
                       xmlfileLocation = System.IO.Path.GetDirectoryName(Program.GetValue("XMLFileLocation", ""));
                   return xmlfileLocation;
               }
           }
   	       
   	public XmlParserClass(XmlDocument xmlDocument)
           {
               xmlDoc = xmlDocument;
               listOfAttributes = new List<Attribute>();
               listOfAttributeValues = new Dictionary<int, AttributeValue>();
               listOfProducts = new List<Product>();
               listOfFileCategories = new List<FileCategory>();
               listOfFiles = new Dictionary<int, File>();
               listOfDistributors = new List<Distributor>();
   
   			.......
   			
                  //Читаем а с XML ....
                   foreach (XmlNode xmlValue in attr.SelectSingleNode("Values").ChildNodes)
                   {
                      ................
                   }
   
                   listOfAttributes.Add(a);
   
   			
   			.........
   			//Читаем fc с XML....
   			listOfFileCategories.Add(fc);
   			
   			.........
   		}
   	}

   Создали экземпляр чтобы прочитать данные с XML. Куда бы нам их положить? Конечно в статические поля!

   dreamer, 18 Мая 2012

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

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

   −23

   1. 1
   2. 2

   std::bad_alloc e;
   throw std::move(e);

   Написал один новичок. Говорит это особая сильно ускоряющая оптимизация.

   Говногость, 18 Мая 2012

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

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

   −23

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

   bool strictlyLess(std::vector<int> v1, std::vector<int>v2) { // v1 < v2
   	bool strictly = false;
   	for(int i = 0; i < v1.size(); i++) {
   		if(v1[i] > v2[i]) {
   			return false;
   		}
   		if(v1[i] < v2[i]) {
   			strictly = true;
   		}
   	}
   	return strictly;
   }

   movaxbx, 18 Мая 2012

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

8. Pascal / Говнокод #10316

   +100

   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

   function Tf_MainFrame.DateToText(d: TDate; tp: integer): string;
   var s: string;
       g: integer;
   begin
     if DateToStr(d) = '00.00.0000' then
     begin
       result := '';
     end else begin
     case tp of
       2:begin
           s := ''; DateTimeToString(s,'dd',d);
           result := s;
           DateTimeToString(s,'mm',d);
           g := StrToInt(s);
           DateTimeToString(s,'yyyy',d);
           case g of
               1:  begin result := result + ' января ' + s; end;
               2:  begin result := result + ' февраля ' + s; end;
               3:  begin result := result + ' марта ' + s; end;
               4:  begin result := result + ' апреля ' + s; end;
               5:  begin result := result + ' майя ' + s; end;
               6:  begin result := result + ' июня ' + s; end;
               7:  begin result := result + ' июля ' + s; end;
               8:  begin result := result + ' августа ' + s; end;
               9:  begin result := result + ' сентября ' + s; end;
               10: begin result := result + ' октября ' + s; end;
               11: begin result := result + ' ноября ' + s; end;
               12: begin result := result + ' декабря ' + s; end;
           end;
         end
       else begin
          result := DateToStr(d);
       end;
     end;
     end;
   end;

   Календарь ацтеков, месяц майя:)

   Nilud, 18 Мая 2012

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

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

   +86

   1. 1
   2. 2
   3. 3

      public static String serialize3(IModellingWorkerSetup r) {
           return serialize(serialize(serialize(r)));
       }

   как вы понимаете, есть и обратное преобразование

   ykhrustalev, 17 Мая 2012

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

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

    +96

    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

    if (IsPostBack)
                {
                    string PostBackerID = Request.Form.Get("__EVENTTARGET");
                    string PostBackerArg = Request.Form.Get("__EVENTARGUMENT");
    
                    if (PostBackerID == "ctl00$cphMainContent$pagerID")
                    {
                        CurrentReviewPage = int.Parse(PostBackerArg);
                        FillReviews();
                    }
                    else if (PostBackerID == "ctl00$cphMainContent$Like")
                    {
                        FillLike();
                    }
                    else if (PostBackerID == "ctl00$cphMainContent$review")
                    {
                        AddReview(PostBackerArg);
                    }
                    else if (PostBackerID == "ctl00$cphMainContent$Follow")
                    {
                        FillFollow();
                    }
                    else if (PostBackerID == "ctl00$cphMainContent$WebResultpagerID")
                    {
                        CurrentWebResultPage = int.Parse(PostBackerArg);
                        FillWebResults();
                    }
                    else if (PostBackerID == "ctl00$cphMainContent$WebResultCluster" || PostBackerID == "ctl00$cphMainContent$HiddenButton")
                    {
                        CurrentWebResultPage = 0;
                        int SelectedClusterID = int.Parse(PostBackerArg);
                        if (AllClusters != null && AllClusters.ContainsKey(SelectedClusterID))
                        {
                            SelectedClusterName = AllClusters[SelectedClusterID];
                            FillWebResults();
                        }
                    }
                }

    компактная логика всей страницы...

    mangyst, 17 Мая 2012

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

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

• 2109
• …
• 985
• 984
• 983
• 982
• 981
• 980
• 979
• 978
• 977
• …
• 1