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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2360)
4. JavaScript (2030)
5. C# (1928)
6. Java (1502)
7. Си (1115)
8. Pascal (647)
9. Python (583)
10. 1C (538)
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. Swift (27)
22. Haskell (26)
23. Kotlin (14)

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

   −857

   1. 1
   2. 2

   SELECT * FROM [dbo].[Publications]												
   where [UserId] not like 'NULL'

   UserId, естественно, int

   synapse, 01 Апреля 2010

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

2. JavaScript / Говнокод #2911

   +148.2

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

   var tbl = order.group != 0 ?
      (order.group != 1 ?
       limit :
       closed) :
      open;

   asilmacc, 01 Апреля 2010

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

3. PHP / Говнокод #2910

   +162.6

   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

   <?php
   
   
   if($_GET['id'] == 10 ){
   
   $location = "/articles/lyustry/lustra-na-kuhne/";
   }elseif($_GET['id'] == 13 and $_GET['id_type'] == 17){
   $location = "/market/interernye_svetilniki/lyustry/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 22 and $_GET['act'] == 'sub_type'){
   $location = "/market/interernye_svetilniki/bolshie_lyustry/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 34 and $_GET['act'] == 'sub_type'){
   $location = "/market/interernye_svetilniki/lyustry/bolee_6_rozhkov/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 2 ){
   $location = "/market/interernye_svetilniki/podvesnye/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 40 and $_GET['act'] == 'sub_type'){
   $location = "/market/interernye_svetilniki/podvesnye/odinochnyj_podves/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 41 and $_GET['act'] == 'sub_type'){
   $location = "/market/interernye_svetilniki/podvesnye/trojnoj_i_bolee/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 5 ){
   $location = "/market/interernye_svetilniki/nastenno-potolochnye/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 35 and $_GET['act'] == 'sub_type'){
   $location = "/market/interernye_svetilniki/potolochnye/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 52 ){
   $location = "/market/interernye_svetilniki/vlagozawitnye/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 51 ){
   $location = "/market/interernye_svetilniki/podsvetka_kartin/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 7 ){
   $location = "/market/interernye_svetilniki/bra/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 37 and $_GET['act'] == 'sub_type'){
   $location = "/market/interernye_svetilniki/bra/odnorozhkovye/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 38 and $_GET['act'] == 'sub_type'){
   $location = "/market/interernye_svetilniki/bra/dvuhrozhkovye/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 39 and $_GET['act'] == 'sub_type'){
   $location = "/market/interernye_svetilniki/bra/treh_i_bolee_rozhkovye/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 1 ){
   $location = "/market/interernye_svetilniki/tochechnye/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 42 and $_GET['act'] == 'sub_type'){
   $location = "/market/interernye_svetilniki/tochechnye/odinochnye/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 43 and $_GET['act'] == 'sub_type'){
   $location = "/market/interernye_svetilniki/tochechnye/dvojnye/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 44 and $_GET['act'] == 'sub_type'){
   $location = "/market/interernye_svetilniki/tochechnye/tri_i_bolee/";
   }
   elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 8){
   $location = "/market/interernye_svetilniki/nastolnye_lampy/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 46 and $_GET['act'] == 'sub_type'){
   $location = "/market/interernye_svetilniki/nastolnye_lampy/ofisnye/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 47 and $_GET['act'] == 'sub_
   type'){
   $location = "/market/interernye_svetilniki/nastolnye_lampy/prikrovatnye_domashnie_ili_interernye/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 6 ){
   $location = "/market/interernye_svetilniki/torshery/";
   }elseif(
   $_GET['id'] == 13 and $_GET['id_type'] == 9 ){
   $location = "/market/ulichnye_svetilniki/";
   }...
   И так далее..

   Клиент поставил нашу CMS, на старой были ссылки типа /?id=10 и т.д. Сделал ему для примера скрипт редиректа с использованием switch case и вот что я увидел через две недели 0_0 Неприкрытый говнокод.

   VeroLom, 01 Апреля 2010

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

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

   +122.6

   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

   string[] mas_param = a.Split('_');
               this.s_sw_neraspred = mas_param[15];
               s_neraspred.Text = this.s_sw_neraspred;
               int i = a.IndexOf("_", 0, a.Length);
               sw_id = a.Substring(0, i);
               int j = i + 1;
               i = a.IndexOf("_", j, a.Length - j);
               mf_id = a.Substring(j, i - j);
               //mf1=a.Substring(j);
               j = i + 1;
               i = a.IndexOf("_", j, a.Length - j);
               ta_ed = a.Substring(j, i - j);
               j = i + 1;
               i = a.IndexOf("_", j, a.Length - j);
               this.s_op = a.Substring(j, i - j);
               j = i + 1;
               i = a.IndexOf("_", j, a.Length - j);
               s_rm = a.Substring(j, i - j);
               j = i + 1;
               i = a.IndexOf("_", j, a.Length - j);
               s_tm = a.Substring(j, i - j);
               j = i + 1;
               i = a.IndexOf("_", j, a.Length - j);
               s_sw = a.Substring(j, i - j);
               j = i + 1;
               i = a.IndexOf("_", j, a.Length - j);
               ta_sd = a.Substring(j, i - j);
               j = i + 1;
               i = a.IndexOf("_", j, a.Length - j);
               sw_sd = a.Substring(j, i - j);
               j = i + 1;
               i = a.IndexOf("_", j, a.Length - j);
               sw_ed = a.Substring(j, i - j);
               //mf_topsw
               j = i + 1;
               i = a.IndexOf("_", j, a.Length - j);
               mf_topsw = a.Substring(j, i - j);
               //s_op_topsw
               j = i + 1;
               i = a.IndexOf("_", j, a.Length - j);
               s_op_topsw = a.Substring(j, i - j);
               //s_rm_topsw
               j = i + 1;
               i = a.IndexOf("_", j, a.Length - j);
               s_rm_topsw = a.Substring(j, i - j);
               //s_tm_topsw
               j = i + 1;
               i = a.IndexOf("_", j, a.Length - j);
               s_tm_topsw = a.Substring(j, i - j);
               //s_sw_topsw
               j = i + 1;
               i = a.IndexOf("_", j, a.Length - j);
               s_sw_topsw = a.Substring(j, i - j);            
           
   
               //ta
   
               ta_id = a.Split('_')[mas_param.Length-1];

   a - строка вида
   {0}_{1}_{2}_{3}_{4}_{5}_{6}_{7}_{8}_{9}_ {10}_{11}_{12}_{13}_{14}_{15}_{16}

   alex_donetsk, 01 Апреля 2010

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

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

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

   public partial class AbstractPickerBrw : Form
   {
          //...
           public AbstractPickerBrw ()
           {
                 this.Shown += new System.EventHandler(this.AbstractPickerBrw_Shown);
           }
           //...
   
           private void AbstractPickerBrw_Shown(object sender, System.EventArgs e)
           {
               Width = Width % 2 == 0 ? Width + 1 : Width - 1;
           }
   }

   Это мой код, просто не мог добиться перерисовки диалога при смене данных в WinForms и решил не заморачиваться.

   xaoc, 01 Апреля 2010

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

