Где кроется ошибка в попытке наследования?

Question

[Abc Edc]

Это родительский класс

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

        }
    }

};
module.exports = MainController;

Тут у нас дочерний

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

};
util.inherits(UserController, mainController);
module.exports = UserController;

ну вы поняли.. Предполагается получить все родительские методы
но что то не задалось

util.js:556
ctor.prototype = Object.create(superCtor.prototype, {
^
TypeError: Cannot read property 'prototype' of undefined
at Object.exports.inherits (util.js:556:43)
at Object. (/home/azureuser/testproject/controllers/user.js:34:6)
at Module._compile (module.js:456:26)
at Object.Module._extensions..js (module.js:474:10)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:312:12)
at Module.require (module.js:364:17)
at new require (module.js:380:17)
at Object. (/home/azureuser/testproject/routes/index.js:1:84)
at Module._compile (module.js:456:26)
azureuser@testonlyy:~/testproject$

Answer 1

[Константин Китманов]

Количество ошибок просто зашкаливает. Из какого-то кривого источника копипастили, да?(
Попробуйте вот так (не тестил):

// Родительский

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

    }
};

module.exports = MainController;

// дочерний
var util = require('util');
var User = require('../models/user').User;
ObjectID = require('mongodb').ObjectID;
var MainController = require('./parent/index')();

var UserController = function () {
    this.model = User;
};

UserController.prototype = {
    prototype: UserController,
    checkAuth: function (req, res, next) {
        if (req.session.user) {
            res.json({login: req.session.user});
        } else {
            next();
        }
    },
    login: function (req, res, next) {
        var mail = req.body.mail;
        var password = req.body.password;
        User.autorize(mail, password, function (err, result, user) {
            if (result.login == 'yes') {
                req.session.user = user._id;
                res.json(result);
            }
            else {
                res.json({login: "no"});
            }
        }
        )
    }
};

util.inherits(UserController, MainController);
module.exports = UserController;

Comments

[Abc Edc]

Я думаю если бы копипастил их было бы меньше, а тут я так тренируюсь просто:)

[Константин Китманов]

ГЛЕБ ГЛЕБОВ: в общем, делать *.prototype = function() { не надо. В прототип пишут обычный объект литералом.
И делать из конструктора return {someMethod: function () {}} тоже никакого смысла нет (наверное, есть случаи, когда такое нужно, но я не могу их придумать) — возвращенный из конструктора объект заменит то, что должно было возвратиться (и наследование не получится).

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

Константин Китманов: можно придумать случай с замыканием (пример сугубо топорный):

UserController.prototype = (function() {
  var args = arguments;
  return {
    init: function () { foo.apply(this, args); }
  } 
})(a,b,c);

[Константин Китманов]

Илья Шатохин: да, можно так. Но на практике я бы не стал, постарался бы как-нибудь обойти.

[Abc Edc]

Константин Китманов: А как тогда будет выглядеть приватный метод или переменная

[Константин Китманов]

ГЛЕБ ГЛЕБОВ: Во-первых, чтобы эмулировать приватные члены, совсем необязательно возвращать объект из конструктора. Во-вторых, я для себя давно отказался от эмуляции приватных членов. В Python'е без них живут, значит, и нам не позорно:) Только некоторые сущности помечаю как protected и название их начинается с подчеркивания.

Тем не менее, если использовать модульную систему — CommonJS/RequireJS/ES2015 — то можно иметь сущности, которые живут исключительно в рамках модуля и наружу не торчат.

[Abc Edc]

Константин Китманов: Спасибо большое, последний вопрос, можете уточнить как именно вызывать метода из дочернего класса а то что то на андифайнед жалется

[Константин Китманов]

ГЛЕБ ГЛЕБОВ: в доках есть пример, смотрите: https://nodejs.org/api/util.html#util_util_inherit...

[Константин Китманов]

ГЛЕБ ГЛЕБОВ: упс, примерчика-то там и нету. Примерно так:

someMethod: function () {
        return this.constructor.super_.someMethod()
    }

[Abc Edc]

Константин Китманов: app.get('/users', UserController.findAll); Я про вот это

[Abc Edc]

Константин Китманов: или вы и имели ввиду

[Константин Китманов]

ГЛЕБ ГЛЕБОВ: посмотрите внимательно, вы пытаетесь использовать метод у UserController, тогда как этот метод есть только у его прототипа или у инстанса этого класса.

[Abc Edc]

Константин Китманов: Воот я про это говорю, как альтернативно php сделать что если classchild extends classparent то и new classchild () без б вызовет унаследованные функции

[Константин Китманов]

ГЛЕБ ГЛЕБОВ: еще раз перечитайте мой комментарий, пожалуйста. Вы пытаетесь вызвать findAll, как будто это статический метод, но он таковым не является.

[Abc Edc]

someMethod: function () {
return this.constructor.super_.someMethod()
}
То есть каждый метод все равно надо описать в дочернем?

[Константин Китманов]

ГЛЕБ ГЛЕБОВ: нет, только если вы в дочернем что-то переопределяете.

[Константин Китманов]

ГЛЕБ ГЛЕБОВ: я подозреваю, вот вы что хотите на самом деле:

var userController = new UserController();
app.get('/users', userController.findAll.bind(userController));