C++ 编程错误：“{”标记之前的预期 unqualified-id

Question

我是 C++ 的新手，我正在尝试制作一个“计算器”：将两个数字相加，两个数字相减，两个数字相乘，两个数字相除，取一个数字的正弦，取一个的余弦数字，或取数字的正切。这是代码：

#include <iostream>;
#include <cmath>;
#include <string>
int main () 
{}
int ask(std::string operation);
    {
        std::cout<<"Type Addition, Subtraction, Multiplication, Division, Sine, Cosine, or Tangent:\n";
        std::cin>>operation;
            if (operation="Addition") 
            {
                goto Add
                                }
    float Add(float addend1, float addend2, float answer) 
    {
    Add:
        std::cout<<"Insert the first number to be added:\n";
        std::cin>>addend1;
        std::cout << "Insert the second number to be added:\n";
        std::cin>>addend2;
        answer=addend1+addend2;
        std::cout<<addend1<<"+"<<addend2<<"="<<answer<<"\n";
        break
    }
}

以后会有更多的功能，但是我的问题在第7行。有一个错误说：expected unqualified-id before "{" token。我知道我的缩进很糟糕，但是谢谢！

Answer 1

你的代码有很多问题。

首先，正如 Ivan 指出的那样，您试图在函数内部定义一个函数（ask() 内部 main()）。那是无效的。

其次，您有一个goto（为什么？！）试图跳转到另一个函数中的标签。我怀疑您的编译器甚至会允许这样做，但是您希望它如何工作？您正在尝试使用传递给您的函数 addition 的变量，这些变量不存在，因为您从未调用该函数并且从未为其设置堆栈。这个不好，别做，正确调用函数就好了。

第三，#include 预处理器指令以换行符结束，而不是分号。这可能会导致一些（相对）难以追踪的编译错误。

第四，当您打算使用相等运算符== 时，您错误地尝试将const char* "Addition" 分配给operation。但这不会起作用，因为operation 是一个右值，不能像这样分配。如果你想修改它，你需要将它声明为一个指针，但再一次，这不是你想要的语义......

如果您想比较字符串并且（无论出于何种原因...）打算使用指向 char 的指针，那么您应该使用strcmp。也就是说，您在 C++ 领域，所以请改用 std:string。

试试这样的。无论如何我都没有增强你的代码，只是让它可以编译和运行。我做了一些改变。

除了消除一些语法错误之外，您原来的Add 函数将结果作为float 参数。从函数内部分配给它只会修改一个副本。如果您希望调用者看到修改后的值，则需要使用指针或引用，但您根本不需要，因为您可以简单地返回结果。

字符串比较区分大小写，因此您可能希望将其更改为不区分大小写。我假设这里没有本地化:)。我也没有对输入进行错误检查，因此请注意，如果用户输入的不是有效的浮点数，它可能会失败。

#include <iostream>
#include <string>

using namespace std;

void Ask();
float Add( float, float );

int main( size_t argc, char* argv[] )
{
    Ask();
    return 0;
}

void Ask()
{
    cout << "Type Addition, Subtraction, Multiplication, Division, Sine, Cosine, or Tangent:\n";

    string operation;
    cin >> operation;

    if( operation == "Addition" )
    {
        float first = 0, second = 0;
        cout << "enter first operand";
        cin >> first;

        cout << "enter second operand";
        cin >> second;

        cout << "The result is: " << Add( first, second );
    }
}

float Add( float first, float second ) 
{
    return first + second;
}

Answer 2

С++ 不允许嵌套函数。您有函数main() 并试图在其中声明函数ask()。而且编译器不知道你想要什么。

Answer 3

我对你的代码做了一些评论，也许这会让你开始：

#include <iostream>;
#include <cmath>;
#include <string>;
int main () {
    int ask (){         //you cannot nest functions in C++

    char operation [20];    //why not use the string class if you include it anyway 
std::cout<<"Type Addition, Subtraction, Multiplication, Division, Sine, Cosine, or Tangent:\n";
    std:cin>>operation;
    if (operation="Addition"){ //you cannot compare char-strings in C++ like that
    goto Addition;      //don't use goto (I don't want to say "ever", but goto is only used in extremely rare cases) make a function call instead
}
}
    float addition(float addend1, float addend2, float answer)  //you probably want to declare the variables inside the function
{
Addition:           //don't use labels
std::cout<<"Insert the first number to be added:\n";
std::cin>>addend1;
std::cout << "Insert the second number to be added:\n";
std::cin>>addend2;
answer=addend1+addend2;
std::cout<<addend1<<"+"<<addend2<<"="<<answer<<"\n";
}

Answer 4

让我们试着把它分解..
你不应该使用 ;关于预编译器指令。

#include <iostream>;
#include <cmath>;
#include <string>;

应该是

#include <iostream>
#include <cmath>
#include <string>

.

int main () {
    int ask (){

请参阅Ivans answer 了解此内容

char operation [20];
std::cout<<"Type Addition, Subtraction, Multiplication, Division, Sine, Cosine, or Tangent:\n";
std:cin>>operation;
if (operation="Addition"){

您可以改用std::string，这样更容易处理。然后就可以写了

#include <string>

...
std::cout<<"Type Addition, Subtraction, Multiplication, Division, Sine, Cosine, or Tangent:\n";
std::string myString;
getline(cin, myString);
if (myString == "Addition"){

.

goto Addition;
}
}
    float addition(float addend1, float addend2, float answer)
{

不确定这里发生了什么.. 但是让我们打破 Addition 到它自己的函数

void Addition(){
    // do addition here
}

.

Addition:
std::cout<<"Insert the first number to be added:\n";
std::cin>>addend1;
std::cout << "Insert the second number to be added:\n";
std::cin>>addend2;
answer=addend1+addend2;
std::cout<<addend1<<"+"<<addend2<<"="<<answer<<"\n";
}

别忘了你必须定义变量

int addend1;
int addend2;
int answer;

希望这对您有所帮助。

Answer 5

首先 int ask() 那是什么。你为什么在这里开始一个块。 其次，由于 ask()，您有两个 {s 和三个 }。 我认为 c++ 不支持匿名函数。 第三，为什么要使用goto，当你有一个函数时，只需调用该函数。 Fourh 你的附加函数应该是 void 或者删除它的最后一个参数。 另外我认为你不需要 string.h 文件，除非你使用一些相当高级的函数，char 数组应该足够你的程序了。