[已解决]有什么错啊

初莫M

素数是指除了1和它本身以外,不能被任何整数整除的数,例如17就是素数。现在请你求出给定的区间里所有的素数。

输入描述
一行，两个整数a、b，表示区间的端点。

输出描述
输出区间里的所有素数，每行输出五个，每两个数字之间有一个空格，行末无空格。没有素数的话输出“Sad.”(你只需要输出引号里的内容即可)。

1. #include<bits/stdc++.h>
   
2. using namespace std;
   
3. int su(int x)
   
4. {
   
5.     if(x<2) return 0;
   
6.     int i;
   
7.     for(i=2;i<=sqrt(x);i++)
   
8.     {
   
9.         if(x%i==0) return 0;
   
10.     }
    
11.     return 1;
    
12. }
    
13. int main()  
    
14. {
    
15.     int n,m,t,s=0;
    
16.     if(m<n)
    
17.     {
    
18.         t=m;m=n;n=t;
    
19.     }
    
20.     cin>>n>>m;
    
21.     for(int i=n;i<=m;i++)
    
22.     {
    
23.         if(su(i))
    
24.         {   s++;
    
25.             if(s%5==0)
    
26.                 cout<<i<<endl;
    
27.         else cout<<i<<' ';}
    
28.     }
    
29.     if(s==0) cout<<"Sad.";
    
30. }

复制代码

最佳答案

月排行榜 / 总排行榜

isdkz

2023-11-10 20:03:49

你的代码存在一些问题。首先，你在使用  cin  读取  n  和  m  的值之前就对它们进行了比较，这样会导致未定义的行为。你应该在读取完  n  和  m  的值之后再进行比较。

其次，你的素数判断函数  su  没有考虑到输入为0或负数的情况，这会导致不正确的输出。你应该在函数开头添加对于输入为0或负数的判断。

另外，你的输出逻辑也存在一些问题。当输出素数的时候，你没有考虑到每行输出五个数字的要求。你可以通过在输出素数的时候维护一个计数器来实现这个逻辑。

最后，你没有在输出素数结束后考虑到没有素数的情况。

以下是修改后的代码：

1. #include<bits/stdc++.h>
   
2. using namespace std;
   
3. 
   
4. int su(int x)
   
5. {
   
6.     if(x<=1) return 0;
   
7.     for(int i=2;i<=sqrt(x);i++)
   
8.     {
   
9.         if(x%i==0) return 0;
   
10.     }
    
11.     return 1;
    
12. }
    
13. 
    
14. int main()
    
15. {
    
16.     int n, m, t, s=0;
    
17.     cin>>n>>m;
    
18.     if(m<n)
    
19.     {
    
20.         t=m; m=n; n=t;
    
21.     }
    
22.     for(int i=n; i<=m; i++)
    
23.     {
    
24.         if(su(i))
    
25.         {
    
26.             s++;
    
27.             if(s%5==0)
    
28.                 cout<<i<<endl;
    
29.             else
    
30.                 cout<<i<<' ';
    
31.         }
    
32.     }
    
33.     if(s==0)
    
34.         cout<<"Sad.";
    
35. }
    

复制代码

跳转到最佳答案楼层

isdkz

你的代码存在一些问题。首先，你在使用  cin  读取  n  和  m  的值之前就对它们进行了比较，这样会导致未定义的行为。你应该在读取完  n  和  m  的值之后再进行比较。

其次，你的素数判断函数  su  没有考虑到输入为0或负数的情况，这会导致不正确的输出。你应该在函数开头添加对于输入为0或负数的判断。

另外，你的输出逻辑也存在一些问题。当输出素数的时候，你没有考虑到每行输出五个数字的要求。你可以通过在输出素数的时候维护一个计数器来实现这个逻辑。

最后，你没有在输出素数结束后考虑到没有素数的情况。

以下是修改后的代码：

1. #include<bits/stdc++.h>
   
2. using namespace std;
   
3. 
   
4. int su(int x)
   
5. {
   
6.     if(x<=1) return 0;
   
7.     for(int i=2;i<=sqrt(x);i++)
   
8.     {
   
9.         if(x%i==0) return 0;
   
10.     }
    
11.     return 1;
    
12. }
    
13. 
    
14. int main()
    
15. {
    
16.     int n, m, t, s=0;
    
17.     cin>>n>>m;
    
18.     if(m<n)
    
19.     {
    
20.         t=m; m=n; n=t;
    
21.     }
    
22.     for(int i=n; i<=m; i++)
    
23.     {
    
24.         if(su(i))
    
25.         {
    
26.             s++;
    
27.             if(s%5==0)
    
28.                 cout<<i<<endl;
    
29.             else
    
30.                 cout<<i<<' ';
    
31.         }
    
32.     }
    
33.     if(s==0)
    
34.         cout<<"Sad.";
    
35. }
    

复制代码