約束を書いたときのパターンが悪いですか？

Question

カスタムの約束を書いたときに悪い習慣を使っているのだろうかと思います。

私は次のような状況があります。

generateHash(password){ 
    return new Promise((resolve, reject) => { 
     bcrypt.genSalt(10).then(salt => { 
      bcrypt.hash(password, salt).then(hash => { 
       resolve(hash) 
      }).catch(reject) 
     }).catch(reject) 
    }) 
} 

generateHash（）は、ハッシュされたパスワードを含む約束を返さなければなりません。 .genSalt（）は約束を返し、.hash（）も返します。私が必要とするものは、.hash（）によって返される約束を返すためにgenerateHash（）ですが、それは内部の.genSalt（）なので、このようなことは起こりません。私は、この関数がカスタムの約束を返し、その中にあるすべてのものが完全に機能していたと思っていましたが、そのような入れ子の約束が悪い習慣であり、ひどい考え方であるかどうか疑問に思っています。私がやった

もう一つは、そのようなコールバックを使用して、それより伝統的な方法でアプローチです：

return new Promise((resolve, reject) => { 
     bcrypt.genSalt(10, function (err, salt) { 
      bcrypt.hash(password, salt, function (err, hash) { 
       if (err) return reject(err) 
       resolve(hash) 
      }); 
     }); 
    }) 

私は優れていると種類の方法についてのアイデアそう約束に固執したい方法の提案に開いています可能な限り最善の方法を約束してそれをすることは大いに感謝されます。

Answer 1

すでに使用しているメソッドが約束を返す場合は、独自の約束を作成する必要はありません。次のコードは、自分で約束を作成せずに、コンソールに生成したハッシュ値を記録します：あなたは、内側の関数を返す必要が

const bcrypt = require("bcrypt") 
function generateHash(password){ 
    return bcrypt.genSalt(10).then(salt => { 
      return bcrypt.hash(password, salt).then(hash => { 
       return hash; 
      }) 
     }) 
} 

generateHash("foo").then(v => console.log(v)); 

注 - そう基本的に、コードは

function generateHash(password){ 
    return bcrypt.genSalt(10).then(salt => bcrypt.hash(password, salt)); 
} 

に煮詰めるでしょうが

Answer 2

なぜ約束を約束に入れますか？

あなただけはい、

generateHash(password){ 
    return bcrypt.genSalt(10) 
       .then(salt => bcrypt.hash(password, salt)) 
}