有奖纠错。。。。。

飞灰橙

这段简化的代码，输出应该只有103, 104, 105三种，
结果怎么都不对。
昨天花一个晚上才查出这个问题，真是太不可饶恕了。
大家看看，第一个找出问题的加30分

1. 
   
2. #include <stdio.h>
   
3. #include <string.h>
   
4. 
   
5. int some_func()
   
6. {
   
7.         return 100;
   
8. }
   
9. 
   
10. typedef enum
    
11. {
    
12.         CHOOSE_NOBODY = 3,
    
13.         CHOOSE_TOM = 4,
    
14.         CHOOSE_JERRY = 5
    
15. } CHOOSE_E;
    
16. 
    
17. int main( int argc, char **argv )
    
18. {
    
19.         CHOOSE_E choice;
    
20.         int action;
    
21.        
    
22.         if( argc != 2 )
    
23.         {
    
24.                 fprintf( stderr, "Wrong arguments\n" );
    
25.                 return 1;
    
26.         }
    
27. 
    
28.         if( strcmp( argv[1], "tom" ) == 0 )
    
29.                 choice = CHOOSE_TOM;
    
30.         else if ( strcmp( argv[1], "jerry" ) == 0 )
    
31.                 choice = CHOOSE_JERRY;
    
32.         else
    
33.                 choice = CHOOSE_NOBODY,
    
34. 
    
35.        
    
36.         /* Assisn the variable action
    
37.            and print it
    
38.          */
    
39.         action = some_func( ) + choice;
    
40. 
    
41.         printf( "action = %d\n", action );
    
42. 
    
43.         return 0;
    
44. }
    

复制代码

[ 本帖最后由 飞灰橙 于 2008-7-11 13:55 编辑 ]

cugb_cat

choice = CHOOSE_NOBODY,

成逗号表达式了。

net_robber

楼上够快，我看了N久

估计楼上是正确答案了

cookis

还有一个问题
strcmp( argv[0], "tom")

argv[0] 不是程序名称吗.. 这应该不是你的本意吧.

ermu

主要原因还是逗号

smalloc

strcmp( argv[0], "tom" ) == 0
程序编译成tom了吗？

linternt

呵呵，大意所致，往往这种问题，你越找越找不着！

emacsnw

这种问题很容易找。随便用个好点的编辑器，比如emacs，你会发现
    action = some_func( ) + choice
这句话对齐不对劲，仔细看看就发现上面是个逗号而不是分号了。

prolj

LZ不是用的Vim C加亮吧？

youyuanyin

楼主给我们这个程序的时候一定是有些地方写错了


#include <stdio.h>
#include <string.h>

int some_func()
{
        return 100;
}

typedef enum
{
        CHOOSE_NOBODY = 3,
        CHOOSE_TOM = 4,
        CHOOSE_JERRY = 5
} CHOOSE_E;

int main( int argc, char **argv )
{
        CHOOSE_E choice;
        int action;

        if( argc != 2 )
        {
                fprintf( stderr, "Wrong arguments\n" );
                return 1;
        }


        //楼主的初衷应该是要比较命令行的第二个参数是不是"tom"或"jerry"，而argv[1]才代表第二个参数，所以这里if,else中的比较都应该是针对argv[1]进行的，而argv[0]是编译后的可执行文件名
        if( strcmp( argv[1], "tom" ) == 0 )
                choice = CHOOSE_TOM;
        else if ( strcmp( argv[1], "jerry" ) == 0 )
                choice = CHOOSE_JERRY;
        else
                choice = CHOOSE_NOBODY;
                //这里原来是“,”，我想楼主原来的程序应该是“;”，如果不是，那这算一处错误，如果这里是逗号，那么 choice = CHOOSE_NOBODY,action = some_func( ) + choice; 形成一个逗号表达式，所以如果执行到这里choice值是3，action值是100+3=103


        /* Assisn the variable action
           and print it
         */
        action = some_func( ) + choice;

        printf( "action = %d\n", action );

        return 0;
}


修改后我执行后的结果如下
[root@youzi ~]# gcc -o forum forum.c
[root@youzi ~]# ./forum tom
action = 104
[root@youzi ~]# ./forum jerry
action = 105
[root@youzi ~]# ./forum youzi
action = 103
[root@youzi ~]#