11

私はこの1時間、findOne、findOneOrCreateなどのメソッドを使用してpassport.jsのユーザーモジュールを作成しようと試みてきましたが、正しく理解できません。

User.js

var User = function(db) {
  this.db = db;
}

User.prototype.findOne(email, password, fn) {
  // some code here
}

module.exports = exports = User;

app.js

User = require('./lib/User')(db);
User.findOne(email, pw, callback);

私は何十ものエラーを経験してきましたが、ほとんどは

TypeError: object is not a function

また

TypeError: Object function () {
  function User(db) {
    console.log(db);
  }
} has no method 'findOne'

ユーザーのオブジェクト/インスタンスを作成せずに、これらの関数を使用して適切なモジュールを作成するにはどうすればよいですか？

アップデート

私は提案された解決策を検討しました：

var db;
function User(db) {
  this.db = db;
}
User.prototype.init = function(db) {
  return new User(db);
}
User.prototype.findOne = function(profile, fn) {}
module.exports = User;

運がない。

TypeError: Object function User(db) {
  this.db = db;
} has no method 'init'
4

2 に答える 2

16

ここではいくつかのことが起こっています。ソースコードを修正し、途中で説明するコメントを追加しました。

lib / User.js

// much more concise declaration
function User(db) {
    this.db = db;
}

// You need to assign a new function here
User.prototype.findOne = function (email, password, fn) {
    // some code here
}

// no need to overwrite `exports` ... since you're replacing `module.exports` itself
module.exports = User;

app.js

// don't forget `var`
// also don't call the require as a function, it's the class "declaration" you use to create new instances
var User = require('./lib/User');

// create a new instance of the user "class"
var user = new User(db);

// call findOne as an instance method
user.findOne(email, pw, callback);
于 2012-08-02T18:36:46.697 に答える
6

あなたnew User(db)はある時点でする必要があります。

あなたはinitメソッドを作ることができます

exports.init = function(db){
  return new User(db)
}

そしてあなたのコードから：

var User = require(...).init(db);
于 2012-08-02T17:13:57.010 に答える