/*
我用python重构了一把这个代码
大家的反应似乎是过度设计了
好吧,我决定不那么激进,采用更中庸一些的重构
我也有些疑惑:
是否如果重构后的代码比重构前要多,就算过度了呢?
*/
void main()
{
Queue queue = new Queue();
while(Message message=queue.Get())
{
process(message);
}
}
void process(Message message)
{
if(isAllowSend(message, setChargeFlag) == true)
{
setSendFlag(message);
}
// 记录每一个进入系统的message信息
logMessage(message);
}
bool isAllowSend(Message message, setChargeFlag)
{
// 消息发送者在白名单中
if(message.sender in getWhileList())
{
return true
}
// 消息发送者在黑名单中
if(message.sender in getBLackList())
{
return false;
}
// 下面这许多if是判断什么的,我不记得了
if(...)
{
return true;
}
if(...)
{
return false;
}
// 这些代码来自setSendFlag
// 在setSendFlag里return
// 就相当于在这里return false
if(...)
{
return false;
}
if(...)
{
return false;
}
// 设置收费标志
setChargeFlag(message);
return true;
}
void setSendFlag(Message message)
{
message.allowSend = true;
saveMessage(message);
// 设置Message属性,保存它,由别的子系统进行发送
}
void setChargeFlag(Message message)
{
message.isCharge = true;
}