[英]Why is “this” undefined in this class method?
我试图搜索似乎整个互联网的内容,但我仍然对JS类的问题感到烦恼,我正在为微服务写作(仍在学习中)。
所以,我尝试在实例化的对象上调用一个类方法,并根据我的知识和我的(我猜错的)单元测试它应该工作。
好吧,我将从收到的错误开始:
GET /api/users 500 2.863 ms - 2649
TypeError: Cannot read property 'repository' of undefined
at list (C:\Users\<user>\Documents\Programming\node\kaguwa-ngn\kaguwa-user-service\controllers\user-controller.js:20:9)
at Layer.handle [as handle_request] (C:\Users\<user>\Documents\Programming\node\kaguwa-ngn\kaguwa-user-service\node_modules\express\lib\router\layer.js:95:5)
at next (C:\Users\<user>\Documents\Programming\node\kaguwa-ngn\kaguwa-user-service\node_modules\express\lib\router\route.js:137:13)
(还有更多)。
代码调用代码:
user-controller.js
'use strict';
var utils = require('./utils');
class UserController {
constructor(repository) {
this.repository = repository || {};
}
/**
*
* Lists all users.
*
* @param {object} req
* @param {object} res
*/
list(req, res) {
this.repository.list(function (err, users) {
if (err) return res.status(500).json(utils.createError(500));
if (Object.keys(users).length !== 0) {
res.json(users);
} else {
res.status(404).json(utils.createNotFound('user', true));
}
});
}
// more code
}
module.exports = UserController
控制器的来电者
user-api.js
'use strict';
var express = require('express');
var UserController = require('../controllers/user-controller');
var router = express.Router();
module.exports = function (options) {
var userController = new UserController(options.repository);
router.get('/users', userController.list);
// Mode code
return router;
};
我真的不知道为什么this
在UserController
未定义。
任何帮助都会受到极大的关注。
当你这样做:
router.get('/users', userController.list);
传递给路由器的只是对.list
方法的引用。 userController实例丢失了。 这不是路由器所特有的 - 这是Javascript中传递内容的通用属性。 要进一步了解,您实际上在做的是:
let list = userController.list;
// at this point the list variable has no connection at all to userController
router.get('/users', list);
而且,在Javascript的strict
模式下,当你调用一个没有任何对象引用的常规函数时,如上面的调用list()
,那么this
将在函数内部undefined
。 这就是你的例子中发生的事情。 要修复它,您需要确保使用适当的对象引用调用您的方法,如userController.list(...)
以便解释器适当地设置this
值。
有多种方法可以解决此问题:
制作自己的函数包装器
router.get('/users', function(req, res) {
userController.list(req, res);
});
这适用于任何版本的Javascript。
使用.bind()
为您创建一个使用正确对象调用它的包装器
router.get('/users', userController.list.bind(userController));
这适用于ES5 +或.bind()
polyfill。
使用ES6箭头功能快捷方式
router.get('/users', (...args) => userController.list(...args));
这适用于ES6 +
就个人而言,我更喜欢.bind()
实现,因为我认为它比其他任何一个更简单,更具说明性/更清晰,而且ES6“快捷方式”并不是真正的缩短。
router.get()
不会以你认为的方式调用你的类。 你给它一个函数的引用,它将在router.get
的上下文中调用。这意味着它不会在你的userController的上下文中。
您可以通过以下方式解决此问题
router.get('/users', function(){userController.list(...arguments)});
换句话说,没有明确使用userController
对list
的引用,已经明确使用了一个带有给定参数的userController
调用list
的闭包。
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.