コールバックは関数ノードではありませんjs

Question

私はjavascriptを初めて使用しています。このエラーを解決するには問題があります。私は "コールバックは関数ではありません"というメッセージを返します： "return callback（rolesArray）"。

Rol.getAllRoles = function(callback){ 
 
    sql = "select role from Role;"; 
 
    var rolesArray = []; 
 
    var role; 
 
    mysql.connection(function(err,conn){ 
 
     if (err){ 
 
      return callback(err); 
 
     } 
 
     conn.query(sql,function(err,rows){ 
 
      if (err){ 
 
       return callback(err); 
 
      } 
 
      for(var i=0; i < rows.length; i++){ 
 
       role = rows[i].role; 
 
       rolesArray.push(rol); 
 
      } 
 
      console.log("roles: " + rolesArray); 
 
      return callback(rolesArray); 
 
     }); 
 
    }); 
 
}

はconsole.log出力： "役割：管理者、顧客の" データベースの作品との接続そう。

Answer 1

私は次のことをお勧めしたい：

Rol.getAllRoles = function(callback){ 
 
    var sql = "select role from Role;"; 
 
    var rolesArray = []; 
 
    var role; 
 
    callback = callback || function(){}; 
 
    mysql.connection(function(err,conn){ 
 
     if (err){ 
 
      return callback(err); 
 
     } 
 
     conn.query(sql,function(err,rows){ 
 
      if (err){ 
 
       return callback(err); 
 
      } 
 
      for(var i=0; i < rows.length; i++){ 
 
       role = rows[i].role; 
 
       rolesArray.push(rol); 
 
      } 
 
      console.log("roles: " + rolesArray); 
 
      return callback(rolesArray); 
 
     }); 
 
    }); 
 
}

あなたがそのコールバックを強制するこの方法では、常に機能です。 Rol.getAllRoles（）のように実行すると、以前はエラーが発生します。今あなたはいない。あなたはもう一度データを取得しません。

適切なパラメータ（つまり、関数）でRol.getAllRolesを呼び出していることを確認してください。

Answer 2

このエラーは、呼び出すときに関数Rol.getAllRoles(fn)に関数を渡していないことを意味します。そのため、コールバックで適切なエラー処理を行うことができるため、エラーと変更する必要のある実際のデータをより簡単に区別することができます。エラーの有無を示すコールバックに常に最初の引数を渡します。

Rol.getAllRoles = function(callback){ 
    sql = "select role from Role;"; 
    var rolesArray = []; 
    var role; 
    mysql.connection(function(err,conn){ 
     if (err){ 
      return callback(err); 
     } 
     conn.query(sql,function(err,rows){ 
      if (err){ 
       return callback(err); 
      } 
      for(var i=0; i < rows.length; i++){ 
       role = rows[i].role; 
       rolesArray.push(rol); 
      } 
      console.log("roles: " + rolesArray); 
      // make sure the first argument to the callback 
      // is an error value, null if no error 
      return callback(null, rolesArray); 
     }); 
    }); 
} 

そして、あなたはこのようにそれを呼び出すべきである：

Rol.getAllRoles(function(err, rolesArray) { 
    if (err) { 
     // handle error here 
    } else { 
     // process rolesArray here 
    } 
}); 

のように非同期コールバックを呼び出してこのスタイルの第二引数（そうでない場合はエラー）は、このようなあなたの結果の配列にすることができますcallback(err, data)は非常に一般的な非同期コールバックデザインパターンです。これにより、すべての発信者は、エラーがあったかどうか、および最終結果にアクセスするためのエラーがないかどうかを確認できます。