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

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

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

1. PHP (5710)
2. C++ (2742)
3. Куча (2393)
4. JavaScript (2034)
5. C# (1930)
6. Java (1503)
7. Си (1120)
8. Pascal (648)
9. Python (591)
10. 1C (540)
11. SQL (432)
12. ActionScript (292)
13. Objective C (202)
14. bash (200)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (43)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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

   +135

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

   if (TrebContext.WebSession.IsExpiredProperty == "true")
   {
        ...
   }

   Индусы такие индусы.
   bool? Не, не слышал.

   badstarosta, 06 Сентября 2012

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

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

   +134

   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

   public void Open()
           {
               _session.OpenSession();
   
               _log = new LogOperations(_session.CurrentSession);
               if (!_session.DBIsExists)
               {
                   var dictList = new List<string> 
                   {
                   #region Добавление всех текстовых ресурсов в список для записи в словарь базы
                       Resources.AllChannelsSelected,
                       Resources.ArchiveClearDone,
                       Resources.ARMShotdown,
                       Resources.BWConnected,
                       Resources.BWConnecting,
                       Resources.BWSelected,
                       Resources.BWServerName,
                       Resources.Cannot_create_0fly,
                       Resources.Cannot_create_1fly,
                       Resources.Cannot_delete_0fly,
                       Resources.Cannot_delete_1fly,
                       Resources.CannotDeleteImage,
                       Resources.CannotDeleteVideo,
                       Resources.CannotSaveImage,
                       Resources.ColorConnected,
                       Resources.ColorConnecting,
                       Resources.ColorSelected,
                       Resources.ColorServerName,
                       Resources.CoolingBreak,
                       Resources.CreateClientError,
                       Resources.FreeSpace20,
                       Resources.FreeSpace5,
                       Resources.ImageSend,
                       Resources.ImageSendASTK,
                       Resources.MainForm_OPUConnect,
                       Resources.MainForm_RecordStart,
                       Resources.MainForm_RecordStop,
                       Resources.OPUAngle,
                       Resources.OPUConnected,
                       Resources.OPUDisconnected,
                       Resources.OPUSavePoint,
                       Resources.OPUScanning,
                       Resources.OPUScanningNotChangedSet,
                       Resources.OPUWatching,
                       Resources.PanoNotExists,
                       Resources.Panorama_Processing,
                       Resources.Panorama_StreamError,
                       Resources.Panorama_Success,
                       Resources.PresetPointRewrite,
                       Resources.PresetPointSaved,
                       Resources.ReadDataError,
                       Resources.Restart,
                       Resources.RestartFailed,
                       Resources.SaveImage,
                       Resources.ScanSetRewrite,
                       Resources.ScanSetSave,
                       Resources.StartRecord,
                       Resources.StopRecord,
                       Resources.ThermalConnected,
                       Resources.ThermalConnecting,
                       Resources.ThermalCoolingTime,
                       Resources.ThermalOFF,
                       Resources.ThermalON,
                       Resources.ThermalReady,
                       Resources.ThermalSelected,
                       Resources.ThermalServerName,
                       Resources.ThermalWork,
                       Resources.UnknowError,
                       Resources.UnknowError2,
                       Resources.UserTypeCommander,
                       Resources.UserTypeDefault,
                       Resources.UserTypeOperator,
                       Resources.UserTypeSystemPreferences,
                       Resources.VideoArchivePlayText,
                       Resources.VideoSend,
                       Resources.VideoSendASTK
                   #endregion
                   };
                   _log.CreateDictionary(dictList);
               }
           }

   Метод, создающий словарь бд из ресурсов проекта

   Flyperformances, 06 Сентября 2012

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

3. C++ / Говнокод #11713

   +82

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

   bool isOperator( char c ) {
   	return c == '+'
   		|| c == '-'
   		|| c == '*'
   		|| c == '/'
   		|| c == '('
   		|| c == ')';
   }

   Fai, 05 Сентября 2012

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

