当前位置: 代码迷 >> J2SE >> 在步骤中返回集合类型时,是直接返回一个arr好,还是返回一个new ArrayList(arr)好
  详细解决方案

在步骤中返回集合类型时,是直接返回一个arr好,还是返回一个new ArrayList(arr)好

热度:519   发布时间:2016-04-24 00:45:40.0
在方法中返回集合类型时,是直接返回一个arr好,还是返回一个new ArrayList(arr)好?
有这样两段代码:

Java code
第一类private Map<Integer, ArrayList<Question>> questions = new HashMap<Integer,ArrayList<Question>>();public ArrayList<Question> getQuestions(int level) {  return new ArrayList<Question>(questions.get(level));}第二类private Map<Integer, ArrayList<Question>> questions = new HashMap<Integer,ArrayList<Question>>();public ArrayList<Question> getQuestions(int level) {  return questions.get(level);}


请问是像第一类那样,返回时,通过原来的集合,new一个新的ArrayList好呢?
还是像第二类那样,直接返回原来的集合好呢?

貌似比较多的情况是用第一种,请问这么做相比第二种,好处是什么呢?



------解决方案--------------------
第一种肯定是占用内存,重新开辟一块内存,如何你对内存问题不在乎,要么可以使用吧

第二种直接返回,方便。前途是定义好了类型



鄙人愚见,欢迎来喷。。
------解决方案--------------------
无定论,关系到你模块的封装粒度要求,计算依赖要求,性能要求,内存开销要求。

最终权衡而定。

一般来说,如果对封装要求不是非常高(必须使用标准的某种VO结构之类的),其被暴露出去的对象结构并不存在后续的计算依赖(也就是说传递出去后即便被随意修改,也不会对这个类后续的计算造成任何影响),那么用第二种更合适。反之则第一种。

另一种判断方式可以稍微宏观点:该接口如果是在一个大模块内部使用的,用第二种;如果该接口是为其它模块提供服务的,用第一种。
------解决方案--------------------
我个人喜欢第二种,能直接返回一个ArrayList,非要在外面包装一个 没有意义啊。
------解决方案--------------------
这有点像 IO这快的Buffered使用与否

new Bufferedxxx(new xxxx())
------解决方案--------------------
包装的话是重新生成了一个list,跟原来的的map中存放的已经没有关系了
三楼说的很明显了
跟你的代码的业务有关联,
如果后续代码对这个list只有读操作,那么第二种方案好
如果后续代码对这个list进行了增删改操作,那么就要根据业务逻辑来决定了
------解决方案--------------------

如果方法是private,就直接返回,因为你对代码有完全的控制权

如果方法是public,就封一层再返回,因为你不知道别人会如何使用你的返回值


------解决方案--------------------
1.是否允许其它模块修改你的数据
2.是否通过这个返回值保持其它模块与你的数据同步

如果1,2都不满足,那么返回一个新的List.......对应第一类
如果1,2都满足,那么直接返回List............对应第二类
如果需要数据同步,又不希望其它模块修改........对应第三类

Java code
private Map<Integer, ArrayList<Question>> questions = new HashMap<Integer,ArrayList<Question>>();public List<Question> getQuestions(int level) {    return Collections.unmodifiableList(questions.get(level));}
  相关解决方案