为什么我的C++除法程序不能编译

Question

我试图编写一个具有正确除法函数的程序。 我的代码是：

#include <iostream> 

using namespace std; 

double x,y,z,a; 

double divide(x,y) { 
    if (x >= y) { 
     x=z; 
     z=y; 
     y=x; 
     return(x/y); 
    } 
    else 
     return(y/x); 
} 

int main() 
{ 
    double x,y,z ; 
    cout << "Enter x " <<endl; 
    cin >> x; 
    cout << "Enter y " <<endl; 
    cin >> y; 
    a = divide (x,y); 
    cout << a <<endl; 

    system("pause"); 
    return 0; 
} 

而且我有2个错误：

expected `,' or `;' before '{' token 

在 {线

。右键下方的双分频(x, y)线

而另一个错误

divide cannot be used as a function 

在a = divide (x, y);线。 我正在使用代码：块

Answer 1

您需要为函数divide指定一个适当的函数签名。具体来说，函数的自变量是缺少它们的类型：

double divide(double x, double y) 
{ 
    ... 
} 

您还需要在if语句来创建每个块的范围：在

if (x > y) 
{ 
    ... 
} 
else 
{ 
    ... 
} 

Answer 2

大括号if语句不绕过else块。你需要一对独立的大括号。尝试：

if (x >= y){ 
     x=z ; 
     z=y ; 
     y=x ; 
     return(x/y); 
    } 
    else { 
     return(y/x); 
    } 

后的“其他”第二组括号（周围的代码的一条线是不是绝对必要的，你可以离开括号关闭的，如果或者否则，如果该块只有一个线长。但是，当你新的你可能不应该，因为它很容易犯错误的方式。

而且，你没有提供类型在分功能的x和y变量，你必须指定类型对于他们来说，就像你对任何其他变量一样。你写的

double x,y,z,a ; 

...功能之外，但没有帮助;它定义了新的双变量名为x,y,z和a，完全独立于您的函数中的变量。

Answer 3

更正了你的if ... else中的大括号。还需要在函数的参数中定义一个类型。

using namespace std; 

     double x,y,z,a ; 

double divide (double x, double y) 
    { 
     if (x >= y){ 
      x=z ; 
      z=y ; 
      y=x ; 
      return(x/y); 
     } 
     else 
     { 
      return(y/x); 
     } 
    } 

    int main() 
{ 
    double x,y,z ; 
    cout << "Enter x " <<endl; 
    cin >> x ; 
    cout << "Enter y " <<endl; 
    cin >> y ; 
    a = divide (x,y); 
    cout << a <<endl; 

     system("pause"); 
    return 0; 
} 

Answer 4

#include <iostream> 

using namespace std; 

// divides x/y 
double divide (x,y) 
{ 
    if(y != 0) 
    { 
     /*{} <- this is called a scope. 
     it is important to keep track of scopes. 
     each function has it's own scope 
     each loop or an if instruction can have it's own scope 
     in case it does - all the instructions from the scope will be executed 
     in case it doesn't - only the 1st instruction after the if/else/for/while etc. will be executed 

     Here's another funny thing about scopes : 
     { 
      double x; // this variable exists ONLY within this scope 
     } 
     { 
      // y doesn't exist here 
      { 
       double y; // y exists here. it's local 
      } 
      // y doesn't exist here 
     } 
     */ 
     return x/y; 
    } 
    else 
     return 0; 
} 

int main() 
{ 
    double x,y; 
    cout << "Enter x " <<endl; 
    cin >> x ; 
    cout << "Enter y " <<endl; 
    cin >> y ; 
    double a = divide (x,y); 
    cout << a <<endl; 
    cin; 
    return 0; 
}