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

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

+17
1. 1
2. 2
3. 3
4. 4
5. 5
6. 6
7. 7
```
switch (impl->used_hash)
            {
            case false:
                break;
            case true :
                ..........
        }
```
Разбирал сырцы одного "гения" и нашел это...

Запостил: LuCiFer, 13 Мая 2014

Tweet
Комментарии (31) RSS
- guest 13.05.2014 15:44 # 0
  
  И что? Это самое показательное, что было в коде этого "гения"?
  Ответить
  - LuCiFer 13.05.2014 22:01 # 0
    
    там словами не передать... а сюда постить, я боюсь, что хостинг у владельцев упадет от объёмов и идиотизма кода. Это оттуда же http://govnokod.ru/9238
    Я если еще найду, что-то, что можно передать в разумных пределах - я здесь опубликую. Но такие кошмары, как классы созданные с помощью дефайнов - т.е. наборы методов где-то объявлены в с помощью
    #define something void do_smth_1(){} \
    void do_smth_2(){} \
    void do_smth_3(){}
    
    а затем много классов используют этот something выкладывать как-то даже страшно.
    Ответить
    - Abbath 14.05.2014 01:11 # 0
      
      Есть еще говнопрактика дефайнов с ретурнами внутри как кусков функций.
      Ответить
      
      bormand 14.05.2014 05:37 # 0
      
      Есть говно и покруче - макросы с использованием локальных переменных:
      #define ARR_ACS(a) (arr[MOD_L(a)]) //access to arr
      Взято отсюда: http://govnokod.ru/15967
      Ответить
- guest 13.05.2014 15:48 # −1
  
  Толсто.
  Ответить
- Smekalisty 13.05.2014 16:00 # +1
  
  Вероятно гений начитался на форумах, что switch работает быстрее пачки if-ов и начал везде "оптимизировать" код.
  Ответить
  - Vasiliy 13.05.2014 16:08 # 0
    
    Это где такое пишут. Я сколько помню пишут наоборот что пачка if побыстрей будет.
    Ответить
    - roman-kashitsyn 13.05.2014 16:23 # 0
      
      > пачка if побыстрей будет
      В C/C++/Java? Акстись. Не может такого быть.
      Ответить
      
      3.14159265 13.05.2014 16:30 # +1
      
      Я не стал бы говорить "невозможно"
      Смотря как писан свич, смотря как писаны ифы, смотря какой комилер.
      Может тупой компилятор не удалит это
      > case false:
      > break;
      И будет лишний переход в сравнении с простым:
      if (impl->used_hash){
      ....
      }
      Ответить
    - guest 13.05.2014 16:48 # 0
      
      Толсто.
      Ответить
- vistefan 13.05.2014 16:10 # 0
  
  Делал так, но у меня в кейсы были вложены ещё свищи и так на несколько уровней.
  Когда один -- пожалуй, грех. Но может семантически быть яснее такой код, ложиться в какую-то концепцию.
  Ответить
  - 1024-- 13.05.2014 17:33 # +1
    
    Концепция enum bool, не иначе
    Ответить
    - Vasiliy 13.05.2014 18:43 # +1
      
      Ожидается появление состояния maybe
      Ответить
- myaut 13.05.2014 18:26 # +3
  
  А где default?
  Ответить
  - Psionic 13.05.2014 18:30 # 0
    
    А третье дано? Хотя извратится можно - там аж целых 7 бит свободных.
    Ответить
    - bormand 13.05.2014 18:33 # 0
      
      Ну мы же не видим тип impl->used_hash... Вдруг там вообще int?
      Ответить
      
      LuCiFer 13.05.2014 22:06 # 0
      
      там boolean
      Ответить
      
      myaut 14.05.2014 09:33 # 0
      
      boolean - в Java
      Ответить
      
      bormand 14.05.2014 09:49 # +4
      
      typedef int boolean
      Ответить
      
      brutushafens 14.05.2014 10:38 # −4
      
      cleaned
      Ответить
      
      Psionic 14.05.2014 12:47 # 0
      
      Вот кстати навеяно - решил извратится
      http://ideone.com/osUYfn
      Результаты весьма странные.
      Не ну первый тернарник я еще понимаю, но свитч и прямое сравнение с единицой (true) - это странно.
      Ответить
      
      bormand 14.05.2014 13:01 # 0
      
      Хреновый какой-то тест... Ни 0 ни 1 не попали в выборку.
      
      Вот так лучше: http://ideone.com/5i40Ej
      Ответить
      
      Psionic 14.05.2014 14:15 # 0
      
      А не смущает что сравнение с единицей ячейки где не еденица возвращает true. Да и скачек на фолс если в ячейке что-то кроме нуля?
      Ответить
      
      bormand 14.05.2014 15:01 # 0
      
      Мой пример лучше тем, что на нем рассмотрены все важные случаи, а не тупо 5 рандомных.
      
      > А не смущает
      Нет, не смущает. Ибо запихивание чего-то в бул через жопу - это UB. А поведение сишки/крестов при UB'ах смущать не может по определению.
      
      3.9.1:6 Using a bool value in ways described by this International Standard as ‘‘undefined,’’ such as by examining the value of an uninitialized automatic variable, might cause it to behave as if is neither true nor false.
      Ответить
