悠然乱弹:从几个方法的重构讲开去--引言

简介:

引言:

在学习代码的过程中,看到如下几个工具方法:

?
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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
// 获取指定包名下的所有类
     public static List<Class<?>> getClassList(String packageName, boolean isRecursive) {
         List<Class<?>> classList = new ArrayList<Class<?>>();
         try {
             Enumeration<URL> urls = getClassLoader().getResources(packageName.replace( "." , "/" ));
             while (urls.hasMoreElements()) {
                 URL url = urls.nextElement();
                 if (url != null ) {
                     String protocol = url.getProtocol();
                     if (protocol.equals( "file" )) {
                         String packagePath = url.getPath();
                         addClass(classList, packagePath, packageName, isRecursive);
                     } else if (protocol.equals( "jar" )) {
                         JarURLConnection jarURLConnection = (JarURLConnection) url.openConnection();
                         JarFile jarFile = jarURLConnection.getJarFile();
                         Enumeration<JarEntry> jarEntries = jarFile.entries();
                         while (jarEntries.hasMoreElements()) {
                             JarEntry jarEntry = jarEntries.nextElement();
                             String jarEntryName = jarEntry.getName();
                             if (jarEntryName.endsWith( ".class" )) {
                                 String className = jarEntryName.substring( 0 , jarEntryName.lastIndexOf( "." )).replaceAll( "/" , "." );
                                 if (isRecursive || className.substring( 0 , className.lastIndexOf( "." )).equals(packageName)) {
                                     classList.add(loadClass(className, false ));
                                 }
                             }
                         }
                     }
                 }
             }
         } catch (Exception e) {
             logger.error( "获取类出错!" , e);
             throw new RuntimeException(e);
         }
         return classList;
     }
 
     // 获取指定包名下指定注解的所有类
     public static List<Class<?>> getClassListByAnnotation(String packageName, Class<? extends Annotation> annotationClass) {
         List<Class<?>> classList = new ArrayList<Class<?>>();
         try {
             Enumeration<URL> urls = getClassLoader().getResources(packageName.replace( "." , "/" ));
             while (urls.hasMoreElements()) {
                 URL url = urls.nextElement();
                 if (url != null ) {
                     String protocol = url.getProtocol();
                     if (protocol.equals( "file" )) {
                         String packagePath = url.getPath();
                         addClassByAnnotation(classList, packagePath, packageName, annotationClass);
                     } else if (protocol.equals( "jar" )) {
                         JarURLConnection jarURLConnection = (JarURLConnection) url.openConnection();
                         JarFile jarFile = jarURLConnection.getJarFile();
                         Enumeration<JarEntry> jarEntries = jarFile.entries();
                         while (jarEntries.hasMoreElements()) {
                             JarEntry jarEntry = jarEntries.nextElement();
                             String jarEntryName = jarEntry.getName();
                             if (jarEntryName.endsWith( ".class" )) {
                                 String className = jarEntryName.substring( 0 , jarEntryName.lastIndexOf( "." )).replaceAll( "/" , "." );
                                 Class<?> cls = loadClass(className, false );
                                 if (cls.isAnnotationPresent(annotationClass)) {
                                     classList.add(cls);
                                 }
                             }
                         }
                     }
                 }
             }
         } catch (Exception e) {
             logger.error( "获取类出错!" , e);
             throw new RuntimeException(e);
         }
         return classList;
     }
 
     // 获取指定包名下指定父类的所有类
     public static List<Class<?>> getClassListBySuper(String packageName, Class<?> superClass) {
         List<Class<?>> classList = new ArrayList<Class<?>>();
         try {
             Enumeration<URL> urls = getClassLoader().getResources(packageName.replace( "." , "/" ));
             while (urls.hasMoreElements()) {
                 URL url = urls.nextElement();
                 if (url != null ) {
                     String protocol = url.getProtocol();
                     if (protocol.equals( "file" )) {
                         String packagePath = url.getPath();
                         addClassBySuper(classList, packagePath, packageName, superClass);
                     } else if (protocol.equals( "jar" )) {
                         JarURLConnection jarURLConnection = (JarURLConnection) url.openConnection();
                         JarFile jarFile = jarURLConnection.getJarFile();
                         Enumeration<JarEntry> jarEntries = jarFile.entries();
                         while (jarEntries.hasMoreElements()) {
                             JarEntry jarEntry = jarEntries.nextElement();
                             String jarEntryName = jarEntry.getName();
                             if (jarEntryName.endsWith( ".class" )) {
                                 String className = jarEntryName.substring( 0 , jarEntryName.lastIndexOf( "." )).replaceAll( "/" , "." );
                                 Class<?> cls = loadClass(className, false );
                                 if (superClass.isAssignableFrom(cls) && !superClass.equals(cls)) {
                                     classList.add(cls);
                                 }
                             }
                         }
                     }
                 }
             }
         } catch (Exception e) {
             logger.error( "获取类出错!" , e);
             throw new RuntimeException(e);
         }
         return classList;
     }

