[STM32F1] 公司的一个技术经理写的程序,看不懂啊

[复制链接]
2668|20
 楼主| amd1217 发表于 2015-7-21 17:07 | 显示全部楼层 |阅读模式
本帖最后由 amd1217 于 2015-7-28 14:37 编辑

一览 发表于 2015-7-21 17:43 | 显示全部楼层
首先是一个协议处理软件:读写IO口的。pdu 的意思是protocol data unit.
可以使用 ucos
renqinglei 发表于 2015-7-21 18:09 | 显示全部楼层
卡不懂
大秦正声 发表于 2015-7-21 19:26 | 显示全部楼层
只能问写代码的本人
mmuuss586 发表于 2015-7-21 19:29 | 显示全部楼层

还行吧;
qq65411253 发表于 2015-7-21 19:43 | 显示全部楼层
这种“牛B”的程序已经跟不上时代的发展了,易读易懂易维护才是王道。

评论

赞  发表于 2015-7-22 09:48
acguy 发表于 2015-7-21 19:52 | 显示全部楼层
贵公司允许员工把整段代码公开? 小心吃官司。
john_lee 发表于 2015-7-21 20:06 | 显示全部楼层
写的还算比较清晰、规范,不是菜鸟了。
你是想看懂其中的“业务逻辑”?这个还是算了吧。去把设计文档拿来对照看才是靠谱。
fanxsd 发表于 2015-7-21 20:56 | 显示全部楼层
这代码 不是牛 是不易维护,你看看 UCOS 源码 freemodbus 源码 那才是牛,功能复杂结构清晰。。
ecoren 发表于 2015-7-22 09:04 | 显示全部楼层
看不懂,咱是搞硬件的,
diweo 发表于 2015-7-22 09:44 | 显示全部楼层
不要过度注释,也不要为了注释而注释。
我觉得这段代码挺好的。

你看不懂,可能是因为你不知道这段代码所实现功能的内在逻辑,你把内在逻辑弄清楚后,看起来应该就很容易了。
huakue 发表于 2015-7-22 10:35 | 显示全部楼层
看起来挺好的
天奕 发表于 2015-7-22 10:41 | 显示全部楼层
我觉得这段代码挺好的。
myxiaonia 发表于 2015-7-22 10:45 | 显示全部楼层
这段程序是抽象程度比较高的,可以改造到不同的外设类型,属于驱动模型了,不过很可能是从linux这种地方借鉴过来的,或者直接引用过来都有可能
songchenping 发表于 2015-7-22 11:24 | 显示全部楼层
看别人写得非标准代码简直是一种折磨啊。
hnkf118 发表于 2015-7-22 12:31 | 显示全部楼层
看来你是需要加强下基本的知识量了。
cos12a 发表于 2015-7-22 13:05 | 显示全部楼层
不懂装懂。
Leeone 发表于 2015-7-22 15:01 | 显示全部楼层
基本语法掌握了,就是按功能分析代码
幸福至上 发表于 2015-7-22 16:34 | 显示全部楼层
写得很精简,其中用法也看得出很有经验,很有功底啊。
lyzhangxiang 发表于 2015-7-22 17:51 | 显示全部楼层
思想不错
static __align(4)  uint  sg_pack_buffer[CMD_IO_BUFF_CNT][1 + CMD_PACK_MAXSIZE/sizeof(uint)];          // buffer池
static             uint* sg_pack_buff_que = &sg_pack_buffer[0][1];                                        // 首个空闲buffer索引
这里处理的很好

您需要登录后才可以回帖 登录 | 注册

本版积分规则

15

主题

183

帖子

3

粉丝
快速回复 在线客服 返回列表 返回顶部