Question

我的目标是让程序尽可能高效。我一直坚持这个问题很长一段时间，当我搜索它时，我被告知flush / endl cout语句。

当我开始调试时，我推断出问题是for循环。它只会跳过for循环，导致长度，宽度，高度为0。

#include <iostream>
#include <string>
#include <array>
using std::cout;    using std::cin;
using std::string;   using std::flush;
using std::endl;

void main()
{
    int length=0, width=0, height=0, volume=0;
    int VolCalcs[3]={length, width, height};

    string Prompt[3] = {"Please enter length: ", "Please enter width: ", "Please enter height: "};
    string NewResult[3] = {"The new length is ", "The new width is ", "The new height is "};

    for(int i=0;i==3;++i)
    {
        cout<<endl;
        cout<<Prompt[i]<<flush;
        cin>>VolCalcs[i];
    }

    volume=length*width*height;
    cout<<" The Volume is: "<<volume<<endl;

    length++;
    width--;
    height+=10;


    for(int i=0;i==3;++i)
    {
        cout<<NewResult[i] << VolCalcs[i] <<endl;
    }
    volume=length*width*height;
    cout<<" The New Volume is: "<<volume<<endl<<endl;
    cout<<"Press Enter to End Program"<<flush;
    cin.ignore();
}

输出如下：

The Volume is: 0

The New Volume is: -10

Press Enter to End Program

Answer 1

我看到以下错误：

main应始终返回int。
for循环条件应为for (int i = 0; i != 3; ++i)。请注意，您需要更改两个for循环。 for循环条件i != 3表示如果i != 3，循环应该继续。
您将数组的使用与变量的使用混合在一起。有两种选择：
- 使用变量：删除VolCalcs并仅使用纯变量length, width, height。删除循环。
- 使用数组。删除length, width, height并仅使用数组。在下面的修复中，我仅使用length, width, height进行数组初始化，但这些变量不会在代码中进一步使用。

#include <iostream>
#include <string>
#include <array>
using std::cout;    using std::cin;
using std::string;   using std::flush;
using std::endl;
int main()
{
    int length=0, width=0, height=0, volume=0;
    int VolCalcs[3]={length, width, height};
    string Prompt[3] = {"Please enter length: ", "Please enter width: ", "Please enter height: "};
    string NewResult[3] = {"The new length is ", "The new width is ", "The new height is "};

    for (int i=0; i != 3; ++i)
    {
        cout<<endl;
        cout<<Prompt[i]<<flush;
        cin>>VolCalcs[i];
    }

    volume=VolCalcs[0]*VolCalcs[1]*VolCalcs[2];
    cout << " The Volume is: "<<volume<<endl;

    VolCalcs[0]++;
    VolCalcs[1]--;
    VolCalcs[2]+=10;


    for(int i=0;i!=3;++i)
    {
        cout<<NewResult[i] << VolCalcs[i] <<endl;
    }
    volume=VolCalcs[0]*VolCalcs[1]*VolCalcs[2];
    cout<<" The New Volume is: "<<volume<<endl<<endl;
    cout<<"Press Enter to End Program"<<flush;
    cin.ignore();

    return 0;
}

现在让我一点点玩你的代码......

#include <iostream>
#include <string>

using namespace std;


int main()
{
    int VolCalcs[3] = {0, 0, 0};
    string Prompt[3] = {"Please enter length: ", "Please enter width: ", "Please enter height: "};
    string NewResult[3] = {"The new length is ", "The new width is ", "The new height is "};

    for (int i=0; i != 3; ++i) {
        cout << endl;
        cout << Prompt[i] << flush;
        cin >> VolCalcs[i];
    }

    int volume = VolCalcs[0] * VolCalcs[1] * VolCalcs[2];
    cout << " The Volume is: " << volume << endl;

    ++VolCalcs[0];
    --VolCalcs[1];
    VolCalcs[2] += 10;

    for (int i = 0; i != 3; ++i) {
        cout << NewResult[i] << VolCalcs[i] << endl;
    }
    volume = VolCalcs[0] * VolCalcs[1] * VolCalcs[2];
    cout << " The New Volume is: " << volume << endl << endl;
    cout << "Press Enter to End Program" << flush;
    cin.ignore();

    return 0;
}

改变了什么？

首选前缀运算符++i到postfix i++。例如，在++VolCalcs[0]。
在运营商周围写下空格。这是一种很好的格式化练习。
尽可能推迟变量定义。在需要时创建volume。
不需要#include <array>。在普通的旧C风格数组之前首选std::array。它们不在我的代码中。这可能是你的下一步：）

为什么程序跳过包含字符串数组和cin / cout的循环

1 个答案: