Как передать в parent значение в конструкторе?

Question

[Abc Edc]

Родительский класс который никак не может принять model, вернее консоль показывает что this.model вполне себе существует и имеет все функции с монго

function MainController(model) {
        this.model = model;
        console.log(this.model);
    }
    MainController.prototype.findAll = function(req, res) {
            this.model.find({}, function (err, results) {
                if (err)  res.json({"findAll": err});
                res.json(results);
            });
    };
    MainController.prototype.findById = function(req,res) {
        var id = new ObjectID(req.params.id)
        this.model.findById(id, function (err, results) {
            if (err)  res.json({"findById":err});
            res.json(results);
        })
    };
    MainController.prototype.create = function(req, res){
            var saver = new this.model(req.body);
            saver.save(function(err,results, affected){
                if (err) throw  err;
                res.json({'create':'yes'});
            });
    };


module.exports = MainController;

Потомок

var util = require('util');
var User = require('../models/user').User;
ObjectID = require('mongodb').ObjectID;
var MainController = require('./index');
function UserController  () {
    MainController.call(this,User);
    this.model = User;
};
util.inherits(UserController, MainController);
UserController.prototype.checkAuth =  function(req, res, next){
    if (req.session.user) {
        res.json({"login": req.session.user});
    }
    else{
        next();
    }
};
UserController.prototype.login = function(req, res, next){
        var mail = req.body.mail;
        var password  = req.body.password;
        model.autorize(mail,password,function(err, result, user){
                if (result.login == 'yes'){
                    req.session.user = user._id;
                    res.json(result);
                }
                else{
                    res.json({"login":"no"});
                }
            }
        )

};

module.exports =  UserController;

var UserController = require('../controllers/user');
var userController = new UserController();
var uploader =   require('../single/fileTransfer') ;
module.exports = function(app) {
app.get('/users',userController.findAll);

Вот тут он пожалуется что

TypeError: Cannot call method 'find' of undefined
at MainController.findAll (/home/azureuser/test/controllers/index.js:7:24)
at Layer.handle [as handle_request] (/home/azureuser/test/node_modules/express/lib/router/layer.js:82:5)
at next (/home/azureuser/test/node_modules/express/lib/router/route.js:110:13)
at Route.dispatch (/home/azureuser/test/node_modules/express/lib/router/route.js:91:3)
at Layer.handle [as handle_request] (/home/azureuser/test/node_modules/express/lib/router/layer.js:82:5)
at /home/azureuser/test/node_modules/express/lib/router/index.js:267:22
at Function.proto.process_params (/home/azureuser/test/node_modules/express/lib/router/index.js:321:12)
at next (/home/azureuser/test/node_modules/express/lib/router/index.js:261:10)
at /home/azureuser/test/node_modules/express-session/index.js:421:7
at /home/azureuser/test/node_modules/connect-mongo/lib/connect-mongo.js:295:11

Но почему? почему же консоль тогда говорит что User передался и как решить проблему

Answer 1

[Илья Шатохин]

Я не удивлюсь, если this подменяет express. Вообще в документации express рекомендуется замыкать midleware

MainController.prototype.findAll = function() {
  var self = this;
  return function (req, res) {
    self.model.find({}, function (err, results) {
      if (err)  res.json({"findAll": err}); 
        res.json(results); // вы забыли else, здесь будет ошибка повторной отправки данных!
     });
   }
};
/* -------------- */
app.get('/users', userController.findAll());

Comments

[Abc Edc]

А как тогда он примет req,res

[Abc Edc]

а все понял

[Abc Edc]

СПАСИБО ВЫ ПРАВЫ ВСЕ ЗАРАБОТАЛО!!

[Abc Edc]

А как вызвать дочерние методы скажите пожавлуйста?

[Abc Edc]

у меня там так же TypeError: Cannot call method 'autorize' of undefined

[Abc Edc]

app.post('/login', userController.checkAuth, userController.login); Этот к примеру

[Илья Шатохин]

ГЛЕБ ГЛЕБОВ: все middleware должны возвращать замыкание, как я показал. Вместо this внутри них используйте self

[Abc Edc]

Илья Шатохин: Дада все я понял огромное спасибо!А для того чтобы переопределить метод мне нужно просто еще раз объявить его?

[Илья Шатохин]

ГЛЕБ ГЛЕБОВ: не понял вопроса, у кого переопределить?

[Abc Edc]

Илья Шатохин: У юзера переопределить метод к примеру MainController.prototype.findAll = function() {
var self = this;
return function (req, res) {
self.model.find({}, function (err, results) {
if (err) res.json({"findAll": err});
res.json(results);
});
}
};
Я знаю что это отдельный вопрос, но подскажите пожалуйтса

[Илья Шатохин]

ГЛЕБ ГЛЕБОВ: UserController.prototype.findAll = "новый код", только старый app.get('/users') придется отвязать и привязать заново на новый метод.

[Abc Edc]

Илья Шатохин: Все, супер огромное еще раз СПАСССИИИБОО!!

[Abc Edc]

Илья Шатохин: Просто магия когда userController.findById(app.req,app.res) явно не предаешь а просто
userController.findById() а оно видит рек и рес

[Илья Шатохин]

ГЛЕБ ГЛЕБОВ: не магия, а замыкание, которое возвращает другую функцию, которая принимает req и res :)