问题：

在该文的最后，曾提到完成的代码还有进一步改进的余地。本文完成了这个改进。所以本文讨论的并不是初学者代码中的常见错误与瑕疵，而是对我自己代码的改进和优化。标题只是为了保持系列的连续性。

改进

程序的总体思想没有改变，所以main()函数不需要任何改动。

int main( void)

{

unsigned n ;

puts("数据组数=?");

scanf("%u" , &n );while ( n -- > 0)

{intx ;

puts("整数X=?");

scanf("%d", &x );

printf("%d\n" , get_nearest( x ) ); //求最接近x的素数

}return 0;

}

进一步的改进体现在

typedefstructprime_list

{

unsigned prime;struct prime_list *next;

}

Node;int get_nearest( intx )

{int step = 0 ; //步长增量

int sign = -1; //符号

Node * head = NULL ; //素数链表

while ( ! be_prime( x , &head ) )

x+= ( sign = - sign ) * ++step ;

my_free(head) ;returnx ;

}

这里增加了一个链表head，用于存储素数表。这样，在判断素数时只用较小的素数试除就可以了，这可以使计算数量大为减少。因为与自然数相比，素数的数量很少(≈ln n / n 个)。此外，在判断完x是否为素数之后，如果需要判断下一个数(x += ( sign = - sign ) * ++ step ;)是否为素数，这个素数表还可以重复使用，最多再向其中添加一个素数就可以了。(注意最初素数表是空的)

判断素数的方法很简单，小学生都懂。

bool be_prime( int x , Node * * pp ) //根据素数表pp判断x是否为素数

{if ( x <= 1)return false;if ( x == 2)return true;if ( get_remainder( x , pp ) == 0 ) //x对素数表pp中素数逐个求余有0值

return false;return true;

}

但是由于素数表(*pp==NULL)可能是空的，因此

int get_remainder( int x , Node * * pp )//x对素数表pp中素数逐个求余

{while ( * pp == NULL || sqr_less( (*pp) -> prime , x ) )//表中素数个数不足

add_1_prime ( pp ) ; //表中增加一个素数

Node* p = *pp ;while ( p !=NULL )

{if ( x % p -> prime == 0)return 0;

p= p ->next ;

}return !0;

}bool sqr_less ( int n , intx )

{return n * n

}

需要先向其中添加素数

add_1_prime ( pp ) ;

直到

sqr_less( (*pp) -> prime , x )

依从小到大次序最后加入的那个素数的平方不小于x为止。

对于

void add_1_prime( Node * *pp )

{if ( * pp ==NULL )

{

add (2 , pp ); //第一个素数

return;

}int next_p = ( * pp )->prime + 1 ; //从最后一个素数之后开始找下一个素数

while ( !be_prime( next_p , pp ) )

next_p++;

add( next_p , pp );//将下一个素数加入素数表

}

来说，加入第一个素数——2很容易，但是寻找素数表中最大素数后的下一个素数时，却需要判断一个整数是否是素数

be_prime( next_p , pp )

这样，就会发现，这个过程最初是由判断某个数x是否是素数开始，

be_prime( x , & head )

在判断过程中需要建立素数表，

add_1_prime ( pp ) ;

而建立素数表，又需要判断某个数是否是素数

be_prime( next_p , pp )

这样就形成了一个极其复杂的间接递归调用。更为复杂的是，在调用的过程中，素数表本身即不断地被使用，而自身也处于不断的变化状态之中，即不断地被添加进新的素数，与复杂的间接递归一道，构成了比复杂更复杂的复杂的代码结构与复杂的数据结构的复杂的结合体。有兴趣的话可以自己算一下圈复杂度，如此复杂的情况通常并不容易遇到。

这种局面完全是由于精打细算造成的，由于对速度的斤斤计较，从而形成了一幅小猫在拼命咬自己尾巴同时小猫自己又在不断变化的复杂无比的动态画面。由此我们不难理解，为什么有人说，“不成熟的优化是万恶之源”(Premature optimization is the root of all evil!- Donald Knuth)。因为优化往往意味着引人复杂。复杂也是一种成本，而且是一种很昂贵的成本。

就这个题目而言这种成本应该算是值得，因为对于求一个较大的最接近的素数问题而言(例如对于109这个量级)，两套代码的速度有天壤之别。

增强可读性？

如果把建立素数表的要求写在get_nearest()函数中，可能会使代码可读性变得更好些。

int get_nearest( intx )

{int step = 0 ; //步长增量

int sign = -1; //符号

Node * head = NULL ; //素数链表

while ( 建立最大素数平方不小于x的素数表() , ! be_prime( x , &head ) )

x+= ( sign = - sign ) * ++step ;

my_free(head) ;returnx ;

}

但这里的这个这个“,”是免不掉的，且圈复杂度不变。

至于这种写法是否真的改善了可读性，恐怕是见仁见智。

进一步提高效率

