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

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

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

1. PHP (5698)
2. C++ (2733)
3. Куча (2324)
4. JavaScript (2027)
5. C# (1924)
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 (196)
15. Perl (194)
16. Assembler (146)
17. Ruby (142)
18. VisualBasic (138)
19. Lua (49)
20. Go (28)
21. Swift (27)
22. Haskell (26)
23. Kotlin (13)

1. C++ / Говнокод #23679

   +1

   1. 1

   https://habrahabr.ru/post/347688/

   Ученые выяснили, что плюсы медленнее си.

   Запостил: g0cTb, 28 Января 2018

   Tweet

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

   • g0cTb 28.01.2018 15:40 # +3

     Тупые крестобляди каждый элемент очереди с приоритетами создают на куче отдельно. Ха-ха, кресты говно!
     
     

     template <class T> class priority_queue
     {
         struct node
         {   
             unsigned int m_weight;
             T m_data;
         };
     
         using node_ptr = std::unique_ptr<node>;
     
         std::size_t m_capacity;
         std::size_t m_size;
         std::unique_ptr<node_ptr[]> m_nodes;

     Ответить

     • Antervis 28.01.2018 17:57 # 0

       автор статьи же не крестоблядь, а сишник с претензией на базовые знания плюсов.

       Ответить

     • g0_1494089156986 28.01.2018 18:05 # −1

       Хуйчик-хуйчик.

       Ответить

     • Dummy00001 30.01.2018 15:54 # +3

       а это реальная проблема. я слышал обсуждения перевода встроенных проектов на кресты, и это был тихий ужас.
       
       если хочешь пользоватся крестами - то классы это обязательно. и наследование вынуждают кресты тоже использовать. и еще надо операторов перегруженых вхуячить. самое главное - все объекты выделять динамически с new. потом сделать драму из исключений, потому что мы их не поддерживаем.
       
       "какой говняный язык, не будем им пользоватся." - почти буквальная цитата.

       Ответить

       • bormand 30.01.2018 17:33 # +1

         А ещё STL в прошивку не влезает, а буст раздувает код.

         Ответить

         • Antervis 30.01.2018 22:30 # +3

           std::vector<bool> потребляет больше памяти, чем экономит из-за того, что раздувает прошивку!

           Ответить

           • bormand 02.02.2018 07:08 # +1

             Кресты, стандарты, комитет

             Ответить

       • SemaReal 03.03.2018 06:08 # 0

         >> самое главное - все объекты выделять динамически с new.
         хм, это точно сишник в кресты пришел а не джавист или пхпшник?:)
         
         а вы кстати знаете, что в джаву хотят завести объекты, живущие на стеке?

         Ответить

         • Dummy00001 22.03.2018 17:08 # 0

           > хм, это точно сишник в кресты пришел а не джавист или пхпшник?:)
           
           ходят кучи книжек/этц как раз рекламирующие кресты в встроенщине.
           
           с другой стороны, какие зеленые выпускники каких курсов скорее все ничего кроме такого стиля и не видели.
           
           > а вы кстати знаете, что в джаву хотят завести объекты, живущие на стеке?
           
           лучше 20 лет поздно - чем никогда.

           Ответить

           • SemaReal 23.03.2018 03:01 # +1

             >>лучше 20 лет поздно - чем никогда.
             Теперь жабоёбы будут ломать голову над вопросом: "а где же мне разместить этот объект"?

             Ответить

             • Dummy00001 23.03.2018 12:20 # +1

               шо?? ... т.е. кампилер сам догадыватся не будет? ... ну может еще через 20 лет.

               Ответить

               • SemaReal 23.03.2018 13:28 # 0

                 Так сам он давно догадываеца, завезти хотят value types, когда ты будешь не ео указателю работать

                 Ответить

                 • Dummy00001 23.03.2018 14:14 # 0

                   > Так сам он давно догадываеца
                   
                   ... а блин, может быть. я думал просто вечный дроч со стороковыми операциями, что бы в гц лишнего не сорить. а строки то там (как и крестовый стринг) динамические их на стек не положишь. или в жабу какую shortstring завезли уже?

                   Ответить

                   • SemaReal 23.03.2018 15:55 # 0

                     >>динамические
                     Строки в жабе, если я верно помню, это честная обёртка вокруг массива char.
                     Массивы в джаве имеют информацию о размере (они не null terminated, там физически есть инфа).
                     
                     Так что теоретически строку тоже можно сложить на стек, но это implementation details, программист про это думать не должен наверное

                     Ответить

                     • bormand 23.03.2018 16:01 # 0

                       Они же иммутабельные... В итоге любой алгоритм который мало-мальски юзает строки порвёт тебе стек.

                       Ответить

                       • SemaReal 23.03.2018 16:06 # 0

                         если он склеивает строки и если у меня нет хвостовой рекурсии то наверное порвет.
                         
                         Но это вообще не вопрос строк: это вопрос стека и кучи, имхо. Если я знаю что нечто будет расти до неебических размеров то я ложу это на кучу, даже в няшненькой.
                         
                         А вот
                         

                         void foo() {
                         String pitux =  "pituh" + Integer.toString(getCurrentId());
                         System.out.print(pitux);
                         }

                         
                         тут-то зачем pitux на кучу класть?

                         Ответить

                         • bormand 23.03.2018 16:15 # 0

                           Ну тут оптимизнуть можно только если полностью заинлайнится Integer.toString(). Иначе откуда конпелятору знать, что там десяток символов а не мегабайт?

                           Ответить

                           • SemaReal 23.03.2018 16:19 # 0

                             У копелятора может быть какой-то интринсик о том сколько чимволов может быть в десятичном представлении unsigned 32 битного Integer , не?
                             
                             Но вообще я фантазирую, конечно: надо читать сырцы jdk, а мне лень.
                             
                             В принципе это известный факт что иногда некоторые объекты jit может покласть на стек если сможет доказать что ссыль на них никуда не утечет.Делает-лион это для примера выше я хз

                             Ответить

                             • bormand 23.03.2018 16:24 # 0

                               Кстати, интересно как вообще реализован Integer.toString(). Через стрингбилдер поди, чтобы сырую сишную функцию не дёргать. Или вообще лезет напрямую в кишки "иммутабельного" стринга...

                               Ответить

                               • SemaReal 23.03.2018 16:32 # 0

                                 http://hg.openjdk.java.net/jdk8/jdk8/jdk/file/687fd7c7986d/src/share/classes/java/lang/Integer.java
                                 
                                 >> char buf[] = new char[33];
                                 >> return new String(buf, charPos, (33 - charPos));

                                 Ответить

                                 • bormand 24.03.2018 06:56 # 0

                                   > 33
                                   Код написан сишником, который решил оставить место под нолик?
                                   
                                   З.Ы. А, тьфу, это под знак.

                                   Ответить

                       • 1024-- 23.03.2018 17:38 # 0

                         Вот в JS за счёт иммутабельности на словах и сокрытия от программиста всякой ненужной для решения задачи внутренней питушни вроде места, где хранится строка, а ещё за счёт JIT создался вычислительный рай как в функциональных языках, где ты пишешь то, что тебе надо для решения задачи, а оно внутри оптимизирует и раскладывает что нужно куда нужно.
                         

                         for(var i=0, s=''; i<N; i++, s += 'a') // ко-ко-ко, s иммутабельная, O(N^2)
                         // на самом деле спокойно может быть O(N)

                         
                         И это хорошо, что в местах, где сейчас нужна производительность, был впилен такой язык (тормозной по определению, с низким порогом вхождения). За счёт этого мы теперь имеем отличные движки с хитрожопейшими системами оптимизации опередившими своё время и внутренним компилятором, который из исходников готовит рабочий код за доли секунды (привет шаблонам C++).
                         Наконец с программой пердолится и выжимает все соки не человек, а другая программа. Никто из авторов движков не надеется, что на JS напишут хороший быстрый код, и не нагружают программиста деталями реализации.
                         А если позволять пользователю раскладывать переменные туда, куда он хочет их положить, придётся либо попрощаться с пирфомансом, либо мириться со временем и стоимостью разработки, ибо то же самое, что напишет на JS неумеха оператор шаблонизатора за час, дорогой опытный сишный оператор памяти выдаст за 100500 часов, а результат тот же.
                         Если уж такое делать, то стоит оставлять это как директивы компилятора (потом, когда все начнут писать на всякий случай какие-нибудь традиционные или устаревшие директивы, не слушать их по умолчанию и добавить второй комплект, третий, чётвёртый - как в GCC с inline), и чтобы без них всё работало автоматически.

                         Ответить

     • g0cTb 30.01.2018 22:42 # +4

       А внутри каждого элемента там инт и еще указатель. И этими руками он запускает кэшгринд.

       Ответить

       • bormand 31.01.2018 00:17 # +3

         Как всё-таки быстро сишники расслабляются увидев язык высокого уровня...

         Ответить

       • SemaReal 23.03.2018 16:15 # 0

         совсем не Data-Oriented Design, лол

         Ответить

   • bormand 28.01.2018 20:37 # 0

     > плюсы медленнее си
     Ну в общем-то логично. Плюсы - язык более высокого уровня.

     Ответить

     • CaptainPlastid 23.02.2018 19:38 # 0

       Си++глюки)

       Ответить

   • roman-kashitsyn 28.01.2018 21:08 # +1

     влепил пинус.

     Ответить

     • bormand 28.01.2018 21:09 # 0

       За хабру?

       Ответить

       • Soul_re@ver 28.01.2018 21:10 # +3

         На хабре?

         Ответить

         • roman-kashitsyn 28.01.2018 22:03 # +3

           > На хабре?
           
           разумеется

           Ответить

           • g0_1494089156986 29.01.2018 11:57 # −1

             Чух, чух! Чух, чух! Трах-потрах! Трах-потрах!

             Ответить

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

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

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

   Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

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

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

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