9

Надо в GetPersons как значение по-умолчанию передать функцию, всегда возвращающую true, хочется сделать лямбда-выражением. Что тут не так?

        public IEnumerable<Person> GetPersons(Choise func = x => true)
        {
            foreach (var u in ManageUsers.GetUsers())
            {
                Person person = new Person();
                person.ImportProperties(u);
                if (func(person))
                {
                    yield return person;
                }
            }
        }

        private delegate bool Choise(Person person);
12
  • 2
    Я бы переписал код как-то так: ManageUsers.GetUsers().Select(u => u.Clone()).Where(func); А вообще, вы бы сделали их read-only и не клонировали.
    – VladD
    15 дек 2012 в 11:59
  • 1
    Зачем объявлять свой тип данных, если можно использовать тип [Func<T, TResult>][1]: Func<Person, bool> [1]: msdn.microsoft.com/ru-ru/library/bb549151.aspx
    – Shad
    15 дек 2012 в 21:00
  • 2
    @Shad: вопрос вкуса. Например, я использую для хранения возраста структуру Age, а не int -- позволяет избежать случайного ошибочного использования вне позволенной семантики (например, не даёт перемножить два возраста.). Хотя int было бы в принципе достаточно. (А вместо Func<Person, bool> я бы всё же порекомендовал Predicate<Person>.)
    – VladD
    15 дек 2012 в 21:48
  • 2
    @VladD: Пример со структурой Age полностью себя оправдывает - вы обеспечиваете некую безопасность в отношении возраста. Но в случае с функциями это, помоему, overhead. Ведь что Choise, что Func<Person, bool> примет входным параметром Person и вернёт bool. То есть никакой разницы кроме названия типа нету (Тогда уж лучше записать так: using Choise = Func<int, bool>). Соглашусь, однако, что это всё-таки дело вкуса. Дж. Рихтер в своей "CLR via C#" советует использовать Func<> и Action<>, вместо своих типов. и мне почему-то это запомнилось :) А по поводу Predicate<Person> - плюсую
    – Shad
    15 дек 2012 в 23:35
  • 1
    Насчёт using Choise = Func<int, bool> -- хорошая идея! Рихтер голова и монстр.
    – VladD
    15 дек 2012 в 23:39

2 ответа 2

6

Ещё вариант - использовать в качестве значения по умолчанию null:

public IEnumerable<Person> GetPersons(Choise func = null)
    {
        foreach (var u in ManageUsers.GetUsers())
        {
            Person person = new Person();
            person.ImportProperties(u);
            if ((func ?? (x=>true))(person))
            {
                yield return person;
            }
        }
    }
2

Нельзя так делать. Выход из ситуации - перегрузка без параметров.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.