程序运行时出错,代码如下:
#include
main(int argc,char *argv[])
{
char s[100];
int i=0,j;
printf("argc = %d\n",argc);
printf("输入一行字符:");
while(i<100 && ( s[i]=getchar() )!=''#'' )
i++;
if(argv[1][0] == ''-'')
{
for(j = i-(argv[1][1] - ''0'')+1;j<=i;j++)
printf("%c",s[j]);
putchar(''\n'');
}
if(argv[1][0] == ''+'')
{
for(j=0;j printf("%c",s[j]); putchar(''\n''); } if(argc == 1) { for(j = i-9;j<=i;j++) printf("%c",s[j]); putchar(''\n''); } } C++技术网解答: 首先,排版很重要。你贴的代码,每一行空一行,本来代码不多的,却让一屏都显示不下,这很影响阅读的效率,建议改进,删除不必要的空行。 其次,main函数丢失了返回值,我想,这个应该是贴代码时复制漏了。不过这种明显的低级错误,尽量不要发生哦。 虽然上面是小建议,不过细节通常是很重要的,请知晓哦。下面开始解答。 下面是程序运行的截图: 输入足够的字数之后,回车才会进入之后的流程。所以这里需要完善细节。回车之后,报错,提示如下: 在调试时,报错后直接跳转定位到错误的代码。下面开始分析错误原因和提供解决办法。 此程序在IDE如VS中调试运行,默认传入的argc=1,argv[0]="c:\users\wdx\documents\visual studio 2010\Projects\tmp_main\Debug\tmp_main.exe"。也就是说,默认情况下,会得到一个命令行参数,所以argc为1,而且agrv数组的第一个元素值为exe所在的文件路径。 而程序中,却使用这样的语句: if(argv[1][0] == ''-'') {...} 因为argv数组只有一个字符串元素,所以也就是最大的索引为0.这里使用了1,自然就越界崩溃了。因为传入的参数是字符串,所以也可以将字符串当做一维,所以可以将argv当做二维数组使用,也就是代码中看到的用法。 argc单词,arg表示参数的意思,后面的c是count的缩写,意思是计数。也就是argc的值是表示命令行参数的个数,也就是argv数组的元素个数。 argv单词后面的v是values的缩写,意思是值。用复数形式,表示这是一个数组,可以存储很多个命令行参数。参数如果是字符串,就可以将字符串当做一维数组使用,那么此时argv就是二维数组了。 所以,在使用argv数组时,如果预先不知道有多少个参数,请先根据argc的值来判断,如果argc的值为1,则argv的索引值绝对不能超过0,否则就会崩溃。一定要对argc值做检查,这是确保argv正确使用的前提。 所以你的代码就是缺少检查,刚好又越界使用了。如何传递命令行参数和测试命令行程序,请参考《应用程序如何传递参数(命令行)以及如何测试命令行》。【无法提供超链接,请前往网站阅读】 最后,改进后的代码如下:(排版和代码) #include int main(int argc,char *argv[]) { char s[100]; int i=0,j; printf("argc = %d\n",argc); printf("输入一行字符:"); while(i<100 && ( s[i]=getchar() )!=''#'' ) i++; //因为得到的参数为一个,所以,argv数组越界了。 if (argc>1)//改进后的参数数量检查 { if(argv[1][0] == ''-'')//此处发生错误 { for(j = i-(argv[1][1] - ''0'')+1;j<=i;j++) printf("%c",s[j]); putchar(''\n''); } if(argv[1][0] == ''+'') { for(j=0;j printf("%c",s[j]); putchar(''\n''); } } else //此处与原有代码衔接 if(argc == 1) { for(j = i-9;j<=i;j++) printf("%c",s[j]); putchar(''\n''); } } 参数检查到位,是程序健壮性的基本保障,对于提高代码质量,提高软件稳定性是非常基础而重要的,请多加小心。