C++ C++；对象没有'；不值钱

我为

Player

类创建的对象不会改变私有变量，即使它在函数中改变了它。

main（）

函数中使用的对象可能存在一些引用问题，但我不知道在哪里或为什么

具体地说，主文件末尾的循环将在向量

players

中打印玩家赢得的游戏，但所有分数都返回0。当player

object.matchWon（）

函数正确解析和打印分数并显示对私有变量所做的更改时

我怀疑主代码中的

getPlayer（）

函数的使用存在问题

提前谢谢

Player.cpp

：

#include "Player.h"
#include <stdlib.h>
#include <iostream>

using namespace std;
//constructor
Player::Player(string first, string last) {
    Player::first = first;
    Player::last = last;
    gWins = 0;
    gLoss = 0;
    mWins = 10;
    mLoss = 2;
}

//setters
void Player::addLoss(int increment) {
    this->gLoss+=increment;
}

void Player::addWins(int increment) {
    this->gWins+=increment;
}

void Player::matchWon(vector<string> scores) {
    for (int i = 5;i<scores.size()-1;i++){
        cout<<(int)scores[i][0]-'0'<<"-";
        cout<<(int)scores[i][2]-'0'<<endl;
        gWins+=scores[i][0]-'0'; //add games won
        cout<<gWins<<endl;
        gLoss+=(int)scores[i][2]-'0';   //add games lost
    }
    this->mWins++;
}

void Player::matchLost(vector<string> scores) {
    this->mLoss++;
}

double Player::winPercentage() {
    return (double)mWins / mLoss;
}

//accessors
string Player::getFirstname() {
    return this->first;
}

string Player::getLastname() {
    return this->last;
}

int Player::getGameswon() {
    //cout<<gWins;
    return this->gWins;
}

#include <iostream>
#include "player.h"
#include <fstream>
#include <vector>
#include <sstream>

using namespace std;

vector<string> split(const string &s) {
    vector<string> elems;
    istringstream iss(s);
    do
    {
        string sub;
        iss >> sub;
        elems.push_back(sub);

        //cout << "Substring: " << sub << endl;

    } while (iss);

    return elems;
}

Player &getPlayer(vector<Player> &players, const string &first, const string &last){
    for (int i=0;i<players.size();i++){
        if (players[i].getFirstname()==first&&players[i].getLastname()==last){
            return players[i];
        }
    }
    players.push_back(Player(first,last));
    return (players[players.size()-1]);
}

int main(int argc, char *argv[]) {

    ifstream file(argv[1]);
    ofstream ofile(argv[2]);
    if (!file.is_open()){
        cerr <<"Could not open file\n";
        return 0;
    }

    string line;
    vector<Player> players;

    while (getline(file,line).good()){

        vector<string> lineParsed = split(line);
        vector<string> matchData = split(line);

        Player p1 = getPlayer(players,lineParsed[0],lineParsed[1]);
        Player p2 = getPlayer(players,lineParsed[3],lineParsed[4]);

        p1.matchWon(lineParsed);
        cout<<p1.getFirstname()<<"!"<<p1.getGameswon()<<endl;
    }

    for (int i=0;i<players.size();i++){
        //cout<<players.size();
        cout<<players[i].getFirstname()<<":"<<players[i].getGameswon()<<endl;
    }


    return 0;
}

---

如果包含类定义，则会有所帮助。似乎发生了一些奇怪的事情：

Player::first = first;
Player::last = last;

---

这些是静态场吗？如果是，问题是您只存储创建的最后一个玩家的名字。

您正在复制玩家：

Player p1 = getPlayer(players,lineParsed[0],lineParsed[1]);
Player p2 = getPlayer(players,lineParsed[3],lineParsed[4]);

因此，当您

p1.matchWon（）

时，它发生在本地

Player

对象上，而不是向量中的对象

尽管

getplayer（）

的返回类型是

Player&

，但如果将其分配给非引用变量，这并不重要。如果要修改向量中的

Player

实例，应该

Player& p1 = getPlayer(players,lineParsed[0],lineParsed[1]);
Player& p2 = getPlayer(players,lineParsed[3],lineParsed[4]);

---

顺便说一下，

getPlayer（）

是不安全的。当您将推回该向量时，该向量可能会自行调整大小，从而使引用无效。您可能需要存储一个指针向量（或者只需通过例如

resize（）

确保该向量确实有空间容纳您想要推送的所有玩家）。

看看这个答案，谢谢您的回复！！当向量调整大小时，它可能会更改存储变量的内部引用？这就是你所说的嵌入引用的意思吗？向量的内容可能必须移动到不同的内存位置。有关详细信息，请参阅。