各位老师帮我看看这个程序的不足。多谢了。

2019-07-15 21:11发布

各位老师帮我看看这个程序的不足。多谢了。
#include<reg52.h>
#define uchar unsigned char
uchar *p="I love you",flag;
void serial_init()
{
    SCON=0x50;
    PCON=0x00;
    TMOD|=0x20;
    TH1=0xFD;
    TL1=0xFD;
    TR1=1;
    EA=1;
    ES=1;
}
void serial_send(uchar a)
{
    if(flag==1)
    {    flag=0;
        ES=0;
        SBUF=a;
        while(!T1);
        T1=0;
        ES=1;
    }
}
void main()
{   
    serial_init();
    while(1);
}
void serial_tx() interrupt 4
{
    while(*p++!='')
    {
    RI=0;
    flag=1;
    serial_send(SBUF);
    }
}

友情提示: 此问题已得到解决,问题已经关闭,关闭后问题禁止继续编辑,回答。
该问题目前已经被作者或者管理员关闭, 无法添加新回复
13条回答
hobbye501
1楼-- · 2019-07-16 01:33
 精彩回答 2  元偷偷看……
魔西
2楼-- · 2019-07-16 03:59
hobbye501 发表于 2013-12-27 13:10
应该注意的问题:
1.没有项目说明,函数说明,包括作者 功能 时间 输入输出参数,返回值等等;
2.没有注释 ...

牛呀。
hobbye501
3楼-- · 2019-07-16 08:53
魔西 发表于 2013-12-27 13:18
牛呀。

都是最基本的,可以看看LINUX 或者WINDOWS的内核程序,或者看看自己买的开发板的程序,要想着这个东西 别人拿过来一眼就能看懂就可以了!程序是给人看的 ,不是自己吃的!
小分子
4楼-- · 2019-07-16 11:15
hobbye501 发表于 2013-12-27 13:10
应该注意的问题:
1.没有项目说明,函数说明,包括作者 功能 时间 输入输出参数,返回值等等;
2.没有注释 ...

老师,你批评很对。我是一个初学者,这个程序呢,比较小所以我就没有做注释,像您这像的老师,一眼就能明了。我还是想麻烦老师能指出这个程序的逻辑问题。多谢老师了。
dongyumin
5楼-- · 2019-07-16 11:42
1、自定义数据类型名最好用 typedef 不要用#define预处理命令。具体差别可以百度下。
2、函数名最好开头单词大写。比如void Serial_Init(void)。定义函数的时候缩进分开层次,这个你已经做到了。函数定义的前面,注释上功能,调用形式,传递参数以及说明等等。
3.赋值语句,赋值“=”号两边加空格。比如:a = 0;之后调试的时候易发现错误。
4、定义并初始化一个字符串。最好弄个数组。Array[]={"i LOVE YOU"}.51单片机的话在在定义的时候加上修饰关键字"code"告诉编译器,这个数组数据是不易修改数据,保存在代码区。
5、指针和变量不要一起定义。分开定义。
个人意见,仅供参考。
hobbye501
6楼-- · 2019-07-16 13:07
 精彩回答 2  元偷偷看……

一周热门 更多>