高内聚低耦合

模块越独立，就越能够被重用，编译该模块的时间也越少。

衡量软件好坏的主要标准是看它的内聚性与耦合性。“高内聚、低耦合”是我们的目标。以此作为标准，会衍生出很多东西，编程规范是其中一种。我们常会见到的一条是尽量不要使用全局变量。

耦合是分级别的。全局变量的耦合性最大，静态变量次之，接着的是函数参数，局部变量的耦合性最小。因此，才有尽量避免使用全局变量的纪律。

以我们53的项目为例——密码判断：

void EntryPhnsetMyPasswordUseLock(void){    extern U8 g_message_lock_next_screen;    extern FuncPtr g_unlock_func;    g_message_lock_next_screen = 5;    g_unlock_func = EntryPhnsetMyPasswordUse;    EntryMessageLockValidateScreen();}

为了进行密码判断，这个函数用到了两个全局变量和一个全局函数。它对密码判断文件的依赖数为3。密码判断模块暴露了它自己的实现细节。使用者必须要知道g_message_lock_next_screen和g_unlock_func的意思，这不仅增加了使用者负担，而且如果实现方式改变（假设全局变量g_message_lock_next_screen改为g_lock_next_screen），所有使用到的地方都要修改。修改是需要时间的。

改进的第一步，可以把全局变量换成函数参数（参数的耦合性小于全局变量）。

void EntryPhnsetMyPasswordUseLock(void){    extern U8 g_message_lock_next_screen;    FuncPtr unlock_func = EntryPhnsetMyPasswordUse;    g_message_lock_next_screen = 5;    EntryMessageLockValidateScreen(unlock_func);}

g_message_lock_next_screen也可以作为参数传进去，但是发现这个变量在现有的代码中没有任何作用。所以应该删除掉。整理后的代码会是这样子：

void EntryPhnsetMyPasswordUseLock(void){    EntryMessageLockValidateScreen(EntryPhnsetMyPasswordUse);}

在所有需要密码的地方，都会看到同样的代码：

void mmi_vdoply_entry_app(void){#ifdef __MODIFY_FOR_PASSWORD__     S16 error;     extern U8 g_message_lock_next_screen;    extern U16 gLockFlag[];    extern FuncPtr g_unlock_func;    ReadValue(NVRAM_VIDEO_PLAY_FLAG_LOCK, &gLockFlag[8], DS_SHORT, &error);         if(1 == gLockFlag[8])      {         g_message_lock_next_screen = 11;         g_unlock_func = mmi_vdoply_entry_app_Lock;         EntryMessageLockValidateScreen();    }    else    {         mmi_vdoply_entry_app_Lock();    }#else /* */     ....#endif}

密码是否开启的判断不仅存在大量重复，而且暴露了太多实现细节，给用户增加负担。更好的代码可以是下面这个样子：

void mmi_vdoply_entry_app(void){    if (is_need_password(ID_VIDEO_PLAY))      {         EntryMessageLockValidateScreen(mmi_vdoply_entry_app_Lock);    }    else    {         mmi_vdoply_entry_app_Lock();    }    ....}

所有需要密码判断的地方只要调用两个函数，一个函数返回密码是否开启，一个函数进行密码验证。至于函数内部怎么实现的，用户根本不需要关心。

函数is_need_password也可以专心做一件事情，返回标志位：

MMI_BOOL is_need_password(U16 index){    if (gLockFlag[index] == 1)    {         return MMI_TRUE;    }    return MMI_FALSE;}

由于gLockFlag只会在一个文件里用到，根据前面提到低耦合原则，gLockFlag并不需要是全局变量， 加上static关键字将它变成静态变量，这样代码就又向高质量迈进了一步。