Почему считывается с файла только 1 слово, а не весь текст?

Question

[ashot_powershot]

#include <iostream>
#include <string>
#include<Windows.h>
#include<fstream>
#include"Head.h"

using namespace std;

int main()
{
	SetConsoleCP(1251);
	SetConsoleOutputCP(1251);
	setlocale(LC_ALL, "");

	const int size = 4;
	string address1, path = "myfile.txt";;
	int home1, floor1, room1, square1, u=3;
	House arr[size];
	ofstream fout;
	ifstream fin;

	House g("Молокова", 26, 47, 4, 68);
	arr[0].setsuper("Молокова", 26, 47, 4, 68);
	House h("Ленина", 10, 18, 6, 93);
	arr[1].setsuper("Ленина", 10, 18, 6, 93);
	House i(h);
	arr[2].setsuper("Ленина", 10, 18, 6, 93);
	cout << "Улица,номер дома, этаж, кол-во комнат, площадь " << endl;
	for (int a = 3; a < 4; a++)
	{
		cin >> address1;
		cin >> home1;
		cin >> floor1;
		cin >> room1;
		cin >> square1;

		arr[a].setaddress(address1);
		arr[a].sethome(home1);
		arr[a].setfloor(floor1);
		arr[a].setroom(room1);
		arr[a].setsquare(square1);
	}
	cout << endl;
	cout << "Вывод массива: " << endl;
	for (int i = 0; i <= 3; i++)
	{
		arr[i].getinfo();
		cout << endl;
	}
	cout << endl;
	fout.open(path);
	if (!fout.is_open())
	{
		cout << "Ошибка открытия файла!" << endl;
	}
	else
	{
		cout << "Файл открыт!" << endl;
		for (int i = 0; i < size; i++)
		{
			fout.write((char*)&arr[i], sizeof(House));
		}
	}
	fout.close();
	fin.open(path);
	if (!fin.is_open())
	{
		cout << "Ошибка открытия файла!" << endl;
	}
	else
	{
		House pnt;
		cout << "Файл открыт!" << endl;
		while (fin.read((char*)&pnt, sizeof(House)))
		{
			pnt.getinfo();
		}
	}

	fin.close();

}

74 строка, не понимаю почему не работает

Comments

[Александр Ананьев]

fin.read((char*)&pnt, sizeof(House))
Вот здесь, ты что пытаешься вычитать? В текстовом файле нет никаких структур типа House

[ashot_powershot]

Александр Ананьев, у меня класс в файле заголовка находится. Я хочу чтобы в переменную pnt считывалось с файла, а размер ее House(класса)

Answer 1

[res2001]

Не раскрыта тема о классе House. Не видя его трудно что-то сказать.

Но могу предположить, что в House, например, поле address является std::stringом.
А std::string хранит свою строку в динамической памяти и sizeof(std::string) в общем случае совсем не равен размеру хранящейся в нем строки. Записывая в файл sizeof(std::string) байт вы записываете грубо говоря только адрес памяти где лежит строка и размер строки, но не саму строку.
Вам нужно делать нормальную сериализацию/десериализацию полей объекта House.

Comments

[ashot_powershot]

#include <iostream>
#include <string>
#include<Windows.h>
#include<fstream>

using namespace std;

class House
{
private:
	string address;
	int home, floor, room, square;
public:
	House();
	House(string address2, int home2, int floor2, int room2, int square2);
	void getinfo();
	string getaddress();
	int gethome();
	int getfloor();
	int getroom();
	int getsquare();
	void setaddress(string address1);
	void sethome(int home1);
	void setfloor(int floor1);
	void setroom(int room1);
	void setsquare(int square1);
	void setsuper(string address1, int home1, int floor1, int room1, int square1);
};

