结对审查1

• 结对伙伴的源代码
  点此进入GitHub查看源代码
• 审查表

• 对伙伴代码的评价
  通过对刘家兴同学代码的审查，我发现了许多值的我学习的优点，虽然代码较长，但是给人的整体观感较为直白整洁，主函数精干简短，通俗易懂，这是他的一次单片机课设代码，已成功在proteus仿真以及试验箱上正常运行，而且LCD的功能实现也体现了刘家兴同学对单片机开发有一定的掌握，虽然还有许多可圈可点的优点,但是也有些许不足,如版权和版本声明问题，以及函数内容问题。
• 优点
  对变量的名的定义直观可读,甚至有清楚的注释

uchar Page_;//页地址寄存器
uchar Code_;//字符代码寄存器
uchar Command;//指令寄存器
uchar LCDData;//数据寄存器
uchar Column;//列地址

?代码整洁，规范化,对函数以及相关条件语句有注释，值得我学习

void LCDinit()//初始化
void Clear()//清屏
if(k<PD1)//写左屏
else//写右屏

• 缺点
  版权和版本声明不完整。
  延时函数内容空洞，仅是一个简单的while循环，失去了函数存在的意义，而且形参不够直观的表达延时时间。建议参照单片机的机器周期，改为一个数字单位延时1ms。

?刘家兴同学的代码

void Delay(uchar i)

{

    while(i--);

}

?更改后的代码

void Delay(uchar i)

{
    for(n=0;n<i;n++)
    while(110--);
}

原文地址：https://www.cnblogs.com/daheUU/p/10759212.html