React Native DRY Javascript

时间:2016-08-08 21:02:29

标签: javascript react-native

1)我有一个具体的实施,我想遵守DRY原则。 两种方法的结构几乎完全相同。我想知道这是否是一种不重复实施的方式:

addCardToExistingCustomer(cardDetail){
    PaymentUtil.stripeCreateCardToken(cardDetail).then((cardTokenResult)=>{
      if(cardTokenResult.error){
        console.log("There was an error with the card!");

      } else {
          PaymentUtil.addCardToExistingCustomer(cardTokenResult.id).then((card) =>{


          });  
      }
    });
  }  

  addCardToNewCustomer(cardDetail){
    this.stripeCreateCardToken(cardDetail).then((cardTokenResult)=>{
      if(cardTokenResult.error){
        console.log("There was an error with the card!");

      } else {
        console.log("Successfully created card token");
        PaymentUtil.stripeCreateCustomer(cardTokenResult.id)

      }
    });
  }

2)这是连锁承诺的最佳方式吗?具体来说,您应该如何处理链中承诺的异常,这些异常不是链中的最后一个元素。

比如承诺getStripeCustomerId被拒绝。你应该如何处理拒绝。

addCardToExistingCustomer(cardTokenResultId){
    return this.getStripeCustomerId(userDetail).then((customerId) => {
            return  this.removeAllExistingCards(userDetail).then(()=>{
                    return  Stripe.addCardToCustomer(cardTokenResultId,customerId);
            });
        });     
},


getStripeCustomerId(userDetail){
    return FirebaseRESTUtil.getStripeCustomer(userDetail.username)
        .then((fbStripe) => (fbStripe.customerId)); 
},

2 个答案:

答案 0 :(得分:3)

1-这种方式可以更好..它将使用promises链来处理数据。

getCardTokenResult(cardDetail) {
  return PaymentUtil.stripeCreateCardToken(cardDetail)
    .then((cardTokenResult) => {
      if(cardTokenResult.error){
        return Promise.reject('There was an error with the card!!');
      } else {
        return cardTokenResult;
      }
    })
    .catch((error) => console.log(error)):
}

addCardToExistingCustomer(cardDetail){
  return getCardTokenResult(cardDetail)
    .then((cardTokenResult) => {
      PaymentUtil.addCardToExistingCustomer(cardTokenResult.id).then((card) =>{
        // do something
      });
    });
}

addCardToNewCustomer(cardDetail){
  return getCardTokenResult(cardDetail)
    .then((cardTokenResult) => {
      PaymentUtil.stripeCreateCustomer(cardTokenResult.id);
    });
}

2-您可以使用 catch 在Promises链上获取错误。你可以查看上面的代码。我添加了catch以获得cardtokenresult错误。你可以在链上继续相同的方式来获得其他错误。

答案 1 :(得分:0)

我注意到您在then的第一个参数中检查错误。我很确定这是不正确的,因为Stripe在错误时不会返回HTTP 200状态。

话虽如此,我喜欢压扁链接的承诺而不是嵌套它们,因为我发现它使代码更易于维护和阅读:

addCardToExistingCustomer(cardTokenResultId) {
    return this.getStripeCustomerId(userDetail).then((customerId) => {
        return this.removeAllExistingCards(userDetail); // returns a promise
    }).then(() => {
        return Stripe.addCardToCustomer(cardTokenResultId,customerId); // returns a promise
    });     
},

现在任何错误都会冒充主要的外部承诺,因此您可以通过

检查错误
addCardToExistingCustomer(myId).then(() => { 
        console.log('success!');
    }, (error) => {
        console.log(error);
    });