您的位置:首页 > 编程语言

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

2017-03-24 14:19 288 查看
最近接手了一个二次开发项目,要把以前一个单片机发送短信的功能改成用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  */,这是什么逻辑,哪位兄弟能给我指点一下呢?

我看不下去了,求您放过我吧!
内容来自用户分享和网络整理,不保证内容的准确性,如有侵权内容,可联系管理员处理 点击这里给我发消息
标签: