二次开发发现以前代码真是惊人的错误

最近接手了一个二次开发项目,要把以前一个单片机发送短信的功能改成用GPRS通信实现。接受到的工程只有代码,没有任何说明资料。

代码走读了一遍,发现各种各样的难以自圆其说的处理逻辑。怪不得以前设备老是出问题,原来代码是喝醉后写的。

在此把发现的一些问题罗列出来,引以为戒吧。



此函数处理有三个问题,一是创建了一个time2的定时任务,却没有OSTmrStart(),更让人吃惊的是找到mygsm()这个函数实现如下。

static void mygsm(void){
    printf("hello!!\r\n");
}

真不知道,在单片机了做这个定时任务是为什么。

问题2:OSTaskCreate()了一个process_gpio的任务,那去看看该函数的实现吧。

我彻底晕了,搞了一个信号量在这,竟然一直等待信号量变化,变化之后却又没有任何处理。这难道是在跟我开玩笑,还是欺负我第一次做单片机,看不懂C代码。

第三个问题更明显了,判断了err的值是否异常,却没有任何处理逻辑,即使在if语句里面写一个空语句,我也认了呀。

    if(err != OS_ERR_NONE){
    }


接下去看另一个处理函数,还请各位兄弟指点一下,我真是实在没看懂。

此函数中也是使用OSTackCreate()创建了两个任务,分别是uart2_task和uctsk_Blink(),

但我接下去看两个函数的实现,发现真是搞不明白这两个任务到底是什么业务使用?

看到了吧,两个任务竟然都是调用OSTimeDlyHMSM(0, 0, 5, 0);     /* 10 S  */,这是什么逻辑,哪位兄弟能给我指点一下呢?

我看不下去了,求您放过我吧!



  • 1
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 打赏
    打赏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包

打赏作者

ware_soft

你的鼓励将是我创作的最大动力

¥1 ¥2 ¥4 ¥6 ¥10 ¥20
扫码支付:¥1
获取中
扫码支付

您的余额不足,请更换扫码支付或充值

打赏作者

实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值