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

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

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

1. PHP (5703)
2. C++ (2740)
3. Куча (2339)
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# / Говнокод #19174

   +8

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

   public static class StringExtensions
       {
           public static bool IsNulldOrEmpty(this string str)
           {
               return string.IsNullOrEmpty(str);
           }
       }

   why

   antoanelenkov, 12 Декабря 2015

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

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

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

   using System;
   using System.Collections.Generic;
   using System.Data;
   using System.IO;
   using Excel;
   using FPCLib.Models.Broadcasts;
   
   namespace ExcelReadTests.Model.Путевка
   {
       public class MyExcel : IDisposable
       {
           private readonly DataTable table;
   
           public MyExcel(string putevkaFileName)
           {
               table = ReadToTable(putevkaFileName);
           }
   
   		  internal DataTable ReadToTable(string excelFileName)
           {
               var stream = File.Open(excelFileName, FileMode.Open, FileAccess.Read);
               var excelReader = ExcelReaderFactory.CreateBinaryReader(stream);
   
               return excelReader.AsDataSet().Tables[0];
           }
   
           public bool IsTvc()
           {
               if (table.Columns.Count == 10)
                   return true;
               return false;
           }
   
           public bool IsFriday()
           {
               if (table.Columns.Count == 16)
                   return true;
               return false;
           }
   		
         
           public List<Broadcast> GetBroadcastsFriday()
           {
               var broadcasts = new List<Broadcast>();
               for (var i = 0; i < table.Rows.Count; i++)
               {
                   if (table.Rows[i][0].ToString().Length > 1)
                   {
                       broadcasts.Add(new Broadcast
                       {
                           Time = new Time(table.Rows[i][0].ToString()),
                           Title = table.Rows[i][1].ToString(),
                           File = table.Rows[i][3].ToString(),
                           Type = table.Rows[i][7].ToString(),
                           Id = table.Rows[i][2].ToString(),
                           Length = new Time(table.Rows[i][4].ToString())
                       });
                   }
               }
   
               return broadcasts;
           }
   
           public List<Broadcast> GetBroadcastsTvc()
           {
               var broadcasts = new List<Broadcast>();
               for (var i = 0; i < table.Rows.Count; i++)
               {
                   broadcasts.Add(new Broadcast
                   {
                       Time = new Time(table.Rows[i][0].ToString()),
                       Title = table.Rows[i][2].ToString().Trim(),
                       File = table.Rows[i][6].ToString().Trim(),
                       Type = table.Rows[i][9].ToString().Trim(),
                       Id = table.Rows[i][1].ToString().Trim(),
                       Length = new Time(table.Rows[i][5].ToString())
                   });
               }
   
               return broadcasts;
           }
   		
   		  public void Dispose()
           {
           }
         
       }
   }

   вот так открываю эксельку
   DataTable ReadToTable(string excelFileName)
   
   по количеству столбцов, определяю ее источник)
   public bool IsTvc()
   public bool IsFriday()
   
   
   
   и даже поддерживаю интерфейс IDisposable))

   yolozesoja, 08 Декабря 2015

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

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

   +2

   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

   public static string _GetValueFromConfigFile(string sKey)
           {
               string sReturnValue = _scNullString;
               string filePath = System.IO.Directory.GetCurrentDirectory() + @"\App.config";
               // FOR TDOCS
               //string filePath = @"D:\hosting\4856094\html\Bin\App.config";
   
               XmlDocument doc = new XmlDocument();
               XmlNode rootNode;
               XmlNode xmlNode;
   
               try
               {
                   doc.Load(filePath);
   
                   rootNode = doc.DocumentElement;
                   xmlNode = rootNode.SelectSingleNode("descendant::add[@key='" + sKey + "']");
                   if (xmlNode != null) sReturnValue = xmlNode.Attributes["value"].Value;
   
                   return sReturnValue;
                   //return "";
               }
               catch// (Exception Ex)
               {
                   //_ErrorDetail = Ex.Message;
                   return _scNullString;
               }
           }

   Very helpful method to get data from app.config :)
   Жаль что для web не работает :(
   
   Note:
   public const string _scNullString = "";

   iec, 02 Декабря 2015

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

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

   +2

   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

   if(itemsCount <= 2)
   		{
   			if(iteration == 1)
   			{
   				dragItemPos = new Vector3(-20, -55, 0);
   			}else if(iteration == 2){
   				dragItemPos = new Vector3(145, 75, 0);
   			}
   		}else if(itemsCount == 3){
   			if(iteration == 1)
   			{
   				dragItemPos = new Vector3(60, -170, 0);
   			}else if(iteration == 2){
   				dragItemPos = new Vector3(245, -45, 0);
   			}else if(iteration == 3){
   				dragItemPos = new Vector3(20, 90, 0);
   			}
   		}else if(itemsCount == 4){
   			if(iteration == 1)
   			{
   				dragItemPos = new Vector3(60, -170, 0);
   			}else if(iteration == 2){
   				dragItemPos = new Vector3(245, -45, 0);
   			}else if(iteration == 3){
   				dragItemPos = new Vector3(-80, 2, 0);
   			}else if(iteration == 4){
   				dragItemPos = new Vector3(140, 160, 0);
   			}
   		}
   		else{
   			if(iteration == 1)
   			{
   				dragItemPos = new Vector3(60, -170, 0);
   			}else if(iteration == 2){
   				dragItemPos = new Vector3(245, -45, 0);
   			}else if(iteration == 3){
   				dragItemPos = new Vector3(-100, -45, 0);
   			}else if(iteration == 4){
   				dragItemPos = new Vector3(25, 95, 0);
   			}else if(iteration == 5){
   				dragItemPos = new Vector3(190, 180, 0);
   			}
   		}

   Определяем позиции объекта по их количеству и по номеру итерации. Массивы? не не слышал.

   kschingiz, 27 Ноября 2015

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

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

   +7

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

   public static bool CheckBoxValue(bool Checked)
           {
               return Conversions.ToBoolean(Interaction.IIf(Checked, true, false));
           }

   inickvel, 26 Ноября 2015

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

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

   +1

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

   string str3 = Strings.Trim(ID);
    do
    {
        num2 = (short) Strings.InStr(str3, " ", CompareMethod.Binary);
        if (num2 > 0)
        {
               str3 = str3.Substring(0, num2 - 1) + Strings.Mid(str3, num2 + 1);
        }
   }
   while (num2 > 0);

   А зачем нам str3.Replace(" ", string.Empty) ?

   inickvel, 25 Ноября 2015

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

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

   +2

   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

   bool isLiveLine = false;
   bool isQALine = false;
   
   if (lineInfo.IndexOf("QL") != -1)
   {
    isLiveLine = true;
    isQALine = true;
   }
   else if (lineInfo.IndexOf("Q") != -1)
   {
    isLiveLine = false;
    isQALine = true;
   }
   else if (lineInfo.IndexOf("L") != -1)
   {
    isLiveLine = true;
    isQALine = false;
   }
   else
   {
    isLiveLine = false;
    isQALine = false;
   }

   pro687, 24 Ноября 2015

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

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

   +2

   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

   public static byte[] HMACSHA256(ProtectedData key, byte[] data)
   {
       using (var _key = key.Get())
       using (var hmac = new HMACSHA256(_key))
           return hmac.ComputeHash(data);
   }
   
   public static byte[] HMACSHA256(ProtectedData key, Stream stream)
   {
       using (var _key = key.Get())
       using (var hmac = new HMACSHA256(_key))
           return hmac.ComputeHash(stream);
   }
   
   public static byte[] HMACSHA256(byte[] key, byte[] data)
   {
       using (var hmac = new HMACSHA256(key))
           return hmac.ComputeHash(data);
   }
   
   public static byte[] HMACSHA256(byte[] key, Stream stream)
   {
       using (var hmac = new HMACSHA256(key))
           return hmac.ComputeHash(stream);
   }
   
   public static byte[] MD5(byte[] data)
   {
       using (var h = System.Security.Cryptography.MD5.Create())
       { return h.ComputeHash(data); }
   }
   
   public static byte[] MD5(Stream stream)
   {
       using (var h = System.Security.Cryptography.MD5.Create())
       { return h.ComputeHash(stream); }
   }
   
   public static byte[] SHA1(byte[] data)
   {
       using (var h = System.Security.Cryptography.SHA1.Create())
           return h.ComputeHash(data);
   }
   
   public static byte[] SHA1(Stream stream)
   {
       using (var h = System.Security.Cryptography.SHA1.Create())
           return h.ComputeHash(stream);
   }
   
   public static byte[] SHA256(byte[] data)
   {
       using (var h = System.Security.Cryptography.SHA256.Create())
           return h.ComputeHash(data);
   }
   
   public static byte[] SHA256(Stream stream)
   {
       using (var h = System.Security.Cryptography.SHA256.Create())
           return h.ComputeHash(stream);
   }
   
   public static byte[] SHA384(byte[] data)
   {
       using (var h = System.Security.Cryptography.SHA384.Create())
           return h.ComputeHash(data);
   }
   
   public static byte[] SHA384(Stream stream)
   {
       using (var h = System.Security.Cryptography.SHA384.Create())
           return h.ComputeHash(stream);
   }
   
   public static byte[] SHA512(byte[] data)
   {
       using (var h = System.Security.Cryptography.SHA512.Create())
           return h.ComputeHash(data);
   }
   
   public static byte[] SHA512(Stream stream)
   {
       using (var h = System.Security.Cryptography.SHA512.Create())
           return h.ComputeHash(stream);
   }

   Психанул

   yourmom, 20 Ноября 2015

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

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

   +2

   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

   using Microsoft.VisualBasic;
   
           public string ConvertEnoviaNameForDB1(string name, char Separ = '%')
           {
               string functionReturnValue = null;
               functionReturnValue = name;
               //input filename
               //check ENOVIA filenames (example c0234244 --.catpart or c0234244--.catpart)
               //output filename in DB format for operator LIKE(c0234244 --.catpart -> c0234244%.catpart)
               int i = 0;
               int loc1 = Strings.InStr(name, ".CATP", CompareMethod.Text);
               bool NeedToConvert = false;
               char Chr1 = '1';
               char ChrBefore1 = '1';
               //check catparts and catproducts ONLY
               if (loc1 > 0)
               {
                   int NumSymbols = loc1 - 2;
                   if (NumSymbols > 6)
                       NumSymbols = 6;
                   string tmpstr1 = Strings.Mid(name, loc1 - NumSymbols - 1, NumSymbols);
                   //analyse 4 chars max
                   for (i = 0; i <= loc1 - 2; i++)
                   {
                       Chr1 = name[loc1 - 2 - i];
                       //3-string array start from 0(position = count-1)
                       if ((Strings.Asc(ChrBefore1) >= 65 & Strings.Asc(ChrBefore1) <= 90))
                       {
                           if (Chr1 == ' ')
                           {
                               i = i + 2;
                               break; // TODO: might not be correct. Was : Exit For
                           }
                           else if (Chr1 == '-')
                           {
                               ChrBefore1 = Chr1;
                               //means can be  two chars (ex. "AA") max
                           }
                           else if ((Strings.Asc(Chr1) >= 65 & Strings.Asc(Chr1) <= 90) & i < 2)
                           {
                               ChrBefore1 = Chr1;
                           }
                           else
                           {
                               break; // TODO: might not be correct. Was : Exit For
                           }
                       }
                       else if (ChrBefore1 == '-')
                       {
                           if (Chr1 == ' ')
                           {
                               i = i + 2;
                               break; // TODO: might not be correct. Was : Exit For
                               // And i < 3 Then 'means can be "---" - not more
                           }
                           else if (Chr1 == '-')
                           {
                               //ChrBefore1 = Chr1
                           }
                           else
                           {
                               i = i + 1;
                               break; // TODO: might not be correct. Was : Exit For
                           }
                           //ChrBefore1 = Chr1
                           // means start
                       }
                       else if (ChrBefore1 == '1')
                       {
                           if (Chr1 == '-' | (Strings.Asc(Chr1) >= 65 & Strings.Asc(Chr1) <= 90))
                           {
                               ChrBefore1 = Chr1;
                           }
                           else
                           {
                               i = i + 1;
                               break; // TODO: might not be correct. Was : Exit For
                           }
   
                       }
                   }
                   functionReturnValue = Strings.Left(name, loc1 - i) + Separ + Strings.Right(name, Strings.Len(name) - loc1 + 1);
               }
               return functionReturnValue;
           }

   Наличие комментария в 7ой строчке приводит в неописуемый восторг.
   Без него понимать поведение функции пришлось бы с болью.
   И да, в RegExp могут не только лишь все. Мало кто может.

   Szer, 20 Ноября 2015

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

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

    +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

    public class Logger
    	{
    		public static string filePath = Path.Combine(AppDomain.CurrentDomain.BaseDirectory, "log.log");
    
    		public static void Write(string message)
    		{
    			using (var sw = File.AppendText(filePath))
    			{
    				sw.WriteLine(DateTime.Now);
    				sw.WriteLine(message);
    				sw.WriteLine();
    				sw.Flush();
    			}
    		}
    
    		public static void Write(Exception exception)
    		{
    			using (var sw = File.AppendText(filePath))
    			{
    				sw.WriteLine(DateTime.Now);
    				sw.WriteLine("ERROR:");
    				sw.WriteLine(exception.Message);
    				sw.WriteLine(exception.StackTrace);
    				sw.WriteLine();
    				sw.Flush();
    			}
    		}
    	}

    Нафига готовые решения? Вот - образец велосипедостроения! (И, тссс! Не вздумайте использовать его в многопоточной среде ;) А именно там он и используется по факту :) )
    
    PS угадайте какой фортель выкинет сеё чудо при race condition

    leon_mz, 18 Ноября 2015

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

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

• 193
• …
• 164
• 163
• 162
• 161
• 160
• 159
• 158
• 157
• 156
• …
• 1