how to sum particular field of list of Objects

Question

0.00/5 (No votes)

See more:

Hi Friends,
I need to sum particular field of list of Objects. I have written a method with accepts Lists but don't know how to pass the field name.
For Example if i pass MarketValue it should return MarketValue Sum like wise if i pass Quantity it should return Quantity Sum

This is my class

C#

public class TopList
   {
        public static double marketsum = 1;
        public string Symbol{get;set;}
        public double MarketValue{get;set;}
        public double PreviousMarketVale{get;set;}
        public double Quantity{get;set;}
        public double MarketValuePercentage { get { return Math.Round((MarkrtValue / marketsum) * 100,2); } }
        public double ChangeInMarketValue {get { return Math.Round(((MarkrtValue-PreviousMarketVale)/MarkrtValue)*100,2);}}
   }

My Method

C#

public static double MarketSum(List<TopList>tl)//here it should accept one more paramter
        {
            return tl.Sum(s => s.MarketValue);//here i need to pass different fields
        }

This is how i invoke the method

C#

List<TopList> tl = new List<TopList>();
 double ms = MarketSum(tl);

Thanks in advance :)

Posted 28-Jul-15 10:16am

jinesh sam

Updated 28-Jul-15 10:24am

v4

Add a Solution

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Matt T Heffron · Answer 1 · 2015-07-28T11:25:00

I'd just abstract the field selection outside of the method:

C#

public static double MarketSum(List<TopList>tl, Func<TopList, double> accessor)
{
    return tl.Sum(accessor);
}

Calling it as:

C#

List<TopList> tl = new List<TopList>();
  double ms = MarketSum(tl, s => s.MarketValue);

Another option is to pre-define the accessor functions:

C#

private static Dictionary<string, Func<TopList, double>> Accessors = new Dictionary<string, Func<TopList, double>>() {
  { "MarketValue", s => s.MarketValue },
  { "Quantity", s => s.Quantity },
  // etc.
};
public static double MarketSum(List<TopList>tl, string accessorKey)
{
    Func<TopList, double>> accessor;
    if (!Accessors.TryGetValue(accessorKey, out accessor))
      throw new InvalidOperationException("No such accessor: " + accessorKey);
    return tl.Sum(accessor);
}

Calling it as:

C#

List<TopList> tl = new List<TopList>();
  double ms = MarketSum(tl, "MarketValue");

A third option is to use reflection to automatically pre-define the accessor functions:

C#

public static Dictionary<string, Func<TopList, double>> Accessors = GetPropertyAccessDictionary<TopList, double>();
static Dictionary<string, Func<Tclass, Tvalue>> GetPropertyAccessDictionary<Tclass, Tvalue>()
{
  Dictionary<string, Func<Tclass, Tvalue>> accessors = new Dictionary<string, Func<Tclass, Tvalue>>();
  PropertyInfo[] props = typeof(Tclass).GetProperties(BindingFlags.Public);
  Type valueType = typeof(Tvalue);
  foreach (var prop in props)
  {
    MethodInfo getter = prop.GetMethod;
    if (valueType.IsAssignableFrom(getter.ReturnType))
      accessors.Add(prop.Name, o => (Tvalue)(getter.Invoke(o, null)));
  }
  return accessors;
}

public static double MarketSum(List<TopList> tl, string accessorKey)
{
  Func<TopList, double> accessor = null;
  if (!Accessors.TryGetValue(accessorKey, out accessor))
    throw new InvalidOperationException("No such accessor: " + accessorKey);
  return tl.Sum(accessor);
}

Use it exactly the same as above.
The string-based lookup of the accessors is not a great architecture. It's OK if the strings are coming directly from user input. Even then, it's probably better to parse and validate that into something more structured.

Ben J. Boyle · Answer 2 · 2015-07-28T10:41:00

I would probably declare another class to contain your list and include the sum methods on that class.

C#

public class TopListItem
{
    public static double marketsum = 1;
    public string Symbol { get; set; }
    public double MarketValue { get; set; }
    public double PreviousMarketVale { get; set; }
    public double Quantity { get; set; }
    public double MarketValuePercentage { get { return Math.Round((MarketValue / marketsum) * 100, 2); } }
    public double ChangeInMarketValue { get { return Math.Round(((MarketValue - PreviousMarketVale) / MarketValue) * 100, 2); } }
}

public class TopList
{
    public static List<toplistitem> items { get; set; }

    static TopList()
    {
        items = new List<toplistitem>();
    }
    public double sumQuantity()
    {
        //Sum them here
        return 0;
    }
}

To invoke:

C#

TopList t1 = new TopList();
double totalQuantity = t1.sumQuantity();

how to sum particular field of list of Objects

2 solutions

Solution 2

Solution 1

Add your solution here

Preview 0