0

以前にforループを使用してデータを入れようとしましたが、問題が大きくなりました。だから私はループを使用しようとしましたが、動作しますが、デバッグしようとすると、すべてのスロットwhileに入れ続けました。-858993460ファイルが適切な場所にあり、.dat開きます。

#include <iostream>
#include <fstream>
#include <string>
using namespace std;

struct record
{
    int item_id;
    string item_type;
    float item_price;
    int num_stock;
    string item_title;
    string item_author;
    int year_published;
};

void read_all_records(record records[], int &valid_entries);
int num_inventory_of_type(record records[], string type, int &valid_entries);
const int max_array = 100;
int main()
{
    int valid_entries = 0;
    record records[max_array];
    read_all_records(records, valid_entries);

    cout << "Stock Report" << endl;
    cout << "------------" << endl;
    int book = num_inventory_of_type(records, "book", valid_entries);
    cout << "Book's In Stock: " << book << endl;
    int cd = num_inventory_of_type(records, "cd", valid_entries);
    cout << "CD's In Stock: " << cd << endl;
    int dvd = num_inventory_of_type(records, "dvd", valid_entries);
    cout << "DVD's In Stock: " << dvd << endl;

    return 0;
}

void read_all_records(record records[], int &valid_entries)
{
    ifstream invfile;
    invfile.open("inventory.dat"); 
    if (!invfile.is_open())
    {
        cout<<"file open failed";
        exit(1);
    }
    while(invfile.good() && valid_entries < max_array)
    {
        invfile >> records[valid_entries].item_id >> records[valid_entries].item_type
            >> records[valid_entries].item_price >> records[valid_entries].num_stock
            >> records[valid_entries].item_title >> records[valid_entries].item_author
            >> records[valid_entries].year_published;
        if(!invfile.good())
            break;
        valid_entries++;

    }
    invfile.close();

}
int num_inventory_of_type(record records[], string type, int &valid_entries)
{
    int count = 0;
    int holder = 0;
    for (int count = 0; count<valid_entries; count++);
    {       
        if (records[count].item_type == type)
        {
            holder+=records[count].num_stock;

        }
    }

    return holder;
}

.dat ファイルは

123456
book
69.99
16
Problem_Solving_With_C++
Walter_Savitch
2011
123457
cd
9.99
32
Sigh_No_More
Mumford_and_Sons
2010
123458
dvd
17.99
15
Red_State
Kevin_Smith
2011
123459
cd
9.99
16
The_Church_Of_Rock_And_Roll
Foxy_Shazam
2012
123460
dvd
59.99
10
The_Walking_Dead_Season_1
Robert_Kirkman
2011

すべてが新しい行にあり、スペースはありません。

基本的に、起動してread_all_records関数を実行し、.datデータを配列に入れる必要があります。ただし、cout << records[count].item_id;データが実際に入っているかどうかを確認するためだけに while ループに入れて、-858993460毎回取得します。その後、次の関数を 3 回実行し、それぞれの書籍の数を返します。

4

3 に答える 3

1

これは問題に対する直接の答えではありませんが、リファクタリング後になくなる可能性があります。

std::istream& operator>>(std::istream& is, record& r) {
    return is >> r.item_id >> r.item_type >> … >> r.year_published;
}

int main () {
    if (std::ifstream invfile("inventory.dat")) {
        std::vector<record> records((std::istream_iterator<record>(invfile)),
                                     std::istream_iterator<record>());

        num_inventory_of_type(records, "dvd");
        num_inventory_of_type(records, "cd");
        num_inventory_of_type(records, "book");
    }
}

それでも各レコードを読みながら印刷したい場合は、コードを次のように再配置できます。

        std::vector<record> records;
        for (std::istream_iterator<record> i(invfile);
             i != std::istream_iterator<record>(); ++i)
        {
            records.push_back(*i);
            std::cout << i->item_id << "\n";
        }
于 2012-03-28T22:30:18.350 に答える
1

で整数型intを使用しましたitem_price。はの代わりにinvfile >> records[count].item_priceのみ抽出するため、 を抽出しようとするとエラーが発生します。6969.99year_published

代わりにfloatまたはを使用します。double

struct record
{
    int item_id;
    string item_type;
    float item_price;
    int num_stock;
    string item_title;
    string item_author;
    int year_published;
};

/* skipped identical lines */

while(invfile.good() && count < max_array)
{
    invfile >> records[count].item_id >> records[count].item_type
        >> records[count].item_price >> records[count].num_stock
        >> records[count].item_title >> records[count].item_author
        >> records[count].year_published;
        cout << records[count].item_price << endl;
        if(!invfile.good())
            break;
    cout << records[count].item_id << endl;
        count++;
}
invfile.close();

に余分なセミコロンがあることに注意してくださいfor (int count = 0; count<max_array; count++);。意図したものではないので、削除してください。

于 2012-03-28T22:40:36.560 に答える
0

int item_price; を変更する必要があります。float に so -> float item_price;

上記のように、count++ を交換する必要があります。cout << records[count].item_id 行。

これら 2 つの変更の後、正常に動作します。

struct record
{
    int item_id;
    string item_type;
    float item_price; // <--- Needs to be a float
    int num_stock;
    string item_title;
    string item_author;
    int year_published;
};


// This order is required because you are storing in the current 'count' record and then you need to print it.  Then increment the count to store the next record  
cout << records[count].item_id;
count++;
于 2012-03-28T22:39:57.823 に答える