Question

问题是：ABC硬件公司已聘请您为其应收帐款部门编写程序。主文件按客户编号按升序排列，客户名称和余额为20。事务文件包含具有客户编号的每个事务的记录。您将从两个文件中一次读取一个记录，并使用事务文件更新主文件中的信息。在继续下一个主记录之前处理所有事务记录。如果事务记录在第1列中包含“O”，则计算orderamount并将其添加到到期余额中。如果记录在第1列中包含“P”，则从到期余额中减去付款。保持ABC公司的AR余额（每个客户的余额总和）的总计。在处理主记录及其所有交易后，程序应为每个客户准备一份发票，其中列出了客户名称，编号，以前的余额，所有交易以及到期的最终余额。

The output should look like:

CUSTOMER NAME     CUSTOMER NUMBER
                  PREVIOUS BALANCE $XXX.XX
(ALL TRANSACTIONS PER CUSTOMER:)
TRANSACTION#    ITEM ORDERED    $ORDER AMOOUNT
TRANSACTION#    ITEM ORDERED    $ORDER AMOUNT
TRANSACTION#    PAYMENT         $PAYMENT AMOUNT 
                            BALANCE DUE $XXX.XX

我已经尝试过更改数组，if语句等等。现在运行程序时根本没有打印。请帮忙！

这是我到目前为止的代码：

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

struct master {
    double custnum;
    string name;
    double balance;
    };

 struct transactions {
    char transtype;
    int custnum;
    int transnum;
    string item;
    int quantity;
    double price;
    double amountpaid;
    };

 int main ()
 {
   ifstream masterfile ("MASTER.txt");
   ifstream transfile ("TRANSACTION.txt");
   int prevbalance[7];
    master main [7];

for (int i=0; !masterfile.eof(); i++) {
    masterfile>>main[i].custnum>>main[i].name>>main[i].balance;
    }

    for (int i=0;!masterfile.eof();i++) {
        cout << main[i].custnum<<" ";
        cout << main[i].name<<" ";
        cout << main[i].balance<<" "<<
        endl<<endl;
prevbalance[i] = main[i].balance;
    }

double companybalance = 0;
double orderamt=0;

transactions tran[35];
for (int i=0; !transfile.eof(); i++) {{
    transfile>> tran[i].transtype;
    cout<<tran[i].transtype<<" ";

    if (tran[i].transtype == 'O') {
    transfile>>tran[i].custnum;
    cout<<tran[i].custnum<<" ";

    transfile>> tran[i].transnum;
    cout<<tran[i].transnum<<" ";

    transfile>>tran[i].item;
    cout<<tran[i].item<<" ";

    transfile>>tran[i].quantity;
    cout<<tran[i].quantity<<" ";

    transfile>>tran[i].price;
    cout<<tran[i].price<<" "<<endl<<endl;

orderamt= tran[i].price*tran[i].quantity;
main[i].balance+= orderamt;

companybalance += main[i].balance;

    }
    else if (tran[i].transtype == 'P'){
    transfile>>tran[i].custnum;
    cout<<tran[i].custnum<<" ";

    transfile>> tran[i].transnum;
    cout<<tran[i].transnum<<" ";

    transfile>>tran[i].amountpaid;
    cout<<tran[i].amountpaid<<endl<<endl<<endl;

main[i].balance-tran[i].amountpaid;

companybalance += main[i].balance;

    }}
for(int i=0; i<50; i++) {
cout<<"Name: "<< main[i].name <<"   Customer #: "<< main[i].custnum<<endl<<endl;
cout<<"Previous Balance "<<prevbalance[i]<<endl;
for(int j=0; j<7; j++){
cout<<"Transaction #: "<<tran[j].transnum<<"    "<<tran[j].item<<"   $"<<orderamt<<endl; }
cout<<"Balance Due: "<<main[i].balance<<endl;
}

}}

以下是输入的两个文件，Masterfile：

1000 TIFFANY 7000.99
2000 MARY 6500.98
3000 JACOB 6560.99
4000 GENE 4560.98
5000 BELLA 5300.87
6000 ANNA 2340.90
7000 DEMI 4230.45

和交易文件：

O 1000 1000 PENS 20 2
O 1000 2000 CPUS 2 200
O 1000 3000 MONITER 2 100
P 1000 4000 4000
P 1000 5000 300

