在数组中的每个素数后插入“99”（有两个功能）

Question

在构造函数并调用它时，我的参数有些问题（这是我第一次尝试使用两个函数）。

这是调试器的错误：

I:\c++\11_Tombos feladatok-beszuras_torles\19.cpp||In function 'bool isPrime(int*)':|
I:\c++\11_Tombos feladatok-beszuras_torles\19.cpp|11|error: 'i' was not declared in   this scope|
I:\c++\11_Tombos feladatok-beszuras_torles\19.cpp|16|error: 'i' was not declared in   this scope|
I:\c++\11_Tombos feladatok-beszuras_torles\19.cpp||In function 'int main()':|
I:\c++\11_Tombos feladatok-beszuras_torles\19.cpp|34|error: too few arguments to  function 'bool isPrime(int*)'|
I:\c++\11_Tombos feladatok-beszuras_torles\19.cpp|9|note: declared here|
||=== Build failed: 3 error(s), 0 warning(s) (0 minute(s), 0 second(s)) ===|

这里是代码：

#include<iostream>
#include<stdlib.h>
#include<math.h>
using namespace std;

bool isPrime(int t[])
{
    for(int j=2; j<=sqrt(t[i]); j++)
    {
        if(t[i]%j==0)
            return false;
    }

    if(t[i]<2)
        return false;

    return true;
}

int main()
{
    int t[50], n, i;
    cout<<"Size of array: ";
    cin>>n;
    for (int i=1; i<=n; i++)
    {
        cout<<i<<". Element: ";
        cin>>t[i];
    }
    i=1;
    while(i<=n)
    {
        if(isPrime())
        {
            for(int j=n; j>=i+1; j--)
            {
                t[j+1]=t[j];
            }
            t[i+1]=99;
            n++;
            i=i+2;
        }
        else
        {
            i++;
        }
    }
    for(int i=1; i<=n; i++)
    {
        cout<<t[i]<<", ";
    }
    cout<<"\n";
    system("pause");
    return 0;
}

我知道 system("pause") 很慢，不推荐使用等等。不要怪我使用它（建议使用其他方法）。

Answer 1

这是代码审查。 :)

1. isPrime 应该采用 int t 参数，而不是数组。您还可以在此函数中使用 t[i]，但 i 超出范围！你在main 中声明了i。将t[] 和t[i] 都替换为t。或者，更好的是，将其命名为 number，这样就非常清楚了。
2. 您在main 的第一行声明int i，然后在它正下方的for 循环中再次声明t 数组。我很惊讶这并没有导致编译错误。
3. 在以for(int i = 1;... 开头的for 循环中，您每次都会错过数组的第一个元素！ 记住：数组是从 0 开始索引的，所以应该是 for(int i = 0;...。
4. 不对用户输入进行错误检查（main 中的声明 cin >> n）。如果我输入 100 会怎样？ main 中的数组 t 包含 50 个整数，因此 t[100] 会导致程序因段错误而崩溃。
5. 在main 的while 循环中，if(isPrime()) 行将无法编译，因为您定义了isPrime 以获取参数，但您没有使用任何参数调用它。我想你的意思是if(isPrime(i))？
6. 在下面的for循环中，初始化int j = n，然后尝试访问未分配的t[j+1]。这也会导致您的程序崩溃。
7. 为什么在while循环中有n++？我可以看到这会导致问题/意外行为，因为这是您的 while 循环的界限。
8. 在编写 C++ 代码时，最好使用 C++ 样式的标头。所以用#include <cstdlib>代替#include <stdlib.h>，用#include <cmath>代替#include <math.h>。

可能还有一些错误/问题，但这应该会让你感动。让我们知道您的想法！

Answer 2

调试器的错误告诉你到底哪里出了问题！

1. i 在方法 isPrime() 中不存在。仅仅因为你在 main() 中有一个 i，你就不能在另一个函数中使用它（阅读 Scope）
2. 您在没有参数的情况下调用 isPrime() 函数。

我建议你从更简单的程序开始，同时阅读functions。

Answer 3

你应该使用类似的东西

cin.ignore();

而不是system("PAUSE");