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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2340)
4. JavaScript (2030)
5. C# (1930)
6. Java (1503)
7. Си (1103)
8. Pascal (649)
9. Python (585)
10. 1C (536)
11. SQL (434)
12. ActionScript (294)
13. Objective C (204)
14. bash (200)
15. Perl (196)
16. Assembler (148)
17. Ruby (144)
18. VisualBasic (140)
19. Lua (51)
20. Go (32)
21. Swift (29)
22. Haskell (28)
23. Kotlin (15)

1. C# / Говнокод #1981

   +136.7

   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

   protected void WriteLoginBlock()
       {
           string s = "";
           if (SiteSecurity.IsAuthenticated)
           {
               s = @"<div style=""margin-top: 3px;""><a href=""/login/"" style=""text-decoration: underline; color:#000; background:none;"">" + SiteSecurity.CurrentUser.Name + "</a></div>";
           }
           else
           {
               s = @"
   			
   			<form action=""/login/"" method=""POST"" id=""gLoginForm"">
   		<div class=""rel_i"">
   			    <input class=""txtinp"" type=""text"" value=""логин"" id=""gLogin"" name=""cllgn"" />
   			    <input class=""txtinp"" type=""text"" name=""clpwd"" id=""gPwd"" value=""пароль"" onfocus=""if (this.value==this.defaultValue) {this.value='';this.type='password'}"" onblur=""if(this.value=='') {this.value=this.defaultValue;this.type='text'}"" />
   				
   				
   				
   				
   				<!--<label for=""clpwd"" class=""pwd_label"" id=""gPwdLabel"">пароль</label>-->
   			    <a href=""/login/"" id=""blogin"">&#160;</a>
   				</div>
   		    </form>
   			
   		    <script type=""text/javascript"">
   			    function isChanged(obj) { return $(obj)[0]._changed; }
   			    function isValue(obj, value) { return $.trim($(obj).val()).toLowerCase() == value.toLowerCase(); }
   			    $(""#gLogin"")
   			    .keydown ( function() { this._changed = true; } )
   			    .focus( function() { if (!isChanged(this) && isValue(this, 'логин')) $(this).val(''); } )
   			    .blur( function() { if(isValue(this, '')) { $(this).val('логин'); $(this)[0]._changed=false; } } );
   
                   $(""#blogin"").click(function(){
   			    if(!isChanged(""#gLogin"") || isValue(""#gLogin"", '') || isValue(""#gPwd"",'')) return false;
   			    $('#gLoginForm').submit();
   			    return false; });
   		    </script>";
           }
           Response.Write(s);
       }

   bniwredyc, 15 Октября 2009

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

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

   +127.4

   1. 1

   idPlansList = ViewState["idPlansList"] is List<int> ? (List<int>)ViewState["idPlansList"] : new List<int>();

   ilya_lysenko, 14 Октября 2009

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

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

   +139

   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

   // export in csv - part of the code
   // ...
               foreach (users_view _item in _users_view)
               {
                   _writer.Write(String.Format("{1}{0}{2}{0}{3}{0}{4}{0}{5}{0}{6}{0}{7}{0}{8}{0}{9}{0}{10}{0}{11}{0}{12}{0}{13}{0}{14}{0}{15}{0}{16}{0}{17}{0}{18}{0}{19}{0}{20}{0}{21}{0}{22}{0}{23}{0}{24}\n",
                       AppSettingsReader.GetValue("CSVFileSeparator"),
                        @"""" + _item.title_name + @"""", @"""" + _item.first_name + @"""",
                        @"""" + _item.last_name + @"""", @"""" + _item.job_title + @"""",
                       @"""" + _item.user_type_name + @"""",
                                               
                       @""""+_item.company_name+@"""",
                       (_item.telephone != null) ? (@"""" + _item.telephone + @"""") : (""),
                       (_item.fax != null) ? (@"""" + _item.fax + @"""") : (""),
                       @"""" + _item.email + @"""",
                       (_item.account_email != null) ? (@"""" + _item.account_email + @"""") : (""),
                       @"""" + _item.site_address + @"""",
                       @"""" + _item.advertisement_source_name+@"""",
   
                       @"""" + _item.address_1+@"""",
                       (_item.address_2 != null) ? (@"""" + _item.address_2+@"""") : (""),
                       @"""" + _item.country_name+@"""",
                      
                       //_item.email_format_name,
   
                       (_item.postcode != null) ? (@"""" + _item.postcode+@"""") : (""),
                       (_item.county != null) ? (@"""" + _item.county+@"""") : (""),
                       @"""" + _item.town + @"""",
                   //    (_item.is_active == false) ? ("No") : ("Yes"),
   
                       @"""" + _item.username+@"""",
                       @"""" + _item.password+@"""",
                       @"""" + _item.account_type_name + @"""",
   
                       @"""" + _item.creation_date + @"""",
   
                       (_item.is_newsletter_subscriber == false) ? ("No") : ("Yes"),
                       (_item.is_marketing_subscriber == false) ? ("No") : ("Yes")
                       )
                   );
               }

   Вот что нашел :)

   bugotrep, 14 Октября 2009

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

4. C# / Говнокод #1969

   +132.9

   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

   //  Этот метод проверяет, создано ли окошло лога, если не создано, то создает его
           public void CheckLogWnd()
           {
               try
               {
                   if (logWnd.IsAccessible)
                   {
                   }
               }
               catch
               {
                   logWnd = new LogWnd();
               }
           }

   Не помню как сделать это правильно :(

   nolka4, 13 Октября 2009

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

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

   +131.8

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

   public string EncodeParameter_1251(string param) {
   	var result = new StringBuilder();
   	foreach (var bt in Encoding.GetEncoding("windows-1251").GetBytes(param)) result.AppendFormat("%{0}", bt.ToString("X02"));
   	return result.ToString(); 
   }

   метод кодирующий значение параметра запроса в "windows-1251" в формате URL (%FF%FF%FF...).
   у проекта кодировка запросов в "UTF-8", но у остальных проектов - "windows-1251", и в другой кодировке они некорректно обрабатывают кириллицу в строке запроса. знаю, наговнокодил, ну а шо поделаеш...

   xeonix, 12 Октября 2009

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

6. C# / Говнокод #1957

   +105.5

   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

   //одногрупнику надо было проверить, является ли "obj" - "А"
   //наблюдал за процессом, и плакал
   //----------------------------------------------------------------------------------
   //1 версия
   static bool IsA(object obj) {
   if (obj.GetType().Name.Equals("A", StringComparison.InvariantCultureIgnoreCase))
       return true;
       else return false;
   }
   //----------------------------------------------------------------------------------
   //2 версия
   static bool IsA(object obj) {
       A a = new A();
       if (obj.GetType().Equals(a.GetType()))
           return true;
       else return false;
   }
   //----------------------------------------------------------------------------------
   //3 версия
   static bool IsA(object obj) {
       if (obj.GetType().Equals(typeof(A)))
           return true;
       else return false;
   }
   //----------------------------------------------------------------------------------
   //потом он вспомнил, что от "A" могут наследоваться другие классы
   static bool IsA(object obj) {
       Type typeObj = obj.GetType();
       do {
           if (typeObj.Equals(typeof(object)))
               return false;
           else if (typeObj.Equals(typeof(A)))
               return true;
           else typeObj = typeObj.BaseType;
       } while (true);
   }
   
   //плачу, смеюсь и плачу, а с виду одногрупник вроде не Индус...
   //...и весь этот говнокод был написан, вместо простого:
   static bool IsA(object obj) { return obj is A; }

   via xeonix

   striker, 12 Октября 2009

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

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

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

   private void TestWorksheetFunction() 
   {
     ...
    
     Excel.WorksheetFunction wsf = ThisApplication.WorksheetFunction;
     ws.get_Range("Min", Type.Missing).Value2 = wsf.Min(rng, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing);
     ws.get_Range("Max", Type.Missing).Value2 = wsf.Max(rng, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing);
     ws.get_Range("Median", Type.Missing).Value2 = wsf.Median(rng,
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing);
     ws.get_Range("Average", Type.Missing).Value2 = wsf.Average(rng, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing);
     ws.get_Range("StDev", Type.Missing).Value2 = wsf.StDev(rng, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing, Type.Missing, Type.Missing, Type.Missing, 
       Type.Missing);
   }

   Если вы - разработчик на C#, вам придется привыкнуть к обилию значений Type.Missing в вызовах методов. Поскольку объектную модель Excel писали в расчете на VBA, многие ее методы принимают необязательные параметры - иногда до 30. Используйте либо многочисленные экземпляры значения Type.Missing или указывайте для каждого параметра определенное значение по умолчанию.
   (c) http://www.gotdotnet.ru/LearnDotNet/NETFramework/22054.aspx

   zerkms, 12 Октября 2009

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

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

   +950

   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

   public string OperatorName
   {
       get
       {
           if (m_operator == null)
           {
               try
               {
                   int idUser = 1;
                   m_operator = (idUser > 0) ? "Василий" : string.Empty;
               }
               catch
               {
                   m_operator = string.Empty;
               }
           }
           return m_operator;
       }
   }

   Василий решает )))

   tonic, 05 Октября 2009

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

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

   +139.6

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

   var complete = new ManualResetEvent(false);
   
   ThreadPool.QueueUserWorkItem(
   	delegate
   		{
   			service.StartDownloadUpdatesProcess(complete);
   		});
   
   complete.WaitOne();

   Запустим поток, а потом подождем пока он закончиться.

   Mike Chaliy, 04 Октября 2009

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

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

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

    private ArrayList GetSubscribers(string condition)
            {
               ArrayList subscribers = new ArrayList();
               ArrayList lst =  new user_category_notification().Factory.GetItems(condition, "user_category_notification.id_user");
                ArrayList distinc_lst = new ArrayList();
                ArrayList distinc_lst_ids = new ArrayList();
                ArrayList lst_ids = new ArrayList();
                for (int i = 0; i < lst.Count; i++)
                {
                    lst_ids.Add(((user_category_notification) lst[i]).id_user);
                }
                for (int i = 0; i < lst_ids.Count;i++ )
                {
                    if (distinc_lst_ids.Contains(lst_ids[i])) continue;
                    else
                    {
                        distinc_lst.Add(lst[i]);
                        distinc_lst_ids.Add(lst_ids[i]);
                    }
                }
                foreach (user_category_notification _un in distinc_lst)
                {
                    user _current = (user)new user().Factory.GetByID(_un.id_user);
                    subscribers.Add(_current);
                }
                
                return subscribers;
            }

    Филтрация :)

    bugotrep, 02 Октября 2009

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

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

• 193
• …
• 22
• 21
• 20
• 19
• 18
• 17
• 16
• 15
• 14
• …
• 1