由一段代码发表一点想法.doc

由一段代码发表一点想法今天在公司发现了一段很怪异的代码，为此还跟公司员工争执了一下，但由于自身是新员工，我只有无奈的屈服了，心里确实不爽，在这里发表一下自己的看法。先看一下代码，代码已经我已经简化了，只有两个类，一个是action层，另一个是Service层，具体如下： Action： Java代码 1. package com.yf.test; 2.3. import java.util.List; 4.5. public class Action 6. public void excute() 7. List list=null; 8. list=Service.doSomething(); 9. if(list.isEmpty() 10. /* 11. *此处省略其他操作 12. */ 13. 14. 15. Service Java代码 1. package com.yf.test; 2.3. import java.util.ArrayList; 4. import java.util.List; 5.6. public class Service 7. public static List doSomething() 8. List list = new ArrayList(); 9. if(1=2) 10. /* 11. * 此处省略对list的其他操作 12. * list=. 13. */ 14. 15. return list; 16. 17. 当我看到if(list.isEmpty()时，发现潜在产生空指针异常的可能，然后就给同事说了一下，这里应该先判断List是否为null，然后他让我看Service层，说Service不会返回null，我看了一下，确实不会返回为null，但是关于这段代码，我觉得写的实在太烂，为什么烂，主要有一下几个原因： 1、在Service层，每次调用doSomething方法时，都实例化一个List，虚拟机都会在堆中为这个list开辟内存，这无疑实在浪费内存和虚拟机的，而且这个list只有在if条件成立时，才需要，如果if不成立，虚拟机还得在方法调用结束后，回收这块内存，这难道不是没事找事吗？ 2、在Action层，action不对返回的list做非null判断，这也是一种很恶心的做法，首先，根据面向对象的封装性，Service层中的实现对Action而言，应该是不可见的，Action层应该对其返回值的可能情况做判断，即list!=null必须在Action做，如果后续Service层单独抽出，以API提供Jar包的形式，即我们无法知道里面的具体细节，这时，Action层还得做非空判断。因此，本人觉得，这段代码应该做如下重构 Action： Java代码 1. package com.yf.test; 2.3. import java.util.List; 4.5. public class Action 6. public void excute() 7. List list=null; 8. list=Service.doSomething(); 9. if(list!=null&list.isEmpty() 10. /* 11. *此处省略其他操作 12. */ 13. 14. 15. Service Java代码 1. package com.yf.test; 2.3. import java.util.ArrayList; 4. import java.util.List; 5.6. public class Service 7. public static List doSomething() 8. List list=null; 9. if(1=2) 10. list= new ArrayList(); 11. /* 12. * 此处省略对list的其他操作 13. * list=. 14. */ 15. 16. return list; 17. 18. 或许并不是每个人都认同我这种做法，不过我个人觉得这样比较合理，软件设计的时候要讲究层次，各层应该