анонимных делегатов и общие списки в C#

Question

Можете ли вы объяснить мне код ниже:

private static List<Post> _Posts; 
public static Post GetPost(Guid id) 
{ 
    return _Posts.Find(delegate(Post p) 
    { 
     return p.Id == id; 
    }); 
} 

1. Какой смысл найти объект в общем списке по этому пути? Он может просто перебирать список.

2. Как этот делегированный метод вызывается для каждого элемента списка?

Примечание:, если это имеет общее название, вы можете обновить название моего вопроса в?

Спасибо!

Answer 1

Вы совершенно правы, он может пройти по списку, вы можете думать о коде в вашем вопросе как концептуально такой же, как следующее:

private static Post GetPost(Guid id) 
{ 
    Post p = default(Post); 

    foreach (Post post in _Posts) 
    { 
     if (post.Id == id) 
     { 
      p = post; 
      break; 
     } 
    } 

    return p; 
} 

Это требует меньше кода, чтобы написать фрагмент кода и главное вы теперь говорят, что вы хотите найти и не точно, как его найти:

private static Post GetPost(Guid id) 
{ 
    return _Posts.Find(delegate(Post p) 
    { 
     return p.Id == id; 
    }); 
} 

В C# 3.0 это может быть сокращено в дальнейшем использовать то, что называется «lambda expression», чтобы:

private static Post NewGetPost(Guid id) 
{ 
    return _Posts.Find(p => p.Id == id); 
} 

Используя наименьшее количество читаемого кода для достижения той же цели делает и писателей и читателей, что код счастливее.

Answer 2

Он пользуется анонимным делегатом. Он мог бы использовать вместо lambda expression:

Posts.Find(p => p.Id == id) 

Кроме того, упаковка доступа к списку в методе ничего не дает в этом случае и предоставляет элементы списка внешних абонентов. Это плохая практика.

Answer 3

Если вы используете C# 3.0 или более позднюю версию, вы можете использовать Linq для быстрого поиска объектов в списке.

public static Post GetPost(Guid id) 
{ 
    return (from p in _Posts 
      where p.Id == id 
      select p).First(); 
} 

Answer 4

1. Список в основном проходит через каждый элемент и проверяет, возвращает ли элемент верно для этого Predicate<T>. Это, по сути, ярлык, поэтому вам не нужно перебирать список. List<T>.Find(Predicate<T>) может также иметь встроенные оптимизации.
2. Вы называете делегата, используя следующий синтаксис:

delegateInstance(arg1,arg2);

Answer 5

List.Find (Predicate match) НЕ является методом расширения LINQ, потому что этот метод был в структуре с 2.0, как указано MSDN. Во-вторых, нет ничего плохого в использовании Find().Это имеет тенденцию быть более читаемым, чем его альтернативы:

Classic:

public static Post GetPost(Guid id) 
{ 
    bool found = false; 

    foreach(post in _Posts) 
    { 
    if post.Id == id return post; 
    } 
    return default(Post); 
} 

LINQ:

public static Post GetPost(Guid id) 
{ 
    var post = (
    from p in _Posts 
    where p.Id = id 
    select p 
).FirstOrDefault(); 
} 

Использование List.Find() говорит вам сразу, что вы посмотрите для элемента, в то время как другой вы должны следовать логике, чтобы убедиться в этом. Find() в основном инкапсулирует итерацию по элементам списка. Если у вас был метод в классе Post, например public bool HasId(Guid id), тогда напишите

_Post.Find(post.HasId(id));