Struktura jednostki opuściła złączenie

84

Jak zmienić to zapytanie, aby zwracało wszystkie u.usergroups?

from u in usergroups
from p in u.UsergroupPrices
select new UsergroupPricesList
{
UsergroupID = u.UsergroupID,
UsergroupName = u.UsergroupName,
Price = p.Price
};
Lasse Edsvik
źródło
1
może to pomoże. to było na innym pytaniu tutaj w SO
Menahem

Odpowiedzi:

135

zaadaptowane z MSDN, jak opuścić złączenie przy użyciu EF 4

var query = from u in usergroups
            join p in UsergroupPrices on u.UsergroupID equals p.UsergroupID into gj
            from x in gj.DefaultIfEmpty()
            select new { 
                UsergroupID = u.UsergroupID,
                UsergroupName = u.UsergroupName,
                Price = (x == null ? String.Empty : x.Price) 
            };
Menahem
źródło
2
Podoba mi się to lepiej niż gdzie gj.DefaultIfEmpty () na końcu, ponieważ mogę użyć x w miejscu lub wybrać!
Gary
1
Czy możesz wyjaśnić wiersz „from x in gj.DefaultIfEmpty ()”?
Alex Dresko
@AlexDresko ta część pobiera wszystkie wyniki ze złączenia, a dla tych, które nie mają wartości po prawej stronie, daje null (domyślnie obiekt ma wartość null). hth
Menahem
2
A jeśli jest więcej niż dwie tabele?
MohammadHossein R
1
Zmieniło się to nieznacznie z efcore; from x in gj.DefaultIfEmpty()staje się from p in gj.DefaultIfEmpty(). docs.microsoft.com/en-us/ef/core/querying/…
carlin.scott
33

Może to być trochę przesada, ale napisałem metodę rozszerzającą, więc możesz to zrobić LeftJoinużywając Joinskładni (przynajmniej w notacji wywołania metody):

persons.LeftJoin(
    phoneNumbers,
    person => person.Id,
    phoneNumber => phoneNumber.PersonId,
    (person, phoneNumber) => new
        {
            Person = person,
            PhoneNumber = phoneNumber?.Number
        }
);

Mój kod nie robi nic więcej niż dodanie GroupJoini SelectManywywołanie do bieżącego drzewa wyrażeń. Niemniej jednak wygląda to na dość skomplikowane, ponieważ muszę samodzielnie zbudować wyrażenia i zmodyfikować drzewo wyrażeń określone przez użytkownika w resultSelectorparametrze, aby całe drzewo było możliwe do przetłumaczenia przez LINQ-to-Entities.

public static class LeftJoinExtension
{
    public static IQueryable<TResult> LeftJoin<TOuter, TInner, TKey, TResult>(
        this IQueryable<TOuter> outer,
        IQueryable<TInner> inner,
        Expression<Func<TOuter, TKey>> outerKeySelector,
        Expression<Func<TInner, TKey>> innerKeySelector,
        Expression<Func<TOuter, TInner, TResult>> resultSelector)
    {
        MethodInfo groupJoin = typeof (Queryable).GetMethods()
                                                 .Single(m => m.ToString() == "System.Linq.IQueryable`1[TResult] GroupJoin[TOuter,TInner,TKey,TResult](System.Linq.IQueryable`1[TOuter], System.Collections.Generic.IEnumerable`1[TInner], System.Linq.Expressions.Expression`1[System.Func`2[TOuter,TKey]], System.Linq.Expressions.Expression`1[System.Func`2[TInner,TKey]], System.Linq.Expressions.Expression`1[System.Func`3[TOuter,System.Collections.Generic.IEnumerable`1[TInner],TResult]])")
                                                 .MakeGenericMethod(typeof (TOuter), typeof (TInner), typeof (TKey), typeof (LeftJoinIntermediate<TOuter, TInner>));
        MethodInfo selectMany = typeof (Queryable).GetMethods()
                                                  .Single(m => m.ToString() == "System.Linq.IQueryable`1[TResult] SelectMany[TSource,TCollection,TResult](System.Linq.IQueryable`1[TSource], System.Linq.Expressions.Expression`1[System.Func`2[TSource,System.Collections.Generic.IEnumerable`1[TCollection]]], System.Linq.Expressions.Expression`1[System.Func`3[TSource,TCollection,TResult]])")
                                                  .MakeGenericMethod(typeof (LeftJoinIntermediate<TOuter, TInner>), typeof (TInner), typeof (TResult));

        var groupJoinResultSelector = (Expression<Func<TOuter, IEnumerable<TInner>, LeftJoinIntermediate<TOuter, TInner>>>)
                                      ((oneOuter, manyInners) => new LeftJoinIntermediate<TOuter, TInner> {OneOuter = oneOuter, ManyInners = manyInners});

        MethodCallExpression exprGroupJoin = Expression.Call(groupJoin, outer.Expression, inner.Expression, outerKeySelector, innerKeySelector, groupJoinResultSelector);

        var selectManyCollectionSelector = (Expression<Func<LeftJoinIntermediate<TOuter, TInner>, IEnumerable<TInner>>>)
                                           (t => t.ManyInners.DefaultIfEmpty());

        ParameterExpression paramUser = resultSelector.Parameters.First();

        ParameterExpression paramNew = Expression.Parameter(typeof (LeftJoinIntermediate<TOuter, TInner>), "t");
        MemberExpression propExpr = Expression.Property(paramNew, "OneOuter");

        LambdaExpression selectManyResultSelector = Expression.Lambda(new Replacer(paramUser, propExpr).Visit(resultSelector.Body), paramNew, resultSelector.Parameters.Skip(1).First());

        MethodCallExpression exprSelectMany = Expression.Call(selectMany, exprGroupJoin, selectManyCollectionSelector, selectManyResultSelector);

        return outer.Provider.CreateQuery<TResult>(exprSelectMany);
    }

