質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.61%

  • Node.js

    1796questions

    Node.jsとはGoogleのV8 JavaScriptエンジンを使用しているサーバーサイドのイベント駆動型プログラムです。

  • Express

    232questions

    ExpressはNode.jsのWebアプリケーションフレームワークです。 マルチページを構築するための機能セットおよびハイブリッドのWebアプリケーションを提供します。

Node.js(express)で関数の途中で処理を終わらせる方法がわからない

解決済

回答 2

投稿

  • 評価
  • クリップ 1
  • VIEW 2,063

twipg

score 23

ユーザーIDをGetパラメータで受け取り、DBのテーブルを検索後、一致していれば後続の処理を行い、一致しなければ処理を中断させるという関数を作っております。

具体的にはrouter.getの最初の行でidCheck関数を呼び、idがDBのテーブルに一致すればtrue、一致しなければfalseを返します。

その後、returnでrouter.getに戻り、

if(!idCheck(req, res)) return;

という文でidの一致、不一致を判定しようと思ったのですが、どちらもif文に引っかかり、returnとなり、一致した場合でも後続の処理に入れません。

どのような書き方をすればtrueとfalseのif文を上手く動かすことが出来るのでしょうか?

よろしくお願いします。



/* Function test */
function idCheck(req, res) {
    User.find({ userid       : req.params.id }, function(err, items){
        if (err) {
            console.log(err);
            res.writeHead(401, {
                'Content-Type': 'text/plain'
            });
            res.end('failed.');
            return false;
        } else {
            if (items.length === 0) {
                console.log('failed. Not registered!');
                res.writeHead(401, {
                    'Content-Type': 'text/plain'
                });
                res.end('failed. Not registered!');
                return false;
            } else {
                console.log('successed!!');
                return true;
            }
        }
    });
}


/* GET users */
router.get('/api/users/:id', function(req, res) {
    if(!idCheck(req, res)) return;
    // 以下、処理
});

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+2

次のようなコードを手元で動かしてみてください。
function idCheck(req, res) {
    User.find({ userid: req.params.id }, function(err, items){
        console.log("Userテーブルの検索が終了しました");
    });
}

if (!idCheck(req, res)) {
    console.log("idCheck関数の処理が終了しました");
}
コマンドラインには次のように表示されると思います。
idCheck関数の処理が終了しました
Userテーブルの検索が終了しました
これは、idCheck関数がUser.findメソッドにfunction(err, items){...}という関数を登録し終えると、即座に関数を抜けてしまうからです。「登録する」という部分が重要で、登録される関数は、即座に実行されるとは限りません。User.findメソッドはその典型です。登録した関数は、DBのテーブルを検索し終えたあとで実行されます。


このように、非同期な動きをする関数を扱うのは少し厄介です。過去にも沢山の人が困ったらしく、ECMAScript 6から、このような問題の解決手段としてPromiseという仕組みが用意されました。

Promiseは、非同期処理を同期的に扱うための手段の一つとして用意されました。Promiseコンストラクタを使うと、質問者さんのコードは次のように書き換わります。
function idCheck(req, res) {
    return new Promise(function(resolve, reject) {
        User.find({ userid: req.params.id }, function(err, items) {
            if (err) reject({ errno: 1, message: err });
            if (items.length === 0) reject({ errno: 2 });
            resolve();
        });
    });
}

router.get('/api/users/:id', function(req, res) {
    idCheck(req, res).then(
    function() {
        /* 成功した時の処理を書く */
    },
    function(obj) {
        switch (obj.errno) {
        case 1:
            console.log(obj.message);
            break;
        case 2:
            console.log('failed. Not registered!');
            break;
        }
    });
});
(routerオブジェクトが何であるかを把握できなかったので、writeHeadメソッドやendメソッドはないものとして考えました)

質問者さんのコードとの主な違いは、

idCheck関数はPromiseコンストラクタのインスタンスを返す
②DBテーブル検索終了後の処理が.then()メソッドの引数の関数内に書かれている
resolve関数やreject関数などの見慣れない関数がある

などです。最初はこれらの仕様に戸惑うかもしれませんが、理解すると、非同期処理を同期的に書く手段として優れたものの一つだということがわかるかと思います。

各メソッドや関数の意味は、MDNのPromiseのリファレンスなどを参考にしてみてください。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/06/14 13:35

    丁寧な回答有難うございます。
    とても助かりました。

    非同期処理のプログラミングがわかっていなかったので、
    非常に勉強になりました。

    徐々に非同期処理のプログラミングに慣れていきたいと思います。

    キャンセル

0

User.find関数が非同期なのでidCheck関数がUser.find関数の終了前に終わってしまっているのだと思います。
promiseを戻せると思うので、それの成功・失敗で切り分けると良いかなと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/06/14 13:35

    回答有難うございました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.61%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Node.js

    1796questions

    Node.jsとはGoogleのV8 JavaScriptエンジンを使用しているサーバーサイドのイベント駆動型プログラムです。

  • Express

    232questions

    ExpressはNode.jsのWebアプリケーションフレームワークです。 マルチページを構築するための機能セットおよびハイブリッドのWebアプリケーションを提供します。