第一感觉,觉得这几个方法肯定有问题,但是问题在哪里呢?

首先看直接看到的问题:

  • 代码重复得比较多
  • 代码圈复杂度比较大
  • 异常处理模式不能说错,但是是否合理值得商榷

代码重复得比较多,是大家一眼就可以看得到的,但是怎么改?确实也是狗咬刺猬无从下口。
代码圈复杂度,用来标示一个代码的复杂程度,不知道概念的话,问下度娘就知道了。
异常处理模式确实不能算错,也就是说只要有一个类出错,就导致整个处理中断。但是我个人对于处理Class,注解什么的,比较倾向于有错记下来,但是不要影响别的类的处理的模式。只能说两种模式各有优缺点,因此可以商榷。

再来分析下深层次方面的问题

  • 性能问题
  • 扩展问题

说到性能问题,我们都知道,应用大到一定程度的时候,Jar文件,类文件都是比较多的,如果每处理一个注解之类的就扫描一次,就会化大量的时候在重复的目录(JarEntry)遍历上;我在合计20M的Jar里遍历所有的class文件,在本人笔记本上大概是2S时间,如果注解多了,那可是一个注解2秒的时间呀。

说到扩展问题,现在我们处理的都是Jar文件,那当然可能有本地文件,也可能有URL外部的文件,还可能有自已写的ClassLoader加载的其它文件,上面的处理无疑只能处理前面两种,如果是自己写的ClassLoader肯定是不在扫描之列了。另外,现在处理的是class文件,当然也可能处理的是一些xml文件或国际化文件等等。这样子一来,这里的工具方法岂不是要爆炸式增长??

当然,写此代码作者决非一般人等,如果简单,他不会让它留着的。

因此,在这短短的几个方法里都就隐藏着怎样的秘密呢?,

相关文章
|
10月前
|
算法
谈一谈|浅谈单纯形法其中奥妙
谈一谈|浅谈单纯形法其中奥妙
120 0
|
数据库 开发者
读书·技术 |《重构》· 重构的原则
读书·技术 |《重构》· 重构的原则
135 0
|
小程序 数据安全/隐私保护 C语言
爱上c++的第九天-聊一聊类和对象的特性
总结如下:1.基于过程的程序设计:由一个个函数组成,每个函数都是独立的,一个函数对应一个操作,适合小程序开发。讲究每一步都要详细的写出,写出来,问题也就解决了。
47 0
爱上c++的第九天-聊一聊类和对象的特性
|
Java Spring
9条消除if...else的锦囊妙计,助你写出更优雅的代码(下)
9条消除if...else的锦囊妙计,助你写出更优雅的代码(下)
|
设计模式 算法 Java
9条消除if...else的锦囊妙计,助你写出更优雅的代码
9条消除if...else的锦囊妙计,助你写出更优雅的代码
|
设计模式 移动开发 安全
与其硬啃“屎山”代码,不如用这六步有条不紊实现代码重构 李慧文
对大规模系统进行重构,如果一个人对着又臭又长的代码硬刚,即使花了大量的时间进行手工验证,最后仍然会有很多问题,特别是一些深路径及特殊场景下的问题。其实,大规模的系统级别重构时是有方法的。我们采访了 Thoughtworks 数字化转型与运营 资深咨询师黄俊彬(QCon+案例研习社讲师),请他来分享 MV*模式重构演进的方法和经验。
521 0
与其硬啃“屎山”代码,不如用这六步有条不紊实现代码重构 李慧文
|
存储 运维 Kubernetes
独家交付秘籍之招式拆解(第一回)
上一回说到经历种种交付难题的王小锤一行人,意外发现一本交付秘籍,打开了新世界。本次他们带着具体交付场景来到阿里云,与交付宗师阿莫探讨秘籍中的招式以及招式背后的秘密。
独家交付秘籍之招式拆解(第一回)
|
XML 测试技术 程序员
人月神话札记:整体部分
人月神话札记:整体部分
97 0
人月神话札记:整体部分