    private class LeftJoinIntermediate<TOuter, TInner>
    {
        public TOuter OneOuter { get; set; }
        public IEnumerable<TInner> ManyInners { get; set; }
    }

    private class Replacer : ExpressionVisitor
    {
        private readonly ParameterExpression _oldParam;
        private readonly Expression _replacement;

        public Replacer(ParameterExpression oldParam, Expression replacement)
        {
            _oldParam = oldParam;
            _replacement = replacement;
        }

        public override Expression Visit(Expression exp)
        {
            if (exp == _oldParam)
            {
                return _replacement;
            }

            return base.Visit(exp);
        }
    }
}
fero
źródło
2
Dzięki za to rozszerzenie fero.
Fergers
To wciąż jest świetne. Dzięki!
TheGeekYouNeed
1
Przetestowano to w .NET Framework 4.6.2 i działa zgodnie z oczekiwaniami (tj. Generuje LEFT OUTER JOIN). Zastanawiam się jednak, czy to działa na .NET Core. Dzięki.
Aleksiej
24

Ułatw sobie życie (nie używaj dołączania do grupy):

var query = from ug in UserGroups
            from ugp in UserGroupPrices.Where(x => x.UserGroupId == ug.Id).DefaultIfEmpty()
            select new 
            { 
                UserGroupID = ug.UserGroupID,
                UserGroupName = ug.UserGroupName,
                Price = ugp != null ? ugp.Price : 0 //this is to handle nulls as even when Price is non-nullable prop it may come as null from SQL (result of Left Outer Join)
            };
Tomasz Skomra
źródło
2
Unikanie przyłączania się do grupy to kwestia opinii, ale z pewnością jest to ważna opinia. Price = ugp.Pricemoże się nie powieść, jeśli nie Pricejest właściwością dopuszczającą wartość null, a lewe sprzężenie nie daje jednak żadnych wyników.
1
Zgadzam się z powyższym, ale przy więcej niż dwóch tabelach to podejście jest o wiele łatwiejsze do odczytania i utrzymania.
Tomasz Skomra
1
Możemy sprawdzić, czy ugp == NULLi ustawić domyślną wartość dla Price.
Hp93,
po prostu idealne :)
MohammadHossein R
1
Niesamowite! Wolę to rozwiązanie ze względu na czytelność. Ponadto znacznie ułatwia to więcej połączeń (tj. Z 3 lub więcej tabel)! Użyłem go z powodzeniem dla 2 lewych złączeń (tj. 3 tabel).
Jeremy Morren
4

Jeśli wolisz notację wywołania metody, możesz wymusić lewe złączenie, używając w SelectManypołączeniu z DefaultIfEmpty. Przynajmniej na Entity Framework 6 z SQL Server. Na przykład:

