emule 代码 的 无责任 分析,设计模式角度

 昨天因为想分析一下文件句柄的占用问题就 下了emule的代码来看，emule 作为sourceforge上most active 的项目影响是很大的。但是就如大多数由少数牛人写的代码一样，在很不负责任的看了一遍之后，我的感受是该项目的代码写的相当hackery，第一点是几乎就没有文档来描述架构之间的关系，连Doxegen也毛有。就有一个耶路撒冷的大哥的貌似毕设论文的东东对protocol进行了详细的阐述，不过代码中变量的名字含义还是很清晰的。可是如果说道重用代码等等的话，那就很难了，因为项目中的C++类的基本作用是作为数据结构使用的.....

在使用了ACE/TAO一段时间后，能感受到设计模式对软件工程强大作用，也因为在使用以及分析了符合设计模式的代码，我现在才能开始看得懂emlue 这样的代码，在以前，功能组件之间的联系就把我转晕了，但是现在因为知道其代码实现的目的才能返过来找到实现的细节之处。emlue是四，五个充满奉献精神的大哥4年多的无私贡献，我如果还说其不符合学院派的设计模式未免站着说话不腰疼，但是正如文学评论一样，还是有代码分析的必要的嘛，嘿嘿。

如果遵循设计模式的话：

1.上传线程的算法应该包装到strategy的类里
UploadQueue.h

2.下载监听采用reactor模式，通过下载工厂激活下载连接
3.对于跨越代理的连接而言不应该跟socket绑定，而应该作为继承的layer在socket连接后处理
4.part file的生成以及管理应该由part file工厂来管理，其与knwonfile的关系不单只是继承而应该是聚合
5.客户-服务器，客户-客户间协议似乎应该由xml tag来描述，以便于extend 以及log
6.tcp 连接 udp 连接应该作为 不同imp 桥接到连接类上
7.与GUI界面代码应该作到完全分离，信息传输通过注册回调方法完成
8.higID 与lowID的用户似乎也应该用桥接的模式到Client类上，尽管二者行为是如此不同

 

完全可分离的代码资源：

1.stall lipman的Parser 代码
2.md4的asm代码
3.一个扩展了CSocket的代理socket
4.XP自带防火墙的COM接口代码
5.Base64 的转换代码
6.QuickSort实现的代码
等等...........