打印
[STM32F1]

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

[复制链接]
2353|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 | 只看该作者
只能问写代码的本人

使用特权

评论回复
5
mmuuss586| | 2015-7-21 19:29 | 只看该作者

还行吧;

使用特权

评论回复
6
qq65411253| | 2015-7-21 19:43 | 只看该作者
这种“牛B”的程序已经跟不上时代的发展了,易读易懂易维护才是王道。

使用特权

评论回复
评论
bhuner 2015-7-22 09:48 回复TA
赞 
7
acguy| | 2015-7-21 19:52 | 只看该作者
贵公司允许员工把整段代码公开? 小心吃官司。

使用特权

评论回复
8
john_lee| | 2015-7-21 20:06 | 只看该作者
写的还算比较清晰、规范,不是菜鸟了。
你是想看懂其中的“业务逻辑”?这个还是算了吧。去把设计文档拿来对照看才是靠谱。

使用特权

评论回复
9
fanxsd| | 2015-7-21 20:56 | 只看该作者
这代码 不是牛 是不易维护,你看看 UCOS 源码 freemodbus 源码 那才是牛,功能复杂结构清晰。。

使用特权

评论回复
10
ecoren| | 2015-7-22 09:04 | 只看该作者
看不懂,咱是搞硬件的,

使用特权

评论回复
11
diweo| | 2015-7-22 09:44 | 只看该作者
不要过度注释,也不要为了注释而注释。
我觉得这段代码挺好的。

你看不懂,可能是因为你不知道这段代码所实现功能的内在逻辑,你把内在逻辑弄清楚后,看起来应该就很容易了。

使用特权

评论回复
12
huakue| | 2015-7-22 10:35 | 只看该作者
看起来挺好的

使用特权

评论回复
13
天奕| | 2015-7-22 10:41 | 只看该作者
我觉得这段代码挺好的。

使用特权

评论回复
14
myxiaonia| | 2015-7-22 10:45 | 只看该作者
这段程序是抽象程度比较高的,可以改造到不同的外设类型,属于驱动模型了,不过很可能是从linux这种地方借鉴过来的,或者直接引用过来都有可能

使用特权

评论回复
15
songchenping| | 2015-7-22 11:24 | 只看该作者
看别人写得非标准代码简直是一种折磨啊。

使用特权

评论回复
16
hnkf118| | 2015-7-22 12:31 | 只看该作者
看来你是需要加强下基本的知识量了。

使用特权

评论回复
17
cos12a| | 2015-7-22 13:05 | 只看该作者
不懂装懂。

使用特权

评论回复
18
Leeone| | 2015-7-22 15:01 | 只看该作者
基本语法掌握了,就是按功能分析代码

使用特权

评论回复
19
幸福至上| | 2015-7-22 16:34 | 只看该作者
写得很精简,其中用法也看得出很有经验,很有功底啊。

使用特权

评论回复
20
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

粉丝