Свойство IsReadOnly в ICollection<T>

#c_sharp #коллекции

Пытаюсь разобраться в написании собственных коллекций (да и вообще по глубже разобраться
в коллекциях). 

Я сейчас в учебных целях пишу реализацию коллекции, которая наследует интерфейс ICollection. 

Такой вопрос, как должен себя вести метод add если в коллекции по умолчанию задано
IsReadOnly = true?

Есть такие варианты как:


выбрасывать исключение если он был вызван (но это ужасно глупо);
не иметь ни какой реализации (получается метод пустышка только для интерфейса, я
не уверен, что это нормально);
скрывать его под явной реализацией интерфейса и делать реализацию пустой (то же самое,
что и в предыдущем варианте только может чуть лучше).

    

Ответы

Ответ 1

Именно так, бросать исключение. Ничего глупого в этом нет: если операцию невозможно
выполнить, нужно как можно громче заявить об этом, а не притворяться, что всё идёт
как надо. В противном случае программист добавит элементы в коллекцию, не заметит ошибку,
и будет считать, что ничего странного не происходит. Отловить логическую ошибку («я
добавил элементы в коллекцию, но они там не появились») на порядок сложнее, чем поймать
креш в виде необработанного исключения.

Наличие свойства IsReadOnly — ошибка в проектировании, сделанная ещё во времена .NET
1.0. Правильнее было бы иметь отдельный интерфейс IReadOnlyList (который появился
в .NET 4.5).