嵌套 for 循环 (C++) 无法正常工作

Nested For Loop (C++) Not Working Properly

本文关键字:常工作 工作 for C++ 嵌套 循环      更新时间:2023-10-16

我正在做一些编程作业,我正在尝试使用for-loop来促进编码过程。这是循环:

#ifndef DIVSALES_H
#define DIVSALES_H
class DivSales
{
    public:
        DivSales(){ quarterSales[4] = {0}; };
        double getTotalSales() { return totalSales;}
        static void setTotalSales(double);
        static void addTotalSales(double);
        double getQuarterSales(int numQuarter) {return quarterSales[numQuarter];}
        void setQuarterSales(int numQuarter, double numAmount) { quarterSales[numQuarter] = numAmount;}
    private:
        static double totalSales;
        double quarterSales[];
};
double DivSales::totalSales = 0;
void DivSales::setTotalSales(double totalAmount) {totalSales = totalAmount; }
void DivSales::addTotalSales(double addAmount) {totalSales += addAmount; }
#endif // DIVSALES_H

#include <iostream>
#include "DivSales.h"
using namespace std;
int main()
{
    const int NUMDIVS = 6;
    const int NUMQUARTERS = 4;
    double amount = 0;
    DivSales divs[NUMDIVS];
    for(int division = 0; division < NUMDIVS; division++)
    {
        cout << "Division " << (division + 1) << endl;
        for(int quarter = 0; quarter < NUMQUARTERS; quarter++)
        {
            cout << "Quarter " << (quarter + 1) << ": ";
            cin >> amount;
            divs[division].setQuarterSales(quarter, amount);
            DivSales::addTotalSales(amount);
        }
    }

    return 0;
}

输出示例:

Division 1
Quarter 1: 500
Quarter 2: 500
Quarter 3: 500
Quarter 2: 500
Quarter 3: 500
Quarter 2: 500
Quarter 3: 500
Quarter 2: 500
Quarter 3: 500
Quarter 2: 

我试图做的是,当我输入一个部门的 4 个季度的数字时,它将移动到下一个部门。但是,在 4 个输入之后,它不会递增for-loop的除法变量,而是继续请求更多输入。这是怎么回事?

我已经找到了导致该问题的原因,它在文件 DivSales.h 中:

更改此行:

double quarterSales[];

对于此行:

double quarterSales[4];

问题是您没有为 4 个元素的数组分配内存。要初始化它,请将构造函数更改为:

DivSales():quarterSales({0}){ };

您还应该将以下行移动到 DivSales.cpp,否则我会收到多重定义错误:

double DivSales::totalSales = 0;

在这里:

divs[division].setTotalSales(amount);

您可能希望:

divs[division].addTotalSales(amount);

但这不是导致您的问题的原因。我无法复制。