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

    −1

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    mId = core.Material.Materials.FirstOrDefault(x => x.Value.article == m.article && x.Value.title == m.title && x.Value.category_id == m.category_id).Key;
if (mId == 0)
	mId = core.Material.Materials.FirstOrDefault(x => x.Value.article == m.article && x.Value.title == m.title).Key;
if (mId == 0)
	mId = core.Material.Materials.FirstOrDefault(x => x.Value.article == m.article && x.Value.category_id == m.category_id).Key;
if (mId == 0)
	mId = core.Material.Materials.FirstOrDefault(x => x.Value.title == m.title && x.Value.category_id == m.category_id).Key;
if (mId == 0)
	mId = core.Material.Materials.FirstOrDefault(x => x.Value.article == m.article).Key;
if (mId == 0)
	mId = core.Material.Materials.FirstOrDefault(x => x.Value.title == m.title).Key;

    Есть 3 поля.
    Поиск сначала по 3. Затем по 2, затем по 2 и еще по 2. И от безысходности по 1
    Реально ли это оптимизировать?

    Запостил: yakov_255, 29 Января 2016

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

    • ava tucvbif 30.01.2016 00:18 # +2

      В любой непонятной ситуации строй граф.
      Ответить

    • ava bormand 30.01.2016 02:06 # 0

      А материалс это таблица в бд или тупо енумерейбл?
      Ответить

      • ava yakov_255 30.01.2016 11:23 # 0

        Енумерейбл (в Dictionary)
        Class Material {
        public string Title {get; set;}
        ....
        }
        Ответить

        • ava bormand 30.01.2016 11:46 # 0

          Хотел предложить что-то типа (считаем rank для каждого элемента и выбираем лучший):
          core.Material.Materials.Max(x =>
    (x.Value.article == m.article ? 1 : 0) +
    (x.Value.title == m.title ? 1 : 0) +
    (x.Value.category_id == m.category_id ? 1 : 0));
          Но оказалось, что Max вернёт тупо циферку, а не сам элемент... По-хорошему Max должен был юзать функцию только для сравнения, а возвращать исходный элемент... Но почему-то майкрософт сделал бесполезную херню, которая только для чисел работает. А аналог через Aggregate уже не няшен.
          Ответить

          • ava nihau 30.01.2016 21:26 # 0

            Недавно также факапнулся. Тоже думал что Max использует функцию как критерий. Сделал ордербайдесцендинг(функция).ферстордефол т()
            Ответить

            • ava bormand 31.01.2016 07:32 # 0

              Ну причём текущую версию Max можно выразить тупо через Select + Max. А ту, которую надо - невозможно никак. Только руками писать новый экстеншн метод или через Aggregate костылить.

              > ордербайдесцендинг(функция).ферстордефол т()
              А оно, случаем, не отсортирует весь список ради одного элемента?
              Ответить

              • ava nihau 31.01.2016 09:14 # 0

                пьюр, вернет ордереденамерабл. Так весь линк работает.
                Ответить

                • ava 1024-- 31.01.2016 12:06 # +2

                  > ордербайдесцендинг(функция).ферстордефолт()
                  > ордереденамерабл
                  Пипец, хоть бы цамелЦасе использовали что-ли.
                  Ответить

    • ava gost 31.01.2016 11:22 # 0

      >Есть 3 поля.
      Куда сам сядешь, куда мать положишь?
      Ответить

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