Проверка на null, прежде чем добавлять в список


Есть ли способ, чтобы выполнить рефакторинг этого?

public IEnumerable<Option> Options
{
    get
    {
        {
            List<Option> ListOption = new List<Option>();

            if (!String.IsNullOrEmpty(Option1)) 
            {
                ListOption.Add(new Option() {Name=Option1 });
            }
            if (!String.IsNullOrEmpty(Option2))
            {
                ListOption.Add(new Option() { Name = Option2 });
            }
            if (!String.IsNullOrEmpty(Option3))
            {
                ListOption.Add(new Option() { Name = Option3 });
            }
            if (!String.IsNullOrEmpty(Option4))
            {
                ListOption.Add(new Option() { Name = Option4 });
            } 

            return ListOption;
        }
    }
}

public class Option
{
    public string Name { get; set; }
}


38567
12
задан 5 июля 2011 в 04:07 Источник Поделиться
Комментарии
3 ответа

Это немного аккуратнее:

List<Option> ListOption = new List<Option> { };
Action<string> add = x => { if (!String.IsNullOrEmpty(x)) ListOption.Add(new Option { Name = x }); }
add(Option1);
add(Option2);
add(Option3);
add(Option4);
return ListOption;

Это, пожалуй, даже лучше:

return (new string[] {Option1, Option2, Option3, Option4})
.Where(x => !String.IsNullOrEmpty(x))
.Select(x => new Option { Name = x })
.ToList();

31
ответ дан 5 июля 2011 в 04:07 Источник Поделиться

Это хорошая возможность использовать доходность оператора:

public IEnumerable<Option> Options
{
get
{
if (!string.IsNullOrEmpty(Option1)) yield return new Option{Name=Option1};
if (!string.IsNullOrEmpty(Option2)) yield return new Option{Name=Option2};
if (!string.IsNullOrEmpty(Option3)) yield return new Option{Name=Option3};
if (!string.IsNullOrEmpty(Option4)) yield return new Option{Name=Option4};
}
}

10
ответ дан 5 июля 2011 в 09:07 Источник Поделиться

Просто какой-то эпатаж, вот версия, используя стандартный механизм зацикливания:

static List<Option> GetNonNullOptions(params string[] options)
{
var results = new List<Option>();
if (options != null)
{
foreach (var option in options)
{
if (option != null)
{
results.Add(new Option { Name = option });
}
}
}
return results;
}

Оставляя вас, чтобы сделать что-то вроде этого:

var nonNullOptions = GetNonNullOptions(Option1, Option2, Option3, Option4);

Обратите внимание, что я только проверить на null значение элементов, как пустая строка и null-значение-это две разные вещи.

6
ответ дан 5 июля 2011 в 12:07 Источник Поделиться