Linq imbriqué sélectionnez nouveau ne fonctionne pas

Question

J'essaye d'obtenir le chargement impatient de travailler avec Subsonic, et il a rendu null pour moi.

Dans la méthode ci-dessous, j'essaie d'hydrater un modèle de domaine (UserModel) qui contient un autre modèle de domaine (CompanyModel). Cependant, avec le code ci-dessous, UserModel.Company est toujours null.

Qu'est-ce qui me manque ici? Toute aide serait appréciée.

public IList<UserModel> GetUsers() 
{    
    return (from u in SubsonicSqlServer.Users.All() 
      select new UserModel 
         { 
          UserId= u.UserId, 
          Company = (from c in u.Companies 
            select new CompanyModel 
               { 
                CompanyId = c.CompanyId, 
                CompanyName = c.CompanyName 
               }).SingleOrDefault(), 
          FirstName = u.FirstName, 
          LastName = u.LastName, 
          BirthDate = u.BirthDate 
         }).ToList(); 

} 

Mise à jour (08/11/09):

Plus avec le toying autour de code, je trouve que la mise en COMPANYID dans l'exemple suivant ne fonctionne pas non plus. J'ai d'abord pensé que c'était un problème avec Subsonic, mais si le code ci-dessous ne fonctionne pas, je suppose que cela a quelque chose à voir avec ma déclaration Linq. Des idées?

public IList<UserModel> GetUsers() 
{    
    return (from u in SubsonicSqlServer.Users.All() 
      select new UserModel 
         { 
          UserId= u.UserId,        
          CompanyId = Guid.NewGuid(), 
          FirstName = u.FirstName, 
          LastName = u.LastName, 
          BirthDate = u.BirthDate 
         }).ToList(); 

} 

Mise à jour (11/17/2009):

n'a pas encore trouvé une solution. Mais nous passons à nHibernate (pas à cause de ce problème).

Answer 1

Deux choses

1. vous retournez un List<UserModel> lorsque la ligne de signature de votre méthode dit IList<User>-t UserModel Hériter de User?

2. Ai-je manqué quelque chose, d'où vient e?

FirstName = e.FirstName, LastName = e.LastName, BirthDate = e.BirthDate Blockquote

Answer 2

"UserModel.Company est toujours nulle."

étant donné que vous définissez ceci avec une expression qui se termine par .SingleOrDefault(), je vais suggérer que la requête ne renvoie pas un seul élément. Commencez à enquêter là-bas. Si vous attendez exactement un élément dans u.Companies, passez à .Single() et forcez un échec anticipé.

Vous pouvez faire le .Single() avant de créer le nouvel objet CompanyModel, je pense. En ce qui concerne le style, j'aime la syntaxe de compréhension de la requête ("from x in y select") mais je trouve cela bizarre quand on la combine avec la syntaxe traditionnelle de la notation par points. C'est juste difficile à lire. (LINQ - Fluent and Query Expression - Is there any benefit(s) of one over other?).

Envisagez d'utiliser let dans la compréhension de la requête pour le rendre plus clair.

De plus, comme une requête renvoie déjà un IEnumerable<T> et que l'appel ToList() force tous les éléments à réaliser, je modifierais ma méthode pour renvoyer IEnumerable<T> si possible.

Ainsi, dans votre cas, je serais factoriser le premier à dire:

public IEnumerable<User> GetUsers() 
{ 
    return from u in SubsonicSqlServer.Users.All() 
     let c = u.Companies.Single() 
     select new UserModel 
     { 
      UserId = u.UserId, 
      Company = new CompanyModel 
      { 
       CompanyId = c.CompanyId, 
       CompanyName = c.CompanyName 
      }, 
      FirstName = e.FirstName, 
      LastName = e.LastName, 
      BirthDate = e.BirthDate 
     }; 
} 

S'il est logique dans votre modèle objet, vous pouvez modifier User d'avoir un constructeur qui prend tout type u est, et il est encore plus simple:

return from u in SubsonicSqlServer.Users.All() 
    select new UserModel (u); 

ou même

return SubsonicSqlServer.Users.All().Select(u => new UserModel (u)); 

Answer 3

S'il vous plaît vérifier ma fourchette @ github (http://github.com/funky81/SubSonic-3.0/commit/aa7a9c1b564b2667db7fbd41e09ab72f5d58dcdb) pour cette solution, en fait il y a un problème lorsque subsonic essayer de projeter une nouvelle classe de type, donc il n'y a rien de mal avec votre code: D