Как я могу вернуть пустой IEnumerable? - PullRequest
302 голосов
/ 12 июля 2010

Учитывая следующий код и предложения, приведенные в этом вопросе , я решил изменить этот оригинальный метод и спросить, есть ли какие-либо значения в IEnumarable, вернуть его, если не вернуть IEnumerable без значения.

Вот метод:

public IEnumerable<Friend> FindFriends()
        {
            //Many thanks to Rex-M for his help with this one.
            //https://stackoverflow.com/users/67/rex-m

            return doc.Descendants("user").Select(user => new Friend
            {
                ID = user.Element("id").Value,
                Name = user.Element("name").Value,
                URL = user.Element("url").Value,
                Photo = user.Element("photo").Value
            });
        }

Поскольку все находится внутри оператора return, я не знаю, как я мог это сделать. Будет ли что-то вроде этой работы?

public IEnumerable<Friend> FindFriends()
        {
            //Many thanks to Rex-M for his help with this one.
            //https://stackoverflow.com/users/67/rex-m
            if (userExists)
            {
                return doc.Descendants("user").Select(user => new Friend
                {
                    ID = user.Element("id").Value,
                    Name = user.Element("name").Value,
                    URL = user.Element("url").Value,
                    Photo = user.Element("photo").Value
                });
            }
            else
            { 
                return new IEnumerable<Friend>();
            }
        }

Вышеуказанный метод не работает, и фактически он не должен; Я просто чувствую, что это иллюстрирует мои намерения. Мне кажется, я должен указать, что код не работает, потому что вы не можете создать экземпляр абстрактного класса.

Вот код вызова, я не хочу, чтобы он получал нулевой IEnumerable в любое время:

private void SetUserFriends(IEnumerable<Friend> list)
        {
            int x = 40;
            int y = 3;


            foreach (Friend friend in list)
            {
                FriendControl control = new FriendControl();
                control.ID = friend.ID;
                control.URL = friend.URL;
                control.SetID(friend.ID);
                control.SetName(friend.Name);
                control.SetImage(friend.Photo);

                control.Location = new Point(x, y);
                panel2.Controls.Add(control);

                y = y + control.Height + 4;
            } 

        }

Спасибо за ваше время.

Ответы [ 5 ]

525 голосов
/ 12 июля 2010

Вы можете использовать list ?? Enumerable.Empty<Friend>() или иметь FindFriends return Enumerable.Empty<Friend>()

146 голосов
/ 12 июля 2010

Вы можете вернуть Enumerable.Empty<T>().

92 голосов
/ 12 июля 2010

Как по мне, самый элегантный способ это yield break

8 голосов
/ 12 июля 2010

Это, конечно, только вопрос личных предпочтений, но я бы написал эту функцию, используя возвращаемый доход:

public IEnumerable<Friend> FindFriends()
{
    //Many thanks to Rex-M for his help with this one.
    //http://stackoverflow.com/users/67/rex-m
    if (userExists)
    {
        foreach(var user in doc.Descendants("user"))
        {
            yield return new Friend
                {
                    ID = user.Element("id").Value,
                    Name = user.Element("name").Value,
                    URL = user.Element("url").Value,
                    Photo = user.Element("photo").Value
                }
        }
    }
}
1 голос
/ 12 июля 2010

Я думаю, что самый простой способ будет

 return new Friend[0];

Требования возврата состоят в том, что метод возвращает объект, который реализует IEnumerable<Friend>.Тот факт, что при разных обстоятельствах вы возвращаете два разных типа объектов, не имеет значения, поскольку оба реализуют IEnumerable.

...