默认参数:在非静态成员函数之外无效使用'this'

时间:2015-03-26 18:30:38

标签: c++ arguments default

我尝试在我的函数中有一个默认参数,但编译器说有一个错误:

invalid use of 'this' outside of a non-static member function

我该如何解决这个问题?

编辑: @RSahu,这是两个重载函数,你能解释一下我如何管理这个问题,因为显然我不明白如何修复它。

Game.hpp:

class Game {
 private :
  int** board;

  vector<pair <int, int> > listPiecesPosition();
  vector<pair <int, int> > listPiecesPosition(int** board);

  // What doesn't work
  vector<pair <int, int> > listPiecesPosition(int** board = this->board); 

Game.cpp:

//Here I need to write more or less two times the same function, how can I do it only once ?

   vector<pair <int, int> > Game::listPiecesPosition() {
vector<pair <int, int> > listPiecesPosition;
for (int i=0; i < getSize(); i++)
  for (int j=0; j < getSize(); j++)
    if (getBoard()[i][j] == nextPlayer.getColor()) // Here I don't use the parameter
      listPiecesPosition.push_back(make_pair(i,j));
return listPiecesPosition;
  }

  vector<pair <int, int> > Game::listPiecesPosition(int** board) {
    vector<pair <int, int> > listPiecesPosition;
    for (int i=0; i < getSize(); i++)
      for (int j=0; j < getSize(); j++)
        if (board[i][j] == nextPlayer.getColor()) // Here I use the parameter
          listPiecesPosition.push_back(make_pair(i,j));
    return listPiecesPosition;
  }

谢谢你的帮助!

1 个答案:

答案 0 :(得分:4)

this只能在非静态成员函数体内使用。因此,使用this->board作为输入的默认值是不正确的。

我建议创建一个重载来解决问题。

class Game {
 private :
  int** board;

  vector<pair <int, int> > listPiecesPosition(int** board);
  vector<pair <int, int> > listPiecesPosition()
  {
     return listPiecesPosition(this->board);
  }

PS this可以在有限的上下文中显示在成员函数体外。这不是其中之一。

更新,以回应OP的评论

更改

vector<pair <int, int> > Game::listPiecesPosition() {
   vector<pair <int, int> > listPiecesPosition;
   for (int i=0; i < getSize(); i++)
      for (int j=0; j < getSize(); j++)
         if (getBoard()[i][j] == nextPlayer.getColor()) // Here I don't use the parameter
            listPiecesPosition.push_back(make_pair(i,j));
   return listPiecesPosition;
}

vector<pair <int, int> > Game::listPiecesPosition() {
   return listPiecesPosition(this->board);
}

通过这样做,可以避免重复实现函数主逻辑的代码。