Re: Минутка WTF-8: не делайте так.
От: samius Япония http://sams-tricks.blogspot.com
Дата: 28.04.16 17:38
Оценка:
Здравствуйте, Sinix, Вы писали:

S>Подсказка: на это раз вопрос не про производительность.

  спойлер
Может быть речь о том, что цикл работал не по IEnumerable реализации, в то время как IEnumerable либо не была реализована, либо была реализована как-то по-другому?
Re: Минутка WTF-8: не делайте так.
От: rameel https://github.com/rsdn/CodeJam
Дата: 28.04.16 17:49
Оценка: 3 (1) +1
Здравствуйте, Sinix, Вы писали:

  Скрытый текст
Ну если речь идет не о производительности, и вопрос с подвохом, то случайно не сравнение для double/float имеется в виду? Ну там double.NaN, double.PositiveInfinite и т.п. ?
... << RSDN@Home 1.0.0 alpha 5 rev. 0>>
Re[2]: Минутка WTF-8: не делайте так.
От: rameel https://github.com/rsdn/CodeJam
Дата: 28.04.16 17:52
Оценка: +1 :)
Здравствуйте, rameel, Вы писали:

R>Ну если речь идет не о производительности, и вопрос с подвохом, то случайно не сравнение для double/float имеется в виду? Ну там double.NaN, double.PositiveInfinite и т.п. ?


Похоже был прав, увидел правки в проекте

ЗЫ. И в форуме библиотеки
... << RSDN@Home 1.0.0 alpha 5 rev. 0>>
Отредактировано 30.04.2016 22:37 VladD2 . Предыдущая версия .
Re[2]: Минутка WTF-8: не делайте так.
От: Sinix  
Дата: 28.04.16 17:56
Оценка:
Здравствуйте, rameel, Вы писали:

R>Ну если речь идет не о производительности, и вопрос с подвохом...


Бинго! Ну и раз уже очевидно — на каких ещё типах может вылезти похожий баг?

Ну и обещанный спойлер для ув -n1-
  спойлер:
Таки да, цикл который ищет максимальный элемент
Автор: AndrewVK
Дата: 28.04.16
.
Re[3]: Минутка WTF-8: не делайте так.
От: Jack128  
Дата: 28.04.16 18:03
Оценка: 81 (2) +2
Здравствуйте, Sinix, Вы писали:

S>Здравствуйте, rameel, Вы писали:


R>>Ну если речь идет не о производительности, и вопрос с подвохом...


S>Бинго! Ну и раз уже очевидно — на каких ещё типах может вылезти похожий баг?


  Скрытый текст
Nullable<XXX>, очевидно
Re[3]: Минутка WTF-8: не делайте так.
От: rameel https://github.com/rsdn/CodeJam
Дата: 28.04.16 18:07
Оценка: +1
Здравствуйте, Sinix, Вы писали:

S>Ну и раз уже очевидно — на каких ещё типах может вылезти похожий баг?


Да вроде все, ну разве что DateTime может быть с разными зонами.
... << RSDN@Home 1.0.0 alpha 5 rev. 0>>
Re[3]: Минутка WTF-8: не делайте так.
От: samius Япония http://sams-tricks.blogspot.com
Дата: 28.04.16 18:13
Оценка:
Здравствуйте, Sinix, Вы писали:

S>Здравствуйте, -n1l-, Вы писали:


N>>Почему просто не использовать Max?


S>Потому что это максимально упрощённый пример, в реальном коде был цикл.

Значит, не максималььно уппрощенный. Aggregate в нем явно избыточен.
Re[3]: Минутка WTF-8: не делайте так.
От: rameel https://github.com/rsdn/CodeJam
Дата: 28.04.16 18:26
Оценка:
Здравствуйте, Sinix, Вы писали:

S>Ну и раз уже очевидно — на каких ещё типах может вылезти похожий баг?


Не так очевидно, раз пропустили этот момент) Заставило задуматься
... << RSDN@Home 1.0.0 alpha 5 rev. 0>>
Подождите ...
Wait...
Пока на собственное сообщение не было ответов, его можно удалить.