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

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

PHP (5708)
C++ (2740)
Куча (2389)
JavaScript (2034)
C# (1930)
Java (1503)
Си (1117)
Pascal (648)
Python (591)
1C (539)
SQL (432)
ActionScript (292)
Objective C (202)
bash (200)
Perl (194)
Assembler (148)
Ruby (145)
VisualBasic (138)
Lua (49)
Haskell (37)
Go (31)
Swift (27)
Kotlin (14)

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

+21
1. 01
2. 02
3. 03
4. 04
5. 05
6. 06
7. 07
8. 08
9. 09
10. 10
11. 11
12. 12
13. 13
14. 14
15. 15
16. 16
```
if(m_bitRate < DSS_LOW_RATE/1000000.)
{
    m_bitRate = DSS_LOW_RATE/1000000.;
}
else if(m_bitRate > DSS_HIGH_RATE/1000000.)
{
    m_bitRate = DSS_HIGH_RATE/1000000.;
}
else if(m_bitRate > (.5*(DSS_HIGH_RATE - DSS_LOW_RATE) + DSS_LOW_RATE)/1000000.)
{
    m_bitRate = DSS_HIGH_RATE/1000000.;
}
else
{
    m_bitRate = DSS_LOW_RATE/1000000.;
}
```
умиляет. ну хотя бы, упростить .5*(DSS_HIGH_RATE - DSS_LOW_RATE) + DSS_LOW_RATE) до (DSS_HIGH_RATE + DSS_LOW_RATE)/2 -- понятнее, да? тогда и if(m_bitRate > DSS_HIGH_RATE/1000000.) можно убрать за ненадобносью (ибо число которое больше чем максимальное из двух точно больше чем среднее из этих жвух), и самое верхнее if(m_bitRate < DSS_LOW_RATE/1000000.) можно убрать ибо в else у нас то же самое произойдет - и сразу как бы читаестя с ходу и понимается, не надо напрягаться.
Боже, как я люблю крупные проприетарные проекты написанные людьми выбравшими для себя не свою профессию разработчика софта...

Запостил: Pencioner, 11 Октября 2012

Tweet
Комментарии (16) RSS
- absolut 11.10.2012 16:47 # +6
  Как-то так, без лишних слов:
  
  const double mega = 1000000.; double low = DSS_LOW_RATE/mega; double high = DSS_HIGH_RATE/mega; double mid = (low + high)/2; m_bitRate = m_bitRate <= mid ? low : high;
  Ответить
- Steve_Brown 11.10.2012 18:04 # 0
  Хорошо бы числа можно было писать через пробел.
  
  const double MinimumValue = 0.000 015, MaximumValue = 1 000 000;
  Ответить
  - defecate-plusplus 11.10.2012 23:05 # +2
    
    const double min = 15.e-6, max=1.e+6;
    Ответить
  - eth0 12.10.2012 06:22 # 0
    
    Через апостроф лучше.
    Ответить
    - absolut 12.10.2012 09:46 # +5
      
      или сразу так:
      
      const double MinimumValue = (double)"пятнадцать миллионных";
      
      чтоб ещё с проверкой синтаксиса
      Ответить
    - Steve_Brown 12.10.2012 10:10 # 0
      
      double million_with_tail = 1,000,000.000,1;
      Ответить
      
      eth0 12.10.2012 12:24 # 0
      
      Как-то по-американски.
      Ответить
  - TarasB 12.10.2012 10:46 # +3
    
    Ada:
    Integer i = 1_000_000;
    Ответить
    - absolut 12.10.2012 10:49 # +1
      
      а 1_0_0_1_0_1 можно?
      Ответить
      
      TarasB 12.10.2012 15:41 # +1
      
      Кажется, да.
      Ответить
      
      guest 12.10.2012 16:27 # −1
      
      Фуууу.
      Ответить
      
      absolut 12.10.2012 16:40 # +2
      
      можно, не значит, что нужно
      Ответить
    - roman-kashitsyn 12.10.2012 11:26 # +4
      
      В жабе 1.7 ввели эту супер-полезную фичу, ни дня без неё.
      Ответить
      
      3.14159265 12.10.2012 15:39 # +4
      
      И 30 лет не прошло.
      Ответить
    - Pencioner 12.10.2012 13:13 # +1
      
      в руби так же можно, да
      Ответить
- guest 12.10.2012 22:45 # 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++ / Говнокод #11913

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

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