Говнокод #19530 — C++ — Говнокод.ру

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

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

C++ / Говнокод #19530

if ( LIKELY( _mode == normal ))

_mode задается один раз в начале программы по конфигурационному файлу.

Запостил:

govnokoderatata, 25 Февраля 2016

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

govnokoderatata 25.02.2016 12:06 # +1

Для полноты картины:
1) условие используется в цикле;
2) у _mode всего 2 различных значения
Ответить
- Elvenfighter 25.02.2016 20:47 # +3
  
  Ну и из данного кода видно, что _mode скорее всего будет "normal", по этому компилятор сгенерирует код с минимумов прыжков для такого варианта событий.
  
  В где говно?
  Ответить
  - govnokoderatata 26.02.2016 08:45 # 0
    
    Мысли следующие:
    1) как было сказано, _mode задается конфигом. Зачем так портить жизнь другому значению параметра конфига, отличного от normal? Что за дискриминация?
    2) раз уж пошла такая отимизационная пьянка, что мешает запилить два различных цикла - для одного значения параметра конфига, и для другого. И вынести это условие из цикла наверх?
    3) да и вообще, начать запиливать likely и при этом на каждой итерации цикла сравнивать одно и то же с одним и тем же (_mode никогда не меняется) выглядит как-то иронично, что ли...
    Ответить
    - xaionaro 09.04.2016 17:59 # 0
      
      Видимо автору было лень сопровождать два одинаковых куска кода, отличающейся только одной строчкой. Однако он знал, что в >99.99% случаев _mode == normal. Тогда почему бы не поставить likely?
      Ответить
      - govnokoderatata 20.05.2016 16:36 # 0
        
        Как он может об этом знать? В конфиг выносят параметры как раз для того, чтобы подстроиться к переменчивой среде без пересборки кода. Если сегодня normal чаще всего используется, кто сказал, что завтра его вообще не перестанут использовать, а будут использовать другое значение параметра?
        Ответить
        
        xaionaro 20.05.2016 17:52 # 0
        
        Полагаю, кроме normal, там ещё бывает а-ля «debug». И автору насрать тормозит или нет в debug-режиме, но не хочется, чтобы этот if вообще чувствовался в normal-режиме. Но это лишь догадка. Однако, пока это не опровергнуто, не вижу причин называть код говнокодом.
        Ответить
ponchic 25.02.2016 18:54 # −5

показать все, что скрытоНикогда не понимал этот блять С++, винапи в 100 раз проще
Ответить
- 4e1 25.02.2016 20:11 # +3
  
  эм ... где здесь С++, ponchic ?
  Ответить
- guest 09.04.2016 18:25 # +1
  
  признайся
  ты это просто так написал? от балды?
  ты ведь ни в С++ ни в винапи ничерта не поним
  Ответить
  - ponchic 12.04.2016 19:12 # 0
    
    В винапи чуть-чуть понимаю
    Ответить
    - guest 12.04.2016 19:58 # +1
      
      а на каком ЯП под winapi пишешь?
      Ответить
      - ponchic 18.04.2016 19:29 # 0
        
        C++
        Ответить
        
        bormand 18.04.2016 19:39 # +1
        
        Си-с-классами или с++?
        Ответить
        
        3_14dar 18.04.2016 19:45 # 0
        
        >Си-с-классами
        Что за зверь?
        Ответить
        
        bormand 18.04.2016 19:46 # +1
        
        Кресты в руках сишника, не осилившего RAII.
        Ответить
        
        3_14dar 18.04.2016 19:48 # 0
        
        Напомни, на крестах дрова/ядро уже можно писать?
        Ответить
        
        bormand 18.04.2016 19:49 # +1
        
        Идеологически - нет. Технически - да.
        
        З.Ы. Точно не помню, но вроде в макоси дрова на крестах.
        Ответить
        
        guesto 18.04.2016 22:09 # 0
        
        да, на сабсете C++
        точнее в XNU
        а еще точнее в mach:)
        
        IOKit
        Ответить
        
        bormand 18.04.2016 19:53 # 0
        
        Да, на крестах:
        C++ was chosen for several reasons. The C++ compiler is mature and the language provides support for system programming. In addition, there is already a large community of Macintosh (and BSD) developers with C++ experience.
        
        The restricted subset disallows certain features of C++, including
        - Exceptions
        - Multiple inheritance
        - Templates
        - Runtime type information (RTTI)
        Ответить
        
        3.14159265 18.04.2016 20:27 # +1
        
        > Multiple inheritance
        > Runtime type information (RTTI)
        Не нужно.
        > Exceptions
        Нужно в исключительных ситуациях ЕВПОЧЯ.
        Ответить
        
        bormand 18.04.2016 20:36 # 0
        
        Не пойму только за что шаблоны запретили.
        Ответить
        
        3.14159265 18.04.2016 20:38 # +1
        
        Долго компилируются, имеют свойство распидорашиваться в мегабайты.
        Хз. Шаблоны - это ж один из китов на которых стоят кресты.
        Ответить
        
        bormand 18.04.2016 20:40 # 0
        
        Ну это если их абузить и всякие MPL хуярить. А если простенький контейнер или смартпоинтер - ничё там не распидорашивает...
        Ответить
        
        3.14159265 18.04.2016 20:41 # 0
        
        >всякие MPL хуярить
        Лол, я как раз хотел сказать "чтоб буст в дрова не тащили".
        Да понятно что штука полезная. А разосрать и препроцессором можно.
        Ответить
        
        defecate-plusplus 18.04.2016 20:44 # 0
        
        имхо это ограничение на экспорт
        Ответить
        
        bormand 18.04.2016 20:45 # 0
        
        > ограничение на экспорт
        Ну можно же шаблоны юзать только для удобства реализации, а в интерфейсы не выставлять...
        Ответить
        
        guesto 18.04.2016 22:09 # 0
        
        https://en.wikipedia.org/wiki/I/O_Kit
        Ответить
        
        ponchic 19.04.2016 16:45 # 0
        
        C++
        Ответить

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

C++ / Говнокод #19530

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

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