一、问题代码
看到BLink上有人问C语言编写出错怎解决,截图如下:
为了方便我把代码按原样写出来
#include <stdio.h>
int main()
{
int x,y;
printf("input x:");
scanf("%d",&x);
if (x < 0)
y = -2x-1;
else if
(x = 0)
y = 0;
else
y = 2x+1;
printf("y = %d\n",y);
return 0;
}
二、分析
如果直接按照我们的数学思维来说,上面的程序是没有问题的,但是如果按C语言的角度来说是有问题的。
也这是很多学生党犯的错误,却又很难找出是错在哪里的原因
2.1 缺少了一个乘号产生的错误
在用CLion写出上面的代码,在2x都直接出现红线,鼠标放在上面,得到如下提示:
Invalid suffix 'x' on integer constant:整数常数后缀'x'无效
从代码上分析,2x其实是想写在2乘以x的意思,但是如果你作C来直接写数学的2x,它会识别不出来,没有这种写法,如果是字符你得用+号连接,如果是数字做乘法,你得用星号*
所以得写成
//把 y = -2x-1;改为:
y = -2*x-1;
...
//把 y = 2x+1;改为
y = 2*x+1;
2.2 把赋值运算号当成等于号
这个也是初学者很容易犯的,以为赋值就是等于号。正确的是==,这个是逻辑错误,IDE不能帮你检查出来的!所以这个代码也需要修改一下
//把(x = 0)改为
(x == 0)
修改完上面的,整个代码如下:
#include <stdio.h>
int main()
{
int x,y;
printf("input x:");
scanf("%d",&x);
if (x < 0)
//把2x改为2*x
y = -2*x-1;
else if
//把x=0改为x==0
(x == 0)
y = 0;
else
//把2x改为2*x
y = 2*x+1;
printf("y = %d\n",y);
return 0;
}
再次运行一下,看是否有错误,发现程序能正常运行,但是怎么看都总觉得怪怪的,阅读性不够啊!
2.3 程序规范化、美化
嵌套的if..else没有缩进,让人阅读很费力,那个if小()应该挨一起,最后变成如下
#include <stdio.h>
int main()
{
int x,y;
printf("input x:");
scanf("%d",&x);
if (x < 0)
y = -2*x-1;
else
if(x == 0)
y = 0;
else
y = 2*x+1;
printf("y = %d\n",y);
return 0;
}
三、最终正确代码
是不是好看多了,易阅读多了,为了更直观还可以加花括号,主要是防止你写多条语句,不写花括号只识别一条的。所以比较危险,为了防止这种事件发现,所以加{}号,最终代码如下:
#include <stdio.h>
int main()
{
int x,y;
printf("input x:");
scanf("%d",&x);
if (x < 0)
{
y = -2*x-1;
}
else
{
if(x == 0)
{
y = 0;
}
else
{
y = 2*x+1;
}
}
printf("y = %d\n",y);
return 0;
}