CODEWITHSUNDEEP
c#listmaxindexof
AJAY KUMAR
AJAY KUMAR

Reputation: 77

How to Get the index of the Element which is Maximum in the List <T>

I have a List with class name Product and I want to know the index of the element which has the maximum value?

class Product
{
    public int ProductNumber { get; set; }
    public int ProductSize { get; set; }
}

List<Product> productList = new List<Product>();

int Index = productList.Indexof(productList.Max(a => a.ProductSize));

I have tried this but it is not getting answer! And getting an error:

"Unable to Cast as Product"

Upvotes: 0

Views: 301

Answers (7)

par
par

Reputation: 1121

productList.Max(a=>a.ProductSize) will return max ProductSize value , not Product object. that condition should be in WHERE condition.

Upvotes: -1

Pablo notPicasso
Pablo notPicasso

Reputation: 3161

Assuming list is not empty:

productList.Indexof(productList.OrderByDescending(a => a.ProductSize).First());

Upvotes: 0

Slaven Tojić
Slaven Tojić

Reputation: 3014

Here is a solution with Enumerable.Range:

int index = Enumerable.Range(0, productList.Count)
                      .FirstOrDefault(i => productList[i].ProductSize == productList.Max(x => x.ProductSize));

DEMO HERE

Upvotes: 0

Dmitrii Bychenko
Dmitrii Bychenko

Reputation: 186668

You are looking for ArgMax which is not implemented in Linq but can be easily emulated via Aggregate:

  int Index = productList
    .Select((item, index) => new { item, index })
    .Aggregate((s, v) => v.item.ProductSize > s.item.ProductSize ? v : s)
    .index;

Upvotes: 1

Marc
Marc

Reputation: 3959

The method Max will give you the maximum ProductSize, not an instance of Product. This is why you are receiving this error.

You could do it with OrderByDescending:

var item = productList.OrderByDescending(i => i.ProductSize).First();
int index = productList.IndexOf(item);

Upvotes: 0

Sweeper
Sweeper

Reputation: 270790

You can first map each item so that each product is associated with its index, then order by descending and get the first item:

int Index = productList
    .Select((x, index) => new { Index = index, Product = x })
    .OrderByDescending(x => x.Product.ProductSize).First().Index;

You don't need another call to IndexOf!

Upvotes: 2

Marco Siffert
Marco Siffert

Reputation: 555

This requires a sort

var maxObject = productList.OrderByDescending(item => item.ProductSize).First();
var index = productList.IndexOf(maxObject);

There are other, easier ways to do this. For example: There is an extension method in MoreLINQ that does this.

See this question

Upvotes: 0

Related Questions