大神帮我看一下这个代码哪里出问题了啊？

[miecuiqi8751]

void getPrime(int n) {
    if(n <= 0) return;
    bool arr[n+1];

    for(int i = 2; i <= n; i++)
        arr[i] = true;
    for(int i = 2; i <= n; i++) {
        if(arr[i]) {
            for(int m = i+1; m <= n; m++) {
                if(m%i==0)
                    arr[m]==false;
            }
        }
    }
    int cnt = 0;
    for(int i = 1; i <= n; i++){
        if(arr[i]) {
            cout << i << " ";
            cnt++;
        }
        if(cnt%10==0)
            cout << endl;
    }
    cout << endl << cnt << endl;

}

int main()
{
    getPrime(10);
    return 0;
}

[ztenv]

请说出你的问题

[棉猴]

getPrime()函数中声明arr数组时有错误。


在声明arr数组时，数组的维数n+1不是常量表达式。
可以考虑使用new实现动态分配内存的方式

void getPrime(int n) {
	if (n <= 0) return;
	bool *arr = new bool[n + 1];//--------动态分配内存-------

	for (int i = 2; i <= n; i++)
		arr[i] = true;
	for (int i = 2; i <= n; i++) {
		if (arr[i]) {
			for (int m = i + 1; m <= n; m++) {
				if (m%i == 0)
					arr[m] == false;
			}
		}
	}
	int cnt = 0;
	for (int i = 1; i <= n; i++) {
		if (arr[i]) {
			cout << i << " ";
			cnt++;
		}
		if (cnt % 10 == 0)
			cout << endl;
	}
	cout << endl << cnt << endl;
	delete[]arr;//---------释放内存--------------
	arr = NULL;//---------复位指针-------------
}

程序运行的效果如下所示

请参考《C++中使用关键字new动态分配数组》
希望能够帮到你

[zhongchengli]

用数组就用动态数组 ，用其他容器list等。

[汗水与勇气]

顶楼上，
你这里 bool arr[n+1]; 的n不是常量，数组是不能这样定义的，应该使用动态数组