c++ - 为什么不保存变量中的更改？

Question

我对 OOP 有点菜鸟，所以我可能犯了一个错误，但我找不到它，这是失败的代码部分:

util 文件只是一个包含错误消息的文件

在 main.cc 中:

int main(){

 Ship imperialDestroyer(IMPERIAL);
 Ship rebelShip(REBEL);

 cout<<imperialDestroyer<<endl; //this just print the ship, you can ignore it
 cout<<rebelShip<<endl;

 imperialDestroyer.improveFighter(); //this is what fails

 cout<<imperialDestroyer<<endl;
}

在 Ship.cc 中:

bool Ship::improveFighter(){

int num, cantidad, cost;
char option, respuesta;
bool improved = false;

cout << "Select fighter number: ";
cin >> num;
num = num-1;

if(num > this->fleet.getNumFighters() || num < 0)
  Util::error(WRONG_NUMBER);

else{
  cout << "What to improve (v/a/s)?";
  cin>>option;

if(option!='v' && option!='a' && option!='s')
  Util::error(UNKNOWN_OPTION);

else{
  cout << "Amount: ";
  cin >> cantidad;

  if(option == 'v')
cost = 2 * cantidad;

  else if(option == 'a')
cost = 3 * cantidad;

  else if(option == 's')
cost = (cantidad + 1) / 2;

  if(this->fleet.getCredits() < cost)
Util::error(NO_FUNDS);

  else{
cout << "That will cost you "<< cost <<" credits. Confirm? (y/n)";
cin >> respuesta;

if(respuesta == 'y'){
  this->fleet.improveFighter(num, option, cantidad, cost);
  improved = true;
}

  }
 }
}

return improved;
}

在 Fleet.cc 中:

void Fleet::improveFighter(int nf, char feature, int amount, int cost){

  if(feature == 'v'){
    getFighter(nf).increaseVelocity(amount);
  }

  else if(feature == 'a'){
    getFighter(nf).increaseAttack(amount); 
  }

  else if(feature == 's'){
    getFighter(nf).increaseShield(amount);   
  }
}

在 Fighter.cc 中:

Fighter Fleet::getFighter(int n) const{

  return fighters[n];
}

void Fleet::improveFighter(int nf, char feature, int amount, int cost){

  if(feature == 'v'){
    getFighter(nf).increaseVelocity(amount);
  }

  else if(feature == 'a'){
    getFighter(nf).increaseAttack(amount); 
  }

  else if(feature == 's'){
    getFighter(nf).increaseShield(amount);   
  }
}

由于某些原因，当我尝试改进某些功能时，它不会被保存。

Answer 1

Fighter Fleet::getFighter(int n) const
{
  return fighters[n];
}

这将返回位置 n 处的 Fighter 的拷贝。修改拷贝不会影响原件。

您可以返回一个Fighter&(即一个reference)，但是因为您的函数是const，所以这将不起作用。您将必须决定您希望此功能成为什么。