Пытаюсь разобраться в написании собственных коллекций (да и вообще по глубже разобраться в коллекциях).
Я сейчас в учебных целях пишу реализацию коллекции, которая наследует интерфейс ICollection
Такой вопрос, как должен себя вести метод add если в коллекции по умолчанию задано IsReadOnly = true?
Есть такие варианты как:
выбрасывать исключение если он был вызван (но это ужасно глупо);
не иметь ни какой реализации (получается метод пустышка только для интерфейса, я не уверен, что это нормально);
скрывать его под явной реализацией интерфейса и делать реализацию пустой (то же самое, что и в предыдущем варианте только может чуть лучше).
Ответ
Именно так, бросать исключение. Ничего глупого в этом нет: если операцию невозможно выполнить, нужно как можно громче заявить об этом, а не притворяться, что всё идёт как надо. В противном случае программист добавит элементы в коллекцию, не заметит ошибку, и будет считать, что ничего странного не происходит. Отловить логическую ошибку («я добавил элементы в коллекцию, но они там не появились») на порядок сложнее, чем поймать креш в виде необработанного исключения.
Наличие свойства IsReadOnly — ошибка в проектировании, сделанная ещё во времена .NET 1.0. Правильнее было бы иметь отдельный интерфейс IReadOnlyList
Комментариев нет:
Отправить комментарий