using(var ctx = new MyDatabaseContext())
{
    var data = ctx
    .MyTable1
    .SelectMany(a => ctx.MyTable2
      .Where(b => b.Id2 == a.Id1)
      .DefaultIfEmpty()
      .Select(b => new
      {
        a.Id1,
        a.Col1,
        Col2 = b == null ? (int?) null : b.Col2,
      }));
}

(Zwróć uwagę, że MyTable2.Col2jest to kolumna typu int). Wygenerowany kod SQL będzie wyglądał następująco:

SELECT 
    [Extent1].[Id1] AS [Id1], 
    [Extent1].[Col1] AS [Col1], 
    CASE WHEN ([Extent2].[Col2] IS NULL) THEN CAST(NULL AS int) ELSE  CAST( [Extent2].[Col2] AS int) END AS [Col2]
    FROM  [dbo].[MyTable1] AS [Extent1]
    LEFT OUTER JOIN [dbo].[MyTable2] AS [Extent2] ON [Extent2].[Id2] = [Extent1].[Id1]
Diego
źródło
Dla mnie jest to generowanie bardzo powolnego zapytania z "CROSS APPLY".
Meekohi
2

Dla 2 i więcej złączeń po lewej stronie (dołączanie po lewej stronie creatorUser i initialatorUser)

IQueryable<CreateRequestModel> queryResult = from r in authContext.Requests
                                             join candidateUser in authContext.AuthUsers
                                             on r.CandidateId equals candidateUser.Id
                                             join creatorUser in authContext.AuthUsers
                                             on r.CreatorId equals creatorUser.Id into gj
                                             from x in gj.DefaultIfEmpty()
                                             join initiatorUser in authContext.AuthUsers
                                             on r.InitiatorId equals initiatorUser.Id into init
                                             from x1 in init.DefaultIfEmpty()

                                             where candidateUser.UserName.Equals(candidateUsername)
                                             select new CreateRequestModel
                                             {
                                                 UserName = candidateUser.UserName,
                                                 CreatorId = (x == null ? String.Empty : x.UserName),
                                                 InitiatorId = (x1 == null ? String.Empty : x1.UserName),
                                                 CandidateId = candidateUser.UserName
                                             };
Dmitrii Matunin
źródło
1

Udało mi się to zrobić, wywołując DefaultIfEmpty () w głównym modelu. To pozwoliło mi pozostawić dołączenie do leniwych załadowanych bytów, wydaje mi się bardziej czytelne:

        var complaints = db.Complaints.DefaultIfEmpty()
            .Where(x => x.DateStage1Complete == null || x.DateStage2Complete == null)
            .OrderBy(x => x.DateEntered)
            .Select(x => new
            {
                ComplaintID = x.ComplaintID,
                CustomerName = x.Customer.Name,
                CustomerAddress = x.Customer.Address,
                MemberName = x.Member != null ? x.Member.Name: string.Empty,
                AllocationName = x.Allocation != null ? x.Allocation.Name: string.Empty,
                CategoryName = x.Category != null ? x.Category.Ssl_Name : string.Empty,
                Stage1Start = x.Stage1StartDate,
                Stage1Expiry = x.Stage1_ExpiryDate,
                Stage2Start = x.Stage2StartDate,
                Stage2Expiry = x.Stage2_ExpiryDate
            });
William Robinson
źródło
1
Tutaj wcale nie potrzebujesz .DefaultIfEmpty(): wpływa tylko na to, co dzieje się, gdy db.Complainsjest pusty. db.Complains.Where(...).OrderBy(...).Select(x => new { ..., MemberName = x.Member != null ? x.Member.Name : string.Empty, ... }), bez żadnego .DefaultIfEmpty(), wykonałoby już lewe sprzężenie (zakładając, że Memberwłaściwość jest oznaczona jako opcjonalna).
1

Jeśli UserGroups ma relację jeden do wielu z tabelą UserGroupPrices, to w EF, po zdefiniowaniu relacji w kodzie takim jak:

//In UserGroups Model
public List<UserGroupPrices> UserGrpPriceList {get;set;}

//In UserGroupPrices model
public UserGroups UserGrps {get;set;}

Możesz wyciągnąć lewy połączony zestaw wyników w ten sposób:

var list = db.UserGroupDbSet.ToList();

zakładając, że DbSet dla lewej tabeli to UserGroupDbSet, który będzie zawierał UserGrpPriceList, czyli listę wszystkich powiązanych rekordów z prawej tabeli.

Jaggan_j
źródło