没什么更好的办法，只能用点“赖皮”手段，即充分运用已有的素数知识，帮计算机算出一部分素数。

void add_1_prime( Node * *pp )

{if ( * pp ==NULL )

{

add (2 , pp ); //第一个素数

return;

}switch ( ( * pp ) ->prime )

{case 2: add ( 3, pp );return;case 3: add ( 5, pp );return;/*这里可以依样写多个case，只要是按照素数从小到大的次序*/

default:

{int next_p = ( * pp )->prime + 1 ; //从最后一个素数之后开始找下一个素数

while ( !be_prime( next_p , pp ) )

next_p++;

add( next_p , pp );//将下一个素数加入素数表

return;

}

}

}

这里switch语句的结构非常有趣。

重构

/*

问题：

素数

在世博园某信息通信馆中，游客可利用手机等终端参与互动小游戏，与虚拟人物Kr. Kong 进行猜数比赛。

当屏幕出现一个整数X时，若你能比Kr. Kong更快的发出最接近它的素数答案，你将会获得一个意想不到的礼物。

例如：当屏幕出现22时，你的回答应是23；当屏幕出现8时，你的回答应是7；

若X本身是素数，则回答X;若最接近X的素数有两个时，则回答大于它的素数。

输入：第一行：N 要竞猜的整数个数

接下来有N行，每行有一个正整数X

输出：输出有N行，每行是对应X的最接近它的素数

样例：输入

4

22

5

18

8

输出

23

5

19

7

作者：薛非

出处：http://www.cnblogs.com/pmer/   “C语言初学者代码中的常见错误与瑕疵”系列博文

版本：V 2.1

*/

#include #includetypedefstructprime_list

{

unsigned prime;struct prime_list *next;

}

Node;int get_nearest( int);bool be_prime( int , Node * *);int get_remainder( int , Node * *) ;void add_1_prime( Node * *);bool sqr_less ( int , int);void add ( int , Node * *);void my_malloc( Node * *);void my_free( Node *);int main( void)

{

unsigned n ;

puts("数据组数=?");

scanf("%u" , &n );while ( n -- > 0)

{intx ;

puts("整数X=?");

scanf("%d", &x );

printf("%d\n" , get_nearest( x ) ); //求最接近x的素数

}return 0;

}int get_nearest( intx )

{int step = 0 ; //步长增量

int sign = -1; //符号

Node * head = NULL ; //素数链表

while ( ! be_prime( x , &head ) )

x+= ( sign = - sign ) * ++step ;

my_free(head) ;returnx ;

}bool be_prime( int x , Node * * pp ) //根据素数表pp判断x是否为素数

{if ( x <= 1)return false;if ( x == 2)return true;if ( get_remainder( x , pp ) == 0 ) //x对素数表pp中素数逐个求余有0值

return false;return true;

}int get_remainder( int x , Node * * pp )//x对素数表pp中素数逐个求余

{while ( * pp == NULL || sqr_less( (*pp) -> prime , x ) )//表中素数个数不足

add_1_prime ( pp ) ; //表中增加一个素数

Node* p = *pp ;while ( p !=NULL )

{if ( x % p -> prime == 0)return 0;

p= p ->next ;

}return !0;

}bool sqr_less ( int n , intx )

{return n * n

}//“偷奸耍滑”的add_1_prime()

void add_1_prime( Node * *pp )

{if ( * pp ==NULL )

{

add (2 , pp ); //第一个素数

return;

}switch ( ( * pp ) ->prime )

{case 2: add ( 3, pp );return;case 3: add ( 5, pp );return;/*这里可以依样写多个case，只要是按照素数从小到大的次序*/

default:

{int next_p = ( * pp )->prime + 1 ; //从最后一个素数之后开始找下一个素数

while ( !be_prime( next_p , pp ) )

next_p++;

add( next_p , pp );//将下一个素数加入素数表

return;

}

}

}//老老实实的add_1_prime()//void add_1_prime( Node * * pp )//{//if ( * pp == NULL )//{//add ( 2 , pp );//第一个素数//return ;//}//

//int next_p = ( * pp )->prime + 1 ;//从最后一个素数之后开始找下一个素数//

//while ( !be_prime( next_p , pp ) )//next_p ++ ;//

//add( next_p , pp );//将下一个素数加入素数表//}

void add ( int prime , Node * *pp )

{

Node*temp ;

my_malloc(&temp );

temp-> prime =prime ;

temp-> next = *pp ;* pp =temp ;

}void my_malloc( Node * *p_p )

{if ( ( * p_p = malloc( sizeof (* * p_p) ) ) ==NULL )

exit(1);

}void my_free( Node *p )

{

Node*temp ;while ( ( temp = p ) !=NULL )

{

p= p->next;

free( temp );

}

}

相关博客：

偶然发现Jingle Guo网友后来研究同一问题的一篇博文，我感觉对阅读此文的网友可能有一定的参考价值，故在此给出相关链接：