技术宅,fat-man

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

导航

< 2025年3月 >
23 24 25 26 27 28 1
2 3 4 5 6 7 8
9 10 11 12 13 14 15
16 17 18 19 20 21 22
23 24 25 26 27 28 29
30 31 1 2 3 4 5

统计

一段代码,欢迎讨论

复制代码
/*
从朋友那看来的,因为他十分警惕的守护自己的代码
所以下面的代码都是凭着我的记忆写下的伪代码,
别问我这是什么语言,大致就是类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   codestyle  阅读(777)  评论(1编辑  收藏  举报

编辑推荐:
· 如何编写易于单元测试的代码
· 10年+ .NET Coder 心语,封装的思维:从隐藏、稳定开始理解其本质意义
· .NET Core 中如何实现缓存的预热?
· 从 HTTP 原因短语缺失研究 HTTP/2 和 HTTP/3 的设计差异
· AI与.NET技术实操系列:向量存储与相似性搜索在 .NET 中的实现
阅读排行:
· 周边上新:园子的第一款马克杯温暖上架
· Open-Sora 2.0 重磅开源!
· .NET周刊【3月第1期 2025-03-02】
· 分享 3 个 .NET 开源的文件压缩处理库,助力快速实现文件压缩解压功能!
· [AI/GPT/综述] AI Agent的设计模式综述
点击右上角即可分享
微信分享提示