2010-01-04 3 views
7

О, я просто обнаружил, что ошибка вызвана другой частью кода. Корпус закрыт.Использование Linq для выбора максимального значения в группе

У меня есть 2 таблицы

1- UserInfo

id uid name 
1 11 Billy 
2 22 Paul 
3 33 Joshua 

2- Score

id uid score 
1 11 30 
2 22 40 
3 11 50 
4 11 60 
5 33 20 
6 33 70 
7 33 80 

У меня есть класс с именем ScoreUser

public class ScoreUser{ 
public long uid{get; set;} 
public string name{get;set;} 
public int score{get;set;} 
} 

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

from s in Scores 
join i in UserInfos 
on s.uid equals i.uid 
group uscore by new { s.uid, i.name} into g 
let maxScore = g.Max(p => p.score) 
select new ScoreUser 
{ 
uid = g.Key.uid, 
name = g.Key.name, 
score = maxScore 
} 

Однако этот код действительно не работает. Он создает 7 объектов вместо 3. Что мне делать?

+0

что возвращается? Вы получаете сообщение об ошибке? – Theresa

+0

Не следует ли переключаться на таблицы «Оценки» и «UserInfos»? «from i in UserInfos присоединяется к Scores ...» В принципе, сейчас ваш основной выбор - 7 записей. –

ответ

9

Вы также группируете score, когда это должен быть агрегатор. Попробуйте это:

from s in Scores 
join i in UserInfos on s.uid equals i.uid 
group by new { s.uid, i.name } into g 
select new ScoreUser 
{ 
    uid = g.Key.uid 
    name = g.Key.name, 
    score = g.Max(p => p.score) 
} 

(обновление)

Я вижу, что вы нашли проблему. Однако я оставляю вас здесь тест на этот запрос:

class UserInfo 
    { 
     public int Id { get; set; } 
     public int UId { get; set; } 
     public string Name { get; set; } 
    } 

    class Score 
    { 
     public int Id { get; set; } 
     public int UId { get; set; } 
     public int SScore { get; set; } 
    } 

    public class ScoreUser 
    { 
     public int uid { get; set; } 
     public string name { get; set; } 
     public int score { get; set; } 

     public override string ToString() 
     { 
      return string.Format("UId:{0} Name:{1} Score:{2}", uid, name, score); 
     } 
    } 


    static void Main(string[] args) 
    { 

     List<UserInfo> infos = new List<UserInfo>() 
     { 
      new UserInfo {Id = 1, UId = 11, Name = "Billy"}, 
      new UserInfo {Id = 2, UId = 22, Name = "Paul"}, 
      new UserInfo {Id = 3, UId = 33, Name = "Joshua"} 
     }; 

     List<Score> scores = new List<Score>() 
     { 
      new Score {Id = 1, UId = 11, SScore = 30}, 
      new Score {Id = 2, UId = 22, SScore = 40}, 
      new Score {Id = 3, UId = 11, SScore = 50}, 
      new Score {Id = 4, UId = 11, SScore = 60}, 
      new Score {Id = 5, UId = 33, SScore = 20}, 
      new Score {Id = 6, UId = 33, SScore = 70}, 
      new Score {Id = 7, UId = 33, SScore = 80} 
     }; 

     var qry = from s in scores 
        join i in infos on s.UId equals i.UId 
        group s by new { s.UId, i.Name } into g 
        select new ScoreUser 
        { 
         uid = g.Key.UId, 
         name = g.Key.Name, 
         score = g.Max(p => p.SScore) 
        }; 

     foreach (var su in qry) 
     { 
      Console.WriteLine(su); 
     } 
    } 

Печать:

UId:11 Name:Billy Score:60 
UId:22 Name:Paul Score:40 
UId:33 Name:Joshua Score:80 
Смежные вопросы