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

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

PHP (5703)
C++ (2740)
Куча (2337)
JavaScript (2030)
C# (1929)
Java (1503)
Си (1103)
Pascal (649)
Python (585)
1C (536)
SQL (434)
ActionScript (294)
Objective C (204)
bash (200)
Perl (196)
Assembler (148)
Ruby (144)
VisualBasic (140)
Lua (51)
Go (32)
Swift (29)
Haskell (28)
Kotlin (15)

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

−16
1. 1
2. 2
3. 3
```
void setOn(const bool &on) {
    // ...
}
```
А вот чего только не встретишь в коде...

Запостил: glook, 29 Июня 2012

Tweet
Комментарии (18) RSS
- bormand 29.06.2012 13:11 # 0
  
  А еще есть bool on() const { ... } ?
  Ответить
  - defecate-plusplus 29.06.2012 13:26 # +3
    
    судя по стилю, должно быть const bool & getOn() const { ...
    Ответить
  - glook 29.06.2012 13:35 # +1
    
    И не только, есть ещё bool isOn() const { return on(); }
    Ответить
    - bormand 29.06.2012 13:57 # 0
      
      А класс называется On?
      Ответить
      
      roman-kashitsyn 29.06.2012 13:59 # 0
      
      class She { };
      Ответить
      
      glook 29.06.2012 14:05 # +1
      
      Класс носит незамысловатое название Guard. =)
      Ответить
      
      guest 29.06.2012 18:48 # +3
      
      class Polizei: virtual public Guard{ }; bool Fire = true; Polizei Paul; Paul.setOn(Fire);
      Ответить
      
      guest 30.06.2012 16:04 # −3
      
      Ааааааааааааааааа!!!!!!!!
      Ты меня зажёг, бой, я буду с тобой %))))))))
      Ответить
- TarasB 29.06.2012 18:55 # 0
  
  Может это дословный перевод с Ады. Там всё по умолчанию передаётся как const& (правда для boolean Ада это компилирет в передачу значения, а не указателя, а кресты -хз)
  Ответить
  - glook 29.06.2012 18:58 # 0
    
    Да если бы. Просто кое-кто так написал.
    Ответить
  - bormand 29.06.2012 19:12 # +1
    
    Уважающие себя крестокомпиляторы для таких примитивных случаев const & тоже превращают в передачу копии.
    
    Так что не так и ужасен этот const bool & (хотя, конечно, вне шаблонов это говно).
    Ответить
- Psionic 29.06.2012 19:17 # +1
  
  А нефиг тимлидам так активно мозги полоскать джуниорам, по поводу константных ссылок - вот и имеем такую дурную паранойю.
  Ответить
  - bormand 29.06.2012 19:43 # +6
    
    Плохие тимлиды значит, раз полоскание про константные ссылки до конца не довели.
    Ответить
- Fai 30.06.2012 06:56 # 0
  
  On& setOn( const On& on ) const;
  Ответить
  - Fai 30.06.2012 06:59 # 0
    
    enum On {
    on_off = 0;
    on_on = 1;
    on_err = 2;
    };
    Ответить
  - bormand 30.06.2012 07:21 # +2
    
    Эм. А как он будет set если он const?
    Ответить
    - defecate-plusplus 30.06.2012 10:35 # +1
      
      внутри член mutable, либо const_cast в теле
      собственно, раз это какой то guard, то смысл есть -быть атомарным даже при константном обращении
      Ответить
    - Fai 30.06.2012 23:13 # +1
      
      setOn( любая_хрень ) = on_off;
      
      Ну всё-таки уровень индуизма должен зашкаливать.
      Ответить
Добавить комментарий
Ошибка компиляции комментария:

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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