Обработайте ошибку с помощью NLog и попробуйте Catch

Во-первых, большинство людей решают эту ошибку, не перехватывая исключение. Таким образом, исключение распространяется на ASP.NET, который отображает веб-страницу "500 Internal Error", и вся соответствующая информация регистрируется.

• Если ваш сервер настроен для работы, на странице ошибки будет просто указано «произошла ошибка, подробности были зарегистрированы».

• Если сервер настроен для разработки, вы получите знаменитую желтую страницу с типом исключения, сообщением и трассировкой стека.

Проглатывание исключения и ручное перенаправление на страницу с ошибкой — плохая практика, поскольку она скрывает ошибки. Есть инструменты, которые проверяют ваши журналы и дают вам хорошую статистику, например, о процентах успешных/неудачных запросов, и они больше не будут работать.

Таким образом, не проглатывание исключения — это то, что люди делают, и, по крайней мере, это решает вашу проблему.

Теперь я нахожу это очень неуклюжим, потому что мне не нравится вручную искать исходные файлы, упомянутые на желтой странице, и вручную переходить к указанным номерам строк. Жёлтая страница мне практически ни к чему, с тем же успехом она могла бы просто сказать «произошла ошибка, плачь мне рекой, нах-нах-нах». Я не читаю желтую страницу.

Вместо этого мне нравится регистрировать исключения самостоятельно, и мой регистратор начинает каждую строку с full-path-to-source-filename(line):, так что каждая строка в журнале отладки в Visual Studio кликабельна, и нажатие на строку автоматически открывает правильный исходный файл, и прокручивается до той строки, которая выдала сообщение журнала. Если вам нужна эта роскошь, то идите вперед и перехватите исключение, но сразу после регистрации исключения вы должны повторно создать его, чтобы все шло своим чередом.

Поправка

Вот некоторая информация, которая была добавлена ​​в комментарии:

Итак, вы можете сделать следующее:

try
{
   ...
}
catch (Exception e)
{
    log( "information" );
    throw; //special syntax which preserves original stack trace
}

Or

try
{
   ...
}
catch (Exception e)
{
    throw new Exception( "information", e ); //also preserves original stack trace
}

Не делайте не этого: catch( Exception e ) { log( "information" ); throw e; }, потому что при этом теряется исходная информация о трассировке стека e.

В вашем коде ошибка возникает в части разделения (x/=x), поэтому не выполняется строка перенаправления (индексная страница) и не выполняется переход к части перехвата, выполняющей регистратор. Вы также должны определить перенаправление на Error.cshtml в части перехвата.

Примечание. При использовании блока try catch ошибка не возникает на уровне ASP.NET, что приводит к отсутствию перенаправления на страницу Error.cshtml

using NLog;

private static Logger _logger = LogManager.GetCurrentClassLogger();

public virtual ActionResult Edit(Client client)
{
  try
  {
        // FORCE ERROR
        var x = 0;

        x /= x; /// error occur here

        return RedirectToAction(MVC.Client.Index()); /// no execution of this line
  }
  catch (Exception e)
  {
    _logger.Error("[Error in ClientController.Edit - id: " + client.Id + " - Error: " + e.Message + "]");
    /// add redirect link here 
     return RedirectToAction(MVC.Client.Error()); /// this is needed since the catch block execute mean no error at ASP.net level resulting no redirect to default error page

  }
}

Это упростит обработку исключений и позволит более лаконично управлять процессом. Создайте такой атрибут:

 public class HandleExceptionAttribute : System.Web.Mvc.HandleErrorAttribute
    {
        // Pass in necessary data, etc
        private string _data;
        public string Data
        {
            get { return _data; }
            set { _data = value; }
        }

        public override void OnException(System.Web.Mvc.ExceptionContext filterContext)
        {            
            // Logging code here
            // Do something with the passed-in properties (Data in this code)
            // Use the filterContext to retrieve all sorts of info about the request
            // Direct the user

            base.OnException(filterContext);
        }
    }

Теперь вы можете использовать его на уровне контроллера или метода с таким атрибутом:

[HandleException(Data="SomeValue", View="Error")]

Или зарегистрируйте его глобально (global.asax) следующим образом:

GlobalFilters.Filters.Add(new HandleExceptionAttribute());