Понимание попытки.. выгода в JavaScript

Это хорошая практика. что-то делать с отловленными исключениями.

Здесь происходит то, что если возникает ошибка (например, загрузка страницы не удалась), в одном из ваших блоков try генерируется исключение.Соответствующий блок catch захватывает его и сообщает, что «это исключение было обработано», но на самом деле вы ничего с ним не сделали.

Попробуйте поставить печать (e.Message); внутри блоков catch, чтобы узнать, какая именно ошибка приводит к тому, что страница не загружается, а затем добавьте код в блок catch для устранения этой ошибки.

Думаю, ваша основная проблема в том, что вы проглатываете исключения, что очень плохо . Поэтому «иногда это срабатывает». Что-то вызывает исключение, и вы его поймаете, но после этого вы больше ничего не делаете. По крайней мере, я бы отобразил какое-то сообщение об ошибке в вашем блоке catch .

Еще несколько проблем:

• Вы уверены, что вам нужны эти несколько блоков try..catch ? Текущее предположение в вашем коде состоит в том, что каждая строка, заключенная в try..catch , не зависит от других, и выполнение может продолжаться, если что-то пойдет не так в одном (или нескольких) из этих операторов. . Вы уверены, что это то, что вам нужно? Если да, то определенно есть лучший способ справиться с этим.
• Если операторы не независимы друг от друга, и если сбой в какой-либо момент означает, что выполнение не может продолжиться, то вы можете заключить все эти операторы в один try..catch блок и отображение сообщения об ошибке в catch
• Как я уже сказал, проглатывание исключений очень плохо ! Вы скрываете проблему и ничего не добиваетесь. Это также чрезвычайно усложняет отладку, потому что все перестанет работать, и вы не поймете, почему (без исключения, без регистрации, без сообщений об ошибках). Исключения используются, когда происходит что-то непредвиденное, прерывающее нормальный ход выполнения программы. Это то, с чем вы обязательно захотите справиться.

Я думаю, что то, что вы хотите, можно сделать следующим образом:

try {
    if(window.opener.hideRecordReload){
        window.opener.hideRecordReload(pg.recordID, pg.recordTypeID);

    } else {
        window.opener.pg.hideRecord(pg.recordID, pg.recordTypeID);

    }

    window.opener.pg.hideEncounter(pg.recordID);
    window.opener.pg.hideRecordResponse(pg.hideReasonID.value == 0 ? pg.otherReason.value : pg.hideReasonID.options[pg.hideReasonID.selectedIndex].text);
    window.opener.pg.hideRecord_Response(pg.recordID, pg.recordTypeID);
    window.opener.pg.hideRecord_Response(pg.recordID, pg.recordTypeID);
    window.opener.window.parent.frames[1].pg.loadQualityMeasureRequest();
    window.opener.pg.closeWindow();

}

catch(e) {
   console.log(e);
}   

Таким образом, если в этой серии операторов возникает исключение, блок catch обработает это.

Javascript также не имеет истинных проверенных исключений. Вы можете обойти это, имея единственный блок попытки и проверив полученный объект исключения ^* .

В дополнение к тому, о чем я говорил ранее, есть два способа обработки исключений. Первый способ, как я показал ранее, предполагает, что при возникновении исключения код находится в недопустимом / неопределенном состоянии, и это означает, что код обнаружил неисправимую ошибку. Другой способ обработки исключений - это если вы знаете, что от этого можно избавиться. Вы можете сделать это с помощью флага. Итак:

try {
   doSomething();
}

catch(e) {
   error = true;
}

if(error) {
   doStuffToRecoverFromError();
}

else {
   doOtherStuff();
}

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

^* Вот несколько надуманный пример, демонстрирующий проверенные исключения. У меня есть два исключения, называемые VeryBadException и ReallyBadException , которые могут быть выброшены (случайным образом) из двух функций. Блок catch обрабатывает исключение и определяет тип исключения, используя оператор instanceof ):

function VeryBadException(message) {
   this.message = message; 
}

function ReallyBadException(message) {
   this.message = message;
}

function foo() {
   var r = Math.floor(Math.random() * 4);
   if(r == 2) {
      throw new VeryBadException("Something very bad happened!");
   }
}

function bar() {
   var r = Math.floor(Math.random() * 4);
   if(r == 1) {
      throw new ReallyBadException("Something REALLY bad happened!");
   }
}

try {
  foo();
  bar();
}

catch(e) {
   if(e instanceof VeryBadException) {
      console.log(e.message);
   }

   else if(e instanceof ReallyBadException) {
      console.log(e.message);
   }
}