- guest 13.05.2014 21:22 # −1
  ░░░░░░░░░░░░░░░░▄▄▄▄▄▄▄▄▄▄▄░░░░░░░░░░░░░ ░░░░░░░░░▄▄▄▄█▀▀▀░░░░░░▄▄█████▄▄░░░░░░░░ ░░░░░░▄█████░░░░░░░░░░▀███████████▄░░░░░ ░░░░▄█▀▀███▄▄██▄░░▄▄▀███████████████▄░░░ ░░▄███▄█▄█████▀▀█████████████████████▄░░ ░▄███▄██████▀░░░░▀▀████████████████████░ ▄█████████▀░░█▄▄▄▄░░▀▀▀▀░░░▀▀▀▀████████▄ ████████▀░▄██▀▀████▀█░░░░▄▄████░░▀██████ ████████░░░░░░░░░░░▀▀░░░░█░░░░░░░░▀▀████ ████████░██▄░░░░░░░░░░░░░░█░░░░░▄▄░░████ ████████░▀█▄██▄▄▄░░▀░▀░░░░▀░░░░▄█░░▄███░ ████████░░▀███▄▄██▀▀▀▄▄▄▄▄▄▄▄▀▀░█░░████░ ▀███████░░░░▀█▀▀█████▄█▄▄▄▄▄█████░░███▀░ ░▀███████▄░░░░▀▀▄▄▄░░▀▀▀▀▀▀▀█▀▀▀█░▄███░░ ░░░██████░▀▄░░░░░▄█▀▀▀█▄▄▄▄▄▄▀▀▀▄░████▄░ ░░░▀██████░░▀▀▄▄░░░▀▀▀▀▄▄▄▄▄▄▄▄▀░░████▀░ ░░░░██████▄░░░░░▀▀▄▄▄░░░░░░░░░░░░█▀███▄░ ░░░░▀███▀██░░░░░░░░░░▀▀▀▄▄▄▄▄▄▄█▀░░▀░░▀░ ░░░░░█▀▀░▀█░░░░░░░░░░░░░░░░░░░░░░░░░░░░░
  Ответить
  - bormand 13.05.2014 21:28 # +4
    
    ☑ Выпилить ботов (список)
    
    P.S. Ты таки вынуждаешь меня довести скрипт до ума, спасибо за мотивацию ;)
    Ответить
    - guest 13.05.2014 21:44 # 0
      
      Пожалуйста ;) Вот вам еще, для стимула:
      