O 2000 6000 CPUS 3 500
O 2000 7000 MOUSE 3 50
O 2000 8000 WIRES 5 8
P 2000 9000 600
P 2000 1100 798

O 3000 1200 MONITERS 6 60
O 3000 1300 CPUS 7 300
O 3000 1400 MOUSE 30 40
O 3000 1500 SPEAKERS 20 20
P 3000 1600 5000

O 4000 1001 SPEAKERS 2 50
O 4000 2002 CABLES 4 20
P 4000 3003 400
P 4000 4004 500
P 4000 5005 68

P 5000 6001 600
P 5000 4002 55
P 5000 2003 450
O 5000 4004 SPEAKERS 4 60
O 5000 1005 LAPTOP 3 300

O 6000 6001 TVS 5 400
O 6000 8002 SPEAKERS 5 70
P 6000 6003 2000
P 6000 8004 1000
O 6000 8005 CABLES 10 15

O 7000 5001 PENS 50 2
O 7000 7002 PAPER 400 2
P 7000 4003 150
P 7000 3004 230
P 7000 6005 450

Answer 1

您使用masterfile.eof()作为循环条件两次。

for (int i=0; !masterfile.eof(); i++) {
    masterfile>>main[i].custnum>>main[i].name>>main[i].balance;
}

for (int i=0;!masterfile.eof();i++) {
    cout << main[i].custnum<<" ";
    cout << main[i].name<<" ";
    cout << main[i].balance<<" "<<
    endl<<endl;
    prevbalance[i] = main[i].balance;
}

想想这怎么可能不起作用。在第一个循环结束时，您已经到达文件的末尾。第二个循环即将开始之前完成！

当然，您可以将文件指针重置为开头，但更好的方法是保留您拥有的记录数，并将该计数用于第二个循环。

int iRecordCount = 0;
for (iRecordCount = 0; !masterfile.eof(); iRecordCount++) {
    masterfile>>main[iRecordCount].custnum>>main[iRecordCount].name>>main[iRecordCount].balance;
}

for (int i=0; i < iRecordCount;i++) {
    cout << main[i].custnum<<" ";
    cout << main[i].name<<" ";
    cout << main[i].balance<<" "<<
    endl<<endl;
    prevbalance[i] = main[i].balance;
}

另请注意，您正在使用固定数组（master main[7]）来存储记录 - 如果您的输入文件中包含超过7条记录，会发生什么？你将超出数组的范围。实际上您应该更改为使用std::vector以便数组可以动态增长，但如果您不想这样做，则应在读取记录时进行额外检查以确保不会溢出阵列：

int iRecordCount = 0;
for (iRecordCount = 0; iRecordCount < sizeof(main) / sizeof(main[0]) && !masterfile.eof();
    iRecordCount++) {
    masterfile>>main[iRecordCount].custnum>>main[iRecordCount].name>>main[iRecordCount].balance;
}

在阅读交易时，你也做了一些奇怪的事情。你有类似的东西：

orderamt= tran[i].price*tran[i].quantity;
main[i].balance+= orderamt;

如果i是交易号（tran[i]），那么它也可以是客户编号（main[i]）？你被允许使用你知道的i以外的变量名。

您的代码底部也有：

for(int i=0; i<50; i++) {
....
}

同样，您应该使用数组中的实际元素数（iRecordCount）而不是固定数字（因为您的数组大小只有7个元素，所以50来自哪里？）。

Answer 2

除了@JonathanPotter in his answer提出的建议之外，我想补充一点，在for循环中使用<div class="table"> <div class="header">Header</div> <div class="header">Header</div> </div>容易出错。

有用的阅读材料：Why is “while ( !feof (file) )” always wrong?。

在ifstream.eof()循环的条件下使用!masterfile.eof()出于完全相同的原因是错误的。

最好使用：

for

您也可以使用类似的逻辑来读取交易记录。

另外，你有一句话：

std::vector<master> masterData;
while ( masterfile )
{
   // Try to read the data into a temporary object.
   master m;
   masterfile >> m.custnum >> m.name >> m.balance;

   // If the input was successful, add it to masterData
   if ( masterfile )
   {
      masterData.push_back(m);
   }
}

我确信这是一个错字，你打算使用：

 main[i].balance - tran[i].amountpaid;

我无法弄清楚我的c ++程序有什么问题

2 个答案: