...
var combo = view.ActiveEditor as ComboBoxEdit;
Debug.Assert(combo != null, "Column editor supposed to be a ComboBoxEdit");
if (combo == null)
return;
...
Это совсем @овнокод или нет?
На всякий случай: это не в модели и не в логике, т.е. ничего и не покорруптится, и по-тихому для пользователя не сделается / не несделаеся.
Мотив проверять: в реализации "view" кто-то что-то изменить (или оно уже сейчас не всегда работает так, как я предполагаю).
Мотив в релизе не кидать эксэпшн: у пользователя просто не будет работать один контрол
но все остальное будет доступно и ничего не потеряется
А если добавить логи и репорты в релизе?