Вот мой класс. fin.read((char*)&pnt этим способом я пытался записать строку в переменную pnt, а размер у меня House. И после этого просто вывести переменную pnt

[res2001]

ashot_powershot, Так и есть. Проблемы у вас из-за string address;
std::string нельзя вот так на прямую сохранять в файл и читать.
Причину и решение описывал выше.

Когда вы пишите std::string в файл, то писать нужно не сам объект, а std::string.c_str() и еще перед этим надо записать длину строки, чтоб потом знать сколько байт из файла надо прочитать для строки.
Конкретно для вашего класса, можно поступить немного по другому, перед записью класса вычисляете общую длину класса, затем пишете home, floor, room, square как обычно. Длина address вычисляется из длины класса - sizeof(home, floor, room, square). Длина класса при этом это не sizeof(Home), а sizeof(home, floor, room, square) + address.size().

В общем добавляете к классу 2 метода сериализатор и десериализатор.
Сериализатор принимает в параметре ostream и пишет класс в поток, как я описал выше.
Десериализатор принимает в параметре istream и загружает класс из потока в обратном от записи порядке.

Это касается не только std::string, но и любых объектов, у которых в данных есть указатели или ссылки. Например std::vector, да и любой контейнер из std. А так же любой ваш собственный класс. Все подобные классы требуют специального подхода для записи и чтения.
Аналогично классы с виртуальными методами, т.к. в таких класса, помимо данных самого класса, есть еще и указатель на таблицу виртуальных функций, которую не нужно сохранять в файл (и читать из файла).

[res2001]

ashot_powershot, Писать напрямую, можно только те классы, которые содержат в качестве членов данных только простые типы (POD) и не содержат виртуальных функций.
Таким образом можно немного схитрить. Например заменить string address, на char address[MAX_ADDRESS_LENGTH]; И тогда можно не писать сериализатор, и записывать в файл напрямую (как сейчас).
Но я бы не советовал так делать. Вам нужно понять проблему и научится ее решать, т.к. на практике это встречается очень часто и не всегда эту проблему можно обойти.

[res2001]

ashot_powershot, Да и еще.
Вам нужно определится вы хотите записывать файл в двоичном режиме или в текстовом.
Если в двоичном, то и открывать файлы нужно в двоичном и применять подход с сериализатором, о котором я писал.
Если в текстовом, то с длинной address можно поступить еще другим способом. address.c_str() писать после всех числовых данных, как последнее поле. И после адреса добавлять в файл перевод строки. Таким образом у вас на одной строке будет распологаться 1 объект.
При чтении, сначала читаете числа, затем оставшуюся строку до символа конца строки - это и будет address.

Вообще при чтении данных из любого потока (файл, сеть, буфер и т.д.) есть только 3 варианта отделить одно поле от другого в потоке:
1. Вы изначально знаете размер поля в потоке и читаете именно столько байт, сколько требуется. Это работает для всех стандартных числовых типов.
2. Если данные переменной длины (std::string), то
2.1. Сначала пишем длину данных. Сама длина данных должна иметь фиксированный размер, затем читаете данные. Длину надо писать до самих данных, что бы при чтении сначала прочитать длину и определить сколько после нее идет данных.
2.2. Используете разделитель. Т.е. пишете данные переменной длинны и в конце данных вставляете символ разделитель. Это как раз вариант с переводом строки, где символ перевода строки является разделителем. Если бы у вас в классе Home было несколько строк, то их можно было бы писать друг за другом вставляя между ними какой-то другой разделитель, например символ '\0'
Других вариантов не существует.

Answer 2

[Wataru]

Если просто через >> читать std::string из std::cin, то он будет читать одно слово.
Чтобы прочитать всю стороку используйте cin.getline.

Comments

[ashot_powershot]

#include <iostream>
#include <string>
#include<Windows.h>
#include<fstream>
#include"Head.h"

using namespace std;

int main()
{
	SetConsoleCP(1251);
	SetConsoleOutputCP(1251);
	setlocale(LC_ALL, "");

	const int size = 4;
	string address1, path = "myfile.txt", pnt;
	int home1, floor1, room1, square1, u=3;
	House arr[size];
	ofstream fout;
	ifstream fin;

	House g("Молокова", 26, 47, 4, 68);
	arr[0].setsuper("Молокова", 26, 47, 4, 68);
	House h("Ленина", 10, 18, 6, 93);
	arr[1].setsuper("Ленина", 10, 18, 6, 93);
	House i(h);
	arr[2].setsuper("Ленина", 10, 18, 6, 93);
	cout << "Улица,номер дома, этаж, кол-во комнат, площадь " << endl;
	for (int a = 3; a < 4; a++)
	{
		cin >> address1;
		cin >> home1;
		cin >> floor1;
		cin >> room1;
		cin >> square1;

		arr[a].setaddress(address1);
		arr[a].sethome(home1);
		arr[a].setfloor(floor1);
		arr[a].setroom(room1);
		arr[a].setsquare(square1);
	}
	cout << endl;
	cout << "Вывод массива: " << endl;
	for (int i = 0; i <= 3; i++)
	{
		arr[i].getinfo();
		cout << endl;
	}
	cout << endl;
	fout.open(path);
	if (!fout.is_open())
	{
		cout << "Ошибка открытия файла!" << endl;
	}
	else
	{
		cout << "Файл открыт!" << endl;
		for (int i = 0; i < size; i++)
		{
			fout.write((char*)&arr[i], sizeof(House));
		}
	}
	fout.close();
	fin.open(path);
	if (!fin.is_open())
	{
		cout << "Ошибка открытия файла!" << endl;
	}
	else
	{
		cout << "Файл открыт!" << endl;
		while (fin.read((char*)&pnt, sizeof(House)))
		{
			getline(cin, pnt);
		}
	}

	fin.close();

}
Не помогло, ничего не выводит и читает только 1 слово.