Не перезаписываются ссылки

@Eumi · Регистрация: 12.06.2012

Author24 — интернет-сервис помощи студентам

Добрый вечер! помоги, пожалуйста.

Вычитываю из файла значения, и хочу переписать вектор ссылок на новый. Но! когда встаю в отладке на выходе из метода все хорошо, но в момент когда я пытаюсь обратиться к ним, для вывода на экран. Приложение крашиться. В отладке видно, что элемент есть, но значения полей некорректные. Подскажите, что я делаю не так?

Перезаписыаю в методе void Coordinator::loadFromFile()
Обращаюсь по методу QTableWidgetItem* Coordinator::getFio(int playerNumber)

Кликните здесь для просмотра всего текста

C++ (Qt)

Coordinator::Coordinator()
{
    footballTeam = new Team();
}
 
void Coordinator::addPlayerToTeam(Player *aPlayer){
    footballTeam->addPlayerToTeam(aPlayer);
}
 
int Coordinator::getTeamSize()
{
    return footballTeam->getSizeTeam();
}
 
QTableWidgetItem* Coordinator::getFio(int playerNumber)
{
    QTableWidgetItem *fioWidget = new QTableWidgetItem(QString::fromStdString(footballTeam->getPlayer(playerNumber)->getFio()));
    return fioWidget;
}
 
QTableWidgetItem* Coordinator::getBirthDay(int playerNumber)
{
    QTableWidgetItem *birthDayWidget = new QTableWidgetItem(QString::fromStdString(footballTeam->getPlayer(playerNumber)->getBirthDay()));
    return birthDayWidget;
}
 
QTableWidgetItem* Coordinator::getTeamRole(int playerNumber)
{
    QTableWidgetItem *teamRoleWidget = new QTableWidgetItem(QString::fromStdString(footballTeam->getPlayer(playerNumber)->getTeamRole()));
    return teamRoleWidget;
}
 
QTableWidgetItem* Coordinator::getPlayedGames(int playerNumber)
{
    QTableWidgetItem *playedGamesWidget = new QTableWidgetItem(QString::number(footballTeam->getPlayer(playerNumber)->getNumberPlayedGames()));
    return playedGamesWidget;
}
 
QTableWidgetItem* Coordinator::getWonGames(int playerNumber)
{
    QTableWidgetItem *wonGamesWidget = new QTableWidgetItem(QString::number(footballTeam->getPlayer(playerNumber)->getNumberWinGames()));
    return wonGamesWidget;
}
 
QTableWidgetItem* Coordinator::getTeamNumber(int playerNumber)
{
    QTableWidgetItem *teamNumberWidget = new QTableWidgetItem(QString::number(footballTeam->getPlayer(playerNumber)->getTeamNumber()));
    return teamNumberWidget;
}
 
void Coordinator::loadFromFile()
{
    ifstream file("team.txt", std::ios::binary);
    size_t count = 0;
    file.read(reinterpret_cast<char*>(&count), sizeof(size_t));
    footballTeam->teamClear();
    for (int i=0; i<count; ++i)
    {
        Player testPlayer;
        file.read(reinterpret_cast<char*>(&testPlayer), sizeof(testPlayer));
        footballTeam->addPlayerToTeam(&testPlayer);
    }
    file.close();
}

Кликните здесь для просмотра всего текста

C++ (Qt)

#include "team.h"
#include <player.h>
#include <QDebug>
#include <algorithm>
 
Team::Team()
{
    teamList.reserve(10);
}
 
Team::~Team()
{
 
}
 
vector<Player*> Team::getTeamList() const
{
    return teamList;
}
 
Player* Team::getPlayer(int playerNumber)
{
    return teamList[playerNumber];
}
 
int Team::getSizeTeam() const
{
    return teamList.size();
}
 
void Team::sortTeam()
{
    qDebug()<<"Sort!";
    sort(teamList.begin(), teamList.end(), [](const Player* p1, const Player* p2) -> bool { return p1->getTeamNumber() < p2->getTeamNumber() ; });
 
}
 
void Team::setTeamList(const vector<Player *> &value)
{
    teamList = value;
}
 
void Team::teamClear()
{
    teamList.clear();
}
 
void Team::addPlayerToTeam(Player *aPlayer){
    teamList.push_back(aPlayer);
}

nd2 · 22.06.2017, 00:57

Сообщение от Eumi

хочу переписать вектор ссылок на новый

Вектор указателей?

Сообщение от Eumi

C++ (Qt)

for (int i=0; i<count; ++i) 
{ 
      Player testPlayer; 
      file.read(reinterpret_cast<char*>(&testPlayer), sizeof(testPlayer)); 
      footballTeam->addPlayerToTeam(&testPlayer); 
}

Сохраняешь адрес локального объекта, который исчезает в конце итерации цикла.

Добавлено через 3 минуты
И что из себя представляет класс Player?

@Eumi · 22.06.2017, 01:05 **[ТС]**

Сообщение от nd2

Сохраняешь адрес локального объекта, который исчезает в конце итерации цикла.

И что из себя представляет класс Player?

как же быть? если мне надо считать N объектов

Player

Кликните здесь для просмотра всего текста

C++ (Qt)

#include "player.h"
#include <algorithm>
#include <fstream>
 
Player::Player(QTextEdit *debugEdit)
{
    debugEdit->append("Работает конструктор сущности Player");
}
 
