Anybody know why the Output of this program is like this?(using iterator in c#)

Question

using System;
using System.Collections;
namespace Iterator_test
{
 class Day
 {
    int days_idx = -1;
    private String[] days = { "mon", "tue", "wed","thu","fri","sat","sun" };
    public IEnumerable getdays()
    {
        days_idx++;
        yield return days[days_idx];
    }
 }
 class Program
 {
    static void Main(string[] args)
    {
        Day d = new Day();
        foreach (string day in d.getdays())
        {
            Console.WriteLine(day);
        }
    }
  }
}

Actually the output should be,

mon
tue
wed
thu
fri
sat 
sun

but its printing only "mon" as,

mon

What will be the reason?

Answer 1

This is happening because there's no loop in your getdays method. You just yield once, returning the first item - "mon" - and that's it!

Here's an easy fix. (If possible, change the IEnumerable return type to IEnumerable<string> too.)

public IEnumerable getdays()
{
    foreach (string day in days)
    {
        yield return day;
    }
}

Answer 2

You need to have a loop around the yield return:

public IEnumerable getdays()
{    
    while (days_idx < 6)
    {
        days_idx++;
        yield return days[days_idx];
    }    
}

Answer 3

Luke and Gonzalo are correct.

as an alternate approach as your getdays seems to be readonly and doesn't particularly do much else (from your example)

class Day
{
    public IEnumerable days
    {
        get
        {
            return new string[] { "mon", "tue", "wed", "thu", "fri", "sat", "sun" };
        }
    }

}
class Program
{
    static void Main(string[] args)
    {
        Day d = new Day();
        foreach (string day in d.days)
        {
            Console.WriteLine(day);
        }
    }
}