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

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

+63.2
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
17. 17
18. 18
19. 19
```
bool FindActualQtyOfWeightedReceptacle
                                 (      std::map<double,double> &ReceptacleWeighings,
                                  const double                   ReceptacleId,
                                        double                  &Weight)
{
   Weight          = 0.0;
   bool recepFound = false;
   
   for (std::map<double,double>::iterator iter  = ReceptacleWeighings.begin();
                                          iter != ReceptacleWeighings.end() && !recepFound;
                                          iter++)
   {  if (iter->first == ReceptacleId)
      {  recepFound = true;
         Weight     = iter->second;
      }
   }

   return(recepFound);
}
```
Во Франции одного ведущего (!) С++ программиста с ироничной фамилией Паскаль попросили таки ознакомиться со стандартными контейнерами из библиотеки STL. В частности с std::map. В итоге из-под палки он выдал вот такой "код". Возникает как минимум два вопроса:
1. производительность?
2. и зачем так многа букаф?

Пардон, Жан-Люк, не обижайся. Наговнокодил...

Запостил: azabluda, 15 Марта 2010

Tweet
Комментарии (29) RSS
- Orfest 15.03.2010 10:35 # +3.8
  
  Senior девелоперы бывают и такими и это грустно
  Ответить
- guest 15.03.2010 10:42 # +4
  
  Меня больше всего умиляет, что для ID используется double. Если это его осознанное решение - к коду его больше пускать нельзя. Дать в руки Эксель и Визио, и пусть принимает великие архитектурные решения (кстати, все будут довольны).
  Ответить
  - azabluda 15.03.2010 10:50 # +1.8
    
    насчет double ID... Боюсь это было решение было принято задолго до него. Так что, неверно не стану приписывать этот грешок a cher ami.
    Ответить
  - guest 17.03.2010 03:46 # +2.2
    
    +1. На@#$%утся они с отладкой этого.
    Ответить
- guest 15.03.2010 18:42 # −14
  
  показать все, что скрытоГде здесь С++?
  Ответить
  - azabluda 15.03.2010 19:12 # +2.6
    
    странный вопрос. Ну std::map например.
    Ответить
    - xaionaro 15.03.2010 21:21 # −0.8
      
      А за что минусуете "azabluda" то?
      
      Всегда бесило, когда какой-то пост минусуют, а никаких аргументов не предоставят.
      Ответить
  - xaionaro 15.03.2010 21:23 # +1.4
    
    Мне уже начинает казаться, что некий guest просто пытается побить рекорд по минусовому рейтингу, и для этого постоянно повторяет этот вопрос везде где это только можно и нельзя.
    Ответить
  - azabluda 16.03.2010 07:20 # +1.2
    
    xaionaro озвучил мнение многих, и моё в том числе. Вопрос действительно повторяется кучу раз на сайте и зачастую совсем не к месту. Возможно даже это бот. Надеюсь достаточный аргумент?
    
    ps: и честно говоря, все равно не вижу смысла "беситься" по поводу минусов...
    Ответить
- guest 15.03.2010 20:35 # +8
  
  > Во Франции одного ведущего (!) С++ программиста с ироничной фамилией Паскаль
  
  Проклятие Паскаля
  Ответить
- guest 17.03.2010 03:45 # +2
  
  Так тщательно написано. Пока до тела цикла не дочитал я думал, что это не говнокод и собирался минусовать.
  Ответить
  - azabluda 17.03.2010 06:39 # +1.2
    
    ага, тщательно написанный говнокод :0)
    Ответить
  - guest 02.04.2010 08:38 # 0
    
    А чё ReceptacleWeighings не map const&, зато ReceptacleId - const double?
    Ответить
- guest 17.03.2010 15:06 # +3.6
  
  Где здесь Паскаль?
  Ответить
  - guest 17.03.2010 20:20 # +2.6
    
    [басом] Ну я Паскаль, чего тебе?
    Ответить
    - guest 18.03.2010 16:59 # +3.6
      
      Ты сегодня пол моешь
      Ответить
- guest 18.03.2010 10:46 # +1.2
  
  Помимо контейнеров еще бы и алгоримы изучали, так нет ведь, все велосипеды изобретают.
  Ответить
  - guest 19.03.2010 03:05 # +2.2
    
    В данном случае алгоритмы не нужны. Хватит и map::find().
    Ответить
    - guest 20.03.2010 00:20 # −1
      
      Ничего что числа double нельзя сравнивать на == ?
      Ответить
      
      azabluda 20.03.2010 06:24 # +1.8
      
      А хорошо бы по этому утвержению ссылочку на стандарт?
      
      Весьма верю, что после сложной операции с double, в значение могут закрасться "младшие биты", и тогда например
      myDouble != sqrt(myDouble)*sqrt(myDouble)
      В данной проге, судя по всему, сравниваются только ID, над которыми ни один даже самый отчаяный паскалист не станет так извращаться. Тобишь единственно, что с ними могло происходить, это серия присвоений. Или у тебя есть основания полагать, что результат не гарантирован даже в этом случае? В любом случае ссылка не помешает.
      Ответить
      
      guest 20.03.2010 10:33 # −2
      
      Так и есть. Тут ссылка на стандарт FPU.
      Ответить
      
      azabluda 20.03.2010 10:39 # +2
      
      сорри, брат, не вижу ссылки. Может сайт вырезает?
      Ответить
      
      guest 20.03.2010 10:33 # 0
      
      Только find для поиска использует operator<, так что это не страшно.
      Ответить
      
      azabluda 20.03.2010 10:42 # 0
      
      дык если биты похерились, то find тогда тоже не найдет
      Ответить
      
      crox 02.04.2010 16:37 # +2
      
      к чему этот спор, если автор всё равно ищет сравнением...
      Ответить
- crox 02.04.2010 16:36 # +2.2
  
  >> 2. и зачем так многа букаф?
  он же француз, они любят многабукаф. Ты просто половину букофф не читай и всё сразу станет нормально ))
  Ответить
- guest 05.04.2010 02:41 # −1
  
  оллоолоо!
  выучил чувак STL. первая double определена как const ибо у map ключ const.
  
  по поводу ==. map и set юзают понятие эквивалетности. иными словами должно быть
  
  if((! (iter->first < ReceptacleId) && ! (ReceptacleId < iter->first) )
  бла-бла;
  
  а вообще чувак жжот. нахрена он хранит Weight если ему нужно лишь выяснить наличие или отсутствие данного ID в map?
  Ответить
  - azabluda 05.04.2010 06:08 # +1
    
    Из данного говнокода однозначно видно, что чувак не только ищет i->first и возвращает i->second через переменную Weight.
    Ответить
    - guest 05.04.2010 13:41 # +2
      
      внатуре! хитер француз!
      не обратил внимание на ссылку
      Ответить
Добавить комментарий
Ошибка компиляции комментария:

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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