帮我检查一下这段代码的错误

[kearo]

题目是要求用二维数组来存储三年来每月的书的销售数量，并且求出每年的总量，以及三年的全部总量。
我编译得出的结果是一大串负数，不知道问题出在哪，实在是找不出来。


#include <iostream>
#include <string>
const int months = 12;
const int years = 3;

int main()
{
using namespace std;
const string month[months] =
{
"January ",
"Feburary ",
"March ",
"April ",
"May ",
"June ",
"July ",
"August ",
"September",
"October ",
"November ",
"December "
};
int sales[years][months];
int sum1 ,sum2, sum3;
sum1 = 0;
sum2 = 0;
sum3 = 0;
int sum = 0;
cout << "请输入02-04年每月的书销量" << endl;
for (int Month = 0; Month < months; ++Month)
{
cout << month[Month] << ": \t";
for (int year = 0; year < years; ++year)
{
cin >> sales[years][months];
cout << "\t";
};
cout << endl;
};
for (int i = 0; i < months; ++i)
{
sum1 += sales[0][months];
sum2 += sales[1][months];
sum3 += sales[2][months];
};
sum = sum1 + sum2 +sum3;
cout << "02年总销售量： " << sum1 << endl;
cout << "03年总销售量： " << sum2 << endl;
cout << "04年总销售量： " << sum3 << endl;
cout << "02-04年的总销售量： " << sum << endl;

return 0;
}

[kearo]

不对吧，我这个数组下标应该没问题吧？应该是这样啊。

for (int Month = 0; Month < months; ++Month)
{
cout << month[Month] << ": \t";
for (int year = 0; year < years; ++year)
{
cin >> sales[year][Month]; //新定义的两个变量，让它们在之前定义的常量范围内递增。
cout << "\t";
};

[realrukawa]

int sales[years][months]={0}；

//sum1 += sales[0][months];//数组下标错误
//sum2 += sales[1][months];
//sum3 += sales[2][months];
这个地方我也略有疑问，同上面解答，“这里的数组下标用i,你用的是最大值months”

[realrukawa]

虽然没仔细看，我也觉得
for (int i = 0; i < months; ++i)
{
//sum1 += sales[0][months];//数组下标错误
//sum2 += sales[1][months];
//sum3 += sales[2][months];
sum1 += sales[0][i]; //这里的数组下标用i,你用的是最大值months
sum2 += sales[1][i];
sum3 += sales[2][i];
};
这个循环看起来十分古怪，把12月的sales连续加12次？

[ReachZh]

Error:
1:
cin >> sales[years][months];
2:
sum1 += sales[0][i];
sum2 += sales[1][i];
sum3 += sales[2][i];

[kearo]

对对对，是下标错误，粗心了，多谢大家指点！

[healer_kx]

应该是某个变量没有初始化,
但是总得来讲,楼主的习惯很好.继续努力.

[healer_kx]

int sales[years][months];
这个没有清零.

[kearo]

怎么清零啊？

[DonaldKnuth]

下面两个地方错了，改了就好了

#include <iostream>
#include <string>
const int months = 12;
const int years = 3;

int main()
{
using namespace std;
const string month[months] =
{
"January ",
"Feburary ",
"March ",
"April ",
"May ",
"June ",
"July ",
"August ",
"September",
"October ",
"November ",
"December "
};
int sales[years][months];
int sum1 ,sum2, sum3, sum;
sum1 = sum2 = sum3 = sum =0;

cout << "请输入02-04年每月的书销量" << endl;
for (int Month = 0; Month < months; ++Month)
{
cout << month[Month] << ": \t";
for (int year = 0; year < years; ++year)
{
//cin >> sales[years][months]; //数组下标错误
cin >> sales[year][Month]; //这里的两个数组下标变量的问题
cout << "\t";
};
cout << endl;
};
for (int i = 0; i < months; ++i)
{
//sum1 += sales[0][months];//数组下标错误
//sum2 += sales[1][months];
//sum3 += sales[2][months];
sum1 += sales[0][i]; //这里的数组下标用i,你用的是最大值months
sum2 += sales[1][i];
sum3 += sales[2][i];
};
sum = sum1 + sum2 +sum3;
cout << "02年总销售量： " << sum1 << endl;
cout << "03年总销售量： " << sum2 << endl;
cout << "04年总销售量： " << sum3 << endl;
cout << "02-04年的总销售量： " << sum << endl;

return 0;
}

[sharpoo7]

for (int year = 0; year < years; ++year)
{
cin >> sales[years][months];
~~~~~~~year?

cout << "\t";
};

[lyy1089]

是 啊，怎么清0。我也想知道，楼主得编译方法很正规啊。