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

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

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

1. PHP (5706)
2. C++ (2739)
3. Куча (2372)
4. JavaScript (2031)
5. C# (1929)
6. Java (1502)
7. Си (1117)
8. Pascal (648)
9. Python (588)
10. 1C (539)
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. Haskell (29)
22. Swift (27)
23. Kotlin (14)

1. Лучший говнокод

   В номинации:

   • Лучший рейтинг
   • Самый комментируемый

   За время:

   • за сегодня
   • за неделю
   • за месяц
   • за всё время

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

   +159

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

   request.done(function(data){			 
   			 if (data){
                                           <...много кода с кучей вложенных if else ..>
   				    	noty({type: "success", text: 'Заказ взят с биржи'});
   				  	setTimeout(location.reload(), 2000);
   				  }
   		   	 } else if (data.error) {
   				 console.log(data);
   				 noty({type: "error", text: data.error["message"]});				
   			 }
   	});

   i had n1ce debug time

   heyzea1, 03 Декабря 2014

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

3. Си / Говнокод #17234

   +133

   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

   bool findImageToleranceIn(CTSInfo *info, bitmap *imageToFind, int32_t *x, int32_t *y, int32_t x1, int32_t y1, int32_t x2, int32_t y2, uint16_t tolerance)
   {
       int I, J, XX, YY;
       info->tol = tolerance;
       int dX = (x2 - x1) - (imageToFind->width - 1);
       int dY = (y2 - y1) - (imageToFind->height - 1);
       for (I = 0; I < dY; ++I)
       {
           for (J = 0; J < dX; ++J)
           {
               for (YY = 0; YY < imageToFind->height; ++YY)
               {
                   for (XX = 0; XX < imageToFind->width; ++XX)
                   {
                       rgb32* pixel = &imageToFind->pixels[YY * imageToFind->width + XX];
                       rgb32* targetPixel = &info->targetImage->pixels[(YY + I) * info->targetImage->width + (XX + J)];
                       if (pixel->a != 0)
                       {
                           if (!(*info->ctsFuncPtr)(info, pixel, targetPixel))
                           {
                               goto Skip;
                           }
                       }
                   }
               }
               *x = J + x1;
               *y = I + y1;
               return true;
               Skip:
               continue;
           }
       }
       *x = -1;
       *y = -1;
       return false;
   }

   В чём здесь сакральный смысл GoTo?

   Cynicrus, 01 Декабря 2014

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

4. C++ / Говнокод #17207

   +53

   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

   BOOL IsNameInExpression(PCWSTR Expression, PCWSTR Name, DWORD len)
   {
   	WCHAR c, b;
   
   	while (c = *Expression++) 
   	{
   		if (c == L'*') 
   		{
   __loop:
   			switch (*Expression)
   			{
   			case '*':
   				Expression++;
   				goto __loop;
   			case 0:
   				return TRUE;
   			}
   
   			if (len)
   			{
   				do 
   				{
   					if (IsNameInExpression(Expression, Name++, len--)) return TRUE;
   				} while (len);
   			}
   
   			return FALSE;
   		}
   
   		if (!len--)
   		{
   			return FALSE;
   		}
   
   		b = *Name++;
   
   		if (c != L'?' && b != c) 
   		{
   			return FALSE;
   		} 
   	}
   
   	return !len;
   }
   
   BOOL IsNameInExpression(PCWSTR Expression, PUNICODE_STRING Name)
   {
   	RtlUpcaseUnicodeString(Name, Name, FALSE);
   	return IsNameInExpression(Expression, Name->Buffer, Name->Length >> 1);
   }

   zhukas, 28 Ноября 2014

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

5. JavaScript / Говнокод #17196

   +160

   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

   var action = $(this).attr("action");
   var id_taxi = $('#id_taxi').val();
   var id_car = $('#id_car').val();
   var car_number = $('#number-car').val();
   var car_code = $('#number-car-code').val();
   <....>
   if ( action == "add" ){
       var url = '/api/4/*****/service/taxi/addAvto';
       var data_send={ 'id_taxi':id_taxi, 'car_number':car_number, 'model':model, 'color':color, 'licence_number': licence_number, 'id_type_car_body': id_type_car_body, 'type_car_comfort' : comfort, 'car_options' : option, 'isActive': is_active };
   
   }else if ( action == "edit"  ){
       var url = '/api/4/****/service/taxi/updateAvto';
       var data_send={ 'id_car': id_car, 'car_number':car_number, 'model':model, 'color':color, 'licence_number': licence_number,'id_type_car_body': id_type_car_body,  'type_car_comfort' : comfort, 'car_options' : option, 'isActive': is_active};
   
   }
   <...>
   var request = $.ajax({
     url: url,  
     type: "POST",
     dataType: 'json',
     data: data_send
   });

   Сабмитим формы.
   И так на всем проекте, 3к строк JS кода, которые подобным образом описывают ВСЕ формы на проекте.
   web service architecture, йобана

   heyzea1, 27 Ноября 2014

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

