/* 从朋友那看来的,因为他十分警惕的守护自己的代码 所以下面的代码都是凭着我的记忆写下的伪代码, 别问我这是什么语言,大致就是类C语言吧 这是一个消息转发系统的一个子系统,这个系统通过转发消息收取费用 子系统的功能是读取消息,判断能不能转发,并设置收费标志 由别的子系统完成收费,转发的动作 问题:这个程序需要重构吗?如果需要怎么重构? */ void main() { Queue queue = new Queue(); while(Message message=queue.Get()) { process(message); } } void process(Message message) { if(isAllowSend(message) == true) { setSendFlag(message); } // 记录每一个进入系统的message信息 logMessage(message); } bool isAllowSend(Message message) { // 消息发送者在白名单中 if(message.sender in getWhileList()) { return true } // 消息发送者在黑名单中 if(message.sender in getBLackList()) { return false; } // 下面这许多if是判断什么的,我不记得了 if(...) { return true; } if(...) { return false; } // 设置收费标志 setChargeFlag(message); return true; } void setSendFlag(Message message) { // 下面这些if判断什么的,我也不记得了 if(...) { return; } if(...) { return; } message.allowSend = true; saveMessage(message); // 设置Message属性,保存它,由别的子系统进行发送 } void setChargeFlag(Message message) { message.isCharge = true; }