[code review了]一个函数，看了眼，吓坏了。

image_z

1. 
   
2. int next_token(struct parse_state *state)
   
3. {
   
4.     char *x = state->ptr;
   
5.     char *s;
   
6. 
   
7.     if (state->nexttoken) {
   
8.         int t = state->nexttoken;
   
9.         state->nexttoken = 0;
   
10.         return t;
    
11.     }
    
12. 
    
13.     for (;;) {
    
14.         switch (*x) {
    
15.         case 0:
    
16.             state->ptr = x;
    
17.             return T_EOF;
    
18.         case '\n':
    
19.             x++;
    
20.             state->ptr = x;
    
21.             return T_NEWLINE;
    
22.         case ' ':
    
23.         case '\t':
    
24.         case '\r':
    
25.             x++;
    
26.             continue;
    
27.         case '#':
    
28.             while (*x && (*x != '\n')) x++;
    
29.             if (*x == '\n') {
    
30.                 state->ptr = x+1;
    
31.                 return T_NEWLINE;
    
32.             } else {
    
33.                 state->ptr = x;
    
34.                 return T_EOF;
    
35.             }
    
36.         default:
    
37.             goto text;
    
38.         }
    
39.     }
    
40. 
    
41. textdone:
    
42.     state->ptr = x;
    
43.     *s = 0;
    
44.     return T_TEXT;
    
45. text:
    
46.     state->text = s = x;
    
47. textresume:
    
48.     for (;;) {
    
49.         switch (*x) {
    
50.         case 0:
    
51.             goto textdone;
    
52.         case ' ':
    
53.         case '\t':
    
54.         case '\r':
    
55.             x++;
    
56.             goto textdone;
    
57.         case '\n':
    
58.             state->nexttoken = T_NEWLINE;
    
59.             x++;
    
60.             goto textdone;
    
61.         case '"':
    
62.             x++;
    
63.             for (;;) {
    
64.                 switch (*x) {
    
65.                 case 0:
    
66.                         /* unterminated quoted thing */
    
67.                     state->ptr = x;
    
68.                     return T_EOF;
    
69.                 case '"':
    
70.                     x++;
    
71.                     goto textresume;
    
72.                 default:
    
73.                     *s++ = *x++;
    
74.                 }
    
75.             }
    
76.             break;
    
77.         case '\\':
    
78.             x++;
    
79.             switch (*x) {
    
80.             case 0:
    
81.                 goto textdone;
    
82.             case 'n':
    
83.                 *s++ = '\n';
    
84.                 break;
    
85.             case 'r':
    
86.                 *s++ = '\r';
    
87.                 break;
    
88.             case 't':
    
89.                 *s++ = '\t';
    
90.                 break;
    
91.             case '\\':
    
92.                 *s++ = '\\';
    
93.                 break;
    
94.             case '\r':
    
95.                     /* \ <cr> <lf> -> line continuation */
    
96.                 if (x[1] != '\n') {
    
97.                     x++;
    
98.                     continue;
    
99.                 }
    
100.             case '\n':
     
101.                     /* \ <lf> -> line continuation */
     
102.                 state->line++;
     
103.                 x++;
     
104.                     /* eat any extra whitespace */
     
105.                 while((*x == ' ') || (*x == '\t')) x++;
     
106.                 continue;
     
107.             default:
     
108.                     /* unknown escape -- just copy */
     
109.                 *s++ = *x++;
     
110.             }
     
111.             continue;
     
112.         default:
     
113.             *s++ = *x++;
     
114.         }
     
115.     }
     
116.     return T_EOF;
     
117. }
     

复制代码

提几个问题：
1，此处无注释（本来也没有，不是没贴出来），有谁能在10分钟内大概推测下该接口的功能。
2，如果你作为该段代码的维护者，测试人员测试出该段代码有bug，有谁能在1天内，尝试入手修改调试。（不需要你立即定位，解掉，看到这段代码的逻辑结构，你在那个地方添加语句，能预测该语句对整个接口的影响）
3，你能够在多长时间内大概的理清这一小段代码的逻辑，流程？

safedead

很正常的代码啊，goto用得好，驱动级代码

image_z

回复 2# safedead


    我要再告诉你这个是我写的，估计会被黑出翔吧

pppStar

功能不知道, 代码还蛮整齐呀,

hellioncu

还行吧，就是出口多了点

image_z

没人气啊。标题不够长！

电子学习员

哪里被吓到了

cascle

这个是解析脚本的吧，android里边init.rc文件的解析程序就是这个这个样的

cascle

这种程序拿个具体文件对比走一遍就全都清楚了——面向过程的嘛

noword2k

解析文本？