Correct syntax for creating an array from a foreach loop

Question

I have the following linq query which gives me values per day, however I've had to hardcode it as I can't work out how to change it into a loop:

var result = from c in groupedJobs

    select new
    {
        Name = c,
        Data = new int[] {
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[0]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[1]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[2]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[3]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[4]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[5]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[6]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[7]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[8]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[9]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[10]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[11]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[12]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[13]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[14]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[15]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[16]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[17]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[18]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[19]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[20]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[21]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[22]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[23]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[24]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[25]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[26]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[27]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[28]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[29]),
        groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[30]),

    }
    };

I'm trying to do something like this but the syntax is wrong:

select new ClaimProviderCountAggregate
{
    Name = c,
    Data = new int[] {
        foreach (var item in daysInMonth)
        {
            groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==item
        }

More info:

Days is an array for the number of days in the current month

 var daysInMonth = DateTime.DaysInMonth(DateTime.Now.Year, DateTime.Now.Month);
 var days = Enumerable.Range(1, daysInMonth).ToArray();

This is my definition for groupedjobs

var groupedJobs = new List<ClaimProviderCount>();
public class ClaimProviderCount
{
    public DateTime Date { get; set; }
    public string ClaimProvider { get; set; }
}

A question now that you've put in more details. Are you sure this is what you want? In your anonymous object c (and hence Name) is not going to be a string, yet you are comparing it with one. I think what you want is Name = c.ClaimProvider. Is that right? — Sнаđошƒаӽ
– Sнаđошƒаӽ, Commented Aug 7, 2018 at 4:11

Sнаđошƒаӽ · Accepted Answer · 2018-08-07 06:18:42Z

3

You can try this, considering your days is an enumerable:

from c in groupedJobs
select new
{
    Name = c,
    Data = days.Select(day => groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==day))
               .ToArray()
}

edited Aug 7, 2018 at 6:18

answered Aug 7, 2018 at 3:54

Sнаđошƒаӽ

17.8k13 gold badges78 silver badges95 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

mehran · Accepted Answer · 2018-08-07 03:52:30Z

0

You can create the object first:

var a=select new ClaimProviderCountAggregate
{
    Name = c,
    Data = new int[30]
}

Then populate the array with loop:

for(int i=0;i<31;i++)
        {
            a.Data[i++]=groupedJobs.Count(d => d.ClaimProvider == c && d.Date.Day==days[i])
        }

answered Aug 7, 2018 at 3:52

mehran

1,4945 gold badges23 silver badges42 bronze badges

Collectives™ on Stack Overflow

Correct syntax for creating an array from a foreach loop

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related