之前C#代码的重新设计

/*
我用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;
}

 

转载于:https://www.cnblogs.com/code-style/p/3503917.html

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值