2017-04-10 8 views
0

学習用にNodeJsでテストREST APIを記述します。 現在、私はパラメータを受け入れる1つのルートしかありません(これはうまくいきます)。クラスのメソッドで使用される場合、クラスのプロパティは未定義です

私はエクスプレスルータを使用して、GET要求をコントローラにルーティングしています。 ルーティングはすべて期待通りに機能しています。

私のServiceControllerは現在、2つのパラメータを受け入れるctor関数を持っています。これらのパラメータの両方は、インスタンス化中にルータによってctor関数に渡されます。

ServiceController ctorでは、パラメータをフィールドに格納します。

問題は、クラスメソッドでこれらのフィールドにアクセスしようとすると、 "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

関連する問題