6. JavaScript / Говнокод #17174

   +156

   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

   var move_container_after = function(toMove, toAfter) {
   	  $(toMove).insertAfter(toAfter);
   };
   
   var move_container_before = function(toMove, toBefore) {
   	  $(toMove).insertBefore(toBefore);
   };
   
   move_container_before(".toggle.basic1-tog", ".basic1 .store-order-item-group-action-list");
   move_container_after(".toggle.basic1rn-tog", ".basic1rn .store-order-item-description");
   
   move_container_before(".toggle.basic3-tog", ".basic3 .store-order-item-group-action-list");
   move_container_after(".toggle.basic3rn-tog", ".basic3rn .store-order-item-description");
   
   move_container_before(".toggle.bundle1-tog", ".bundle1 .store-order-item-group-action-list");
   move_container_after(".toggle.bundle1rn-tog", ".bundle1rn .store-order-item-group-action-list");
   
   move_container_before(".toggle.bundle12-tog", ".bundle12 .store-order-item-group-action-list");
   move_container_after(".toggle.bundle12rn-tog", ".bundle12rn .store-order-item-group-action-list");
   
   move_container_before(".toggle.bundle3-tog", ".bundle3 .store-order-item-group-action-list");
   move_container_after(".toggle.bundle3rn-tog", ".bundle3rn .store-order-item-group-action-list");
   
   move_container_before(".toggle.bundle6-tog", ".bundle6 .store-order-item-group-action-list");
   move_container_after(".toggle.bundle6rn-tog", ".bundle6rn .store-order-item-group-action-list");
   
   move_container_before(".toggle.desktop1-tog", ".desktop1 .store-order-item-group-action-list");
   move_container_after(".toggle.desktop1rn-tog", ".desktop1rn .store-order-item-description");
   
   move_container_before(".toggle.desktop12-tog", ".desktop12 .store-order-item-group-action-list");
   move_container_after(".toggle.desktop12rn-tog", ".desktop12rn .store-order-item-description");
   
   move_container_before(".toggle.desktop3-tog", ".desktop3 .store-order-item-group-action-list");
   move_container_after(".toggle.desktop3rn-tog", ".desktop3rn .store-order-item-description");
   
   move_container_before(".toggle.desktop6-tog", ".desktop6 .store-order-item-group-action-list");
   move_container_after(".toggle.desktop6rn-tog", ".desktop6rn .store-order-item-description");
   
   move_container_before(".toggle.desktopbus12-tog", ".desktopbus12 .store-order-item-group-action-list");
   move_container_after(".toggle.desktopbus12rn-tog", ".desktopbus12rn .store-order-item-description");   
   
   move_container_before(".toggle.desktopbus6-tog", ".desktopbus6 .store-order-item-group-action-list");
   move_container_after(".toggle.desktopbus6rn-tog", ".desktopbus6rn .store-order-item-description");
   
   move_container_before(".toggle.premium1-tog", ".premium1 .store-order-item-group-action-list");
   move_container_after(".toggle.premium1rn-tog", ".premium1rn .store-order-item-description");
   
   move_container_before(".toggle.premium12-tog", ".premium12 .store-order-item-group-action-list");
   move_container_after(".toggle.premium12rn-tog", ".premium12rn .store-order-item-description");
   
   move_container_before(".toggle.premium3-tog", ".premium3 .store-order-item-group-action-list");
   move_container_after(".toggle.premium3rn-tog", ".premium3rn .store-order-item-description");
   
   move_container_before(".toggle.premium6-tog", ".premium6 .store-order-item-group-action-list");
   move_container_after(".toggle.premium6rn-tog", ".premium6rn .store-order-item-description");
   
   move_container_before(".toggle.premiumbus12-tog", ".premiumbus12 .store-order-item-group-action-list");
   move_container_after(".toggle.premiumbus12rn-tog", ".premiumbus12rn .store-order-item-description");
   
   move_container_before(".toggle.premiumbus6-tog", ".premiumbus6 .store-order-item-group-action-list");
   move_container_after(".toggle.premiumbus6rn-tog", ".premiumbus6rn .store-order-item-description");

   "Перенос" формы, для каждого ID, в HTML есть соответствующее колово форм подобного типа
   
   <div class="toggle basic1-tog">Auto Renew: Off
   <form method="POST" action="https://sites.fastspring.com/store/api/order">
   <input type="hidden" name="operation" value="update"/>
   <input type="hidden" name="destination" value="checkout"/>
   <input type="hidden" name="product_1_path" value="/basicsubscription1monthrn"/>
   <input type="hidden" name="product_2_path" value="/basicsubscription1month"/>
   <input type="hidden" name="product_2_quantity" value="0"/>
   <input type="submit" value="Turn On"/>
   </form>
   </div>
   <div class="toggle basic1rn-tog">Auto Renew: On
   <form method="POST" action="https://sites.fastspring.com/store/api/order">
   <input type="hidden" name="operation" value="update"/>
   <input type="hidden" name="destination" value="checkout"/>
   <input type="hidden" name="product_1_path" value="/basicsubscription1month"/>
   <input type="hidden" name="product_2_path" value="/basicsubscription1monthrn"/>
   <input type="hidden" name="product_2_quantity" value="0"/>
   <input type="submit" value="Turn Off"/>
   </form>
   </div>

   IRIDON, 25 Ноября 2014

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

