包括

Question

这是我的第一个问题。我真的很喜欢编程，当我从WTF codemonkey代码中识别出结构良好的代码时，我很难挣扎。在创建这个ubber简单程序时，我应该做些什么吗？我主要担心的是这个功能。

        #include <iostream>
        using namespace std;

        void enter_numbers ( int & iNum1, int & iNum2 ) {
        cout << "Enter first number" << endl;
        cin >> iNum1;
        cout << "Enter second number" << endl;
        cin  >> iNum2;
        }

        float calc_avg ( int iNum1, int iNum2){
            float fRes;
            fRes =(float)(iNum1 + iNum2)/2;
            return fRes;
        }

        void show_avg ( float fRes ) {
        cout << "Average is: " << fRes;
        }

        void main () {
        int iNum1;
        int iNum2;
        enter_numbers ( iNum1, iNum2);
        float fRes = calc_avg (iNum1, iNum2);
        show_avg ( fRes );
        fflush(stdin);
        getchar ();

       }

Answer 1

您不应该输入输入流，如下所述：http://www.gidnetwork.com/b-57.html
您的main函数应该将int作为返回类型并返回0，以便成功执行： What should main() return in C and C++?

休息时，你的代码似乎很好。请注意，对于初学者，你应该感到高兴，因为它是非常好的。我刚才提到了一些你可能想要做的改进，并指出了一些进一步文档的来源，因为我想你想了解更多。

Answer 2

每个企业文化 - 或者其他任何一起编写代码的人 - 都会有一套编码风格指南，变量和函数命名约定等等。关于如何使代码可读，你会听到很多很多意见（可能很快就会发现= D），以下问题主要是宗教性的：

匈牙利表示法（变量名称如lpctszName）vs普通英语。我的观点是，它有时间和地点。查找“Apps匈牙利语”与“匈牙利语系统”。
压痕。我们都同意（几乎 - 反对者？）复合语句（例如函数定义）的主体应该缩进。那样做吧。然而，关于支撑线的哪一行，确实存在激烈的争论。我喜欢线路末端的开口支架。查找“一个真正的缩进风格”。
ReadableVariable vs readableVariable vs readable_variable依此类推。

最重要的是，与任何其他文本正文一样，代码也有受众群体。这是与您合作的人员，或接管维护代码或测试代码的人员。在个人项目中，就是你，在你不记得变量是什么的几个星期之后。您的编码风格应与观众所知的一致。

除此之外，如果没有异常或编译时错误，您的代码就是好的。将来，工作代码会在代码审查StackExchange上进行，概念性问题将由程序员StackExchange提供。欢迎使用StackOverflow = D.

Answer 3

只需撰写num1，num2，res而不是iNum1，iNum2，fRes。该类型已作为声明的一部分可见。

Answer 4

主要功能不能无效。
你必须使用“Return 0;”结束主要功能。
不要让自己声明变量，例如“iNum或fRes”......

嗯，这就是我修复你的代码的方式，希望它有所帮助...

包括

    using namespace std;

    void enter_numbers ( int &Num1, int &Num2) {
    cout << "Enter first number" << endl;
    cin >> Num1;
    cout << "Enter second number" << endl;
    cin  >> Num2;
    }

    float calc_avg ( int Num1, int Num2){
        float Res;
        Res =(Num1 + Num2)/2;
        return Res;
    }

    void show_avg ( float Res) {
    cout << "Average is: " << Res;
    }

    int main (){
    int Num1;
    int Num2;
    enter_numbers ( Num1,Num2);
    float Res = calc_avg (Num1, Num2);
    show_avg ( Res );
    getchar ();
    return 0;
   }

这个功能很好地宣布了吗？

4 个答案:

包括