1. C# / Говнокод #19867

    +4

    1. 1
    2. 2
    IEnumerable<SmartCurrencyPriceRule> rules = _smartRuleService.GetCurrencyPriceRules(id).OrderBy(r => r.Title).ThenBy(r => r.Updated).ThenBy(r => r.Created);
var oldPriceRule = rules.SingleOrDefault(r => r.RuleId == postData.RuleId) ?? new SmartCurrencyPriceRule();

    Давайте три раза отсортируем, а то мало ли что, а потом возьмём тот, у которого id совпадает.
    Вот что значит - любит человек порядок.

    Запостил: kerman, 23 Апреля 2016

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

    • ava kerman 23.04.2016 10:14 # 0

      Строго говоря, это разворачивается в одну сортировку, но всё равно...
      Ответить

    • ava bormand 23.04.2016 10:22 # +2

      Ну, может быть, там id не уникальное?
      Ответить

      • ava kerman 23.04.2016 22:36 # 0

        О_о
        это же айдишник. Если он не уникальный, тогда он не айдишник... Впрочем, ещё путает наименование, обычно ShitId означает ссылку на айди.
        Ответить

        • ava bormand 23.04.2016 22:41 # 0

          > Если он не уникальный, тогда он не айдишник
          Не факт. Если в базе констрейнта на уникальность нету - я бы не был так уверен...

          Ну и RuleId может быть просто ссылкой на primary key другой таблицы (в которой он таки уникален).
          Ответить

          • ava kerman 23.04.2016 23:27 # +1

            > Не факт. Если в базе констрейнта на уникальность нету - я бы не был так уверен...
            да при чём тут констрейнт? В уникальности глубинный смысл айдишника - по нему можно объект однозначно идентифицировать!

            >> путает наименование, обычно ShitId означает ссылку на айди
            > Ну и RuleId может быть просто ссылкой на primary key другой таблицы (в которой он таки уникален)
            Например CurrencyPriceRules... oh shi...
            Ответить

        • ava kegdan 23.04.2016 23:15 # +1

          Есть вариант что в таблице многие ко многим, и id там составной из 2 id других таблиц.

          Эх. хоть как то пытаюсь оправдать...
          Ответить

          • ava kerman 23.04.2016 23:32 # 0

            Да если уж даже получится придумать случай, когда с таким RuleId может быть несколько, остаётся восхищаться глубинному смыслу выбора первого объекта в алфавитном порядке с наименьшей датой изменения и наименьшей датой создания. Зачем?

            Because I can, that's why.
            Ответить

            • ava kegdan 23.04.2016 23:34 # 0

              Because I can LINQ ASS PARELLEL!!!
              Ответить

              • ava kerman 23.04.2016 23:46 # 0

                Точно. Я вот сейчас подумал, что можно было бы этот кусок ускорить за счёт Parallel LINQ.
                Ответить

            • ava bormand 23.04.2016 23:38 # 0

              > наименьшей датой изменения и наименьшей датой создания
              > oldPriceRule
              Что-то в этом есть...
              Ответить

              • ava kerman 23.04.2016 23:44 # 0

                Так в приоритете же сортировка по алфавиту.
                Ответить

                • ava kegdan 23.04.2016 23:48 # +1

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

                  Только вот он не учел, что линк выполняется во время обращения к переменной запроса для получения самых актуальных данных
                  Ответить

      • ava Irdis 25.04.2016 16:36 # +2

        >>> rules.SingleOrDefault(r => r.RuleId == postData.RuleId)
        к слову, если таких (в rules которые r.RuleId == postData.RuleId) больше одного, то эта конструкция начнет ругаться эксепшенами. Поэтому сортировка не имеет никакого смысла. Что бы пофиксить это используют FirstOrDefault.
        Ответить

        • ava kerman 25.04.2016 18:30 # 0

          Да, действительно. Впрочем, расследование показало, что данный ГК мог образоваться вследствии инлайна метода получения списка (с сортировкой) в метод, берущий единственное правило.
          Ответить

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