C# simple loop to assign array variable

Question

Is there a simple method with a loop I could use to walk through both arrays at once? I've been working on this for days and I feel like it's a lot simpler than I'm making it out to be.

Something like..

While statsArray and perDayArray subscripts are less than ten, divide statsArray[x] by days and assign to perdayArray[x]

Also, the statsArray subscripts have previously been assigned by textBoxes.

    private double CalculatePerDay(double stats, int days)
    {
        return stats / days;
    }

        perDayArray[0] = CalculatePerDay(statsArray[0], daysPassed);
        perDayArray[1] = CalculatePerDay(statsArray[1], daysPassed);
        perDayArray[2] = CalculatePerDay(statsArray[2], daysPassed);
        perDayArray[3] = CalculatePerDay(statsArray[3], daysPassed);
        perDayArray[4] = CalculatePerDay(statsArray[4], daysPassed);
        perDayArray[5] = CalculatePerDay(statsArray[5], daysPassed);
        perDayArray[6] = CalculatePerDay(statsArray[6], daysPassed);
        perDayArray[7] = CalculatePerDay(statsArray[7], daysPassed);
        perDayArray[8] = CalculatePerDay(statsArray[8], daysPassed);
        perDayArray[9] = CalculatePerDay(statsArray[9], daysPassed);
        perDayArray[10] = CalculatePerDay(statsArray[10], daysPassed);

Will they always contain the same number of items?

Alfie Goodacre
– Alfie Goodacre

2016-04-19 14:33:45 +00:00
Commented Apr 19, 2016 at 14:33 — Alfie Goodacre
– Alfie Goodacre, Commented Apr 19, 2016 at 14:33

Roman · Accepted Answer · 2016-04-19 14:35:27Z

7

for (int i = 0; i < perDayArray.Length; i++)
{
    perDayArray[i] = CalculatePerDay(statsArray[i], daysPassed);
}

edited Apr 19, 2016 at 14:35

answered Apr 19, 2016 at 14:33

Roman

12.3k10 gold badges42 silver badges50 bronze badges

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

3 Comments

Alfie Goodacre Over a year ago

Why not perDayArray.GetLength(0)? (or perDayArray.GetUpperBound(0) - 1)

Roman Over a year ago

Yes, it will be even better. Thanks @AlfieGoodacre

Airvvic Over a year ago

This worked perfectly. I feel like I've tried it so many different ways and this was one of them. Must have screwed something up somewhere. Thanks Roma.

Codor · Accepted Answer · 2016-04-19 14:34:34Z

7

You could use Linq as follows

perDayArray = statsArray.Select( a => CalculatePerDay( a, daysPassed ) ).ToArray();

answered Apr 19, 2016 at 14:34

Codor

17.6k9 gold badges32 silver badges59 bronze badges

1 Comment

Icemanind Over a year ago

@Fabjan - I think leaving it as a function is better, because 1. It's clearer as to what the calculation is doing and 2. there is room in the future to expand the function.

Carra · Accepted Answer · 2016-04-19 14:34:43Z

1

I think you want to do this:

for(int i=0; i<= 10; i++)
{
  perDayArray[i] = CalculatePerDay(statsArray[i], daysPassed);
}

answered Apr 19, 2016 at 14:34

Carra

18k8 gold badges66 silver badges75 bronze badges

Collectives™ on Stack Overflow

C# simple loop to assign array variable

3 Answers 3

3 Comments

1 Comment

Comments

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

1 Comment

Comments

Related