高内聚低耦合

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

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

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

以我们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关键字将它变成静态变量，这样代码就又向高质量迈进了一步。