Исключения не предназначены для показа пользователю. Текст исключения не локализуем, содержит техническую информацию, содержит форматирование, интересное программисту, а не пользователю. Он может выдать секретную или приватную информацию, которую вы вовсе не планировали показывать пользователю. Самое ужасное — текст исключения не помогает пользователю выйти из проблемной ситуации, он лишь описывает непосредственную причину ошибки в терминах технических подробностей случившегося, а не в терминах того, что нужно на самом деле пользователю, и что он сделал не так (или что пошло не так).
Поэтому никогда (то есть действительно никогда) не выводите пользователю в UI текст исключения.
Как делать правильно? В случае, если ваша модель произвела исключение, вы должны определить причину ошибки в терминах вашей задачи, понятных пользователю, и конвертировать это в объект, представляющий нестандартную ситуацию. Затем ваша бизнес-логика должна запустить процедуру информирования пользователя о случившемся и возможно автоматической реакции на него.
Например, если при чтении конфигурационного файла возникла ошибка, связанная с тем, что доступ к файлу запрещён, или что в файле не найдем валидный XML, или что значение "привет земляне"
не может быть преобразовано к типу int
и не может быть значением атрибута version.major
— всё это неинтересно пользователю. Ваш код должен сказать, что инсталляция программы испорчена, и её нужно восстановить, или (ещё лучше!) проинформировать пользователя, что произведена успешная попытка сброса испорченной конфигурации программы.
Это должно выглядеть примерно так:
try
{
await model.ApplySettings();
return;
}
catch (InvalidSettingsException e)
{
Logger.Log(e);
// для старых версий .NET вам придётся вынести await за catch
var corrected = await model.TryRepairSettings();
if (corrected)
{
await PresentSettingsCorrected();
}
else
{
await PresentSettingsUncorrectable();
throw new UnrecoverableSettingsError(e);
}
}
Локализацией сообщения займётся процедура PresentSettingsCorrected
(и PresentSettingsUncorrectable
) обычным образом, как и любая другая процедура, ответственная за взаимодействие с пользователем.
Код
try
{
...
}
catch (Exception e)
{
MessageBox.Show(e.ToString());
}
— преступление. Никогда так не делайте. Программа, написанная так, всеми и всегда будет считаться студенческой поделкой.
С другой стороны, логирование подавленного исключения (вместе с цепочкой вложенных исключений) — это практически обязанность программиста. Вам эта информация пригодится, когда что-то на машине пользователя пойдёт не так. Вы можете даже написать мелкую функциональность по автоматической упаковке в архив и отправке на ваш адрес свежих логов (разумеется, спросив предварительно явного согласия пользователя на это).
(вынес из комментариев) Правильная реакция на ошибочные ситуации — очень важная задача, и притом непростая. Обычно при прототипировании приложения на обработке ошибок экономят время, потому что главное на этом этапе — показать функциональность, работающую «когда всё в порядке». Потом встраивание логики обработки ошибок становится сложнее, ведь она превращает линейное течение логики программы в разветвлённое.
На самом деле в UI-приложении экранов, обслуживающих ошибочные ситуации, может быть и столько же, сколько экранов для нормального течения программы! Поэтому на правильную обработку таких ситуаций придётся потратить время, сравнимое с временем разработки основной логики приложения.
К сожалению, часто этим пренебрегают даже крупные производители ПО, в результате чего мы и видим беспомощные сообщения об ошибках в стиле «Произошло что-то ужасное, я падаю» или «Вот вам текст исключения, разбирайтесь сами». Пользователи часто выносят своё мнение о программе по таким вот недоделкам. Которые, на мой вкус, и отличают хорошо, с любовью сделанный проект от закодированного на коленке прототипа.