程序运行时出错,代码如下:
#include <stdio.h>
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<argv[1][1]-''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 <stdio.h>
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<argv[1][1]-''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'');
}
}
参数检查到位,是程序健壮性的基本保障,对于提高代码质量,提高软件稳定性是非常基础而重要的,请多加小心。