MDK下新的编程习惯，和大家分享一下、、、、

只看该作者 · 2014-11-16 10:38

比如以前写

uint8_t tmp[] = {0,1,2,3,4,5,6,7,8};

在MDK下最好写成 uint8_t tmp[9] = {0,1,2,3,4,5,6,7,8};

还有在一个文件下定义 __IO t = 0;
另一个文件下，可以定义为只读或只写 extern __I t;

还有sizeof这个用的时候要谨慎在KEIL51编译器要注意。

只看该作者 · 2014-11-16 11:42

楼主，可以阐述下详细的理由！

只看该作者 · 2014-11-16 12:17

1.愿闻其区别...
2.
core_cm4.h
#ifdef __cplusplus
  #define __I    volatile          /*!< Defines 'read only' permissions                */
#else
  #define __I    volatile const    /*!< Defines 'read only' permissions                */
#endif
#define    __O    volatile          /*!< Defines 'write only' permissions             */
#define    __IO volatile          /*!< Defines 'read / write' permissions             */
3.sizeof(var)/sizeof(int)

3万 · 2014-11-16 13:19

谢谢楼主分享；

只看该作者 · 2014-11-16 19:34

这个，从编译角度上讲，其实没什么差异
只是早期的编译器可能编译不过。

只看该作者 · 2014-11-16 19:45

uint8_t tmp[] = {0,1,2,3,4,5,6,7,8};

在MDK下最好写成 uint8_t tmp[9] = {0,1,2,3,4,5,6,7,8};

愿闻其详

1万 · 2014-11-16 19:50

只看该作者 · 2014-11-16 19:54

这是要啥意思呢！求解答！

只看该作者 · 2014-11-16 19:58

sxhhhjicbb 发表于 2014-11-16 19:45
uint8_t tmp[] = {0,1,2,3,4,5,6,7,8};

在MDK下最好写成 uint8_t tmp[9] = {0,1,2,3,4,5,6,7,8};

哦。那个mdk检查边界啊。我说的对吧？？

只看该作者 · 2014-11-16 20:00

sxhhhjicbb 发表于 2014-11-16 19:45
uint8_t tmp[] = {0,1,2,3,4,5,6,7,8};

在MDK下最好写成 uint8_t tmp[9] = {0,1,2,3,4,5,6,7,8};

这2种,tmp[18]报错。

只看该作者 · 2014-11-16 20:15

icecut 发表于 2014-11-16 19:50

uint8_t tmp[] = {0,1,2,3,4,5,6,7,8};

/* Private functions -------------------------*/
int main(void)
{
tmp[18] = 0;
===============
..\user\main.c(112): warning: #175-D: subscript out of range

只看该作者 · 2014-11-16 20:16

uint8_t tmp[20] = {0,1,2,3,4,5,6,7,8};

/* Private functions -------------------------*/
int main(void)
{
tmp[18] = 0;
=====================
".\OBJ\firmware.axf" - 0 Error(s), 0 Warning(s).

只看该作者 · 2014-11-16 20:16

难道大家没发现那么？？

只看该作者 · 2014-11-16 20:18

sedatefire 发表于 2014-11-16 19:34
这个，从编译角度上讲，其实没什么差异
只是早期的编译器可能编译不过。 ...

你这没说在点子上。

我们应该充分利用编译器检查功能。根据偏移检查。
原理大概就类似memcpy 里面那个aasertt 判断偏移

只看该作者 · 2014-11-16 20:21

adce 发表于 2014-11-16 12:17
1.愿闻其区别...
2.
core_cm4.h

sizeof 我说的是这个意思在一个文件夹下面定义

U8 TMP[10] = 0;那么在这个文件下sizeof[10]就等于10

如果另一个文件extern U8 TMP[];
那么sizeof[TMP]就等于1.

一般情况下考虑咱们可能考虑通用型只在一个文件下定义个宏指出那个10.
所以就要注意

只看该作者 · 2014-11-16 20:22

xlsbz 发表于 2014-11-16 20:18
你这没说在点子上。

我们应该充分利用编译器检查功能。根据偏移检查。

恩，如果执行
tmp[9] = xx；编译器是会报警的

但是，执行 tmp[index] = xxx， index=无穷大，编译器也不会报警。

只看该作者 · 2014-11-16 20:25

sedatefire 发表于 2014-11-16 20:22
恩，如果执行
tmp[9] = xx；编译器是会报警的

用 tmp[CONSTANT] = xx
这种写法，本身可扩展性差，可移植性也差
一般都是用
struct TYPE *p = (struct TYPE *)tmp;
p->var = xxx;
BigEndianToBuf(&p->yyy, xxx);
去搞的

只看该作者 · 2014-11-16 20:30

sedatefire 发表于 2014-11-16 20:22
恩，如果执行
tmp[9] = xx；编译器是会报警的

嗯我说的有点问题看样子。以前好像不报警啊。难道是编译器版本新了？

准确说法是

另一个文件不报警

/* Includes ------------------------------------------------------------------*/
#include "microcb.h"

extern uint8_t tmp[];

void test(void)
{
tmp[30] = 0;
}

".\OBJ\firmware.axf" - 0 Error(s), 0 Warning(s).

只看该作者 · 2014-11-16 20:31

sedatefire 发表于 2014-11-16 20:25
用 tmp[CONSTANT] = xx
这种写法，本身可扩展性差，可移植性也差
一般都是用

因为在另一个文件不报警所以我在一楼的说法仍然是利大于弊

只看该作者 · 2014-11-16 20:35

sedatefire 发表于 2014-11-16 20:25
用 tmp[CONSTANT] = xx
这种写法，本身可扩展性差，可移植性也差
一般都是用

你跑题了

MDK下新的编程习惯，和大家分享一下、、、、

突出贡献奖章

技术高手奖章

以坛为家

涓涓之细流

技术领袖奖章

十世金身

坚毅之洋流

荣誉元老奖章

无冕之王奖章

沉静之湖泊

伴坛终老

缘定三生

晶莹之水滴

常驻人口

技术新星奖章