求关于java代码优化
[问题点数：200分，结帖人assdust110]
求关于java代码优化
[问题点数：200分，结帖人assdust110]
不显示删除回复
显示所有回复
显示星级回复
显示得分回复
只显示楼主
2011年12月 扩充话题大版内专家分排名第三
2008年2月 总版技术专家分月排行榜第一2008年1月 总版技术专家分月排行榜第一
2008年7月 总版技术专家分月排行榜第三2008年6月 总版技术专家分月排行榜第三
匿名用户不能发表回复！|求救，项目 Java 代码太乱，有什么方法或者工具能够优化下代吗 - V2EX
求救，项目 Java 代码太乱，有什么方法或者工具能够优化下代吗
160 天前 &aino
比如这样：
compareTo = Integer.valueOf(a).compareTo(Integer.valueOf(b));
File file = new
File(getClass().getResource("/test.txt").getFile());
String text = "";
reader = new
BufferedReader(new
FileReader(file));
String line =
line = reader.readLine();
(line == null) {
text += line.trim() + "\n";
reader.close();
(FileNotFoundException e1) {
e1.printStackTrace();
(IOException e) {
e.printStackTrace();
都说编写代码是 30%，维护是 70%，现在这样一个代码维护起来真的让人很头疼，特此求助 V 友支招
2095 次点击所在节点 &
tomczhen160 天前如果真的有工具或者方法能低成本的解决这个问题，那么这个问题就不是难题了。
aino160 天前@ 看着脑子头疼
hqs0417160 天前使用 Java 8 中的工具，File
Files.readAllLines(Path)
aino160 天前@ 不是用法，提的是 JAVA 代码混淆不清晰改如何解决
misaka19000160 天前这代码排版没法看，楼下怎么说？
janus77160 天前你的目的其实就是重构
所以这个问题就转换成了，重构有自动化的工具吗？
目前好像都是人工来的，所以……
arsgm159 天前@ 这个排版，我很怀疑这些代码就是楼主写的。
hubhub159 天前这个代码有什么问题吗？求大神说说
kyuuseiryuu159 天前代码部分应该用 Markdown 的代码块来写啊，这样看的脑壳疼。
ranleng159 天前等等. int 为啥要还 Integer.valueOf ?
aino159 天前@ 不是重构代码一说，重构代码我已完全无力
aino159 天前@ 我现在只想解决一下代码排版的问题，排版都是乱七八糟的
aino159 天前@ 这些代码不是我写的，我接的锅
aino159 天前排版问题
lovedebug159 天前你需要 checkstyle 和 findbugs 以及通过 Intellij Idea 自带的功能重构
当然 单元测试不补全吗？
lovedebug159 天前idea 设置好规则后可以通过 reformat 格式化代码
zzf2017159 天前排版问题 ide 应该都会自带的，你要设置一下，比如｛｝的位置和间距之类的
zzf2017159 天前还有 你要是有强迫症，我建议你去学 python，他的格式排版能让你从此都不想 排版
aino159 天前@ IDEA 的格式化解决不了的问题
ofblyt159 天前用 eclipse 完全够用
第 1 页 ／ 共 2 页
这是一个专为移动设备优化的页面（即为了让你能够在 Google 搜索结果里秒开这个页面），如果你希望参与 V2EX 社区的讨论，你可以继续到
上打开本讨论主题的完整版本。
是创意工作者们的社区，是一个分享自己正在做的有趣事物、交流想法，可以遇见新朋友甚至新机会的地方。
is a community of developers, designers and creative people.没有更多推荐了，
加入CSDN，享受更精准的内容推荐，与500万程序员共同成长！在 SegmentFault，学习技能、解决问题
每个月，我们帮助 1000 万的开发者解决各种各样的技术问题。并助力他们在技术能力、职业生涯、影响力上获得提升。
问题对人有帮助，内容完整，我也想知道答案
问题没有实际价值，缺少关键内容，没有改进余地
1.没有被引用和没有被使用是两码事。
javapublic List&MyConcernFriends& getConcernListByUserId(String userId) {
List&MyBlackList& myBlackLists= this.getMyBlackListByUserId(userId);
HashMap&String,Boolean& hash = new HashMap&String,Boolean&();
for (MyBlackList black : myBlackLists)
hash.put(black.getBlackUserId(),true);
BSONObject query = new BasicBSONObject();
query.put(MY_CONCERN_FRIENDS.myUserId, userId);
DBAccessAPIHelp APIHelp = new DBAccessAPIHelp();
List&String& jsonDataList = APIHelp.getJsonObjects(
DBConstant.USER_INFO_SPACE, DBConstant.MY_COCERN_FRIENDS_DATA, query);
APIHelp.freeDBConnection();
List&MyConcernFriends& list = new ArrayList&MyConcernFriends&();
for (String json : jsonDataList) {
MyConcernFriends myConcernFriends = (MyConcernFriends) JacksonUtil
.jsonToBean(json, MyConcernFriends.class);
if (hash.get(myConcernFriends.getFrUserId())==null){
list.add(myConcernFriends);
对这一段代码如何做出优化？
尽量的在for里面少创建对象。那么如何在for里面尽量少的创建对象呢？
但是不在这里面创建，又在哪里面创建呢？
比如，在List 里面查询出了一万条数据。那么这一万条数据for循环然后json反序列化变成java bean，最后放在list里面返回出去。如果是一万条数据性能就回很低，要创建一万个对象了
这段代码该如何优化呢?
答案对人有帮助，有参考价值
答案没帮助，是错误的答案，答非所问
首先，代码这种编写方式相对来说还是比较清晰的，除了查询Json数据应该封装到DAO层去，不是对性能有极端要求，不建议做过多优化。
但是如果真有必要，提供几个优化思路：
如果你的BlackList很大，
1.1 可以预估取出值的数量，在新建的时候可以调整capacity，甚至调整load factor，以免多次开辟内存空间；
1.2 或者创建一次就直接进行缓存，用触发式的更新机制，避免重复创建；
1.3 用TreeMap替代HashMap，可以略微提升查询性能，其实你这个场景完全可以用Set，可以略微减小内存开销；
1.4 可以把这个List的Id写入到redis或者memcached中去，这样不占用本应用的内存空间。
Json反序列化可以使用性能更高的类库，比如Jackson，fastjson
如果你获取数据的那端可以控制，可以考虑性能更高的序列化方法，比如protobuff。
查询结果的List如果数量可以预估，那么也可以设定初始的capacity。
感觉你自己提的想要1w个对象，却不想创建1w个对象，这感觉自我矛盾的事情，貌似是没有任何办法的。
同步到新浪微博
分享到微博？
关闭理由：
删除理由：
忽略理由：
推广（招聘、广告、SEO 等）方面的内容
与已有问题重复（请编辑该提问指向已有相同问题）
答非所问，不符合答题要求
宜作评论而非答案
带有人身攻击、辱骂、仇恨等违反条款的内容
无法获得确切结果的问题
非开发直接相关的问题
非技术提问的讨论型问题
其他原因（请补充说明）
我要该，理由是：
在 SegmentFault，学习技能、解决问题
每个月，我们帮助 1000 万的开发者解决各种各样的技术问题。并助力他们在技术能力、职业生涯、影响力上获得提升。如何优化JAVA代码及提高执行效率_百度知道
如何优化JAVA代码及提高执行效率
我有更好的答案
可供程序利用的资源（内存、CPU时间、网络带宽等）是有限的，优化的目的就是让程序用尽可能少的资源完成预定的任务。优化通常包含两方面的内容：减小代码的体积，提高代码的运行效率。本文讨论的主要是如何提高代码的效率。在Java程序中，性能问题的大部分原因并不在于Java语言，而是在于程序本身。养成好的代码编写习惯非常重要，比如正确地、巧妙地运用java.lang.String类和java.util.Vector类，它能够显著地提高程序的性能。下面我们就来具体地分析一下这方面的问题。1、
尽量指定类的final修饰符带有final修饰符的类是不可派生的。在Java核心API中，有许多应用final的例子，例如java.lang.String。为String类指定final防止了人们覆盖length()方法。另外，如果指定一个类为final，则该类所有的方法都是final。Java编译器会寻找机会内联（inline）所有的final方法（这和具体的编译器实现有关）。此举能够使性能平均提高50% 。2、
尽量重用对象。特别是String 对象的使用中，出现字符串连接情况时应用StringBuffer 代替。由于系统不仅要花时间生成对象，以后可能还需花时间对这些对象进行垃圾回收和处理。因此，生成过多的对象将会给程序的性能带来很大的影响。3、
尽量使用局部变量，调用方法时传递的参数以及在调用中创建的临时变量都保存在栈（Stack）中，速度较快。其他变量，如静态变量、实例变量等，都在堆（Heap）中创建，速度较慢。另外，依赖于具体的编译器/JVM，局部变量还可能得到进一步优化。请参见《尽可能使用堆栈变量》。4、
不要重复初始化变量
默认情况下，调用类的构造函数时， Java会把变量初始化成确定的值：所有的对象被设置成null，整数变量（byte、short、int、long）设置成0，float和double变量设置成0.0，逻辑值设置成false。当一个类从另一个类派生时，这一点尤其应该注意，因为用new关键词创建一个对象时，构造函数链中的所有构造函数都会被自动调用。5、
在JAVA + ORACLE 的应用系统开发中，java中内嵌的SQL语句尽量使用大写的形式，以减轻ORACLE解析器的解析负担。6、
Java 编程过程中，进行数据库连接、I/O流操作时务必小心，在使用完毕后，即使关闭以释放资源。因为对这些大对象的操作会造成系统大的开销，稍有不慎，会导致严重的后果。7、
由于JVM的有其自身的GC机制，不需要程序开发者的过多考虑，从一定程度上减轻了开发者负担，但同时也遗漏了隐患，过分的创建对象会消耗系统的大量内存，严重时会导致内存泄露，因此，保证过期对象的及时回收具有重要意义。JVM回收垃圾的条件是：对象不在被引用；然而，JVM的GC并非十分的机智，即使对象满足了垃圾回收的条件也不一定会被立即回收。所以，建议我们在对象使用完毕，应手动置成null。8、
在使用同步机制时，应尽量使用方法同步代替代码块同步。9、
尽量减少对变量的重复计算例如：for(int i = 0;i & list. i ++) {
…}应替换为：for(int i = 0,int len = list.size();i & i ++) {
…}10、尽量采用lazy loading 的策略，即在需要的时候才开始创建。
String str = “aaa”;
if(i == 1) {
list.add(str);}应替换为：
if(i == 1) {String str = “aaa”;
list.add(str);}11、慎用异常异常对性能不利。抛出异常首先要创建一个新的对象。Throwable接口的构造函数调用名为fillInStackTrace()的本地（Native）方法，fillInStackTrace()方法检查堆栈，收集调用跟踪信息。只要有异常被抛出，VM就必须调整调用堆栈，因为在处理过程中创建了一个新的对象。异常只能用于错误处理，不应该用来控制程序流程。12、不要在循环中使用：Try {} catch() {}应把其放置在最外层。13、StringBuffer 的使用：
StringBuffer表示了可变的、可写的字符串。有三个构造方法 :StringBuffer ();
//默认分配16个字符的空间StringBuffer (int size);
//分配size个字符的空间StringBuffer (String str);
//分配16个字符+str.length()个字符空间你可以通过StringBuffer的构造函数来设定它的初始化容量，这样可以明显地提升性能。这里提到的构造函数是StringBuffer(int length)，length参数表示当前的StringBuffer能保持的字符数量。你也可以使用ensureCapacity(int minimumcapacity)方法在StringBuffer对象创建之后设置它的容量。首先我们看看StringBuffer的缺省行为，然后再找出一条更好的提升性能的途径。StringBuffer在内部维护一个字符数组，当你使用缺省的构造函数来创建StringBuffer对象的时候，因为没有设置初始化字符长度，StringBuffer的容量被初始化为16个字符，也就是说缺省容量就是16个字符。当StringBuffer达到最大容量的时候，它会将自身容量增加到当前的2倍再加2，也就是（2*旧值+2）。如果你使用缺省值，初始化之后接着往里面追加字符，在你追加到第16个字符的时候它会将容量增加到34（2*16+2），当追加到34个字符的时候就会将容量增加到70（2*34+2）。无论何事只要StringBuffer到达它的最大容量它就不得不创建一个新的字符数组然后重新将旧字符和新字符都拷贝一遍――这也太昂贵了点。所以总是给StringBuffer设置一个合理的初始化容量值是错不了的，这样会带来立竿见影的性能增益。StringBuffer初始化过程的调整的作用由此可见一斑。所以，使用一个合适的容量值来初始化StringBuffer永远都是一个最佳的建议。14、合理的使用Java类 java.util.Vector。简单地说，一个Vector就是一个java.lang.Object实例的数组。Vector与数组相似，它的元素可以通过整数形式的索引访问。但是，Vector类型的对象在创建之后，对象的大小能够根据元素的增加或者删除而扩展、缩小。请考虑下面这个向Vector加入元素的例子：Object obj = new Object();Vector v = new Vector(100000);for(int I=0;I&100000; I++) { v.add(0,obj); }　　除非有绝对充足的理由要求每次都把新元素插入到Vector的前面，否则上面的代码对性能不利。在默认构造函数中，Vector的初始存储能力是10个元素，如果新元素加入时存储能力不足，则以后存储能力每次加倍。Vector类就象StringBuffer类一样，每次扩展存储能力时，所有现有的元素都要复制到新的存储空间之中。下面的代码片段要比前面的例子快几个数量级：Object obj = new Object();Vector v = new Vector(100000);for(int I=0; I&100000; I++) { v.add(obj); }　　同样的规则也适用于Vector类的remove()方法。由于Vector中各个元素之间不能含有“空隙”，删除除最后一个元素之外的任意其他元素都导致被删除元素之后的元素向前移动。也就是说，从Vector删除最后一个元素要比删除第一个元素“开销”低好几倍。　　假设要从前面的Vector删除所有元素，我们可以使用这种代码：for(int I=0; I&100000; I++){　v.remove(0);}　　但是，与下面的代码相比，前面的代码要慢几个数量级：for(int I=0; I&100000; I++){　v.remove(v.size()-1);}　　从Vector类型的对象v删除所有元素的最好方法是：v.removeAllElements();　　假设Vector类型的对象v包含字符串“Hello”。考虑下面的代码，它要从这个Vector中删除“Hello”字符串：String s = &Hello&;int i = v.indexOf(s);if(I != -1) v.remove(s);　　这些代码看起来没什么错误，但它同样对性能不利。在这段代码中，indexOf()方法对v进行顺序搜索寻找字符串“Hello”，remove(s)方法也要进行同样的顺序搜索。改进之后的版本是：String s = &Hello&;int i = v.indexOf(s);if(I != -1) v.remove(i);　　这个版本中我们直接在remove()方法中给出待删除元素的精确索引位置，从而避免了第二次搜索。一个更好的版本是：String s = &Hello&; v.remove(s);　　最后，我们再来看一个有关Vector类的代码片段：for(int I=0; I++;I & v.length)　　如果v包含100,000个元素，这个代码片段将调用v.size()方法100,000次。虽然size方法是一个简单的方法，但它仍旧需要一次方法调用的开销，至少JVM需要为它配置以及清除堆栈环境。在这里，for循环内部的代码不会以任何方式修改Vector类型对象v的大小，因此上面的代码最好改写成下面这种形式：int size = v.size(); for(int I=0; I++;I&size)　　虽然这是一个简单的改动，但它仍旧赢得了性能。毕竟，每一个CPU周期都是宝贵的。15、当复制大量数据时，使用System.arraycopy()命令。16、代码重构：增强代码的可读性。
例如：public class ShopCart {
public void add (Object item) {
if(carts == null) {
carts = new ArrayList();}crts.add(item);}public void remove(Object item) {
if(carts. contains(item)) {
carts.remove(item);}}public List getCarts() {
//返回只读列表
return Collections.unmodifiableList(carts);}//不推荐这种方式//this.getCarts().add(item);
}17、不用new关键词创建类的实例用new关键词创建类的实例时，构造函数链中的所有构造函数都会被自动调用。但如果一个对象实现了Cloneable接口，我们可以调用它的clone()方法。clone()方法不会调用任何类构造函数。在使用设计模式（Design Pattern）的场合，如果用Factory模式创建对象，则改用clone()方法创建新的对象实例非常简单。例如，下面是Factory模式的一个典型实现：public static Credit getNewCredit() {
return new Credit();}改进后的代码使用clone()方法，如下所示：private static Credit BaseCredit = new Credit();public static Credit getNewCredit() {
return (Credit) BaseCredit.clone();}上面的思路对于数组处理同样很有用。18、乘法和除法考虑下面的代码：for (val = 0; val & 100000; val +=5) {alterX = val * 8; myResult = val * 2;}用移位操作替代乘法操作可以极大地提高性能。下面是修改后的代码：for (val = 0; val & 100000; val += 5) {alterX = val && 3; myResult = val && 1;}修改后的代码不再做乘以8的操作，而是改用等价的左移3位操作，每左移1位相当于乘以2。相应地，右移1位操作相当于除以2。值得一提的是，虽然移位操作速度快，但可能使代码比较难于理解，所以最好加上一些注释。19、在JSP页面中关闭无用的会话。
一个常见的误解是以为session在有客户端访问时就被创建，然而事实是直到某server端程序调用HttpServletRequest.getSession(true)这样的语句时才被创建，注意如果JSP没有显示的使用 &%@pagesession=&false&%& 关闭session，则JSP文件在编译成Servlet时将会自动加上这样一条语句HttpSession session = HttpServletRequest.getSession(true);这也是JSP中隐含的session对象的来历。由于session会消耗内存资源，因此，如果不打算使用session，应该在所有的JSP中关闭它。对于那些无需跟踪会话状态的页面，关闭自动创建的会话可以节省一些资源。使用如下page指令：&%@ page session=&false&%&20、JDBC与I/O如果应用程序需要访问一个规模很大的数据集，则应当考虑使用块提取方式。默认情况下，JDBC每次提取32行数据。举例来说，假设我们要遍历一个5000行的记录集，JDBC必须调用数据库157次才能提取到全部数据。如果把块大小改成512，则调用数据库的次数将减少到10次。[p][/p]21、Servlet与内存使用许多开发者随意地把大量信息保存到用户会话之中。一些时候，保存在会话中的对象没有及时地被垃圾回收机制回收。从性能上看，典型的症状是用户感到系统周期性地变慢，却又不能把原因归于任何一个具体的组件。如果监视JVM的堆空间，它的表现是内存占用不正常地大起大落。解决这类内存问题主要有二种办法。第一种办法是，在所有作用范围为会话的Bean中实现HttpSessionBindingListener接口。这样，只要实现valueUnbound()方法，就可以显式地释放Bean使用的资源。另外一种办法就是尽快地把会话作废。大多数应用服务器都有设置会话作废间隔时间的选项。另外，也可以用编程的方式调用会话的setMaxInactiveInterval()方法，该方法用来设定在作废会话之前，Servlet容器允许的客户请求的最大间隔时间，以秒计。22、使用缓冲标记一些应用服务器加入了面向JSP的缓冲标记功能。例如，BEA的WebLogic Server从6.0版本开始支持这个功能，Open Symphony工程也同样支持这个功能。JSP缓冲标记既能够缓冲页面片断，也能够缓冲整个页面。当JSP页面执行时，如果目标片断已经在缓冲之中，则生成该片断的代码就不用再执行。页面级缓冲捕获对指定URL的请求，并缓冲整个结果页面。对于购物篮、目录以及门户网站的主页来说，这个功能极其有用。对于这类应用，页面级缓冲能够保存页面执行的结果，供后继请求使用。23、选择合适的引用机制在典型的JSP应用系统中，页头、页脚部分往往被抽取出来，然后根据需要引入页头、页脚。当前，在JSP页面中引入外部资源的方法主要有两种：include指令，以及include动作。include指令：例如&%@ include file=&copyright.html& %&。该指令在编译时引入指定的资源。在编译之前，带有include指令的页面和指定的资源被合并成一个文件。被引用的外部资源在编译时就确定，比运行时才确定资源更高效。include动作：例如&jsp:include page=&copyright.jsp& /&。该动作引入指定页面执行后生成的结果。由于它在运行时完成，因此对输出结果的控制更加灵活。但时，只有当被引用的内容频繁地改变时，或者在对主页面的请求没有出现之前，被引用的页面无法确定时，使用include动作才合算。24、及时清除不再需要的会话为了清除不再活动的会话，许多应用服务器都有默认的会话超时时间，一般为30分钟。当应用服务器需要保存更多会话时，如果内存容量不足，操作系统会把部分内存数据转移到磁盘，应用服务器也可能根据“最近最频繁使用”（Most Recently Used）算法把部分不活跃的会话转储到磁盘，甚至可能抛出“内存不足”异常。在大规模系统中，串行化会话的代价是很昂贵的。当会话不再需要时，应当及时调用HttpSession.invalidate()方法清除会话。HttpSession.invalidate()方法通常可以在应用的退出页面调用。25、不要将数组声明为：public static final 。26、HashMap的遍历效率讨论经常遇到对HashMap中的key和value值对的遍历操作，有如下两种方法：Map&String, String[]& paraMap = new HashMap&String, String[]&();................//第一个循环Set&String& appFieldDefIds = paraMap.keySet();for (String appFieldDefId : appFieldDefIds) {String[] values = paraMap.get(appFieldDefId);......}//第二个循环for(Entry&String, String[]& entry : paraMap.entrySet()){String appFieldDefId = entry.getKey();String[] values = entry.getValue();.......}第一种实现明显的效率不如第二种实现。分析如下 Set&String& appFieldDefIds = paraMap.keySet(); 是先从HashMap中取得keySet代码如下：public Set&K& keySet() {Set&K& ks = keySreturn (ks != null ? ks : (keySet = new KeySet()));}private class KeySet extends AbstractSet&K& {public Iterator&K& iterator() {return newKeyIterator();}public int size() {}public boolean contains(Object o) {return containsKey(o);}public boolean remove(Object o) {return HashMap.this.removeEntryForKey(o) !=}public void clear() {HashMap.this.clear();}}其实就是返回一个私有类KeySet, 它是从AbstractSet继承而来，实现了Set接口。再来看看for/in循环的语法for(declaration : expression_r)statement在执行阶段被翻译成如下各式for(Iterator&E& #i = (expression_r).iterator(); #i.hashNext();){declaration = #i.next();statement}因此在第一个for语句for (String appFieldDefId : appFieldDefIds) 中调用了HashMap.keySet().iterator() 而这个方法调用了newKeyIterator()Iterator&K& newKeyIterator() {return new KeyIterator();}private class KeyIterator extends HashIterator&K& {public K next() {return nextEntry().getKey();}}所以在for中还是调用了在第二个循环for(Entry&String, String[]& entry : paraMap.entrySet())中使用的Iterator是如下的一个内部类private class EntryIterator extends HashIterator&Map.Entry&K,V&& {public Map.Entry&K,V& next() {return nextEntry();}}此时第一个循环得到key，第二个循环得到HashMap的Entry效率就是从循环里面体现出来的第二个循环此致可以直接取key和value值而第一个循环还是得再利用HashMap的get(Object key)来取value值现在看看HashMap的get(Object key)方法public V get(Object key) {Object k = maskNull(key);int hash = hash(k);int i = indexFor(hash, table.length); //Entry[] tableEntry&K,V& e =while (true) {if (e == null)if (e.hash == hash && eq(k, e.key))return e.
e = e.}}其实就是再次利用Hash值取出相应的Entry做比较得到结果，所以使用第一中循环相当于两次进入HashMap的Entry中而第二个循环取得Entry的值之后直接取key和value，效率比第一个循环高。其实按照Map的概念来看也应该是用第二个循环好一点，它本来就是key和value的值对，将key和value分开操作在这里不是个好选择。
为您推荐：
其他类似问题
您可能关注的内容
勃兰兑斯的相关知识
换一换
回答问题，赢新手礼包
个人、企业类
违法有害信息,请在下方选择后提交
色情、暴力
我们会通过消息、邮箱等方式尽快将举报结果通知您。}