更新时间:2016-08-15 10:27:51浏览次数:1+次
程序运行时出错,代码如下:
#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'');
}
}
首先,排版很重要。你贴的代码,每一行空一行,本来代码不多的,却让一屏都显示不下,这很影响阅读的效率,建议改进,删除不必要的空行。
其次,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当做二维数组使用,也就是代码中看到的用法。最后,改进后的代码如下:(排版和代码)
#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'');
}
}
参数检查到位,是程序健壮性的基本保障,对于提高代码质量,提高软件稳定性是非常基础而重要的,请多加小心。
相关资讯