[编程好习惯] 使用sizeof减少内存操作失误

来源：李云责任编辑：栏目编辑发表时间:2013-07-01 19:35　点击:次

在编程过程中，在大量的情况下需要使用memset()函数对内存进行置零初始化（除了这里说的memset()其实很多函数，比如snprintf()、strncpy()等都可以借用这里谈到的方法），以下三种情况是在工作中经常在这方面碰到的错误，大部分错误是因为疏忽而造成的。
例1

char *buf [FAIP_WITHDOT_MAX_LEN+1];
memset (buf, 0, FAIP_WITHDOT_MAX_LEN + 1);

例2

#define DIGEST_LEN 17
#define DIGEST_MAX 16

char digest [DIGEST_MAX];
memset (digest, 0, DIGEST_LEN);

例3

dll_node_t *p_node = malloc (sizeof (dll_node_t));
if (p_node == 0)
return;
memset (p_node, 0, sizeof (dll_t));

下面示例了如何采用一般化的方法更正上面的三种错误。
例1修订1

char *buf [FAIP_WITHDOT_MAX_LEN+1];
memset (buf, 0, sizeof (char *) * (FAIP_WITHDOT_MAX_LEN + 1));

例2修订1

#define DIGEST_LEN 17
#define DIGEST_MAX 16

char digest [DIGEST_MAX];
memset (digest, 0, DIGEST_MAX);

例3修订1

dll_node_t *p_node = malloc (sizeof (dll_node_t));
if (p_node == 0)
return;
memset (p_node, 0, sizeof (dll_node_t));

需要思考的是，如何在工作中尽可能的避免这类“低级”错误。虽然可以通过采用code review的方式增加错误的检出率，但这种效果并不好。更为好一点的方法是程序员养成一定的编程习惯，比如采用这里将要主张的sizeof()方法就能显著地降低这类错误。下面示例了如何采用sizeof()来更正错误。