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

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

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

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

+21

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

Комментарии (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

"людьми выбравшими для себя не свою профессию разработ"
- А куда же нам деться?
Ответить

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

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

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

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