Lambda Max et Max et Max

Question

Rapide et probablement facile Question Lambda:

J'ai un restaurant avec commentaires. Je veux interroger pour celui avec le:

• Max (AverageRating)
• Et Max (reviewCount)
• Et Max (NewestReviewDate)
• Et le Min (DistanceAway)

Quelque chose comme ceci:

var _Result = AllRestaurants 
    .Max(x => x.AverageRating) 
    .AndMax(x => x.ReviewCount) 
    .AndMax(x => x.NewestReviewDate) 
    .AndMin(x => x.DistanceAway); 

Maintenant, je sais que c'est un pseudo code. Mais il le décrit parfaitement!

Bien sûr, dans plusieurs instructions, c'est simple.

Je me demandais simplement si cela est possible dans une instruction sans tuer la lisibilité.

Merci d'avance. Je sais que certains d'entre vous aiment les questions d'interrogation!

Answer 1

Vous ne pouvez pas avoir plusieurs maxes ou minutes, cela n'a aucun sens. Vous aurez besoin d'une sorte d'heuristique comme:

.Max(x => x.AverageRating * x.ReviewCount - x.DaysSinceLastReview - x.DistanceAway) 

Answer 2

Peut-être que ce serait faire?

var bestRestaurant = AllRestaurants 
    .OrderByDescending(r => r.AverageRating) 
    .ThenByDescending(r => r.ReviewCount) 
    .ThenByDescending(r => r.NewestReviewCount) 
    .ThenBy(r => r.DistanceAway) 
    .FirstOrDefault(); 

Vous devez modifier l'ordre des instructions pour indiquer lequel est le plus important.

Answer 3

Si je comprends bien votre question, je pense que la meilleure approche va être écrit des déclarations individuelles que vous mentionnez ...

var HighestRating = AllRestaurants.Max(x => x.AverageRating); 
var HighestReviewCount = AllRestaurants.Max(x => x.ReviewCount); 
var LatestReviewDate = AllRestaurants.Max(x => x.NewestReviewDate); 
var ShortestDistanceAway = AllRestaurants.Min(x => x.DistanceAway); 

Récupération divers Maxes et minutes à partir d'une seule requête Linq obtiendrait assez désordonné et je ne suis pas sûr qu'il y aurait un avantage avec l'efficacité, soit.

Answer 4

Tenir compte quelque chose comme ça ...

List<RestaurantRecord> _Restaurants; 

public RestaurantRecord Best() 
{ 
    return _Restaurants.Where(
       x => 
        x.AverageRating >= _BestRating && 
        x.ReviewCount >= _MinReviews && 
        x.Distance <= _MaxDistance) 
         .GetFirstOrDefault(); 
} 

Cela étant dit, en utilisant un lambda dans ce cas aura des conséquences maintenabilité sur la route. Ce serait une bonne idée de refactoriser ceci, de sorte que si d'autres critères apparaissent à l'avenir (par exemple: accès au smartphone, type de cuisine?), Votre application peut être plus facilement modifiée pour s'adapter à ceux-ci.

Sur cette note, une mise en œuvre un peu mieux peut-être quelque chose comme:

public RestaurantRecord Best() 
{ 
    IQueryable temp = _Restaurants.Clone(); 

    temp = temp.Where(x => x.AverageRating >= _BestRating); 
    temp = temp.Where(x => x.ReviewCount >= _MinReviews); 
    // ...snip... 

    return temp.GetFirstOrDefault(); 
} 

J'espère que cela vous met sur la bonne voie. :)

Answer 5

Une alternative à avoir une heuristique pondérée est de classer par AverageRating, puis par ReviewCount, puis ...

Quelque chose comme cela devrait fonctionner:

var _Result = AllRestaurants 
    .OrderByDescending(x => x.AverageRating) 
    .ThenByDescending(x => x.ReviewCount) 
    .ThenByDescending(x => x.NewestReviewDate) 
    .ThenByDescending(x => x.DistanceAway); 
    // using *Descending so you get the higer-valued ones first