6. Java / Говнокод #2907

   +69

   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

   if (isRenderVerifyClaimTab()) {
   			addContrToInit("com.cs.creditecspert.webjsf.controllers.ddeControllers.VerifyClaimController");
   		}
   		if (isRenderVerifyAvtoTab()) {
   			addContrToInit("com.cs.creditecspert.webjsf.controllers.ddeControllers.VerifyAvtoController");
   		}
   		if (isRenderUntipicalClaimTab()) {
   			addContrToInit("com.cs.creditecspert.webjsf.controllers.ddeControllers.UntipicalClaimController");
   		}
   		if (isRenderTypeClaimTab()) {
   			addContrToInit("com.cs.creditecspert.webjsf.controllers.ddeControllers.TypeClaimController");
   
   		}

   Профтыкал что можно было делать так:
   
   if (isRenderCheckClaimTab()) {
   addContrToInit(CheckClaimController.clas s.getName());
   }
   
   :)

   Lockdog, 01 Апреля 2010

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

7. Куча / Говнокод #2906

   +131.6

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

   <div style="width:100%; text-align:center; height:95%; position: absolute ; left:0; top:0;  text-align:center;  "  >
         
         
               <table id="TableHead" cellpadding="0" cellspacing="0" style="text-align:center;width: 400px; margin-top:-150px; margin-left:-200px;    position:absolute; top: 50%; " align="center">
   
   ...
   </div>

   ну не красотень

   alex_donetsk, 01 Апреля 2010

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

8. Java / Говнокод #2905

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

   private String getNullString(int kol){
           String nullstr="";
           for(int i=0;i<kol;i++){
               nullstr+="0";
           }
           return nullstr;
       }
   
   public String getNumber(int idd){
           String regid="";
           String idds=(idd+"");
           if(idds.length()<getNumberLenth()){
               regid+=getNullString(getNumberLenth()-idds.length());
               regid+=idds;
           }
           String regnum="";
           regnum+=getNumberPrefix()+regid;
           if(getNumberPeriod()!=NUM_PERIOD_NOPERIOD){
               GregorianCalendar c=new GregorianCalendar();
               switch(getNumberPeriod()){
                   case NUM_PERIOD_MONTH:
                       int m=c.get(Calendar.MONTH)+1;
                       regnum+="-"+m;
                       regnum+=".";
                       regnum+=c.get(Calendar.YEAR);
                       break;
                   case NUM_PERIOD_YEAR:
                       regnum+="-"+c.get(Calendar.YEAR);
               }
           }
           return regnum;
       }

   Ну незнал я про String.format (((..

   maxt, 01 Апреля 2010

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

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

   +102

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

   function TicksBetween(t1, t2 : dword) : dword;
   begin
   	if t2 > t1
   	then Result:=t2 - t1 + 1
   	else Result:=dword(-1) - t1 + t2 + 1;
   end;

   Никогда не пишите с бодуна!

   whiskey, 01 Апреля 2010

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

10. Pascal / Говнокод #2903

    +143.6

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

    for i:=1 to 10000000 do
    begin
    {...}
    if m=0 then break;
    end;

    kozzztya, 31 Марта 2010

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

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

• 2105
• …
• 286
• 285
• 284
• 283
• 282
• 281
• 280
• 279
• 278
• …
• 1