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

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

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

1. PHP (5701)
2. C++ (2737)
3. Куча (2338)
4. JavaScript (2028)
5. C# (1928)
6. Java (1502)
7. Си (1102)
8. Pascal (647)
9. Python (582)
10. 1C (534)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (198)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (30)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

1. Список говнокодов пользователя Lokich

   Всего: 35

2. Куча / Говнокод #27690

   −1

   1. 1

   Здесь будут опубликованы пароли от учётных записей

   Lokich, 29 Сентября 2021

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

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

   −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

   protected void ASPxUploadControl1_FilesUploadComplete(object sender, DevExpress.Web.ASPxUploadControl.FilesUploadCompleteEventArgs e)
           {
               if (Request.Cookies["LID"] != null)
               {
                   int ListId = Convert.ToInt32(Request.Cookies["LID"].Value);
                   Response.Cookies["LID"].Expires = DateTime.Now.AddDays(-1);
                   foreach (DevExpress.Web.ASPxUploadControl.UploadedFile uf in ASPxUploadControl1.UploadedFiles)
                   {
                       uf.SaveAs(@"C:\TEMP\ListUploads\" + uf.FileName, true);
                       ListInfo.ImportList(Convert.ToInt16(ListId), @"C:\TEMP\ListUploads\" + uf.FileName);
                   }
               }
           }

   блять, это же каким инвалидом надо быть, чтобы так сделать загрузку файла в базу?

   Lokich, 06 Июня 2017

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

4. JavaScript / Говнокод #22097

   −48

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

   // fuckin IE8/9/10/11 sometimes returns 0 for dimensions. this weird hack was the only thing that worked
           both.css({
               position: 'relative', // cause a reflow, which will force fresh dimension recalculation
               left: -1 // ensure reflow in case the el was already relative. negative is less likely to cause new scroll
           });

   Lokich, 01 Февраля 2017

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

5. Куча / Говнокод #21576

   −21

   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

   <form id="form1" runat="server" >
       <div id="something" style="position: absolute;     height: 200px;     width: 400px;     margin: -100px 0 0 -200px;     top: 50%;     left: 50%;">
       <table width="100%"  >
       <tr>
           <td colspan="3">
           </td>
       </tr>
       <tr>
           <td width="40%">
           </td>
           <td width="20%">
               <asp:Login ID="Login4" runat="server" BackColor="#F7F7DE" BorderColor="#CCCC99" 
                   BorderStyle="Solid" BorderWidth="1px" 
                   DisplayRememberMe="False" FailureText="Введите правильные логин и пароль" 
                   Font-Names="Verdana" Font-Size="10pt" 
                   onloggedin="Login1_LoggedIn" PasswordLabelText="Пароль" 
                   UserNameLabelText="Логин" Width="100%">
                   <TitleTextStyle BackColor="#6B696B" Font-Bold="True" 
                       ForeColor="#FFFFFF" />
               </asp:Login>
           </td>
           <td width="40%">
           </td>
           </tr>
           <tr>
               <td colspan="3">
               </td>
           </tr>
       </table>
       </div>
       </form>

   верстка, уроверь - бог!

   Lokich, 26 Октября 2016

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

6. SQL / Говнокод #20493

   −48

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

   CREATE OR REPLACE PACKAGE BODY REPORT."PKG_COMMIT" 
   	IS
   	PROCEDURE P_COMMIT
   		IS
   	BEGIN
   		COMMIT;
   	END;
   END PKG_COMMIT;

   откопал чудный пакет

   Lokich, 08 Августа 2016

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

7. Си / Говнокод #20410

   −50

   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

   static PyObject *
   binascii_b2a_qp_impl(PyModuleDef *module,
                        Py_buffer *data,
                        int quotetabs,
                        int istext,
                        int header)
   {
     Py_ssize_t in, out;
     const unsigned char *databuf;
     ....
     if ((databuf[in] > 126) ||
         (databuf[in] == '=') ||
         (header && databuf[in] == '_') ||
         ((databuf[in] == '.') && (linelen == 0) &&
         (databuf[in+1] == '\n' || databuf[in+1] == '\r' ||
                                    databuf[in+1] == 0)) ||
         (!istext && ((databuf[in] == '\r') ||
                      (databuf[in] == '\n'))) ||
         ((databuf[in] == '\t' || databuf[in] == ' ') &&
              (in + 1 == datalen)) ||
         ((databuf[in] < 33) &&
          (databuf[in] != '\r') && (databuf[in] != '\n') &&
          (quotetabs ||
         (!quotetabs && ((databuf[in] != '\t') && // <=
                (databuf[in] != ' '))))))
     {
     ....
     }
     ....
   }

   Pyhon source https://habrahabr.ru/company/pvs-studio/blog/306204/
   теперь я ничему не удивляюсь

   Lokich, 22 Июля 2016

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

8. SQL / Говнокод #20129

   −45

   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

   if P_ID_STANDART is null then -- Это отвязка
   
     
     MANAGER_OBJECT_DSD.SET_RELATION_VARIANT(V_ID_OBJECT,ID_VARIANT,P_ID_STANDART,P_USER_NM);
     
    
     
     
     
   
   
   else -- Иначе привязка
   
   
     MANAGER_OBJECT_DSD.SET_RELATION_VARIANT(V_ID_OBJECT,ID_VARIANT,P_ID_STANDART,P_USER_NM);
     
   
     
     
   
   end if;

   кусок PL/SQL процедуры. форматирование сохранено.
   
   для чего нужно было вставлять столько переносов строк мне вообще непонятно.

   Lokich, 03 Июня 2016

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

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

   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

   public enum EventType {
               Error,
               Notification,
               Success
           }
   ...
                           String sEventType = String.Empty;
                           switch (eType) {
                               case EventType.Error:
                                   sEventType = "Error: ";
                                   break;
                               case EventType.Notification:
                                   sEventType = "Notification: ";
                                   break;
                               case EventType.Success:
                                   sEventType = "Success: ";
                                   break;
                           }

   Lokich, 11 Апреля 2016

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

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

    +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

    public class FileStorage {
            public FileStorage() {
                try {
                    if (String.IsNullOrEmpty(Common.GlobalVariables.FileStorage)) {
                        throw new Exception(Common.GuiHelper.ShowErrorMessage("FileStorage", String.Empty));
                    }
    
                    if (!Directory.Exists(Common.GlobalVariables.FileStorage)) {
                        DirectoryInfo di = Directory.CreateDirectory(Common.GlobalVariables.FileStorage);
                        if (!di.Exists) {
                            throw new Exception(Common.GuiHelper.ShowErrorMessage("FileStorageDir", String.Empty));
                        }
                    }
                } catch (Exception er) {
                    throw new Exception(er.Message);
                }
            }
    
            public String CreateFileDirectory() {
                try {
                    int iIdx = 0;
                    Boolean bIsCreated = false;
                    while (!bIsCreated && iIdx < 10) {
                        String sDir = Guid.NewGuid().ToString("N").ToLower();
                        sDir = sDir.Substring(0, 2);
                        if (!Directory.Exists(Common.GlobalVariables.FileStorage + sDir + @"\")) {
                            DirectoryInfo di = Directory.CreateDirectory(Common.GlobalVariables.FileStorage + sDir + @"\");
                            if (di.Exists) {
                                return Common.GlobalVariables.FileStorage + sDir + @"\";
                            }
                        }
    
                        iIdx++;
                    }
    
                    return null;
                } catch (Exception er) {
                    throw new Exception(er.Message);
                }
            }
        }

    может я чего не понимаю, но зачем?

    Lokich, 08 Апреля 2016

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

11. C# / Говнокод #19669

    +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

    public class ResultBase {
            private ColumnCollections _columns;
            private ArrayList _rows;
    ...
    }
    ...
    public class ColumnCollections : System.Collections.Specialized.NameObjectCollectionBase {}
    ...
            public int Compare(object x, object y) {
                if (x != null && y != null) {
                    object[] objX = x as object[];
                    object[] objY = y as object[];
    
                    if (objX != null && objY != null && objX.Length > 0 && objX.Length == objY.Length) {
                        switch (_ColumnType.FullName) {
                            case "System.Int16":
                                if (_Direction == DirectionType.ASC) {
                                    return ((short)objX[_ColumnIndex]).CompareTo((short)objY[_ColumnIndex]);
                                } else {
                                    return ((short)objY[_ColumnIndex]).CompareTo((short)objX[_ColumnIndex]);
                                }
                            case "System.Int32":
                                if (_Direction == DirectionType.ASC) {
                                    return ((int)objX[_ColumnIndex]).CompareTo((int)objY[_ColumnIndex]);
                                } else {
                                    return ((int)objY[_ColumnIndex]).CompareTo((int)objX[_ColumnIndex]);
                                }
                            case "System.Int64":
                                if (_Direction == DirectionType.ASC) {
                                    return ((long)objX[_ColumnIndex]).CompareTo((long)objY[_ColumnIndex]);
                                } else {
                                    return ((long)objY[_ColumnIndex]).CompareTo((long)objX[_ColumnIndex]);
                                }
                            case "System.Double":
                                if (_Direction == DirectionType.ASC) {
                                    return ((double)objX[_ColumnIndex]).CompareTo((double)objY[_ColumnIndex]);
                                } else {
                                    return ((double)objY[_ColumnIndex]).CompareTo((double)objX[_ColumnIndex]);
                                }
                            case "System.Decimal":
                                if (_Direction == DirectionType.ASC) {
                                    return ((decimal)objX[_ColumnIndex]).CompareTo((decimal)objY[_ColumnIndex]);
                                } else {
                                    return ((decimal)objY[_ColumnIndex]).CompareTo((decimal)objX[_ColumnIndex]);
                                }
                            case "System.Single":
                                if (_Direction == DirectionType.ASC) {
                                    return ((float)objX[_ColumnIndex]).CompareTo((float)objY[_ColumnIndex]);
                                } else {
                                    return ((float)objY[_ColumnIndex]).CompareTo((float)objX[_ColumnIndex]);
                                }
                            case "System.String":
                                String sX = (!Convert.IsDBNull(objX[_ColumnIndex])) ? (String)objX[_ColumnIndex] : String.Empty;
                                if (String.IsNullOrEmpty(sX)) {
                                    sX = "";
                                }
    
                                String sY = (!Convert.IsDBNull(objY[_ColumnIndex])) ? (String)objY[_ColumnIndex] : String.Empty;
                                if (String.IsNullOrEmpty(sY)) {
                                    sY = "";
                                }
    
                                if (_Direction == DirectionType.ASC) {
                                    return sX.CompareTo(sY);
                                } else {
                                    return sY.CompareTo(sX);
                                }
                            case "System.DateTime":
                                if (_Direction == DirectionType.ASC) {
                                    return ((DateTime)objX[_ColumnIndex]).CompareTo((DateTime)objY[_ColumnIndex]);
                                } else {
                                    return ((DateTime)objY[_ColumnIndex]).CompareTo((DateTime)objX[_ColumnIndex]);
                                }
                            case "System.Boolean":
                                if (_Direction == DirectionType.ASC) {
                                    return ((Boolean)objX[_ColumnIndex]).CompareTo((Boolean)objY[_ColumnIndex]);
                                } else {
                                    return ((Boolean)objY[_ColumnIndex]).CompareTo((Boolean)objX[_ColumnIndex]);
                                }
                        }
                    }
                }
    
    
                return 0;
            }

    достался мне в наследство код, который я года два поддерживал. никогда не вдавался в код той тулзы, которая из базы результаты возвращяла, пока не пришлось описать проект комментариями, чтобы отдать его клиенту.
    компарер просто божественен...
    я конечно точно не знаю, кто это писал, но за то знаю, кому пренадлежил авторство конкретной версии, но за то знаю, кто писал первую... и этот человек мне еще говорил, что мой экстеншен в 50 строк, который через рифлексию миллион строк из IDataReader за секунду в коллекцию объектов возвращял, уверял меня, что это медленное и корявое говно. нет, это медленное и корявое говно!
    
    господи, за что?

    Lokich, 21 Марта 2016

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

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

• 4
• 3
• 2
• 1