IA_函数设计推荐.pptVIP

函数设计 IA事业部 一个编写的不好的函数 Void HandleStuff(CORP_DATA* pInputRec, int CrntQtr, EMP_DATA EmpRec, float* pEstimRevenue, float YTDRevenue, int ScreenX, int ScreenY, COLOR_TYPE * pNewColor, COLOR_TYPE PrevColor, STATUS_TYPE* pStatus , int ExpenseType) { int i; for( i = 1; i= 100; i++){ pInputRec-revenue[i] = 0; pInputRec-expense[i] = corpExpense [CrntQtr][i]; } UpdateCorpDatabase( EmpRec ) ; *pEstimRevenue = YTDRevenue * 4.0 /(( float )CrntQtr); *pNewColor = PrevColor; *pStatus = Success; if( ExpenseType == 1){ Profit[i] =Revenue[i] - Expense.Type1[i]; } else if( ExpenseType == 2){ Profit[i] = Revenue[i] - Expense.Type2[i]; } else (if ExpenseType == 3){ Profit[i] = Revenue[i] - Expense.Type3[i]; } } 上一个函数的缺点 函数的名字不好，不能说明函数是干什么的 函数进行了全局变量的读写操作。 CorpExpense、profit都是全局变量 函数的功能不是单一的。 它初始化了某些变量；对一个数据库进行写操作；又进行了某些计算工作。而这些功能之间又看不出任何联系 函数中没有采取预防非法数据的措施 如果CrntQtr的值为“0”，那么，表达式YTDRevenue* 4.0／real（CrntQtr）就会出现被零除的错误 指针没有检查是否为空 在函数中仅使用了CORP-DATA型参数的两个域，却传入了整个结构化变量 函数中的一些参数没有使用过 ：ScreenX和ScreenY 函数中的参数太多 函数中使用了几个常数(神秘数字): 100，4．0，2和 3 函数设计的话题 函数设计概述 大原则 内聚性 耦合性 信息隐藏 不好搞小动作 小细节 函数名 参数 返回值 函数内部设计 函数设计概述 函数设计主要在详细设计中做 创建函数的理由（1） 降低复杂性 避免重复代码段 限制改动带来的影响 隐含顺序 改进性能 实行集中控制 隐含数据结构 隐含指针操作 创建函数的理由（2） 隐含全局变量 代码段复用 计划开发一个软件族 改善某一代码段的可读性 改善可移植性 分隔复杂操作 独立非标准语言函数的使用 简化复杂的布尔测试 大原则——内聚性 内聚性是指一个函数内的各种操作之间互相联系的紧密程度 可以接受的内聚性 功能内聚性 顺序内聚性 通讯内聚性 临时内聚性 不可接受的内聚性 过程内聚性 逻辑内聚性 偶然内聚性 函数设计的目标：强内聚 可以接受的内聚性 功能内聚性 一个好的函数应该具有功能内聚性，即Do one thing and do it well 举例：Sin(),GetCustomerName(),EraseFile()等 顺序内聚性 特点：函数内的操作不能形成完整的功能，只是包含需要按特定顺序进行的、逐步分享数据的一些操作 举例：下面五个操作完成一个功能：打开文件、读文件、进行两个计算、输出结果、关闭文件。如果DoStep1()打开文件、读文件和计算操作，而DoStep2()进行输出结果和关闭文件操作。Dostep1和DoStep2只是把操作分隔开来，并没有产生出独立的功能，所以具有顺序内聚性 可以接受的内聚性 通讯内聚性 特点：函数中的两个操作只是使用相同数据，而没有其它任何联系 举例：GetNameandChangePhoneNumber()，它取得的Name和PhoneNumber在同一个用户记录中，GetName和ChangePhoneNumber之间没有其他的关系 临时内聚性 特点：函数