吻你不厌 发表于 2020-11-25 13:00:30

为什么我的程序有问题啊?

大佬们,看一下咯!
1.本题要求实现一个判断素数的简单函数、以及利用该函数计算给定区间内素数和的函数。

素数就是只能被1和自身整除的正整数。注意:1不是素数,2是素数。

函数接口定义:
int prime( int p );
int PrimeSum( int m, int n );
其中函数prime当用户传入参数p为素数时返回1,否则返回0;函数PrimeSum返回区间内所有素数的和。题目保证用户传入的参数m≤n。

裁判测试程序样例:
#include <stdio.h>
#include <math.h>

int prime( int p );
int PrimeSum( int m, int n );

int main()
{
    int m, n, p;

    scanf("%d %d", &m, &n);
    printf("Sum of ( ");
    for( p=m; p<=n; p++ ) {
      if( prime(p) != 0 )
            printf("%d ", p);
    }
    printf(") = %d\n", PrimeSum(m, n));

    return 0;
}

/* 你的代码将被嵌在这里 */
输入样例:
-1 10
输出样例:
Sum of ( 2 3 5 7 ) = 17

            这是我的程序
#include <stdio.h>
#include <math.h>

int prime( int p );
int PrimeSum( int m, int n );

int main()
{
    int m, n, p;

    scanf("%d %d", &m, &n);
    printf("Sum of ( ");
    for( p=m; p<=n; p++ ) {
      if( prime(p) != 0 )
            printf("%d ", p);
    }
    printf( ") = %d\n", PrimeSum(m, n));

    return 0;
}

int prime( int p )
{
        int i;
        int ret = 1;
        if(p > 4)
        {
                for(i = 2;i < p/2;i++)
                {
                        if(p % i == 0)
                        {
                                ret = 0;
                                break;
                        }
                }
        }
        else if( 0 < p <= 4)
        {
                switch(p)
                {
                        case 1:ret = 0;
                        case 2: ;
                        case 3: ;
                        case 4:ret = 0;
                }
        }
        else
        {
                ret = 0;
        }
       
        return ret;
}

int PrimeSum( int m, int n )
{
        int i;
        int sum = 0;
        int flag = 1;
        if(m > 4)
        {
                while(m <= n)
                {
                        for(i = 2; i < m/2;i++)
                        {
                                if(m % i == 0)
                                {
                                        flag = 0;
                                }
                        }
                        if(flag == 1)
                        {
                                sum = sum + m;
                        }
                        m++;
                }
        }
        else if(0 < m <= 4)
        {
                sum = 5;
                m = 5;
                while(m <= n)
                {
                        for(i = 2; i < m/2;i++)
                        {
                                if(m++ % i != 0)
                                {
                                        sum = sum + m;
                                }
                        }
                }
        }
        else
        {
                ;
        }
       
        return sum;
}

jackz007 发表于 2020-11-25 14:52:25

int prime(int p)
{
      int k , r = 0                         ;
      if(p > 1) {
                if(p % 2) {
                        for(r = 1 , k = 3 ; k * k < p + 1 ; k += 2) {
                              if(k < p && ! (p % k)) {
                                        r = 0 ;
                                        break ;
                              }
                        }
                } else if(p == 2) r = 1       ;
      }
      return r                              ;
}

int PrimeSum(int m , int n)
{
      int k , s                                              ;
      for(s = 0 , k = m ; k <= n ; k ++) if(prime(k)) s += k ;
      return s                                             ;
}

吻你不厌 发表于 2020-11-25 15:17:48

你的程序我看懂了,但是大佬能不能看看我的问题出在哪里

心驰神往 发表于 2020-11-25 15:34:07

本帖最后由 心驰神往 于 2020-11-25 15:40 编辑

case 1:{ret= 0; break;}这样,其他也加break,不然0/4一直返回ret=0。
至于求和我调试了一下,你的循环有点问题,一直在循环
while(m <= n)
        {
                     for(i = 2; i < m/2;i++)

倒戈卸甲 发表于 2020-11-25 15:34:44

粗略看了一眼,明显的逻辑错误发现一处,PrimeSum这个函数,while循环体内第一句应该是flag=1,因为flag上一次循环变成0后,新一次循环不会自动变回来。
小的语法错误有一处,switch语句的case结束要手动break

吻你不厌 发表于 2020-11-25 16:10:20

倒戈卸甲 发表于 2020-11-25 15:34
粗略看了一眼,明显的逻辑错误发现一处,PrimeSum这个函数,while循环体内第一句应该是flag=1,因为flag上 ...

改过来之后还是有问题,怎么办呢?

倒戈卸甲 发表于 2020-11-25 16:48:48

吻你不厌 发表于 2020-11-25 16:10
改过来之后还是有问题,怎么办呢?

暂时没看到其它错误,你调试看看错误在哪一行。但是说实话,else if(0<m<=4)这种情形下的整段代码,我没搞清楚它的业务逻辑,不知道是想干什么。我现在手边没电脑,不能给你调试

吻你不厌 发表于 2020-11-25 16:53:51

倒戈卸甲 发表于 2020-11-25 16:48
暂时没看到其它错误,你调试看看错误在哪一行。但是说实话,else if(0

调试是什么东西,一直没搞明白。。

吻你不厌 发表于 2020-11-25 16:55:04

我的那个0到4的范围只是相当于为了特指,将判断的核心程序不满足的条件抽出来

倒戈卸甲 发表于 2020-11-25 17:02:08

吻你不厌 发表于 2020-11-25 16:53
调试是什么东西,一直没搞明白。。

你在计算机上编译运行这段代码,运行的软件会告诉你错误在哪一行,语法错误一编译就报,逻辑错误程序崩了才报。一般是在你代码窗口下面的小窗口里看

吻你不厌 发表于 2020-11-25 17:14:19

倒戈卸甲 发表于 2020-11-25 17:02
你在计算机上编译运行这段代码,运行的软件会告诉你错误在哪一行,语法错误一编译就报,逻辑错误程序崩 ...

但是我的代码没有报错,只是单纯的不符合条件

倒戈卸甲 发表于 2020-11-25 17:23:57

吻你不厌 发表于 2020-11-25 17:14
但是我的代码没有报错,只是单纯的不符合条件

没错的话,其它我不清楚,但你else if(0<m<=4)后面
这段单独的代码肯定有问题,看着就不好

吻你不厌 发表于 2020-11-25 17:34:15

jackz007 发表于 2020-11-25 14:52


大佬看一下我的程序吧!救救孩子啊!

吻你不厌 发表于 2020-11-25 21:26:24

{:10_249:}

吻你不厌 发表于 2020-11-25 22:51:00

没人吗{:10_269:}
页: [1]
查看完整版本: 为什么我的程序有问题啊?