ノードコールバック - コードが動作しない

Question

これはノードエクスプレスのコードで、SQLトランザクションが正常に完了すると電子メールを送信します！

router.post('/',function(req,res,next){ 
    sql.connect(config).then(function() { 
     var request = new sql.Request(); 
     request.query(`update projects set CIP_NCIP= '${req.body.cip_noncip}' , Capital_Expense ='${req.body.capital_expensed}' , Approval_Status ='Approved' where Project_ID ='${req.body.projid}'`).then(function(recordset) { 
      console.log('Recordset: ' + recordset); 
      console.log('Affected: ' + request.rowsAffected); 
     }).catch(function(err) { 
      if(err) { 
       console.log('Request error: ' + err); 
      } 
     }) 
    }).then(
     transporter.sendMail(mailOptions, function(error, info){ 
      if (error) { 
       console.log(error); 
      } else { 
       console.log('Email sent: ' + info.response); 
      } 
     })).catch(function(err) { 
     if (err) { 
      console.log('SQL Connection Error: ' + err); 
     } 

    }); 
}); 

SQLのINSERTが動作しているようですが、そのコールバックとES5にかなり新しいコード

transporter.sendMail(mailOptions, function(error, info){ 
       if (error) { 
        console.log(error); 
       } else { 
        console.log('Email sent: ' + info.response); 
       } 
      }) 

のこのビットを実行していません。提案は、あなたが.thenメソッドを介して結合するための機能を実行することにより、

return request.query(`update projects set CIP_NCIP= '${req.body.cip_noncip}' , Capital_Expense ='${req.body.capital_expensed}' , Approval_Status ='Approved' where Project_ID ='${req.body.projid}'`).then(function(recordset) { 
      console.log('Recordset: ' + recordset); 
      console.log('Affected: ' + request.rowsAffected); 
     }).catch(function(err) { 
      if(err) { 
       console.log('Request error: ' + err); 
      } 
     }) 

Answer 1

あなたの現在のコードに基づいて、実際に正しい順序で問題を解決しているわけではありません（これは問題の可能性があります）。このお試しください：上記の

var request = sql.Request(); 
sql.connect(config) 
    .then(function() { 
     console.log('UPDATING RECORD...'); 
     return request.query(`update projects set CIP_NCIP= '${req.body.cip_noncip}' , Capital_Expense ='${req.body.capital_expensed}' , Approval_Status ='Approved' where Project_ID ='${req.body.projid}'`); 
    }) 
    .catch(function(err) { 
     if (err) { 
      console.log(`SQL Connection Error: ${err}`); 
     } 
    }) 
    .then(function(recordset) { 
     console.log('Recordset: ' + recordset); 
     console.log('Affected: ' + request.rowsAffected); 
     console.log('SENDING EMAIL...'); 
     return transporter.sendMail(mailOptions); 
    }) 
    .catch(function(err) { 
     if (err) { 
      console.log(`Nodemailer Err: ${err}`); 
     } 
    }) 
    .then(function() { 
     console.log('COMPLETE'); 
    });   
    console.log('CONNECTING...'); 

を最大限に約束を活用して、問題がどこにある絞り込む助けるべきいくつかのログが含まれています。ここ

FWIWは、それが（デバッグログなし）ES6でどのように見えるかだ

sql.connect(config) 
    .then(() => sql.Request().query('...')) 
    .catch(err => console.error(`SQL Error ${err}`)) 
    .then(() => transporter.sendMail(mailOptions)) 
    .catch(err => console.error(`Nodemailer err: ${err}`); 

Answer 2

が見え歓迎します。あなたが持っているあなたの場合

：

sql.connect(config) 
    .then(function() { ... }) 
    .then(valueNotAFunction); 

thenで実行される関数の戻り値は約束そのものである場合には、次のブロックが前の仕上げを待つことになるが。

コードブロック：

transporter.sendMail(mailOptions, function(error, info){ 
    if (error) { 
     console.log(error); 
    } else { 
     console.log('Email sent: ' + info.response); 
    } 
})) 

は、チェーン内の前の機能が終了していない後、直ちに実行されます。

また、request.queryはプロミスと評価され、あなたはチェーンに戻っていません。 transporter.sendMailを約束するので、あなたがsome promisification libraryでそれをラップすることがあります評価されない場合がありますことを心に留めておいてください

sql.connect(config) 
    .then(function() { .... ; return request.query(...) }) 
    .then(function() { .... ; return transporter.sendMail(....) }) 
    .catch(function(error) { console.error(error); }); 

：

あなたのコードは次のようになります。

Answer 3

約束は約束チェーンを次のAS-約束を返すのを忘れよう