1

我正在编译我的 C 代码并收到两个错误：

警告：传递 strcmp 的参数 2 使指针从整数而不进行强制转换

和

警告：注意：预期 const char * 但参数是 int 类型

这是我的主要：

int main(int argc, char *argv[])
{
    //check to make sure that the command line arguments are valid  
    if(argc!=3) 
    {
        printf("invalid function call try again\n");
    }

    //else to choose the proper command
    else
    {
        //reverse routine A         
        if(strcmp(argv[2],'a'||'A')==0) //line 138
        {
            reva(argv[1]);
        }
        //reverse routine B
        else if(strcmp(argv[2],'b'||'B')==0)  //line 143
        {
            revb(argv[1]);
        }
        //reverse routine C
        else if(strcmp(argv[2],'c'||'C')==0)  //line 148
        {
            revc(argv[1]);
        }
        //unacceptable command line argumant
        else
        {
            printf("unacceptable command line argument for reverse routine try again:\n");
        }
    }

}
4

3 回答 3

6

这意味着它所说的。 'a'||'A'是一个整数——具体来说，它是整数1。的第二个参数strcmp必须是字符串，而不是整数。

您似乎打算与argv[2]和a进行比较A。为此，您需要两个不同strcmp的调用。此外，您需要使用双引号，而不是单引号。

于 2012-01-20T06:07:39.030 回答
3

在“C”中，'||' 运算符是布尔“或”运算，而不是串联运算。此外，撇号的使用表示基本上是“char”类型的单个字符。

我想你想要这样的东西（对于第 148 行）：

if (strcmp(argv[2], "C")==0 || (strcmp(argv[2], "c")==0) ...

或者，如果您的 C 库支持它：

if (strcasecmp(argv[2], "C") == 0)

这是不区分大小写的比较。

于 2012-01-20T06:11:19.410 回答
1

我相信您在这里的目的是将命令行参数（argv[2]）与字符（字符串）“C”或“c”进行比较，也就是说，如果用户在命令行中给出了 c 或 C，那么您就是这样。

SO用户已经提供了解释。你需要使用

(strcmp(argv[2], "C")==0 || (strcmp(argv[2], "c")==0)

为了消除你的警告。

但是，这不是在 C 中解析命令行参数的最佳方式。如果您的程序在解析用户输入时过于复杂，我建议使用库“getopt”。它旨在帮助用户以结构化的方式解析和分析输入。

这是一个小代码截图

opt = getopt_long( argc, argv, optString, longOpts, &longIndex );
    while( opt != -1 ) {
        switch( opt ) {
            case 'I':
                globalArgs.noIndex = 1; /* true */
                break;

            case 'l':
                globalArgs.langCode = optarg;
                break;

            case 'o':
                globalArgs.outFileName = optarg;
                break;

            case 'v':
                globalArgs.verbosity++;
                break;

            case 'h':   /* fall-through is intentional */
            case '?':
                display_usage();
                break;

            case 0:     /* long option without a short arg */
                if( strcmp( "randomize", longOpts[longIndex].name ) == 0 ) {
                    globalArgs.randomized = 1;
                }
                break;

            default:
                /* You won't actually get here. */
                break;
        }

        opt = getopt_long( argc, argv, optString, longOpts, amp;longIndex );
    }

请在一些文档（或 linux 手册页）中搜索 getopt 和 getopt_long。这是来自GNU的示例。

于 2012-01-20T20:30:50.187 回答