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

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

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

1. PHP (5698)
2. C++ (2733)
3. Куча (2324)
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 (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++ / Говнокод #18542

   +1000

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

   uint16_t swap_bytes(uint16_t a){
     uint16_t tmp = *((uint8_t*)&a+1);
     *(((uint8_t*)&tmp)+1) = *((uint8_t*)&a);
     return tmp;
   }

   Запостил: radmir, 28 Июля 2015

   Tweet

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

   • inkanus-gray 28.07.2015 13:57 # 0

     Красиво.

     Ответить

   • guest 28.07.2015 14:04 # +4

     Будет очень медленно работать. Вместо работы с регистрами будет работа с памятью, да еще и побайтовая.

     Ответить

     • inkanus-gray 28.07.2015 14:33 # 0

       Без оптимизации — да.
       
       А с оптимизацией творятся чудеса. gcc и icc с ключом -O2 вставляют сдвиг, но при этом ещё работают с памятью:
       

       swap_bytes(unsigned short):
       	movl	%edi, %eax
       	shrw	$8, %ax
       	movw	%ax, -2(%rsp)
       	movb	%dil, -1(%rsp)
       	movzwl	-2(%rsp), %eax
       	ret

       
       
       Шланг вовсе обходится без работы с памятью:
       

       swap_bytes(unsigned short):                        # @swap_bytes(unsigned short)
       	rolw	$8, %di
       	movzwl	%di, %eax
       	retq

       
       
       P.S. Кто такой DIL?

       Ответить

       • bormand 29.07.2015 10:16 # +1

         > Кто такой DIL
         Как AL, только для DI. В x86_64 добавили.

         Ответить

         • inkanus-gray 29.07.2015 11:06 # 0

           Но при этом DIH, SIH, BPH и прочие с -H не добавили. Зато добавили ненужный SPL.

           Ответить

           • bormand 29.07.2015 11:21 # +2

             Добавишь DIH - начнут просить всякие DIHL, DIHH для старшей половины... Поэтому остановились на минимуме.

             Ответить

             • inkanus-gray 31.07.2015 22:49 # 0

               Инструкции для старшей половины регистра удобны тем, что позволяли обходиться без сдвига на 8.
               
               Если сделать инструкции для старшей половины 32- и 64-битных регистров, то также сдвиги на 16 и на 32 станут не нужны.
               
               А если вообще представить основные регистры как массив 8-, 16- и 32-битных чисел, то в некоторых случаях отпадёт надобность в непонятных расширениях типа SSE.

               Ответить

           • bormand 29.07.2015 11:22 # +1

             > ненужный SPL
             А может я стек не юзаю, и RSP для меня - регистр общего назначения?

             Ответить

   • guest 28.07.2015 20:40 # +3

     uint16_t swap_bytes(uint16_t a) {
     return (a << 8) | (a >> 8);
     }

     Ответить

   • bormand 29.07.2015 07:10 # 0

     Где здесь c++, radmir? А вот где:

     template <typename T>
     void swap_bytes(T & value)
     {
         char * p = reinterpret_cast<char*>(&value);
         std::reverse(p, p + sizeof(T));
     }

     Ответить

   • radmir 29.07.2015 11:24 # 0

     Его тут нет, просто категория с кодом на СИ почти пустая, решил загрузить сюда.

     Ответить

     • mittorn 01.08.2015 19:45 # 0

       А надо было пополнять коллекцию

       Ответить

       • radmir 02.08.2015 15:31 # 0

         будем работать над этим

         Ответить

         • kegdan 02.08.2015 16:30 # 0

           пиши говно на си во славу говнокода!

           Ответить

           • bormand 02.08.2015 17:25 # 0

             Каждый раз, когда ты пишешь говно на си, умирает котёнок.

             Ответить

             • kegdan 02.08.2015 17:45 # 0

               бородатый котенок

               Ответить

               • bormand 02.08.2015 17:57 # 0

                 Бородатый козлёнок.

                 Ответить

                 • kegdan 02.08.2015 18:24 # 0

                   умирает бородатый слоненок аллюзия на импотенцию

                   Ответить

                   • bormand 02.08.2015 18:27 # 0

                     Каждый раз, когда ты пишешь говно на си, умирает страус. А его труп пишет с++.

                     Ответить

                     • Vasiliy 02.08.2015 18:30 # 0

                       Коммент про Петросяна.

                       Ответить

                     • kegdan 02.08.2015 18:38 # 0

                       к сожалению поздно пить боржоми - С++ уже написан а труп проживет еще лет 15-20

                       Ответить

                     • kegdan 02.08.2015 18:54 # 0

                       http://lurkmore.so/images/thumb/d/d8/Shinji_ivan.jpg/800px-Shinji_ivan.jpg
                       
                       Знакомая ситуация

                       Ответить

   • Asen 02.08.2015 20:58 # 0

     По большому счету, нынче все операции с указателями по определению являются говнокодом ;)

     Ответить

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

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

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

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

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

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

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