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 的,所以现有的实现总觉得不够优雅。不知道有没有办法可以解决这个问题?
  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

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

抵扣说明:

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

余额充值