我正在编译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 againn");
}
//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");
}
}
}
意思就是它所说的。'a'||'A'
是整数–具体地说是整数1
。strcmp
的第二个参数必须是字符串,而不是整数。
您似乎打算将argv[2]
与a
和A
进行比较。您需要两个不同的strcmp
调用。此外,您需要使用双引号,而不是单引号。
在"C"中,'||'运算符是布尔'or'运算,而不是串联运算。此外,撇号的使用表示一个基本上是"char"类型的单个字符。
我想你想要这样的东西(148行):
if (strcmp(argv[2], "C")==0 || (strcmp(argv[2], "c")==0) ...
或者,如果你的C库支持它:
if (strcasecmp(argv[2], "C") == 0)
这是不区分大小写的比较。
我相信这里的对象是将命令行参数(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的一个例子。