``````````````¶¶¶¶¶¶¶¶¶¶¶¶¶¶¶¶¶``````````````````` ```````````¶¶¶¶¶¶``````¶`¶¶¶¶¶¶¶¶¶¶``````````````` ````````¶¶¶¶¶```¶¶¶`````````````¶¶¶¶¶¶¶¶¶¶¶``````` ``````¶¶¶````¶¶¶¶¶¶¶¶¶¶````````¶¶¶```¶¶¶¶¶¶¶¶````` ````¶¶¶``¶¶¶¶¶¶¶¶``¶¶``¶¶¶¶¶¶¶¶`````¶¶¶¶¶¶¶¶¶¶¶``` ```¶¶¶`¶¶¶¶¶`¶¶¶¶¶¶¶¶¶¶``````````¶¶¶¶```¶¶¶¶¶````` ``¶¶¶¶¶¶¶¶``¶¶````````¶¶````````¶¶¶````¶¶¶¶`¶¶```` `¶¶````````¶¶``````````¶¶``````¶¶```````````¶¶```` `¶¶````````¶``¶¶¶¶`````¶¶``````¶¶``````````¶¶¶```` `¶```````¶¶¶`¶¶¶¶¶`````¶¶```````¶¶```````¶¶¶`¶¶``` ¶¶`````¶¶¶¶¶¶¶¶¶``````¶¶`````````¶¶¶¶¶¶¶¶¶¶¶¶¶¶``` ¶¶````````¶``¶¶¶¶¶¶¶¶¶¶````````````¶¶¶``````¶¶¶``` ¶¶``````````````¶¶¶¶¶``````````¶¶¶¶¶``¶¶¶¶¶¶``¶¶¶` ¶```````````````````````¶¶¶¶¶¶¶¶`¶¶``¶¶¶```¶¶``¶¶` ¶`````````````````````¶¶¶`¶`````¶¶¶`¶¶¶¶¶¶¶¶¶¶`¶¶` ¶`````````````````````¶¶¶¶¶¶¶¶``¶¶`¶¶¶¶¶¶¶¶¶¶¶`¶¶` ¶```````````````````````````````¶¶`¶¶````````¶`¶¶` ¶¶``````````````````````````````¶¶`¶¶```````¶¶`¶¶` ¶¶`¶¶```````````````````````````¶¶`¶¶```````¶¶¶¶`` `¶``¶``¶``¶``````````````````````¶`¶¶``````¶¶`¶¶`` `¶¶``¶`¶```¶`````````````````````¶`¶¶``````¶``¶``` ``¶¶``¶¶¶```¶¶`````¶¶¶```````````¶`¶¶`````¶¶``¶¶`` ``¶¶```¶¶````¶¶``````¶¶¶¶¶``````¶¶`¶¶``````¶``¶¶`` ```¶¶``¶¶¶````¶````````¶¶¶¶¶````¶¶`¶¶¶¶¶¶``¶¶``¶`` ```¶¶```¶¶¶```¶``````````¶¶¶```¶¶`¶¶````¶¶``¶``¶`` ````¶¶```¶¶¶``¶¶`````¶¶¶```¶¶``¶¶`¶¶``````¶¶¶``¶`` `````¶¶```¶¶¶```¶`````¶¶¶¶¶````¶``¶¶¶¶¶¶¶¶¶¶¶``¶`` ``````¶¶```¶¶¶````¶¶``````¶¶¶``¶``¶¶`¶¶¶`¶`¶¶``¶`` ```````¶¶```¶¶``¶```¶````````¶¶¶``¶¶¶``¶¶¶¶¶``¶¶`` ````````¶¶¶```¶¶`¶¶¶``````````¶¶¶``¶¶¶¶¶¶¶¶``¶¶``` ``````````¶¶¶`````¶¶¶¶¶````````¶¶¶````¶¶¶```¶¶¶``` ```````````¶¶¶¶¶`````¶¶¶¶¶¶¶¶¶¶`¶¶¶¶¶````¶¶¶¶````` ``````````````¶¶¶¶¶¶```````¶¶¶¶¶```¶¶¶¶¶¶¶¶``````` ``````````````````¶¶¶¶¶¶¶````````¶¶¶¶¶¶``````````` ```````````````````````¶¶¶¶¶¶¶¶¶¶¶¶```````````````
Ответить

bormand 13.05.2014 22:24 # 0

Готово (в версии 0.0.16).

Если на коммент бота нет ответов людей, то он выпиливается полностью.
Если есть - просто скрывается.
Ответить

guest 13.05.2014 22:30 # −2

У меня тоже готово: если человек - злодей, его сообщения выпиливаютя со всех страниц; если нет - тоже выпиливаются, чтобы другим неповдно было.
Ответить

1024-- 13.05.2014 22:51 # 0

http://rghost.ru/55137076.view
Ограничения ширины и так хватает. Что хорошо, при этом не теряется контекст и сообщения от неупоротых гостей.
Вот если ещё добавить код, который убирает форматирование и большой красный текст сделает маленьким чёрным, то можно удалять только повторяющиеся сообщения.
Ответить

Vindicar 14.05.2014 08:23 # 0

идея интересная, но смысл?
Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только в понедельник, среду, четверг или воскресение
Семь раз отмерь — один отрежь, guest!

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

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

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

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

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

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

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