- 1
- 2
- 3
- 4
- 5
public static IEnumerable<TItem> GetNextItemFrom<TItem>(IEnumerable<TItem> Collection)
{
foreach (var Item in Collection)
yield return Item;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+126
public static IEnumerable<TItem> GetNextItemFrom<TItem>(IEnumerable<TItem> Collection)
{
foreach (var Item in Collection)
yield return Item;
}
Название неправильное. = The name is wrong.
Название не правильное. = The name isn't right.
В школе Мариванна наверное конечно учила, что правильно только слитно писать?..
public static IEnumerable<TItem> GetNextItemFrom<TItem>(IEnumerable<TItem > Collection)
where TItem : new()
{
// Создать новую коллекцию.
IList<TItem> list = new List<TItem>();
// Скопировать в новую коллекцию все элементы из коллекции принимаемой в параметре метода GetNextItemFrom()
foreach (var Item in Collection)
{
// К примеру, элементы из входящей коллекции не поддерживают клонирование.
TItem newItem = new TItem();
newItem.Prop1 = Item.Prop1;
newItem.Prop2 = Item.Prop2;
newItem.Prop3 = Item.Prop3;
// и т.д.
list.Add(Item);
}
// Вернуть клонированную коллекцию.
return list;
}
Ну не очень элегантно. А если код достался в наследство, то это как вариант. Имя метода канеш такое, не очень.
Ну а так бывает и хуже.
Не, это не говно. -1
эт эквивалентно этому:
public static IEnumerable<TItem> GetNextItemFrom<TItem>(IEnumerable<TItem > Collection)
{
// Создать новую коллекцию.
IList<TItem> list = new List<TItem>();
// Скопировать в новую коллекцию все элементы из коллекции принимаемой в параметре метода GetNextItemFrom()
foreach (var Item in Collection)
list.Add(Item);
// Вернуть клонированную коллекцию.
return list;
}
Да, не очень удачный метод.