Replies: 2 comments
-
I would prefer to stick with Select, Where (ie without the Item suffix) as this is more familiar with what developers are used to coming from more traditional list based linq operations |
Beta Was this translation helpful? Give feedback.
0 replies
-
My ¢2: I'd second @nickrandolph and stick with |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Currently on
ListFeed
the single operator is.Where
and is about item itself (i.e.ListFeed<T>.Where(Predicate<T>)
). As we are about to add.Select
and.SelectAsync
the question is should we suffix operators byItem
to make distinction clearer (.WhereItem
, .SelectItem
,.SelectItemAsync
).Reminder: The
IListFeed<T>
does not implementsIFeed<T>
so there is no possible conflict betweenListFeed<T>.Where(Predicate<T>)
andListFeed<T>.Where(Predicate<IImmutableList<T>>)
(which does not exists and is not planned).Pros:
Feed
andListFeed
ListFeed<T>.Where(collection => collection.Count > 0)
).Cons:
.Where
and.Select
are from LINQ which is already about list and collections, i.e. the "item" implicit.Beta Was this translation helpful? Give feedback.
All reactions