Перечислимые, дающие неожиданный вывод - PullRequest
Новая
       20

Перечислимые, дающие неожиданный вывод

4 голосов
/ 18 марта 2010 
class Foo
{
    public static IEnumerable<int> Range(int start, int end)
    {
        return Enumerable.Range(start, end);
    }

    public static void PrintRange(IEnumerable<int> r)
    {
        foreach (var item in r)
        {
            Console.Write(" {0} ", item);
        }
        Console.WriteLine();
    }
}

class Program
{
    static void TestFoo()
    {
        Foo.PrintRange(Foo.Range(10, 20));
    }

    static void Main()
    {
        TestFoo();
    }
}

Ожидаемый результат: 

10  11  12  13  14  15  16  17  18  19  20

Фактический объем производства: 

10  11  12  13  14  15  16  17  18  19  20  21  22  23  24  25  26  27  28  29

В чем проблема с этим кодом? Что происходит?

Ответы [ 3 ]

10 голосов
/ 18 марта 2010

Второй параметр Enumerable.Range указывает число генерируемых целых чисел, не последнее целое число в диапазоне.

При необходимости достаточно просто создать собственный метод или обновить существующий метод Foo.Range, чтобы сгенерировать диапазон из параметров start и end.

4 голосов
/ 18 марта 2010

Второй параметр Range - это количество предметов для производства.

3 голосов
/ 18 марта 2010

Почему это не конец, а счет?

Как вы перечисляете пустой диапазон, если у вас есть начальная и конечная точки? Например, предположим, что у вас есть текстовый буфер на экране и выделение, и выделение состоит из одного символа, начинающегося с символа 12 и заканчивающегося с символа 12. Как вы перечисляете этот диапазон? Вы перечисляете один символ, начиная с символа 12.

Теперь предположим, что выбор состоит из нулевых символов. Как вы перечисляете это? Если у вас есть начало, размер, вы просто передаете ноль для размера. Если у вас есть начало, конец, что вы делаете? Вы не можете пройти 12 и 12.

Теперь вы можете сказать: «Ну, просто не перечисляйте его, если это пустой диапазон». В итоге вы получаете код, который должен выглядеть следующим образом: 

var results = from index in Range(selectionStart, selectionSize)
              where blah blah blah
              select blah;

и вместо этого писать 

IEnumerable<Chicken> results = null;
if (selectionSize == 0)
{
    results = Enumerable.Empty<Chicken>();
}
else
{
    results = from index in Range(selectionStart, selectionEnd)
              where blah blah blah
              select blah;
}

что ранит мои глаза.

...