技术宅,fat-man

增加语言的了解程度可以避免写出愚蠢的代码

导航

一段代码,欢迎讨论

/*
从朋友那看来的,因为他十分警惕的守护自己的代码
所以下面的代码都是凭着我的记忆写下的伪代码,
别问我这是什么语言,大致就是类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;
}

 

posted on 2013-12-31 14:24  codestyle  阅读(772)  评论(1编辑  收藏  举报