zoukankan      html  css  js  c++  java
  • dotText: 一个代码级别的重构及疑惑

    以下是我在阅读 dotText 过程中碰到的一个小问题。(博客园发布版本的代码,原版我不了解)

    下面这个类:

    namespace Dottext.Framework.EntryHandling
    {
        
    using System.Threading;
        
    using Dottext.Framework.Components;
        
    using Dottext.Framework.Util;

        
    /// <summary>
        
    /// 为了将 IEntryHandlers 加到托管队列里的帮助类
        
    /// </summary>
        public class EntryHanlderQueue
        {
            
    //Current factory and entry
            public EntryHanlderQueue(IEntryFactoryHandler factory, Entry e)
            {
                
    this._factory = factory;
                
    this._entry = e;
            }

            
    private IEntryFactoryHandler _factory = null;
            
    private Entry _entry = null;

            
    public void Enqueue(object state)
            {
                _factory.Process(_entry);
            }

            
    /// <summary>
            
    /// Creates an Instance of FactoryQueue and adds the Factory to the queue.
            
    /// </summary>
            
    /// <param name="factory"></param>
            
    /// <param name="e"></param>
            public static void Enqueue(IEntryFactoryHandler factory, Entry e)
            {
                EntryHanlderQueue queue 
    = new EntryHanlderQueue(factory, e);
                ManagedThreadPool.QueueUserWorkItem(
    new WaitCallback(queue.Enqueue));
            }
        }
    }

    其中的

      public void Enqueue(object state)
      {
       _factory.Process(_entry);
      }


    方法分明是单纯用作一个 callback,仅仅被自身类里面实现的一个 static 方法调用到。该方法在整个 dotText 代码的其他地方也没有引用到,而且从语意上讲也和 "Enqueue" 不相关。因此我认为将该方法重命名为 EntryHandlerCallback 更加妥当,这样可以避免阅读时的概念混淆(我看到这里的时候一开始就被该方法名弄糊涂了)。

    另外,该类的实现依赖于 ManagedThreadPool, 因此必须把该方法作为一个 public 级别的方法公开,但是对于 EntryHandlerQueue 类的 client 来讲,实际上应该不允许他们了解上述作为 Callback 的方法,他们只需要调用
    public static void Enqueue(IEntryFactoryHandler factory, Entry e)
    方法即可。同样该类的 constructor 也不应该是 public 的,所以现有的实现总觉得不够优雅。不知道有没有办法可以解决这个问题?
  • 相关阅读:
    400 Bad Request
    Django 中间件 阅读目录 初步了解 中间件 中间件的介绍 自定义中间件 process_template_response(用的比较少) 中间件的执行流程 中间件版登录验证
    关于python语言优化的一些思考
    从系统角度考虑性能优化 【被面试官吊打】从系统角度考虑性能优化 目录 什么是性能? 定义系统范围 重定义功能 关注价值 总结
    需求设计
    开源软件评选白热化,这些项目冲击 Top 5
    两个向量的outer product
    协同过滤算法中皮尔逊相关系数的计算 C++
    求向量间的点积
    string 类型的翻转
  • 原文地址:https://www.cnblogs.com/RChen/p/248803.html
Copyright © 2011-2022 走看看