Порядок Linq по десятичному полю сортируется как строка?

Question

Порядок Linq по десятичному полю сортируется как строка?

1

Я полностью озадачен и не могу найти что-либо в Интернете об этом, так что я должен делать что-то неправильно?

 _positionsRepo.GetAllTrades().OrderByDescending(x => x.TotalPLPercent).ToList();

TotalPLPercent - это десятичное поле. результат упорядочивается следующим образом:

Очевидно, это неправильно. Я проверил сортировку против другого поля, которое было двойным, и работало, как ожидалось. Что мне не хватает здесь о десятичных знаках в С#?

Я использую драйвер Mongo DB С#.

ijjo 16 июль 2014, в 06:52

Источник

0

Попробуйте добавить .ToList(). между GetAllTrades() и OrderByDescending() . Таким образом, список будет отсортирован в памяти, а не в базе данных (я подозреваю, что это как-то связано с этим).
Blorgbeard 16 июль 2014, в 04:25
0

Убедитесь, что TotalPLPercent не является строкой.
Usman Zafar 16 июль 2014, в 04:27
0

это абсолютно не строка ... но вроде как один, отсюда и путаница.
ijjo 16 июль 2014, в 04:30
0

Хорошо, похоже, MongoDB не имеет собственного десятичного типа. Вы уверены, что это не строка? Он может сериализоваться в десятичный C #, но в базе данных это может быть строка.
Blorgbeard 16 июль 2014, в 04:31
0

эй blorgbeard получил ответ. у меня было подозрение, что я пытаюсь это сделать, но не думаю, что это может быть причиной! сейчас работает нормально. но теперь я не получаю выигрыша в производительности от отложенного выполнения, потому что ToList получает ВСЮ коллекцию из БД, ПОТОМ сортирует. звучит как ошибка в драйвере monggoDB C #.
ijjo 16 июль 2014, в 04:32
0

да blorgbeard, это звучит правильно. Могу поспорить, что это строка в базе данных. похоже на ошибку.
ijjo 16 июль 2014, в 04:35

Показать ещё 4 комментария

Теги:

c#

linq

3 ответа

2

По-видимому, это проблема с MongoDB Driver for C#. Проверьте этот открытый вопрос здесь -

тип "System.Decimal" сохраняет как "String"

Как вы можете видеть, поскольку Decimal сохраняется как String, он, несомненно, будет сортироваться как строка. Я думаю, вы должны попробовать double вместо decimal пока они не исправит эту проблему.

brainless coder 16 июль 2014, в 02:20

0

Опасно использовать double для значений валюты - вы получаете ошибки округления с плавающей запятой.
Blorgbeard 16 июль 2014, в 04:46

1

Вы уверены, что тип x равен десятичному? Похоже, он делает строковый порядок. (Я бы поставил точку останова на "x.TotalPLPercent", а затем оценил тип x, чтобы дважды проверить его, как я думал.)

Harley Pebley 16 июль 2014, в 03:08

0

да, это определенно десятичное число. я думаю, что это ошибка в драйвере mongodb C # с десятичными числами. так как, возможно, Монго смотрит на это как на строку. проверьте комментарии на мой вопрос, решение есть. не идеально, но это работает.
ijjo 16 июль 2014, в 04:34

Ещё вопросы

Попробуйте добавить .ToList(). между GetAllTrades() и OrderByDescending() . Таким образом, список будет отсортирован в памяти, а не в базе данных (я подозреваю, что это как-то связано с этим).
Убедитесь, что TotalPLPercent не является строкой.
это абсолютно не строка ... но вроде как один, отсюда и путаница.
Хорошо, похоже, MongoDB не имеет собственного десятичного типа. Вы уверены, что это не строка? Он может сериализоваться в десятичный C #, но в базе данных это может быть строка.
эй blorgbeard получил ответ. у меня было подозрение, что я пытаюсь это сделать, но не думаю, что это может быть причиной! сейчас работает нормально. но теперь я не получаю выигрыша в производительности от отложенного выполнения, потому что ToList получает ВСЮ коллекцию из БД, ПОТОМ сортирует. звучит как ошибка в драйвере monggoDB C #.
да blorgbeard, это звучит правильно. Могу поспорить, что это строка в базе данных. похоже на ошибку.
Опасно использовать double для значений валюты - вы получаете ошибки округления с плавающей запятой.
да, это определенно десятичное число. я думаю, что это ошибка в драйвере mongodb C # с десятичными числами. так как, возможно, Монго смотрит на это как на строку. проверьте комментарии на мой вопрос, решение есть. не идеально, но это работает.

Blorgbeard · Accepted Answer · 2014-07-16T03-08-00.000Z

Драйвер С# Mongo DB сериализует decimal строку в виде строки (исходный код драйвера подтверждает это).

Он делает это, потому что нет типа BSON для десятичного числа - double не имеет такой же точности. К сожалению, это означает, что вы не можете сравнивать "десятичные" значения как числа.

Вы можете сортировать данные в памяти, например:

_positionsRepo.GetAllTrades()
   .ToList()
   .OrderByDescending(x => x.TotalPLPercent)
   .ToList();

Другим вариантом было бы хранить количество центов (или сотых долей) как длинное целое число. Затем вы можете сортировать их нормально, и вам просто нужно разделить на 100 (или 10 000), чтобы получить реальную ценность.

Благодарю. работает. но, похоже, не идеально подходит для производительности сейчас.
Ну, вы все равно извлекаете все строки, я не думаю, что это будет иметь слишком большое значение, если только ваш набор данных не очень большой.