模块越独立,就越能够被重用,编译该模块的时间也越少。
衡量软件好坏的主要标准是看它的内聚性与耦合性。“高内聚、低耦合”是我们的目标。以此作为标准,会衍生出很多东西,编程规范是其中一种。我们常会见到的一条是尽量不要使用全局变量。
耦合是分级别的。全局变量的耦合性最大,静态变量次之,接着的是函数参数,局部变量的耦合性最小。因此,才有尽量避免使用全局变量的纪律。
以我们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关键字将它变成静态变量,这样代码就又向高质量迈进了一步。