承诺在一组真/假值中失败

Question

我有一系列验证检查，其中一个恰好使用 ajax 来检查地址。在提交表单之前，我需要整个数组返回 true 并按顺序触发。我尝试使用承诺，但无济于事。

这就是问题所在。如果我输入了错误的地址或将输入留空，则它不会提交表单（好）。但是，当我实际输入有效地址时，尽管我在 validations 数组中的其他验证具有错误值，但表单仍会提交。我做错了什么？

var validations = [validateInputPresence, validatePassword, validateAddress];

$continueButton.on('click', function() {    
  toggleSpinner();

  subscribe().then(function() {
    submitForm(); // This is firing even when some values are false in the array
  }, function() {
    toggleSpinner();
  });
});

function subscribe() { 
  var promises = validations.map(function(validation) {
    return validation();
  });

  return Promise.all(promises);
}

function validatePassword() {
  var password = $password.val();
  var format = /^(?=.*?[A-Z])(?=.*?[a-z])(?=.*?[0-9])[^+&\n]{8,}$/;

  return validateInput(format.test(password));
}

function validateAddress() {
  return new Promise(function (resolve, reject) {
    $.ajax({
      type: 'POST',
      url: '/address/validate',
      data: $form.serialize(),
      dataType: 'json',
      success: function(response) {
        var hasValidAddres = response.Data === 200;

        validateInput(hasValidAddres);

        hasValidAddres ? resolve() : reject();
      }, 
      error: function() {
        toggleSpinner();
      }
    });
  });
}

function validateInput(validation) {
  if (validation) {
    return true;
  } else {
    return false;
  }
}

Answer 1

我做错了什么？

您从未测试过数组中的值是true 还是false。您正确地完成了大部分承诺，subscribe() 返回一个布尔值数组的承诺。如果你想测试它们是否都是真的，你需要明确地这样做：

subscribe.then(function(results) {
    if (results.every(Boolean)) // all are true
        submitForm();
    else // some are false
        …;
}, function(err) {
    // some validations threw an exception
});

Answer 2

主要问题是您没有在 validateInput() 函数中返回 Promise。您不能通过在其中一个函数中返回 false 来拒绝 Promise.all 返回的承诺。 Read more about Promise.all，以下引用摘自MDN

如果在可迭代数组中传递的东西不是 Promise，它会被 Promise.resolve 转换为 Promise。

所以实际上任何事情，但 Promise 将被视为已解决。

你应该做的是编写 validateInput 函数来返回一个承诺。

function validateInput(validation) {
  return new Promise(function(resolve, reject) {
    if (validation) { 
      resolve();
    } else {
      reject();
    }  
  });
}