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

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

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

1. PHP (5714)
2. C++ (2747)
3. Куча (2429)
4. JavaScript (2035)
5. C# (1931)
6. Java (1503)
7. Си (1123)
8. Pascal (649)
9. Python (594)
10. 1C (541)
11. SQL (433)
12. ActionScript (292)
13. bash (203)
14. Objective C (202)
15. Perl (194)
16. Assembler (148)
17. Ruby (145)
18. VisualBasic (138)
19. Lua (49)
20. Haskell (48)
21. Go (31)
22. Swift (27)
23. Kotlin (14)

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. Гости могут высказаться только во вторник, пятницу или субботу

   Семь раз отмерь — один отрежь, guest!

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

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

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