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

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

ただいまの
回答率

90.75%

  • JavaScript

    15246questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • MySQL

    5502questions

    MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

  • Node.js

    1729questions

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

  • 非同期処理

    101questions

    非同期処理とは一部のコードを別々のスレッドで実行させる手法です。アプリケーションのパフォーマンスを向上させる目的でこの手法を用います。

Node.js GETパラメータをMySQLで検索したいが非同期処理なので思ったように動きません

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 620

Clover

score 7

http://localhost:3000/home?id=test
GETパラメータのidよりtestを取得し、MySQLのusersテーブルへ接続しカラムidにtestがあった場合はhomeへ、なかった場合はerrorへ。

上記を想定していたのですが、MySQLのconnection.query処理が非同期でreturnされる前に次の処理へ進んでしまい、GETパラメータ(id)による処理の分別(home||errorを表示)が出来ません。

async/awaitなど色々調べたのですが良く分かりませんでした。
どのようにコードを書くのが適切でしょうか?すいませんが、よろしくお願い致します。

var http = require('http');
var fs = require('fs');
var path = require('path');
var mysql = require('mysql');
var url  = require('url');

// サーバー
var mime = {
  ".html": "text/html",
  ".css":  "text/css"
};
var server = new http.createServer(function(req, res) {
    if (req.url.indexOf('home') !== -1) {
        var url_parts = url.parse(req.url, true);
        if (url_parts.query.id !== undefined) {

            if (auth(url_parts.query.id)) {
                filePath = '/home.html';
            } else {
                filePath = '/error.html';
            }
        }
    } else {
        filePath = req.url;
    }
    var fullPath = __dirname + filePath;

    res.writeHead(200, {"Content-Type": mime[path.extname(fullPath)] || "text/plain"});
    fs.readFile(fullPath, function(err, data) {
        if (!err) {
            res.end(data, 'UTF-8');
        } else {
            res.end("Not Found", 'UTF-8');
        }
    });
}).listen(3000);

// データベース
function auth(id) {
    try {
        var connection = mysql.createConnection({
            host     : 'host',
            user     : 'user',
            password : 'password',
            database : 'database'
        });
        connection.connect();

        connection.query({
            sql: 'SELECT * FROM `users` WHERE `id` = ?',
            timeout: 5000,
            values: id
        }, function (error, results, fields) {
            if (results == '') {
                console.log('存在しないユーザーです。');
                return false;
            } else {
                console.log('存在するユーザーです。');
            }
        });
        connection.end();
        return true;
    } catch (err) { }
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

動くコードになるか不安ですが単純にasync-awaitに対応させてみます
asnc-awaitは内部的にpromiseを使っているのでpromiseから学習してみるのがいいと思います。

おすすめは以下のドキュメントです
JavaScript Promiseの本

var http = require('http');
var fs = require('fs');
var path = require('path');
var mysql = require('mysql');
var url  = require('url');

// サーバー
var mime = {
  ".html": "text/html",
  ".css":  "text/css"
};
var server = new http.createServer(async function(req, res) { //変更1
    if (req.url.indexOf('home') !== -1) {
        var url_parts = url.parse(req.url, true);
        if (url_parts.query.id !== undefined) {

            const result = await auth(url_parts.query.id); // 変更2
            if (result) {
                filePath = '/home.html';
            } else {
                filePath = '/error.html';
            }
        }
    } else {
        filePath = req.url;
    }
    var fullPath = __dirname + filePath;

    res.writeHead(200, {"Content-Type": mime[path.extname(fullPath)] || "text/plain"});
    fs.readFile(fullPath, function(err, data) {
        if (!err) {
            res.end(data, 'UTF-8');
        } else {
            res.end("Not Found", 'UTF-8');
        }
    });
}).listen(3000);

// データベース
async function auth(id) { //変更3
    return new Promise((resolve, reject) => { // 変更4
        try {
            var connection = mysql.createConnection({
                host     : 'host',
                user     : 'user',
                password : 'password',
                database : 'database'
            });
            connection.connect();

            connection.query({
                sql: 'SELECT * FROM `users` WHERE `id` = ?',
                timeout: 5000,
                values: id
            }, function (error, results, fields) {
                if (results == '') {
                    console.log('存在しないユーザーです。');
                    resolve(false); //変更5
                } else {
                    console.log('存在するユーザーです。');
                    connection.end();
                    resolve(true); //変更6
                }
            });
        } catch (err) { 
            reject(err); //変更7
        }
    });
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/11/17 11:50

    ご回答頂きありがとうございます。

    promise、今週中に本格的に勉強を始めてみます。

    すいません、コードを動作せてみたのですが、正しく動作しませんでした。
    const result = await auth(url_parts.query.id); の関数が実行される前に、resolve(true); が強制的に実行されるようで、resultが必ずtrueになってしまいますOrz

    キャンセル

  • 2017/11/17 11:53

    すいません!
    connection.end(); を function (error, results, fields) { } に含めましたら、問題なく動作しました。

    ありがとうございました!

    キャンセル

  • 2017/11/17 11:59

    自己解決したみたいでよかったです。基本的にasync-awaitによって関数のネスト呼び出しは撲滅できるので本来はこのコードは理想的ではないです。実際このようにバグが発見されてますし。
    ですので理解した上で再修正することをオススメします

    キャンセル

  • 2017/11/17 12:08

    ありがとうございます。

    何度も質問をすいません。
    やはりこのコードですと、
    ネストが深くなる、というものでしょうか?
    それとも、想定しないバグが発生する可能性が高いという事でしょうか?

    キャンセル

  • 2017/11/17 12:14

    例えば http.createServerに渡している無名関数 `` async function(req, res) `` は非同期処理を同期的に扱えるのに、まだ非同期関数を含めています。
    (``fs.readFile(fullPath, function(err, data) `` の箇所。 )
    この箇所も同じようにasync awaitを使って同期的な書き方に直すべきだと思います。

    キャンセル

  • 2017/11/18 13:34

    すいません、返信が遅れました。
    なるほどです!ありがとうございます。

    キャンセル

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

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

関連した質問

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

  • JavaScript

    15246questions

    JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

  • MySQL

    5502questions

    MySQL(マイエスキューエル)は、TCX DataKonsultAB社などが開発するRDBMS(リレーショナルデータベースの管理システム)です。世界で最も人気の高いシステムで、オープンソースで開発されています。MySQLデータベースサーバは、高速性と信頼性があり、Linux、UNIX、Windowsなどの複数のプラットフォームで動作することができます。

  • Node.js

    1729questions

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

  • 非同期処理

    101questions

    非同期処理とは一部のコードを別々のスレッドで実行させる手法です。アプリケーションのパフォーマンスを向上させる目的でこの手法を用います。