C++. Как записать класс в бинарный файл с помощью перегруженного оператора?

Question

[Миша Ягудин]

Мне нужно записать в бинарный файл свой класс:

class Book
{
public:
	Book(void);
	~Book(void);
	
	char *code;
	char *name;
	char *autor;
	char *publisher;
	char *year;
	bool remove;

	bool check(void);
};

Для этого я перегрузил оператор ввода:

ofstream& operator<<(ofstream &of, const Book &book)
{
	of.write(book.code, my_strlen(book.code));
	of.write("\n", 1);
	of.write(book.name, my_strlen(book.name));
	of.write("\n", 1);
	of.write(book.autor, my_strlen(book.autor));
	of.write("\n", 1);
	of.write(book.publisher, my_strlen(book.publisher));
	of.write("\n", 1);
	of.write(book.year, my_strlen(book.year));
	of.write("\n", 1);
	of.write(book.remove ? "1" : "0", 1);
	of.write("\n", 1);
	return (of);
}

Но с этой функцией у меня не компилится программа. И еще вопрос: что нужно возвращать из этой функции?

Comments

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

Какая ошибка при компиляции?

[Миша Ягудин]

Александр Ананьев, ошибся, при таком возвращаемом значении все работает. Но в файл пишет стандартный оператор ostream&operator<<(const void *val), и запись там вот такая: 0027CCA0

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

Вы что то делаете не так. Напишите сюда код записи в файл.

[Миша Ягудин]

bool add_notes(void)
{
	int count;
	Book *book;
	ofstream file;

	file.open(FILE_NAME);
	if (!file.is_open())
		return (false);
	cout << "Записей добавить: ";
	cin >> count;
	while (count-- > 0)
	{
		book = get_book();
		if (book->check())
			file << book;
	}
}

Answer 1

[Евгений Шатунов]

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

Твой operator<<(ofstream&, const Book&) должен быть определен в пространстве имен std, т.к. ofstreamопределен именно в этом пространстве.

Правильная перегрузка должна выглядеть так:

namespace std
{
    // use the `Book` type definitely from global namespace.
    ofstream& operator<<(ofstream &of, const ::Book &book)
    {
        // ...
    }
}

И еще вопрос: что нужно возвращать из этой функции?

Конкретно этот оператор (в терминологии стандартных стримов) возвращает левый операнд по ссылке.

Comments

[Миша Ягудин]

А если я определю этот оператор как дружественный классу Book?

[Евгений Шатунов]

Миша Ягудин, дружественность не спасет, т.к. вывод реализации оператора основывается в первую очередь на левом операнде. ADL работает очень строго и детерминировано.

[Евгений Шатунов]

Евгений Шатунов, к слову, дружественность - это принцип расширения внешнего интерфейса типа через все тот же ADL.
Если ты определяешь друга для Book, ты расширяешь внешний интерфейс именно для Book, но не для какого-либо иного типа.

[Миша Ягудин]

Евгений Шатунов, спасибо