求解毒

umyspl

劣质代码评析[url]http://www.cnblogs.com/pmer/archive/2012/10/20/2731910.html[/url]
一时手痒写了几行代码，但是觉得很难看，求解毒

1. 
   
2. #include<stdlib.h>
   
3. #include<stdio.h>
   
4. #include<time.h>
   
5. 
   
6. #define MASK                    1
   
7. #define MARK_SET(inNum,mark)    (mark|=(MASK<<(inNum)))
   
8. #define CHECK_EXIST(inNum,mark) (mark&(MASK<<(inNum)))
   
9. #define RESET_MARK(mark)        mark=0
   
10. #define CREATE_MARK(mark)       int mark=0
    
11. #define RESET_TIPS(ti)          ti.num=0;ti.A_Counts=0;ti.B_Counts=0
    
12. 
    
13. #define NUM_LENGTH 4
    
14. #define NUM_MOD    10
    
15. #define MAX_TIMES  10
    
16. 
    
17. typedef struct _tips_info{
    
18.         int num;
    
19.         int A_Counts;
    
20.         int B_Counts;
    
21. }tips_info,*ptips_info;
    
22. 
    
23. int  generate(int*);
    
24. void tips(int,int*,int,ptips_info);
    
25. int  show_tips(ptips_info,int);
    
26. void show_num(int*);
    
27. 
    
28. int main(void)
    
29. {
    
30.         srand((unsigned)time(NULL));  //这句是抄袭
    
31.         int num[NUM_LENGTH];
    
32.         int mark = generate(num);
    
33.         tips_info ti;
    
34.         int input;
    
35.         int i = 0;
    
36.         for(;i<MAX_TIMES;i++)
    
37.         {
    
38.                 printf("guess num(%d):",NUM_LENGTH);
    
39.                 scanf("%d",&input);        //此处不知如何处理。假定输入的就是4位整数
    
40.                 RESET_TIPS(ti);
    
41.                 tips(input,num,mark,&ti);
    
42.                 if(show_tips(&ti,1))return 0;
    
43.         }
    
44.         printf("game over!");
    
45.         show_num(num);
    
46.         return 0;
    
47. }
    
48. 
    
49. void show_num(int * numAr)
    
50. {
    
51.         int num = 0;
    
52.         int i = NUM_LENGTH;
    
53.         for(;i>0;i--) num = num*10+numAr[i-1];
    
54.         printf("%d\n",num);
    
55. }
    
56. 
    
57. int show_tips(ptips_info ti,int reset)
    
58. {
    
59.         if(ti->A_Counts==NUM_LENGTH)
    
60.         {
    
61.                 printf("congratulations!(%d)\n",ti->num);
    
62.                 return 1;
    
63.         }
    
64.         else
    
65.                 printf("%dA%dB:[%d]\n",ti->A_Counts,ti->B_Counts,ti->num);
    
66.         return 0;
    
67. }
    
68. 
    
69. void tips(int num,int* numAr,int mark,ptips_info ti)
    
70. {
    
71.         ti->num = num;
    
72.         CREATE_MARK(submark);
    
73.         int i = 0;
    
74.         for(;i<NUM_LENGTH;i++)
    
75.         {
    
76.                 int sub = num%NUM_MOD;
    
77.                 num /= NUM_MOD;
    
78. 
    
79.                 if(sub==numAr[i])
    
80.                 {
    
81.                         ti->A_Counts++;
    
82.                         if(CHECK_EXIST(sub,submark))ti->B_Counts--;
    
83.                 }
    
84.                 else if(!CHECK_EXIST(sub,submark)&&CHECK_EXIST(sub,mark))ti->B_Counts++;
    
85. 
    
86.                 MARK_SET(sub,submark);
    
87.         }
    
88. }
    
89. 
    
90. int generate(int * numAr)
    
91. {
    
92.         CREATE_MARK(mark);
    
93.         int i = NUM_LENGTH;
    
94.         for(;i>0;i--)
    
95.         {
    
96.                 int num;
    
97.                 do
    
98.                 {
    
99.                         num = rand()%NUM_MOD;
    
100.                 }while(CHECK_EXIST(num,mark)&&(mark||num));
     
101.                 numAr[i-1] = num;
     
102.                 MARK_SET(num,mark);
     
103.         }
     
104.         return mark;
     
105. }
     

复制代码

{:3_190:}

hellioncu

LZ对宏有癖好？

pandaiam

1. #include <stdio.h>
   
2. #include <stdlib.h>
   
3. #include <string.h>
   
4. #include <assert.h>
   
5. #include <time.h>
   
6. 
   
7. 
   
8. int main(void)
   
