0

以下のコードを使用して、100 個の空のレコードを持つファイルを初期化しようとしています。

void initializeInventory() {
    std::ofstream out("hardware.dat", std::ios::binary);

    Hardware h;
    for (int i = 0; i < 100; ++i) {
        h.ID = i;
        h.name = "try"; // std::string();
        h.quantity = 0;
        h.price = 0;
        h.notes = "try2"; //std::string();

        out.write(reinterpret_cast<const char*>(&h), sizeof(Hardware));
    }

    out.close();
}

しかし、それらを印刷しようとすると、常に 25 個の要素しか印刷されないか、クラッシュします。
これは要素を出力する関数です:

void readInventory() {
    std::ifstream in("hardware.dat", std::ios::in);
    std::cout << std::setiosflags(std::ios::left) << std::setw(4) << "ID"
              << std::setw(16) << "Name"
              << std::setw(11) << "Quantity"
              << std::setw(10) << std::resetiosflags(std::ios::left)
              << "Price"
              << std::setw(50) << "Notes" << '\n';

    Hardware h;

    while (!in.eof()) {
        in.read(reinterpret_cast<char*>(&h), sizeof(Hardware));

        //if (!in.eof())
            printOut(std::cout, h);
    }

    in.close();
}

void printOut(std::ostream &output, const Hardware& h) {
    output << std::setiosflags(std::ios::left) << std::setw(4) << h.ID
           << std::setw(16) << h.name
           << std::setw(11) << h.quantity
           << std::setw(10) << std::setprecision(2)
           << std::resetiosflags(std::ios::left)
           << std::setiosflags(std::ios::fixed | std::ios::showpoint )
           << h.price
           << std::setw(50) << h.notes << '\n';
}

また、for のサイクル数を増やすと (100 ではなく 400 を入れてみました)、hardware.datファイルが大きくなるように見えるので、印刷機能の問題であると考えました。何か案が?足りないものはありますか?
前もって感謝します。

4

1 に答える 1

0

演算子 << をオーバーロードするのが最善です

friend ostream& operator<<(ostream& out, const Hardware& h) // output
{
    out << "(" << h.id() << ", " << h.whatever() << ")";
    return out;
}

ファイルを 1 行ずつハードウェア オブジェクトに読み込みます。ちなみに、入力演算子 >> のオーバーロードも可能です。

于 2013-06-04T15:44:25.630 に答える