NodeJsのユーザー名とパスワードの確認null

Question

データベースに格納されている入力済みのユーザー名とパスワードを確認しようとしています。 私の解決策は正しくありません。自分のコードよりも優れたものがあるかもしれません。

ここでは、これまでのところです：console.log(callback);戻りヌル、

私のコードで

function login (username, password, callback) { 
    var query = "SELECT * FROM users WHERE username = ?"; 

    connection.query(query, [username], function (err, results) { 
     if (err) return callback(err); 
     if (results.length === 0) return callback(); 
     var user = results[0]; 

     if (!bcrypt.compareSync(password, user.password)) { 
      return callback(); 
     } 

     callback(null, { 
      id:   user.id.toString(), 
     }); 

    }); 
} 
app.get('/salam', function (req, res) { 
    var username = 'mahdi'; 
    var originalPassword = 'a'; 

    login(username , originalPassword,function (callback) { 
     console.log(callback); 
    }); 
}); 

が、ユーザ名とパスワードが正しいことを確認します。これをどうすれば解決できますか？

Answer 1

callback関数では、引数は2つではありますが、エラーコールバックには1つの引数しかありません。

errorとsuccess場合、最初のパラメータの値は常にnullなり、引数として渡された値がないようif (!bcrypt.compareSync(password, user.password)) {場合には、第1引数の値はundefinedになります。

提案：Boolean（false又はtrue）として使用最初の引数と値に基づいて、コールバックを処理します。

function login(username, password, callback) { 
 
    var query = "SELECT * FROM users WHERE username = ?"; 
 

 
    connection.query(query, [username], function(err, results) { 
 
    if (err) return callback(false); 
 
    if (results.length === 0) return callback(); 
 
    var user = results[0]; 
 

 
    if (!bcrypt.compareSync(password, user.password)) { 
 
     return callback(false); 
 
    } 
 
    callback(true, { 
 
     id: user.id.toString(), 
 
    }); 
 

 
    }); 
 
} 
 
app.get('/salam', function(req, res) { 
 
    var username = 'mahdi'; 
 
    var originalPassword = 'a'; 
 

 
    login(username, originalPassword, function(success, value) { 
 
    if (success) { 
 
     console.log(value); 
 
    } 
 
    }); 
 
});

Answer 2

あなたがコールバックで何かを通過しなかったので、それは、する必要があります。このように変更します。

function login (username, password, callback) { 
    var query = "SELECT * FROM users WHERE username = ?"; 

    connection.query(query, [username], function (err, results) { 
     if (err) return callback(err); 
     if (results.length === 0) return callback(null, false); 
     var user = results[0]; 

     if (!bcrypt.compareSync(password, user.password)) { 
      return callback(null, false); 
     } 

     callback(null, true, { 
      id:   user.id.toString(), 
     }); 

    }); 
} 
app.get('/check', function (req, res) { 
    var username = 'mahdi'; 
    var originalPassword = 'a'; 

    login(username , originalPassword,function (err, result, id) { 
     console.log(err); 
     console.log(result); 
     console.log(id); 
    }); 
}); 

resultは、アクションのtrue|falseを見つけるためのものです。そしてresultがcallbackための真

またerrときidは意味あなたは既にNULLとしてコールバックの最初のパラメータを設定している