LINQ to SQL - ліворуч приєднайтесь до кількох умов приєднання


148

У мене є такий SQL, який я намагаюся перекласти на LINQ:

SELECT f.value
FROM period as p 
LEFT OUTER JOIN facts AS f ON p.id = f.periodid AND f.otherid = 17
WHERE p.companyid = 100

Я бачив типову реалізацію лівого зовнішнього з'єднання (тобто. І into x from y in x.DefaultIfEmpty()т . Д.), Але не знаю, як запровадити іншу умову приєднання ( AND f.otherid = 17)

EDIT

Чому AND f.otherid = 17умова є частиною приєднання, а не в пункті WHERE? Тому що fможе не існувати для деяких рядків, і я все ще хочу, щоб ці рядки були включені. Якщо умова застосовується в пункті WHERE, після ПРИЄДНАЙТЕСЬ - тоді я не отримую потрібної мені поведінки.

На жаль це:

from p in context.Periods
join f in context.Facts on p.id equals f.periodid into fg
from fgi in fg.DefaultIfEmpty()
where p.companyid == 100 && fgi.otherid == 17
select f.value

начебто рівнозначно цьому:

SELECT f.value
FROM period as p 
LEFT OUTER JOIN facts AS f ON p.id = f.periodid 
WHERE p.companyid = 100 AND f.otherid = 17

що не зовсім те, що я після цього.


Солодке! Я шукав це деякий час, але не знав, як це зробити. Не знаєте, як додати теги до цієї відповіді. Ось критерії пошуку, які я використав: linq to sql фільтр підключення або з linq до sql, де стаття приєднання або з
Solburn

Відповіді:


243

Перед тим, як дзвонити, потрібно ввести свою умову приєднання DefaultIfEmpty(). Я просто використовую синтаксис методу розширення:

from p in context.Periods
join f in context.Facts on p.id equals f.periodid into fg
from fgi in fg.Where(f => f.otherid == 17).DefaultIfEmpty()
where p.companyid == 100
select f.value

Або ви можете використовувати підзапит:

from p in context.Periods
join f in context.Facts on p.id equals f.periodid into fg
from fgi in (from f in fg
             where f.otherid == 17
             select f).DefaultIfEmpty()
where p.companyid == 100
select f.value

1
Дякуємо за те, що поділилися .Where кваліфікувачем у заяві від .... defaultifempty. Я не знав, що ти можеш це зробити.
Френк Томас

28

це теж працює, ... якщо у вас є кілька приєднань стовпців

from p in context.Periods
join f in context.Facts 
on new {
    id = p.periodid,
    p.otherid
} equals new {
    f.id,
    f.otherid
} into fg
from fgi in fg.DefaultIfEmpty()
where p.companyid == 100
select f.value

12

Я знаю, що " трохи пізно ", але про всяк випадок, якщо комусь потрібно це зробити в синтаксисі методу LINQ ( саме тому я і знайшов цю публікацію спочатку ), це було б так:

var results = context.Periods
    .GroupJoin(
        context.Facts,
        period => period.id,
        fk => fk.periodid,
        (period, fact) => fact.Where(f => f.otherid == 17)
                              .Select(fact.Value)
                              .DefaultIfEmpty()
    )
    .Where(period.companyid==100)
    .SelectMany(fact=>fact).ToList();

2
Дуже корисно побачити версію лямбда!
Учень

2
.Select(fact.Value)має бути.Select(f => f.Value)
Петро Фельцман

5

Іншим правильним варіантом є розподіл з'єднань на декілька пропозицій LINQ таким чином:

public static IEnumerable<Announcementboard> GetSiteContent(string pageName, DateTime date)
{
    IEnumerable<Announcementboard> content = null;
    IEnumerable<Announcementboard> addMoreContent = null;
        try
        {
            content = from c in DB.Announcementboards
              // Can be displayed beginning on this date
              where c.Displayondate > date.AddDays(-1)
              // Doesn't Expire or Expires at future date
              && (c.Displaythrudate == null || c.Displaythrudate > date)
              // Content is NOT draft, and IS published
              && c.Isdraft == "N" && c.Publishedon != null
              orderby c.Sortorder ascending, c.Heading ascending
              select c;

            // Get the content specific to page names
            if (!string.IsNullOrEmpty(pageName))
            {
              addMoreContent = from c in content
                  join p in DB.Announceonpages on c.Announcementid equals p.Announcementid
                  join s in DB.Apppagenames on p.Apppagenameid equals s.Apppagenameid
                  where s.Apppageref.ToLower() == pageName.ToLower()
                  select c;
            }

            // Add the specified content using UNION
            content = content.Union(addMoreContent);

            // Exclude the duplicates using DISTINCT
            content = content.Distinct();

            return content;
        }
    catch (MyLovelyException ex)
    {
        // Add your exception handling here
        throw ex;
    }
}

Хіба це не буде повільніше, ніж виконувати всю операцію за один запит linq?
Умар Т.

@ umar-t, так, швидше за все, враховуючи це було більше восьми років тому, коли я це писав. Особисто я , як корельований підзапит постулированного Dahlbyk тут stackoverflow.com/a/1123051/212950
MAbraham1

1
"Союз" - це інша операція, ніж "перехресне з'єднання". Це як додавання проти множення.
Suncat2000

1
@ Suncat2000, дякую за виправлення. Щасливого Дня Подяки! 👪🦃🙏
MAbraham1

0

Можна записати за допомогою складеного ключа з'єднання. Також, якщо є необхідність вибрати властивості як з лівої, так і з правої сторін, LINQ може бути записаний як

var result = context.Periods
    .Where(p => p.companyid == 100)
    .GroupJoin(
        context.Facts,
        p => new {p.id, otherid = 17},
        f => new {id = f.periodid, f.otherid},
        (p, f) => new {p, f})
    .SelectMany(
        pf => pf.f.DefaultIfEmpty(),
        (pf, f) => new MyJoinEntity
        {
            Id = pf.p.id,
            Value = f.value,
            // and so on...
        });

-1

Мені здається, є користь при розгляді деяких переписувань вашого SQL-коду, перш ніж намагатися його перекласти.

Особисто я б написав такий запит як об'єднання (хоча я би уникав нулів повністю!):

SELECT f.value
  FROM period as p JOIN facts AS f ON p.id = f.periodid
WHERE p.companyid = 100
      AND f.otherid = 17
UNION
SELECT NULL AS value
  FROM period as p
WHERE p.companyid = 100
      AND NOT EXISTS ( 
                      SELECT * 
                        FROM facts AS f
                       WHERE p.id = f.periodid
                             AND f.otherid = 17
                     );

Тож я думаю, що я згоден з духом відповіді @ MAbraham1 (хоча їх код, здається, не пов'язаний із питанням).

Однак, схоже, запит чітко призначений для отримання результату єдиного стовпця, що містить повторювані рядки - дійсно дублювання нулів! Важко не прийти до висновку, що такий підхід є хибним.

Використовуючи наш веб-сайт, ви визнаєте, що прочитали та зрозуміли наші Політику щодо файлів cookie та Політику конфіденційності.
Licensed under cc by-sa 3.0 with attribution required.