愚蠢的语法错误 c++

Question

我对 C++ 完全陌生。

我为这个错误而苦恼了一个多小时。想必有经验的人都能一眼看出来。

以下代码给出了错误：

class TimeTravellingCellar { 

private:

public:
  int determineProfit (int [] profit, int [] decay) { 
    int N = sizeof(profit)/sizeof(decay); 
    int max = 0; 
    for (int i = 0; i < N; i++) { 
      for (int j = 0; j < N; j++) { 
        if (i == j) continue; 
        if (profit [i] - decay [j] > max) 
          max = profit [i] - decay [j]; 
      } 
    } 
    return max; 
  } 
}

Visual Studio Express 在 defineProfit 的参数中的 profit 下放置了一条红线，并表示：

在标识符利润之前需要一个 ')'< /代码>。

我将不胜感激一些帮助。 谢谢！

Answer 1

您正在声明您的数组，就好像这是 C# 一样。应该是

int profit[]

，否则

int *profit

你接下来就会碰到这个。您需要用分号来终止您的课程。

class Foo { 

};  <----

您遇到的下一个问题是逻辑问题，而不是语法问题。这并不像您想象的那样：

int N = sizeof(profit)/sizeof(decay); 

您使用的是 sizeof 两个指针，而不是数组的大小。您实际上有：

int N = 4/4  /* assumes sizeof int == 4 */

您还需要将您的大小传递给函数（或者，更好的是；停止使用数组并使用向量。）

当您采用“数组”时作为函数的参数，它实际上衰减为指向数组类型的指针（数组本身不能传递给函数）。因此，接下来

void Foo( int array[] ) {
    size_t arrSize = sizeof(array);
    // arrSize == 4 for a 32-bit system, i.e., sizeof(int*)

    int a[100];
    size_t actualSizeInBytes = sizeof(a);
    // actualSizeInBytes == 400, i.e., 4 * 100 as an int occupies 4 bytes
}

，这一行会导致您的第一次迭代始终被跳过。不确定这是否是故意的：

if (i == j) continue; 

Answer 2

您不需要像 C++ 中那样声明数组，[] 需要位于名称后面。
另请注意，类声明后需要有一个分号。

class TimeTravellingCellar { 

private:

public:
  int determineProfit (int profit[], int decay[]) { 
    int N = sizeof(profit)/sizeof(decay); 
    int max = 0; 
    for (int i = 0; i < N; i++) { 
      for (int j = 0; j < N; j++) { 
        if (i == j) continue; 
        if (profit [i] - decay [j] > max) 
          max = profit [i] - decay [j]; 
      } 
    } 
    return max; 
  } 
};

编辑：还要记住 sizeof(pointer) 将返回指针类型的字节数，而不是数组中的元素数。因此，如果您有一个 int 数组，则 sizeof(array) == sizeof(int)。您的 N 值将始终等于 1。

Answer 3

这行是错误的：

 int determineProfit (int [] profit, int [] decay) { 

将其更改为：

int determineProfit (int profit[], int decay[]) { 

or

int determineProfit (int* profit, int* decay) { 

并添加结束 ;

如果您这样做并添加 main，当然：

int main() {}

那么您可以编译您的代码 - 我刚刚用 g++ 尝试过。

Answer 4

尝试 int certainProfit (int*profit, int*decay) 因为对于形式参数，数组和指针几乎是相似的。

Answer 5

括号与变量名称相关联，而不是与类型相关联。第一行应该是

int determineProfit (int profit[], int decay[]) { 

Answer 6

关于C 中的数组的教程可能会很有启发，特别是在数组参数的传递方面。

Answer 7

int determineProfit (int[] profit int [] decay 

这是你的错误 - 上面的陈述是错误的；应该是这样的

int determineProfit (int profit[], int decay[])