C#: реализация SkipLast

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

Альтернативным подходом было бы использование foreach, всегда возвращающее ранее возвращенное значение, если вы не были на первой итерации:

public static IEnumerable<T> SkipLast<T>(this IEnumerable<T> source)
{
    T previous = default(T);
    bool first = true;
    foreach (T element in source)
    {
        if (!first)
        {
            yield return previous;
        }
        previous = element;
        first = false;
    }
}

Другой вариант, более близкий к вашему коду:

public static IEnumerable<T> SkipLast<T>(this IEnumerable<T> source)
{
    using (IEnumerator<T> iterator = source.GetEnumerator())
    {
        if(!iterator.MoveNext())
        {
            yield break;
        }
        T previous = iterator.Current;
        while (iterator.MoveNext())
        {
            yield return previous;
            previous = iterator.Current;
        }
    }
}

Это позволяет избежать такой же глубокой вложенности (выполняя ранний выход, если последовательность пуста) и использует «настоящее» условие while вместо while(true)

Ваша реализация выглядит отлично для меня - вероятно, я бы так и сделал.

Единственное упрощение, которое я мог бы предложить в отношении вашей ситуации, — упорядочить список наоборот (т. е. по возрастанию, а не по убыванию). Хотя это может не подходить для вашего кода, это позволит вам просто использовать collection.Skip(1) для получения всех элементов, кроме самого последнего.

Если это невозможно по причинам, которые вы не указали в своем посте, то ваша текущая реализация не представляет проблемы.

Если вы используете .NET 3.5, я думаю, вы могли бы использовать:

public static IEnumerable<T> SkipLast<T>(this IEnumerable<T> source)
{
  return source.TakeWhile((item, index) => index < source.Count() - 1))
}

(Старый ответ удален; этот код был протестирован и работает.) Он печатает
первый
второй
ПЕРВЫЙ
ВТОРОЙ
ТРЕТИЙ

public static class ExtNum{
  public static IEnumerable skipLast(this IEnumerable source){
    if ( ! source.Any())
      yield break;
    for (int i = 0 ; i <=source.Count()-2 ; i++ )
      yield return source.ElementAt(i);
    yield break;
  }
}
class Program
{
  static void Main( string[] args )
  {
    Queue qq = new Queue();
    qq.Enqueue("first");qq.Enqueue("second");qq.Enqueue("third");
    List lq = new List();
    lq.Add("FIRST"); lq.Add("SECOND"); lq.Add("THIRD"); lq.Add("FOURTH");
    foreach(string s1 in qq.skipLast())
      Console.WriteLine(s1);
    foreach ( string s2 in lq.skipLast())
      Console.WriteLine(s2);
  }
}