c coding style之学习篇

1. 使用do-while结构去避免潜在的内存泄漏问题。

do
{
    p1 = malloc(10);
    if (null == p1)
    {
        break;
    }
    p2 = malloc(20);
    if (null == p2)
    {
        break;
    }

    return ok;
}while(0);

de_init();
return fail;

 

2. 使用指针时必须先检查指针的有效性，这样做的一大好处是可以防止误用null pointer

而引起系统crash。

if (p1 != null)
{
    do_something();
}
else
{
    printf("\nnull pointer\n");
}

 

3. free一个指针时，记得将其置为null，否则这个指针就会变成wild pointer，

这会导致系统变得不稳定。

if (p1 != null)
{
    free(p1);
    p1 = null;
}

 

4. 不允许在other thread里面做你自己的事，这可能会block other thread做自己的事。

在后面callback function里面会细说这件事。

 

5. local variable必须要初始化。虽然这一条rule不总是必须的，但是养成好习惯总不是坏事。

int value = 0;

详细可以参考我下面的博文：

http://blog.csdn.net/boyxulin1986/article/details/11474923

 

6. dynamically分配的内存也需要初始化，

p1 = malloc(10);
memset(p1, 0, sizeof(10));

详细可参考我的博文：

 http://blog.csdn.net/boyxulin1986/article/details/11474923

 

7. 你的程序中不应该有可能导致系统卡住的死循环。

这里如果is_done()总是false，那程序就永远卡住了；最坏情况如果is_done()总是false，

那应该要怎样做？

while(false == is_done())
{
    continue;
}

 

8. 不要滥用comments，comments不是用来解释语法的。

而且comments如果与code不match的话，会给代码维护者带来巨大无比的困扰。

 

9. 当你给你的function，给你的variable命名时，请先想一下。

命名规则很重要，它直接会影响到代码的可读性以及可维护性。

 

10. 代码中不要有深层次的递归调用，太深的递归调用可能会block住thread甚至会让

thread的stack爆掉。

 

11. 要用模块化的方法来设计软件，降低模块间的耦合性、提高模块的内聚行。

模块化的好处是使得程序具有更好的重用性、更容易维护以及更容易扩展等。

 

12. function要尽量短而小，一个function实现一个功能。

 

以上是目前本人总结的一些心得体会，实际上coding style远不止于以上几条。

后面会结合自己的学习工作陆续的补充，谢谢。