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

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

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

1. PHP (5698)
2. C++ (2733)
3. Куча (2326)
4. JavaScript (2027)
5. C# (1925)
6. Java (1501)
7. Си (1095)
8. Pascal (647)
9. Python (582)
10. 1C (531)
11. SQL (431)
12. ActionScript (292)
13. Objective C (202)
14. bash (197)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (29)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

1. PHP / Говнокод #22601

   −18

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

   abstract class CAllMain
   {
   ...
   	var $sDocPath2, $sDirPath, $sUriParam;
   	public $oAsset;
   
   ... 
   	var $buffer_man = false;

   Код ядра битрикс
   Кто сколько анитипатеррнов насчитал?

   Запостил: gorynych, 16 Марта 2017

   Tweet

   Комментарии (13) RSS

   • gostinho 16.03.2017 21:35 # −13

     показать все, что скрытоVanished

     Ответить

     • gostinho 16.03.2017 21:43 # −13

       показать все, что скрытоVanished

       Ответить

       • gorynych 16.03.2017 22:56 # +1

         Битриксофилы яростно минусуют, может быть из-за этого ваш содержательный ответ и пропал. Я бы ни за что такого не совершил.
         Ваш комментарий:
         
         Вот его содержание:
         1) комелкейс и ондерскор перемешаны
         2) var нинужон
         3) var и не var перемешаны
         И что-то наверняка с обстрактным классом связано, но я их не юзаю
         
         Все верно. С абстрактными классами именно в этом куске все впорядке, но если поковыряться, думаю найдется что-нибудь

         Ответить

         • 1024-- 16.03.2017 23:09 # +4

           > Битриксофилы яростно минусуют
           Давайте разберёмся.
           1. Для регистрации на этом сайте требуется ввести ровно одну капчу.
           2. Для минусования никогда не нужно вводить капчу.
           3. Имеется набор учётных записей с известным всем паролем 123, с которых пишут анонимусы.
           4. Комментарий пользователь может редактировать пять минут, для этого тоже не надо вводить капчу.
           5. Это сайт программистов, которые умеют автоматизировать отправку HTTP-запросов.
           
           Думаю, из пяти этих мартовских тезисов уже всё понятно. Проясним ситуацию.
           gostinho - общая учётка с известным паролем. Одни шутники автоматизировали отправку бредовых комментариев с подобных учёток, другие - затирание комментариев (скрипт/программа "ванишер") и минусование. Всё выполняется автоматически. Битриксофилы скорее всего дома чай с плюшками пьют; может, и поставили один из минусов данному говнокоду, не более того.

           Ответить

           • guest 16.03.2017 23:23 # −14

             показать все, что скрыто

             type
               TCommRec=record
                 UserName:string;
                 ANumber:string;
                 RNumber:string;
               end;
             ...
             
             for I:=Comms.Count -1 downto 0 do
                 begin
                   Comm:=TGComment(Comms.Objects[i]);
                   try
                     if (IsBadUser(Comm.Author)) and ((Comm.Votes > -5) and (abs(comm.Votes) <= x)) then
                     begin
                       if (SameText(Comm.VoteTime,'òîëüêî ÷òî')) and (not SameText(comm.Comment, 'vanished')) then
                       begin
                         NeedsVanish:=True;
                         Posts.AddObject(Comm.Number, TObject(CommRec));
                         Continue;
                       end
                       else
                       Posts.Add(Comm.Number);
                     end;

             Ответить

             • guest 16.03.2017 23:23 # −13

               показать все, что скрыто>> TObject(CommRec));

               Ответить

             • guestinho 17.03.2017 00:00 # +3

               Ахахах, сразу понятно кто автор)))))))

               Ответить

             • barop 17.03.2017 00:10 # +2

               Стертор, ты же решил уйти?

               Ответить

         • inkanus-gray 17.03.2017 12:10 # +2

           > 1) комелкейс и ондерскор перемешаны
           
           http://phpsadness.com/sad/4
           
           Even between similar/related functions, some use underscores between words, while others do not. Here are some examples:
           
           gettype
           get_class
           
           str_ireplace str_pad str_repeat str_replace str_shuffle str_split str_word_count
           strcasecmp strchr strcmp strcoll strcspn
           
           base64_encode quoted_printable_encode session_encode
           rawurlencode urlencode gzencode
           
           htmlentities
           html_entity_decode
           
           Антипаттерн позаимствован у стандартной библиотеки.

           Ответить

           • gost 18.03.2017 14:27 # 0

             > gettype
             > get_class
             Блять, как на этом говне можно писать???

             Ответить

             • barop 18.03.2017 14:46 # 0

               Так и не пишут на пхп те, кто знает другой язык

               Ответить

         • inkanus-gray 17.03.2017 12:18 # +2

           Мне не нравятся названия переменных.
           
           Кто такой Буферман? Если ему присвоено false, значит, это флажок. Тогда лучше его назвать is_buffer_свойство или isBufferСвойство, где свойство — это краткое название свойства, которым обладает буфер при установленном флажке.
           
           И что такое sDocPath2? Почему именно 2?

           Ответить

   • ASD_77 17.03.2017 01:54 # +3

     ну пхп - это один большой антипатерн :)

     Ответить

   Добавить комментарий

   Ошибка компиляции комментария:

   1. Гости могут высказаться только во вторник, пятницу или субботу

   Где здесь C++, guest?!

   А не использовать ли нам bbcode?

   • [b]жирный[/b] — жирный
   • [i]курсив[/i] — курсив
   • [u]подчеркнутый[/u] — подчеркнутый
   • [s]перечеркнутый[/s] — перечеркнутый
   • [blink]мигающий[/blink] — мигающий
   • [color=red]цвет[/color] — цвет (подробнее)
   • [size=20]размер[/size] — размер (подробнее)
   • [code=<language>]some code[/code] (подробнее)

   Проверочный код: *