JavaScriptのPromiseの挙動について

Question

前提・実現したいこと

Promiseとは下記のように then をつけると、関数1の処理が終わったあとに関数2の処理をやってくれるものだと認識しています。

関数1().then(関数2);

発生している問題・エラーメッセージ

関数1の処理が終わらずに関数2が実行される。

該当のソースコード

var file_path = "....."

var file_upload = (param_path) =>{
    var formData = {
        'file': fs.createReadStream(param_path),
        'timeout': '10000' //milliseconds
    };

    //コールバック関数のpost処理
    request.post({url:url, formData:formData}, function(err, response, body) {
        if(err){
          console.log(err)
        }else{
          console.log(body)
          console.log("post完了")
        };
    });
}


var promiseUpload = function(param_path){
  console.log("start --- promiseUpload ---")
  return new Promise(function(resolve, reject){
    resolve(file_upload(param_path));
  });
}

promiseUpload(file_path).then(function(){
  console.log("end --- promiseUpload ---")
});

期待値

start --- promiseUpload ---
post完了
end --- promiseUpload ---

現実

start --- promiseUpload ---
end --- promiseUpload ---
post完了

推測

request.postのようなコールバック関数がある場合はいかにPromiseといえども、コールバック関数の戻りまでは待ってくれないのでしょうか？

補足情報（FW/ツールのバージョンなど）

Ubuntu18.04
node v14.15.3

Answer 1

関数1の処理が終わったあとに関数2の処理をやってくれるもの

少々違います。resolve関数を実行した後に関数2が実行されます。提示されたコードではrequest.post()のコールバックが呼ばれる前にresolve()を実行しているので、期待どおりになっていません。以下のように、コールバック内でresolve()します。

var file_upload = (param_path, resolve, reject) =>{
    var formData = {
        'file': fs.createReadStream(param_path),
        'timeout': '10000' //milliseconds
    };

    //コールバック関数のpost処理
    request.post({url:url, formData:formData}, function(err, response, body) {
        if(err){
          console.log(err);
          reject(err);
        }else{
          console.log(body);
          console.log("post完了");
          resolve(body);
        };
    });
}


var promiseUpload = function(param_path){
  console.log("start --- promiseUpload ---")
  return new Promise(function(resolve, reject){
    file_upload(param_path, resolve, reject);
  });
}

Comments

[EzrealTrueshot]

ありがとうございます！
postするファイルが1つの場合はご教示いただきました方法で無事にかいけつできました。

ただ、request.post関数の直前でfor文を使う（複数のファイルをPOSTするため）と、最初の１つしか意図する動きになりませんでしたが、これは少し自分で考えてみようと思います！