C # / LINQ: an attempt to optimize performance

Question

This is my setting

class EditorTabViewModel : TabViewModel {
    ...
    public bool CanSave { get; set; };
}

ObservableCollection<TabViewModel> _tabs

I want to check if there are any tabs in _tabswhich EditorTabViewModelwhich property is CanSaveset to true

I did something like ...

var tabs = from t in _tabs
            where t is EditorTabViewModel
            && ((EditorTabViewModel)t).CanSave == true
            select t;
if (tabs.Count() > 0)
    return true;
else
    return false;

I wonder if there is a better way to do this? maybe I don’t need to retrieve all the tabs, or maybe I just need to request an invoice or something else?

Answer 1

What about:

return _tabs.OfType<EditorTabViewModel>().Any(t => t.CanSave);

Here:

• OfType<> is a non-buffered filter that limits us EditorTabViewModel
• Any is short-circuited, so returns true as soon as a match is found

Answer 2

Yes, you can improve. Maybe something like this:

return _tabs.Any(x => x is EditorTabViewModel && ((EditorTabViewModel)x).CanSave);

Answer 3

Using linq extensions you can write something like

_tabs.Any( p => p is EditorTabViewModel && ((EditorTabViewModel)t).CanSave)

Answer 4

Try something like:

return _tabs.FirstOrDefault(y => y is EditorTabViewModel && ((EditorViewModel)t).CanSave) != null;