Это выражение js безопасно: if (! X || doSomething (x [prop], y [prop]) === false)

Из отчета об ошибке я думаю, что следующее выражение может вызвать исключение, если x равен нулю:

if ( !x || doSomething( x[prop], y[prop] ) === false )

Исключение составляют:

Невозможно прочитать свойство 'prop' из null

... как будто правая сторона || оценивается, даже если левая сторона верна. ссылка на JavaScript указывает на то, что это должно не бывает, но я не уверен. Я проверял, что просто запись x = null не (всегда) приводит к сбою, но гарантируется ли это на каждом движке JS?

РЕДАКТИРОВАТЬ:

Тот же вопрос о

if( x && foo( x[prop] ) === true && bar() === false )

Один из способов сказать это:

Если (a && b && c)

... оценивает b или c, если a === false? В этом случае документ не совсем ясно, только для "a && (expr1 && expr2)", а не для "&& expr1 && expr2"

Полный фрагмент кода

var x = null;
var y = {
  "p1": "p1",
  "p2": "p2"
};

function f() {
  return true;
}

for (var propName in y) {

  if (x && f(y[propName]) === true && f(y[propName]) === false) {
    doSomething(x[propName], y[propName]);
  } else if (!x || f(x[propName], y[propName]) === false) {
    console.log(y[propName]);
  }
}

EDIT2: для полноты, настоящий (свернутый) код, который запускается в браузере

  function a(c, b, e, f) {

        for (var d in b) {
          if (c && _.isObject(b[d]) === true && _.isArray(b[d]) === false) {
            a(c[d], b[d], e, d + ".")
          } else {
            if (!c || _.isEqual(c[d], b[d]) === false) {
              e.push({
                name: f + d,
                value: b[d]
              })
            }
          }
        }
        return e
      }