C语言面试题

[xddzccn]

指出这段代码的它的功能与不足
#include <stdio.h>
#define MAX_NUM 100
int main(int argc,char *argv[])
{
unsigned int buf[MAX_NUM];
unsigned int i,j;
for(i=0;i<MAX_NUM;i++)
{
buf[i]=i;
}
for(i=2;i<MAX_NUM;i++)
{

if(buf[i]!=0)
{
for(j=i+i;j<MAX_NUM;j+=i)
{
buf[j]=0;
}

}

}

for(i=2;i<MAX_NUM;i++)
{
if(buf[i]!=0)
{
printf("%d\n",i);
}



}
return 0;
}

[xddzccn]

刚参加完 电子公司的面试
到是看出是 找素数的程序 但是不知道有什么缺点

[quench]

找出数组中2～100之间的质数，并打印。

效率不太好，判断N是否是质数，从2开始除，除到√N就可以了。
所以
这个循环改一下

for(i=2;i <MAX_NUM;i++)
{

if(buf[i]!=0)
{
for(j=i+i;j <MAX_NUM;j+=i)
{
buf[j]=0;
}

}

} 改成
unsigned int k = (int)sqrt(MAX_NUM);
for(i=2;i < k;i++)
{

if(buf[i]!=0)
{
for(j=i+i;j <MAX_NUM;j+=i)
{
buf[j]=0;
}

}

}

[quench]

找出数组中2～100之间的质数，并打印。

效率不太好，判断N是否是质数，从2开始除，除到√N就可以了。
所以
这个循环改一下

for(i=2;i <MAX_NUM;i++)
{

if(buf[i]!=0)
{
for(j=i+i;j <MAX_NUM;j+=i)
{
buf[j]=0;
}

}

} 改成
unsigned int k = (int)sqrt(MAX_NUM);
for(i=2;i < k;i++)
{

if(buf[i]!=0)
{
for(j=i+i;j <MAX_NUM;j+=i)
{
buf[j]=0;
}

}

}

[IlikeEnglish]

不是效率不好，是完全不正确。


最好用筛法

[zeloas]

效率太低了，5楼的是种不错的方法

[xddzccn]

怎么沉下去了阿

[cs_wgh_dn]

感觉上面几楼说的不太正确啊！
测试了一下，原来程序需要计算144次
5楼的方法需要计算347次！！
真正哪儿不好 还不清楚！
不知道对不？

[twilight041132]

判断素数的方法：
用一个数分别去除以2到sqrt(这个数)，如果能被整除，则表明此数不是素数，反之是素数。

所以原代码应改为:
#include <stdio.h >
#include <math.h>
#define MAX_NUM 100
int main(int argc,char *argv[])
{
unsigned int buf[MAX_NUM];
unsigned int i,j,k,flag;
k=(int)sqrt(MAX_NUM);
for(i=0;i <MAX_NUM;i++)
{
buf[i]=i;
}
for(i=2;i <MAX_NUM;i++)
{
flag=1;
for(j=2;j<=k;j++)
{
if(buf[i]%j==0 && buf[i]!=j)
{
flag=0;
break;
}
}
if(flag)
{
printf("%d\n",i);
}
}
return 0;
}

[davelv]

楼主发的代码有点错误，稍微修改了下
另外，效率的确不是很高，最常用的办法，就是从2除到SQRT（N）就好了

引用

[font=Fixedsys]#include <stdio.h >
#define MAX_NUM 100
int main(int argc,char *argv[])
{
unsigned int buf[MAX_NUM];
unsigned int i,j;
for(i=0;i <MAX_NUM;i++)
{
buf[i]=i;
}
for(i=2;i <MAX_NUM;i++)
{

if(buf[i]!=0)
{
for(j=i+i;j <MAX_NUM;j+=i)
{
buf[j]=0;
}

}

}

for(i=2;i <MAX_NUM;i++)
{
if(buf[i]!=0)
{
printf("%d\n",i);
}
}

return 0;
}
[/font]

[liuchangfei]

[Quote=引用 9 楼 davelv 的回复:]
楼主发的代码有点错误，稍微修改了下
另外，效率的确不是很高，最常用的办法，就是从2除到SQRT（N）就好了
引用
[font=Fixedsys]#include <stdio.h>
#define MAX_NUM100
intmain(intargc,char*argv[])
{
       unsigned intbuf[MAX_NUM];
       unsigned inti,j;
       for(i=0;i<MAX_NUM;i++)
        {
               buf[i]=i;
        }
               for(i=2;i<MAX_NUM;i++)
        {

               if(buf[i]!=0)
                {
                       for(j=i+i;j<MAX_NUM;j+=i)
                        {
                               buf[j]=0;
                        }
 
                }

        }

       for(i=2;i<MAX_NUM;i++)
        {
               if(buf[i]!=0)
                {
                       printf("%d\n",i);
                }
        }

       return0;
}
[/font]
[/Quote]效果如何？？

[BePatient]

觉得第一个for循环 for(i=2;i<MAX_NUM;i++)可以改成 for(i=2;i<(MAX_NUM/2);i++)

[gx168853]

i <MAX_NUM这个把效率降低了很多；用MAX_NUM的开方数还好点儿

[wotur]

大家都看出来是找素数的了?
我看这个最大的缺点就是没有写名函数的具体功能,和要实现的目标!
如果你看到一堆代码首先要去猜它是干什么的,那么肯定不会是优良的code

[coolness]

1 #include <stdio.h>
2 #include <string.h>
3 #include <math.h>
4
5 /*
6 gcc -lm
7 */
8 #define SQRT(m) sqrt((m)+1)
9 #define LIMITED 800
10
11 int main()
12 {
13 int m,i,k,h=0,leap=1;
14
15 for( m = 1; m <= LIMITED; m++ ) {
16 k = SQRT(m);
17 for( i = 2; i <= k; i++)
18 if( m % i == 0 ) {
19 leap = 0;
20 break;
21 }
22 if( leap ) {
23 printf("%-4d", m );
24 h++;
25 if( h % 10 == 0 )
26 printf("\n");
27 }
28 leap = 1;
29 }
30 printf("\nThe total is %d\n",h);
31 return 0;
32 }

[akilguo]

我把9楼第二个for语句中的MAX_NUM改成了sqrt(MAX_NUM),但是这样编译时报错了"error C2065: 'sqrt' : undeclared identifier",这是怎么回事，难道这样子不对吗？sqrt()不是个函数吗？

[akilguo]

哦，我知道了，在开头加入：#include<math.h>就行了！

[renzhewh]

知道效率的话，那么sqrt 呢？
计算平方根更费时

[kenyyy]

for(j=i+i, k = 0;j <MAX_NUM && k <= j;j+=i, k ++)
这么改会好些