c语言程序 调错 运行出负数

Question

企业发放的奖金根据利润提成。利润(I)低于或等于10万元时，奖金可提10%；利润高 于10万元，低于20万元时，低于10万元的部分按10%提成，高于10万元的部分，可可提成7.5%；20万到40万之间时，高于20万元的部分，可提成5%；40万到60万之间时高于40万元的部分，可提成3%；60万到100万之间时，高于60万元的部分，可提成1.5%，高于 100万元时，超过100万元的部分按1%提成。该公司共十人，从键盘输入当月利润，求每个人应发放奖金总数？
void main()
{
long int x1,x2,x4,x6,x10,x;
int i;
long int a[10];
i=1;
while(i<=10)
{
scanf("%d",&a[i]);
i++;
}
x1=100000*0.1;
x2=x1+100000*0.075;
x4=x2+200000*0.05;
x6=x4+200000*0.03;
x10=x6+400000*0.015;
for(i=1;i<=10;i++)
{
if(a[i]<=100000)
x=a[i]*0.1;
else if(a[i]<=200000)
x=x1+(a[i]-100000)*0.075;
else if(a[i]<=400000)
x=x2+(a[i]-200000)*0.05;
else if(a[i]<=600000)
x=x4+(a[i]-400000)*0.03;
else if(a[i]<=1000000)
x=x6+(a[i]-600000)*0.015;
else
x=x10+(a[i]-1000000)*0.01;
printf("x=%ld\n",x);
}
}
先谢了

Answer 1

楼上的并没有找到错误。x只是应该最好用实型，不过你用整型也没所谓，当表达式中出现浮点数与整数的二元运算时编译器会自动将整型转换为浮点类型在进行运算。所以楼主的程序的问题并不在这儿。而在数组访问越界。楼主，C语言里面数组的下标是从0开始的。所以一个10元素的数组，合法的下标是从0到9。a[0]是第一个元素，a[9]是第十个元素。你的while循环和for循环里面数组的下标都是从1开始的，所以不管是你往数组里面存东西还是从数组里面取东西，你都是从a[1]（因为你写的都是a[i]）开始的，也就是说数组的第一个元素a[0]你都没有用，然后一直到a[10]，而本来a[9]才是你的数组的最后一个元素，结果你非法访问了你的数组的有效范围以外的一个内存单元，即所谓的a[10]，这个单元里本来就不在你的数组有小范围之内，所以它里面的值是垃圾值，你打印出来当然就可能是任意值了（这个在不同的运行环境下可能有不同的表现，比如说打印一个很大的值或者一个负数，反正不是你想要的值）。你的程序有两种改法：
1. i=0; while( i < 10 )
for ( i = 0; i < 10; i++ )

2. 把循环体里面的a[i]改成a[i-1]，这样你的数组访问就不会越界了，i=1的时候，刚好是a[0]，i=9的时候，刚好到最后一个元素a[9]。

另外，楼上的说得也没错，你最好把你的运算操作数都改成浮点类型(float或者double），不要依赖编译器帮你做类型转换

Answer 2

long int x1,x2,x4,x6,x10,x;
int i;
long int a[10];
i=1;
while(i<=10)
{
scanf("%d",&a[i]);
改为float int x1,x2,x4,x6,x10,x;
int i;
float a[10];
i=1;
while(i<=10)
{
scanf("%f",&a[i]);
就可以了，调试过，已经通过，运行正常
补充：刚才看了楼下day9981 的回答，首先我同意他说的应该说用long int也没有关系，但是我不同意他所说的问题出现在下标越界一说。不信的话你可以在scanf语句下面紧跟着加一句printf("%d",a[i])；让程序把你刚刚输入存入数组的每一个元素都输出来看看？你会发现输出的每个数都不是你输入的数。这应该不是越界造成的吧。
至于为什么long int从理论上讲是可以的，但是这段程序中不行，要改成float才行，我水平有限解释不来了，请高手作答！

第二次补充：
刚才又进行了调试，发现原来是这么低级的一个错误，就是scanf语句中的格式符应该改为%ld而不是%d,还有下标越界也是要改的
最后给一段修改好的吧：
void main()
{
long int x1,x2,x4,x6,x10,x;
int i;
long int a[10];
i=0; （原来是i=0)
while(i<=9) (原来是i<=10)
{
scanf("%ld",&a[i]);(原来是%d)
i++;
}
x1=100000*0.1;
x2=x1+100000*0.075;
x4=x2+200000*0.05;
x6=x4+200000*0.03;
x10=x6+400000*0.015;
for(i=0;i<=9;i++)
{
if(a[i]<=100000)
x=a[i]*0.1;
else if(a[i]<=200000)
x=x1+(a[i]-100000)*0.075;
else if(a[i]<=400000)
x=x2+(a[i]-200000)*0.05;
else if(a[i]<=600000)
x=x4+(a[i]-400000)*0.03;
else if(a[i]<=1000000)
x=x6+(a[i]-600000)*0.015;
else
x=x10+(a[i]-1000000)*0.01;
printf("x=%ld\n",x);
}
}

Answer 3

void main()
{
long int x1,x2,x4,x6,x10,x;
long int m;
printf("please input this month's profit:\n");
scanf("%ld",&m);
x1=100000*0.1;
x2=x1+100000*0.075;
x4=x2+200000*0.05;
x6=x4+200000*0.03;
x10=x6+400000*0.015;
if(m<=100000)
x=m*0.1;
else if(m<=200000)
x=x1+(m-100000)*0.075;
else if(m<=400000)
x=x2+(m-200000)*0.05;
else if(m<=600000)
x=x4+(m-400000)*0.03;
else if(m<=1000000)
x=x6+(m-600000)*0.015;
else
x=x10+(m-1000000)*0.01;
printf("x=%ld\n",x);
getchar();
getchar();
}

Answer 4

void
main()
{
long
int
x1,x2,x4,x6,x10,x;
long
int
m;
printf("please
input
this
month's
profit:\n");
scanf("%ld",&m);
x1=100000*0.1;
x2=x1+100000*0.075;
x4=x2+200000*0.05;
x6=x4+200000*0.03;
x10=x6+400000*0.015;
if(m<=100000)
x=m*0.1;
else
if(m<=200000)
x=x1+(m-100000)*0.075;
else
if(m<=400000)
x=x2+(m-200000)*0.05;
else
if(m<=600000)
x=x4+(m-400000)*0.03;
else
if(m<=1000000)
x=x6+(m-600000)*0.015;
else
x=x10+(m-1000000)*0.01;
printf("x=%ld\n",x);
getchar();
getchar();
}

Answer 5

哦,提个建议啊??
首先我觉得出现负数,
1.是数组越界,多调试可以发现是否越界
2.是由于非法数字出现了!
你的程序结构不合理!
1.c语言中主要是函数,把处理的部分独立出一个函数来!
2.用switch case 语句替代那么多的if语句
这样的话,结构清晰,也不容易出错误.
就剩下很多事情了,那会出现什么错误啊?