Является ли нормальным оставлять пустой catch?

Question

[Алексей Николаев]

Доброго дня.
На моем текущем проекте (Vue + Vuex) среди гребцов сложилась устойчивая практика хэндлить исключения следующим образом: в экшене стора вызываем ErrorNotify (просто отображает popover) и делаем rethrow исключения; затем в компоненте делается так:

async someMethod() {
   try {
      await this.store.a.something();
   catch(e) {}
   finally {
      // some logic
   }
}

На вопрос мне объяснили, что это для того, чтобы исключение не валилось в UI и не портило экспириенс, ибо неперехваченное исключение сломает приложение (спасибо, кэп). А rethrow исключения в сторе (там, где оно, собственно, обрабатывается вызовом ErrorNotify) они делают для того, чтобы иметь возможность где-нибудь его еще перехватить при необходимости.

Я считал и считаю пустые catch плохой практикой, исходящей из кривых углов в архитектуре. Я прав? Или так делать можно?
Заранее спасибо.

Comments

[Ice]

Если нет необходимости обрабатывать catch, зачем тогда использовать try?
Делать можно, но это бесполезная практика.
На релизе такое точно нужно избегать.

[Алексей Николаев]

Ice,

зачем тогда использовать try?

На случай, если в коде, что внутри try, вывалится исключение. Пустой catch подавляет исключение, и оно не приведет к крэшу приложения.

[Ice]

Алексей Николаев, Это был риторический вопрос.
Мысль была в том, catch как раз для логирования, отлова ошибок с последующим исправлением или просто сказать где-то "Ooops".
А работать в режиме "Вот тут валятся ошибки, закрою их, чтобы просто глаза не мозолили", сомнительное применение.