Player::Player()
{
 
}
 
Player::~Player()
{
 
}
 
int Player::getNumberPlayedGames() const
{
    return numberPlayedGames;
}
 
void Player::setNumberPlayedGames(int value)
{
    numberPlayedGames = value;
}
 
int Player::getNumberWinGames() const
{
    return numberWinGames;
}
 
void Player::setNumberWinGames(int value)
{
    numberWinGames = value;
}
 
int Player::getTeamNumber() const
{
    return teamNumber;
}
 
void Player::setTeamNumber(int value)
{
    teamNumber = value;
}
 
string Player::getTeamRole() const
{
    return teamRole;
}
 
void Player::setTeamRole(const string &value)
{
    teamRole = value;
}
 
string Player::getBirthDay() const
{
    return birthDay;
}
 
void Player::setBirthDay(const string &value)
{
    birthDay = value;
}
 
string Player::getFio() const
{
    return fio;
}
 
void Player::setFio(const string &value)
{
    fio = value;
}

nd2 · 22.06.2017, 01:16

Сообщение от Eumi

Player

Не то, player.h покажи.

@Eumi · 22.06.2017, 01:22 **[ТС]**

Сообщение от nd2

Не то, player.h покажи.

Кликните здесь для просмотра всего текста

C++ (Qt)

#ifndef PLAYER_H
#define PLAYER_H
#include <string>
#include <QTextEdit>
#include <algorithm>
 
using namespace std;
 
class Player
{
public:
    Player(QTextEdit *debugEdit);
    Player();
    ~Player();
    int getNumberPlayedGames() const;
    void setNumberPlayedGames(int value);
 
    int getNumberWinGames() const;
    void setNumberWinGames(int value);
 
    int getTeamNumber() const;
    void setTeamNumber(int value);
 
    string getTeamRole() const;
    void setTeamRole(const string &value);
 
    string getBirthDay() const;
    void setBirthDay(const string &value);
 
    string getFio() const;
    void setFio(const string &value);
    string playerToString();
 
private:
    string fio;
    string birthDay;
    string teamRole;
    int teamNumber;
    int numberWinGames;
    int numberPlayedGames;
};
 
#endif // PLAYER_H

nd2 · 22.06.2017, 01:25

И как ты объекты этого класса в файл записываешь?

@Eumi · 22.06.2017, 01:29 **[ТС]**

Сообщение от nd2

И как ты объекты этого класса в файл записываешь?

Считывается вроде нормально, в отладке объект корректный.

C++ (Qt)

void Coordinator::saveToFile()
{
    ofstream out("team.txt",ios::binary|ios::out);
    size_t count = footballTeam->getTeamList().size();
    out.write((char*)(&count), sizeof(int));
    for (int i = 0; i<footballTeam->getSizeTeam(); i++){
        out.write(reinterpret_cast<char*>(footballTeam->getPlayer(i)), sizeof(*(footballTeam->getPlayer(i))));
    }
    out.close();
    helper->setSaveList(footballTeam->getTeamList());
}

nd2 · 22.06.2017, 01:34

Твой класс Player не относится к POD-типу, поэтому не годится для бинарных методов записи/чтения.

@Eumi · 22.06.2017, 01:42 **[ТС]**

Сообщение от nd2

Твой класс Player не относится к POD-типу, поэтому не годится для бинарных методов записи/чтения.

Тогда я не понимаю, если "не годится", почему я в отладке вижу что он считал и считал так как я вводила?

nd2 · 22.06.2017, 01:54

Сообщение от Eumi

почему я в отладке вижу что он считал и считал так как я вводила?

Это всё UB. Зависит от реализации std::string, от длины используемых строк, от того: читается ли записанный файл в этой же программе (без выхода из неё), от фазы луны, активности солнца и т.д.

Добавлено через 2 минуты
Запись и чтение структуры с файла

@TRam_ · 22.06.2017, 02:26

Eumi, почитай про сериализацию/десериализацию данных

@TRam_ зомбяк 1584 / 1218 / 345 Регистрация: 14.05.2017 Сообщений: 3,939
	22.06.2017, 02:26	11
	Eumi, почитай про сериализацию/десериализацию данных 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	22.06.2017, 01:16	4
	Сообщение от Eumi Player Не то, player.h покажи. 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	22.06.2017, 01:25	6
	И как ты объекты этого класса в файл записываешь? 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	22.06.2017, 01:34	8
	Твой класс Player не относится к POD-типу, поэтому не годится для бинарных методов записи/чтения. 0

@Eumi 5 / 5 / 1 Регистрация: 12.06.2012 Сообщений: 60
	22.06.2017, 01:42 [ТС]	9
	Сообщение от nd2 Твой класс Player не относится к POD-типу, поэтому не годится для бинарных методов записи/чтения. Тогда я не понимаю, если "не годится", почему я в отладке вижу что он считал и считал так как я вводила? Миниатюры 0

nd2 3437 / 2816 / 1249 Регистрация: 29.01.2016 Сообщений: 9,426
	22.06.2017, 01:54	10
	Сообщение от Eumi почему я в отладке вижу что он считал и считал так как я вводила? Это всё UB. Зависит от реализации std::string, от длины используемых строк, от того: читается ли записанный файл в этой же программе (без выхода из неё), от фазы луны, активности солнца и т.д. Добавлено через 2 минуты Запись и чтение структуры с файла 0