4. PHP / Говнокод #11712

   +72

   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

   /*
    * Called from a large number of places
    * By Ofer
    */
   static function getOrderStatus( $po_numb='', $supplier='', $shipping='', $id=0,$customer_id=0,$order='',$order_asc='',$item_status_id=0,$order_status=0, $start_date='',$end_date='',
   		$name_begins_with = '', $phone = '', $email = '', $list_mgr_id = 0, $is_corporate = '', $sales_id=0,
   		$first_name = '', $last_name='', $city='', $state='', 
   		$zip='', $store_numb=0, $po_numb='', $release_date='',$supplier=0, 
   		$tracking_numb='', $payment_method='', $shipped_balance='', $avs='', $last4='', 
   		$brand_id=0, $model_numb='',$damage='', $delivery_issue='', $past_damage='', 
   		$coming_back='', $file_claim_ups='', $file_claim_frt='', $rewview='', $balance='', $shipper=0,
   		$has_balance = 0, $sales_account_id=0, $reference_number="", $third_party_order='', $ebay_id='', $trucker_id='',$get_total=false, 
   		$limit=0, $start=0, $use_dates=0) {

   Просто очаровательный комментарий!

   paulrudy, 05 Сентября 2012

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

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

   +157

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

   var ml = ctx.offset().left - $(window).width() / 2 + 35;
   
               if (ml > 230) {
                   ml = 185;
                   ml = ctx.offset().left - $(window).width() / 2 - 289 - 10
                   popup.find('.popup-angle').addClass('popup-angle-right');
               }

   Чудеса нумерологии

   SuperChel, 05 Сентября 2012

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

6. Си / Говнокод #11710

   +116

   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

   #define KRATNOST6_STEPE32_(VAL) \ 
     (                            \ 
     (\ 
     (VAL/32*32)+(((VAL/32)^1)*(\ 
       (VAL/16*16)+(((VAL/16)^1)*(\ 
         (VAL/8*8)+(((VAL/8)^1)*(\ 
           (VAL/4*4)+(((VAL/4)^1)*(\ 
             (VAL/2*2)\ 
           ))\ 
         ))\ 
       ))\ 
     ))\ 
     )\ 
     +\ 
     ((\ 
     (VAL/32)+(((VAL/32)^1)*(\ 
       (VAL/16)+(((VAL/16)^1)*(\ 
         (VAL/8)+(((VAL/8)^1)*(\ 
           (VAL/4)+(((VAL/4)^1)*(\ 
             (VAL/2)\ 
           ))\ 
         ))\ 
       ))\ 
     ))\ 
     )^1)*32\ 
     ) 
   #define KRATNOST6_STEPE32(MAX_STEPEN6,VAL) KRATNOST6_STEPE32_(((int)VAL&((1<<(MAX_STEPEN6+1))-1)))

   Вычисление ближайшей кратности к степ 2 в препроцессоре, такую байду сорудил. (не знаю правильно ли выразился)
   Тут максимальная 32. Если есть выше то вот тут выставляется 32 всёравно.
   
   Проще не варик?

   TarasB, 05 Сентября 2012

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

7. Python / Говнокод #11709

   −100

   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

   def profile_edit(request, template_name='profile/subdiler/profile_edit.html'):
       profile = request.profile
       
       if request.method == "POST":
           form = DilerProfileForm(request.POST, request.FILES, instance=profile)
           if form.is_valid():
               profile = form.save(commit=False)
               try:
                   profile.avatar = profile.avatar[0]
               except:
                   pass
               try:
                   profile.sign = profile.sign[0]
               except:
                   pass
               profile.city = form.city
               profile.save()

   Замечательная цепочка из try-cath
   Не менее замечательно то, что метод profile_edit полностью идентичный лежит в двух файлах subdiler_views.py и diler_views.py
   Разница между ними только в переменной template_name='profile/(subdiler|diler)/profile_edit.html'
   Для чего так сделано, мне нипанять.

   хуита, 05 Сентября 2012

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

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

   +162

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

   try {
                       /**
                        * Нахожу номер "начисления" в массиве "charges" По соответствию поля "adapter".
                        */
                       _.each(currentEmployee.charges, function (charge, num) {
                           if (_.isEqual(charge.adapter, currentCharge.adapter)) {
                               throw num;
                           }
                       }, this);
                   } catch (num) {тут какой то код}

   Самый оригинальный спомоб выйти из цикла
   Используется underscore.js

   nur, 05 Сентября 2012

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

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

   +49

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

   $em = $this->getDoctrine()->getEntityManager();
   $user = $em->getRepository('AdminBundle:AdminUser')
           ->findOneById($id);
   if ($user == $this->get('security.context')->getToken()->getUser()) {
           $this->get('session')->setFlash('admin-delete', 'TODO:TRANSLATE: Suicide is not allowed. Thank you!');
   } else {
           $em->remove($user);
           $em->flush();
           $this->get('session')->setFlash('admin-delete', 'TODO:TRANSLATE: User ' . $user->getEmail(). ' was deleted.');
   }

   Текст ошибок просто супер!

   EugeneC, 05 Сентября 2012

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

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

    +137

    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 MotionEventsController(VideoServer server, MSO2Rotator rotator, Log log, Button recordButton)
        {
            _server = server;
            _rotator = rotator;
            _log = log;
            _recordButton = recordButton;
        }
        public void Record(MotionDetectorFilter filter, MotionObjectsArgs movingObjects)
        {
            _server.StartRecord();
             IsRecording = true;
            _recordButton.BackColor = Color.Red;
            switch (_server.Name)
            {
                case ServerNames.ColorServer:
                    _log(Resources.ColorStartRecord, String.Empty);
                    break;
                case ServerNames.BwServer:
                    _log(Resources.BWStartRecord, String.Empty);
                    break;
                case ServerNames.ThermalServer:
                    _log(Resources.ThermalStartRecord, String.Empty);
                    break;
            }    
        }
    }

    Класс, принимающий кнопку записи в конструкторе.

    Flyperformances, 05 Сентября 2012

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

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

• 2113
• …
• 1056
• 1055
• 1054
• 1053
• 1052
• 1051
• 1050
• 1049
• 1048
• …
• 1