2017-04-10 53 views
0

爲了學習目的,使用NodeJ編寫測試REST API。 目前我只有1路接受參數(工作正常)。在類方法中使用時,類中的屬性未定義

我正在使用快速路由器將GET請求路由到我的控制器。 所有的路由都按預期工作。

我的ServiceController目前有一個ctor函數可以接受2個參數。在實例化過程中,這兩個參數都由路由器傳遞給ctor函數。

在ServiceController中,我將參數存儲到字段中。

問題是,當我嘗試在類方法中訪問這些字段時,出現「TypeError:無法讀取未定義的屬性'exec'」。 我沒有寫這兩個值到控制檯,以確保ServiceController正確接收這些值(它是)。

所以,我不確定爲什麼即使在get方法中嘗試訪問「this.exec」或「this.logger」時出現此錯誤。

路由器

import express from 'express'; 
import { exec } from 'child-process-promise'; 
import ServiceController from '../controllers/serviceController'; 

let routes = (logger) => { 
const router = express.Router(); 
let controller = new ServiceController(exec, logger); 

router.route('/status/:name') 
    .get(controller.get); 

return router; 
}; 

module.exports = routes; 

ServiceController的

export default class ServiceController { 
constructor(childProcess, logger) { 
    this.logger = logger; 
    this.exec = childProcess; 
} 
get(req, res) { 
    if (!req.params.name) { 
     res.status(400).send('A service name was not provided'); 
    } else { 
     this.exec(`sc query ${req.params.name}`).then(result => { 
      if (result.stderr) { 
       this.logger.log.warn(`stderr: ${result.stderr}`); 
      } 
      const regex = /STATE\s+:\s+\d+\s+(\w+)/; 
      let [, status] = result.stdout.toString().match(regex); 
      if (!status) { 
       throw new Error('Status query unsuccessful'); 
      } 

      let service = { 
       name: req.params.name, 
       status: status 
      }; 

      res.json(service); 
      return service; 

     }).catch(error => { 
      this.logger.log.error(`${error.name} ${error.message}`); 
      res.status(500).send('An error occurred while executing command'); 
     }); 
    } 
} 
} 
+0

你是否也在'get'函數中記錄它們? –

回答

1

這是this方面的問題。在不是您的ServiceController實例的上下文中使用get方法。

綁定您的實例方法:

router.route('/status/:name') 
    .get(controller.get.bind(controller)); 

或者你也可以在ServiceController類中定義的arrow function

+0

謝謝。我對自己有點沮喪,我沒有聽清楚。 – WBuck