7. Java / Говнокод #17170

   +74

   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

   if (convertView == null || convertView != null) {
                   if (Position2.get(position)) {
                       t.setVisibility(0);
                       b.setVisibility(4);
                       mlp.setMargins(15, 0, 60, 0);
                       y.setBackgroundColor(getResources().getColor(R.color.app_text_white));
                   } else {
                       mlp.setMargins(60, 0, 15, 0);
                       t.setVisibility(4);
                       b.setVisibility(0);
                       y.setBackgroundColor(getResources().getColor(R.color.app_text_white));                    
                   }
               } else {
                   view = convertView;
               }

   Индусы

   vinnizp, 25 Ноября 2014

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

8. JavaScript / Говнокод #17128

   +156

   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

   // ==UserScript==
   // @name AutoConfirm
   // @namespace test
   // @include http://[CENSORED]/reviewboard/*
   // @version 1
   // @grant none
   // ==/UserScript==
   
   setTimeout(
   function(){
   window.confirm = function(msg){return true;};
   document.getElementById("shipit-link").click();
   },
   2000
   );

   Ускорение code review

   kit, 18 Ноября 2014

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

9. PHP / Говнокод #16883

   +164

   1. 1
   2. 2

   $gender = ($gender == NCL::$MAN ? 
                     "сдал" : "сдала");

   saksmt, 17 Октября 2014

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

10. PHP / Говнокод #16859

    +156

    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

    function mydqs($s="") {
        global $dellink,$origqs;
        $dellink="";
        if (!$s) $s=$_SERVER["QUERY_STRING"];
        $key="qwertyuiopasdfghjklzxcvbnmQWERTYUIOPASDFGHJKLZXCVBNM1234567890 &=";
        $orig="qXbn BrZ7tiJz1Hva4mx0KCY6hfV&LGAejIO=TsdgF5E3uPwkp8NScyWlRQo2MD9U";
        $i=0;
        $len=5;
        $ret="";
        $s=urldecode($s);
        while ($i<strlen($s)) {
            $i2=0;
            $retchar=$s[$i];
            while ($i2<strlen($orig)) {
                if ($s[$i]==$orig[$i2]) {
                    $pos=$i2-$i%$len;
                    if ($pos<0) $pos+=strlen($orig);
                    $retchar=$key[$pos];
                }
                $i2++;
            }
            $i++;
            $ret.=$retchar;
        }
        $pcs=explode("&",$ret);
        foreach ($pcs as $k=>$v) {
            if ($v=="") continue;
            $vs=explode("=",$v);
            $vs[1]=str_replace("'","\\'",$vs[1]);
            @eval("global \$$vs[0];\$$vs[0]='$vs[1]';");
            if (substr($vs[0],0,1)!="_") $dellink.="&$vs[0]=$vs[1]";
        }
        $origqs=$ret;
        return $ret;
    }

    что делает этот метод?

    afwbkbc, 15 Октября 2014

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

11. PHP / Говнокод #16844

    +161

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

    if (in_array($str, array('1', '2', '3', '4', '5', '6', '7', '8', '9', '10', '11', '12'))) {
        return TRUE;
    } else {
        $this->error = "Значение поля 'Месяц' содержит некорректное значение";
        return FALSE;
    }

    AgentSIB, 13 Октября 2014

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

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

• 1971
• …
• 410
• 409
• 408
• 407
• 406
• 405
• 404
• 403
• 402
• …
• 1