c++ - 将文件的内容添加到结构

Question

我应该创建一个程序来为我收藏的书籍编制索引。该结构包含通常的书籍信息:书名、作者、出版商等。但是，我没有得到任何输出。一个问题是标题会有空格。

/* Book Inventory assignment 2 by Heath Martens. */

#include <iostream>
#include <stdlib.h>
#include <fstream>
#include <cstring> // I had to throw this in in order to get memcpy to work.

using namespace std;

typedef struct book{
    char title[100];
    char author[100];
    char publisher[100];
    float price;
    int isbn;
    int pages;
    int copies;
} Book;

Book collection[100];
int currentIndex;


void
indexBook(Book *my_book)
{
    memcpy(&collection[currentIndex], my_book, sizeof(Book));
    currentIndex++;
}

void
readfile(void)
{
    fstream my_stream;
    string line = " ";
    my_stream.open("input.txt");
    int i=0;
    for (i = 0; i < currentIndex; i++)
        {
            while (getline(my_stream, line))
            {
                cin >> line >> collection[i].title;
                cin >> line >> collection[i].author;
                cin >> line >> collection[i].publisher;
                cin >> line >> collection[i].price;
                cin >> line >> collection[i].isbn;
                cin >> line >> collection[i].pages;
                cin >> line >> collection[i].copies;
            }
        }

    my_stream.close();

}

void
printCollection(void)
{
    int i;
    for (i = 0; i < currentIndex; i++)
    {
        cout << "Title: " << "\t\t" << collection[i].title << endl;
        cout << "Author: " << "\t" << collection[i].author << endl;
        cout << "Publisher: " << "\t" << collection[i].publisher << endl;
        cout << "Price: " << "\t\t" << collection[i].price << endl;
        cout << "ISBN: " << "\t\t" << collection[i].isbn << endl;
        cout << "Pages: " << "\t\t" << collection[i].pages << endl;
        cout << "Copies: " << "\t" << collection[i].copies << endl;
    }
}

void printCollection(void);

int
main(void)
{
    currentIndex = 0;

    Book *my_book = new Book;

    indexBook(my_book);

    readfile();

    printCollection();

    delete my_book;

    return 0;
}

这是我要使用的 txt 文件。

Magician: Apprentice
Raymond E. Feist
Spectra (January 1, 1994)
5.02
0553564943
512
1
Magician: Master
Raymond E. Feist
Spectra (January 1, 1994)
7.99
0553564935
499
1

这里是基于所提供的一些示例的更新代码。

    void
    readfile(void)
    {
       fstream my_stream ("input.txt");
       if(!my_stream)
    {
       return;
    }
     string line = " ";
     int i=0;
for (i = 0; i < currentIndex; i++)
    {
        if(!std::getline(my_stream, line))
        {
            break;
        }
        memcpy(collection[currentIndex].title, line.c_str(), std::min(sizeof(collection[currentIndex].title), line.size()));
        memcpy(collection[currentIndex].author, line.c_str(), std::min(sizeof(collection[currentIndex].author), line.size()));
        memcpy(collection[currentIndex].publisher, line.c_str(), std::min(sizeof(collection[currentIndex].publisher), line.size()));
        my_stream >> collection[currentIndex].price;
        my_stream >> collection[currentIndex].isbn;
        my_stream >> collection[currentIndex].pages;
        my_stream >> collection[currentIndex].copies;
        my_stream.ignore();
        if(!my_stream)
        {
            break;
        }
    }

Error in output

Answer 1

主要问题出在 readfile 函数上。如评论中所述， std::fstream 可以将字符串作为第一个参数，因此不必稍后调用 open 。此外，在对文件执行操作之前，应该检查文件是否打开。 std::string 不需要初始化以便稍后在此函数中使用。这导致以下代码。

fstream my_stream("input.txt");
if(!my_stream) {
    return;
}
string line;

readfile 的循环似乎没有正确构造。外循环在集合数组的当前索引范围内递增，而内循环看起来好像要读取整个文件。如果内循环构造正确，则文件的内容将写入集合中的第 0 本书。内部循环从测试是否从文件中读取一行(通过从 std::cin 读取输入而被忽略)是否成功开始。

因为 input.txt 中“Books”的数量未知，外循环似乎与将所有“Books”读入不同集合元素的目标无关。为了读取整个文件，我们将更改循环以检查文件是否仍然可读。

while(my_stream) {

为了读取字符串，我们需要使用 std::getline(my_stream >> 行； 这里会被空格绊倒)读取并存储整行，并使用 memcpy 复制行的内容放入对应的char数组中。因为 std::getline 可能会失败，所以我们在使用 line 的内容之前检查是否成功。例如，这里的标题:

// title
if(!std::getline(my_stream, line)) {
    break;
}
memcpy(collection[currentIndex].title, line.c_str(), std::min(sizeof(collection[currentIndex].title), line.size()));

对于数字，我们可以使用运算符>>读取数字并忽略行尾的任何额外字符。同样，此操作也可能会失败并进行检查。例如，这里的价格:

    my_stream >> collection[currentIndex].price; 
    my_stream.ignore(); 
    if(!my_stream) { 
        break;
    } 

如果当前 Book 的所有成员都已被正确读取，那么在循环体的末尾，我们将增加已读取的 Books 数量 (++currentIndex;)。如评论中所述，在 readfile 的上下文中不需要显式关闭 my_stream，因为在范围末尾调用 my_stream 的析构函数时文件将关闭。

一些额外的小问题。正如上面评论中所讨论的，std::string 应该用于 Book::title、Book::author 和 Book::publisher。这是因为 std::string 可以更优雅地处理字符数未知的情况，而无需显式管理内存。同样，集合更适合标准容器(例如，std::vector)。这确实会导致 indexBook 的当前实现出现问题，可以将其更改为使用 Book 的复制构造函数存储在集合中。对于 Book IO，可以重载 operator>> 和 operator<< 以替换和简化 readfile 和 printCollection 中的代码。