9. {
   
10.         time_t time_now;
    
11.         int i, j, k;
    
12.         int ran_n[4];
    
13.         int guess_a, guess_b;
    
14.         char buf_n[100];
    
15.         
    
16.         int debug_n;
    
17.         
    
18. 
    
19.         
    
20.         debug_n = 0;
    
21.         time_now = time(NULL);
    
22.         srand(time_now);
    
23.         /* random number gen */
    
24.         for ( i = 0; i < 4; ) {
    
25. 
    
26.                 ran_n[i] = rand()%10;
    
27.                 switch (i) {
    
28.                 case 0:
    
29.                         break;
    
30.                 case 1:
    
31.                         if ( ran_n[1] == ran_n[0] )
    
32.                                 goto rang;
    
33.                         break;
    
34.                 case 2:
    
35.                         if ( ran_n[2] == ran_n[0] || ran_n[2] == ran_n[1] )
    
36.                                 goto rang;
    
37.                         break;
    
38.                 case 3:
    
39.                         if ( ran_n[3] == ran_n[0] || ran_n[3] == ran_n[1] || \
    
40.                              ran_n[3] == ran_n[2])
    
41.                                 goto rang;
    
42.                         break;
    
43.                 }
    
44.                 i++;
    
45.         rang:   debug_n++;
    
46.         }
    
47.         
    
48.         printf("guess the number:\n");  
    
49.         for ( i = 0; i < 10 ;) {
    
50.                 memset(buf_n, 0, sizeof(buf_n));
    
51.                 scanf("%s", buf_n);
    
52.                 if ( strlen(buf_n) != 4 ) {
    
53.                         printf("not 4 number, ");
    
54.                 cc:printf("input not legal, again please:\n");
    
55.                         continue;
    
56.                 }
    
57.                 for (j = 0; j < 3; j++) {
    
58.                         if (buf_n[j] > 0x39 || buf_n[j] < 0x30) {
    
59.                                 printf("--%c-- not number, ", buf_n[j]);
    
60.                                 goto cc;
    
61.                         }
    
62.                 }
    
63.                 if ( buf_n[0] == buf_n[1] || buf_n[0] == buf_n[2] || \
    
64.                      buf_n[0] == buf_n[3] || buf_n[1] == buf_n[2] || \
    
65.                      buf_n[1] == buf_n[3] || buf_n[2] == buf_n[3]) {
    
66.                         printf("the four cala should be different, ");
    
67.                         goto cc;
    
68.                 }
    
69.                
    
70.                 /* count the A B */
    
71.                 guess_a = 0;
    
72.                 guess_b = 0;
    
73.                 for ( j = 0; j < 4; j++)  {
    
74.                         for ( k = 0; k < 4; k++) {
    
75.                                 if ( ran_n[j] == buf_n[k]-48 ) {
    
76.                                         if ( j == k)
    
77.                                                 guess_a ++;
    
78.                                         else
    
79.                                                 guess_b ++;
    
80.                                         break;
    
81.                                 }
    
82.                         }
    
83.                 }
    
84.                 if ( guess_a == 4) {
    
85.                         printf("Congratu--, You are right. The answer is:--%s--\n", buf_n);
    
86.                         return 0;
    
87.                 }
    
88.                 printf("%2d--%s--, --%d--A--%d--B\n", 10-i, buf_n, guess_a, guess_b);
    
89.                 i++;
    
90.         }
    
91. 
    
92.         printf("You are so stupid!! The right answer is:--%d%d%d%d--\n", ran_n[0], ran_n[1], ran_n[2], ran_n[3]);
    
93.         return 0;
    
94. }
    

复制代码

自己写的垃圾代码..
猜数字还是xAxB的好玩..

代码虽然差点但平时玩下还是很不错的.

fender0107401

前排围观。

sqfasd

回复 5# pmerofc

用宏来定义变量
linux内核代码，google的开源代码，经常这么干
一是可以稍微缩短代码长度
二是可以在宏里封装复杂的命名方式，防止命名冲突
   

sqfasd

回复 7# pmerofc

那也不至于看不懂吧
你是挑刺挑太久了，现在看什么都是刺
其实很多风格问题，实际项目中有编码规范在手，这些都不成问题

umyspl

hellioncu 发表于 2013-01-23 16:14
LZ对宏有癖好？

有点控

pmerofc 发表于 2013-01-23 23:21
我说的是看不懂他为什么这样写

习惯了。主要是在工程中别人使用时不需要关心mark是个啥东西。方便使用。其次是tips本来也想加个宏。但是懒得动了。如果说不该这么写（脱裤子放屁）。我接受意见。