`
michael8335
  • 浏览: 187610 次
  • 性别: Icon_minigender_1
  • 来自: 北京
社区版块
存档分类
最新评论

由一段代码发表一点想法

阅读更多
今天在公司发现了一段很怪异的代码,为此还跟公司员工争执了一下,但由于自身是新员工,我只有无奈的屈服了,心里确实不爽,在这里发表一下自己的看法。先看一下代码,代码已经我已经简化了,只有两个类,一个是action层,另一个是Service层,具体如下:
Action:
package com.yf.test;

import java.util.List;

public class Action {
	public void excute(){
		List list=null;
		list=Service.doSomething();
		if(list.isEmpty()){
			/*
			 *此处省略其他操作
			 */
		}
	}
}

Service
package com.yf.test;

import java.util.ArrayList;
import java.util.List;

public class Service {
	public static List doSomething() {
		List list = new ArrayList();
		if(1==2){
			/*
			 * 此处省略对list的其他操作
			 *  list=.....
			 */			
		}
		return list;
	}
}

当我看到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:
package com.yf.test;

import java.util.List;

public class Action {
	public void excute(){
		List list=null;
		list=Service.doSomething();
		if(list!=null&&list.isEmpty()){
			/*
			 *此处省略其他操作
			 */
		}
	}
}

Service
package com.yf.test;

import java.util.ArrayList;
import java.util.List;

public class Service {
	public static List doSomething() {
		List list=null;
		if(1==2){
			list= new ArrayList();
			/*
			 * 此处省略对list的其他操作
			 *  list=.....
			 */			
		}
		return list;
	}
}

或许并不是每个人都认同我这种做法,不过我个人觉得这样比较合理,软件设计的时候要讲究层次,各层应该干得事情,就应该在所在层做好,而不是有其下层来保证,这种强依赖下层保证是一种很恶心的做法,如果后续下层代码逻辑变更,还得去上层看看对其的影响,这就很无耻了!!!
21
13
分享到:
评论
58 楼 jayming 2012-10-17  
我同意楼主的观点,空指针很多时候并不是错误,我们不应该让程序在没有错误的情况下抛出空指针异常,参考hibernate返回list的方法,@return a result object returned by the action, or <code>null</code>
57 楼 黯然小伙 2012-10-16  
貌似很多讨论。我的想法是:
项目中最好不要返回null(经历过很多血的教训),尤其是对list和map。
我项目中的解决方法是:
public class Service {  
    public static List doSomething() {  
        if(xxx){  
            List list== new ArrayList();  
            /* 
             * 此处省略对list的其他操作 
             *  
             */   
             return list;
        }  
        return Collections.emptyList();  
    }  
}  

仅供参考,个人愚见。
56 楼 奔三的小生 2012-10-16  
返回一个空对象也不应该返回null,不要对内存太较真
55 楼 maimode 2012-10-16  
kidneyball 写道
谈谈返回null值的问题。在2009年,“null”的发明人就发表了一次演讲,说引入null这个概念是他犯下的一个巨大错误,几十年来造成了软件行业超过十亿美元的损失。(http://www.infoq.com/presentations/Null-References-The-Billion-Dollar-Mistake-Tony-Hoare)。因此我个人的倾向是,除非不得已,不要返回null。

以博主的场景为例,如果我在团队里规定,返回容器类型的方法不能返回null。那么对使用者来说,这个方法总共就三种合法情况:返回非空容器,返回空容器,抛出异常。如果发现返回值为null,那就说明是这个方法本身出错了,使用者可以直接反馈给编写者排错。

反之,如果团队里规定,“返回容器类型的方法里可能返回null”,那么对使用者来说,这个方法有以下几种可能情况:
1. 返回非空容器
2. 返回空容器
3. 抛出异常
4. 返回null,其意义相当于空容器
5. 返回null,其意义相当于抛异常
6. 返回null,有特殊的业务含义
7. 编写者没处理好,不小心返回了null(毕竟空指针是最常见的运行期错误,没有之一)。

使用者拿到一个这样的方法,就必须要在4、5、6、7之间做出选择。如果编写者没有写Javadoc,必须要看具体实现才能确认4和5,要加上一定推理才能确认6。无论选4,5,6,都无法排除可能是7。

说的很透彻!其实楼主和他的同事只是奉行的风格不同罢了,没有说那种风格一定好,但是能确定的是,团队保持一致的风格非常重要。
54 楼 greatghoul 2012-10-16  
kidneyball 写道
1. 容器类型的底层方法应该尽量返回空集,避免返回null。从而避免顶层方法做大量的null判断。具体可参考《Effective Java》。

2. 一个以容器为返回类型的方法,返回空集通常是不常见的异常分支,为一个不常见的分支考虑类创建和垃圾回收层面上的性能问题而导致调用者做大量的额外NP判断并不值得。这类优化应该在有profiling数据支持的情况下再做。

3. “具体细节不可见”的问题,写点Javadoc就能解决。调用者需要参考接口文档知道返回值的具体情况,是很正常的事情。后续维护时要保证接口协议不变,也是很正常的事情。

4. 某些团队都会形成共识,只要是返回容器类型的方法就不返回null。只要大家都遵守这个协议,不会出问题而且可以免去大量无谓的NP判断。博主作为一个新人,应该先了解团队中的开发规范,而不是和现有规则较劲。


对,很多流行框架也是这么做的。
53 楼 skzr.org 2012-10-16  
教人家编程会让其痛苦一辈子
52 楼 jinnianshilongnian 2012-10-16  
mfkvfn 写道
jinnianshilongnian 写道
1、在Service层,每次调用doSomething方法时,都实例化一个List,虚拟机都会在堆中为这个list开辟内存,这无疑实在浪费内存和虚拟机的,而且这个list只有在if条件成立时,才需要,如果if不成立,虚拟机还得在方法调用结束后,回收这块内存,这难道不是没事找事吗??

如果服务层 调用 持久层框架(如hibernate) 即使是空 也是返回一个集合的;
                 自己写业务  如果为null  可以返回Collections.EMPTY_LIST(单例 且不可变的) 
建议:不要过早优化 先保证代码可读和可维护


“为空时返回不一个不可变的Collections.EMPTY_LIST单例”这个是不可以的。
一个方法有可能会返回可变的ArrayList有可能会返回不可变的单例,这是不实现的。

比如
    private static final List<String> XXX = new ArrayList<String>(0);

    private List<String> getList(int a) {
        if (a == 1) {
            return new ArrayList<String>();
        } else {
            return XXX;
        }
    }

    private void b() {
        List<String> b = getList(1);
        b.add("abc");
    }

在编译时不会出错。运行时可能会出错。


这些要看自己怎么处理了,基本没有定式;
如查询 可以返回Collections.EMPTY_LIST
如返回的数据需要修改  建议再做一个List保存处理结果 而不是在之前的List中修改
看自己编码习惯了
51 楼 mfkvfn 2012-10-16  
jinnianshilongnian 写道
1、在Service层,每次调用doSomething方法时,都实例化一个List,虚拟机都会在堆中为这个list开辟内存,这无疑实在浪费内存和虚拟机的,而且这个list只有在if条件成立时,才需要,如果if不成立,虚拟机还得在方法调用结束后,回收这块内存,这难道不是没事找事吗??

如果服务层 调用 持久层框架(如hibernate) 即使是空 也是返回一个集合的;
                 自己写业务  如果为null  可以返回Collections.EMPTY_LIST(单例 且不可变的) 
建议:不要过早优化 先保证代码可读和可维护


“为空时返回不一个不可变的Collections.EMPTY_LIST单例”这个是不可以的。
一个方法有可能会返回可变的ArrayList有可能会返回不可变的单例,这是不实现的。

比如
    private static final List<String> XXX = new ArrayList<String>(0);

    private List<String> getList(int a) {
        if (a == 1) {
            return new ArrayList<String>();
        } else {
            return XXX;
        }
    }

    private void b() {
        List<String> b = getList(1);
        b.add("abc");
    }

在编译时不会出错。运行时可能会出错。
50 楼 lianglaiyang 2012-10-16  
michael8335 写道
finallygo 写道
第一点我支持博主的,因为本来定义的接口就是可能多种实现的,虽然有的人说什么团队规范,但是一个团队规范不能保证所有人都认真的执行,其次还增加了一种潜规则在里面,所以,我采用不信任编程,另外第二个,我觉得还是尽量不要返回null

谢谢了,其实我写这个只是表达一下我的想法,同事想看看大家的看法,不过根据大家的评论来看,有两点是明显的,1、上层要做非空判断;2、下层不应该返回null。感谢大家的评论,从中学习不少。

从人与人相处来说,作为一个新人,楼主应该先向你同事公开道歉,以后遇到意见不合时,就要虚心一点。这个是最重要的,不然如果不汲取这次教训,很难混下去
49 楼 weng 2012-10-16  
yawei 写道
你说的是理论上的东西, 而代码呈现的是实际开发中的最好实现。 server层的实现很合理, list先建立就保证了无论条件如何, 总会返回一个list。

action不检查null是因为,在这种模式下, null意味着重大错误, NPE是可以接受的结果。

这种设计逻辑很多地方都能见到,例如搜索引擎。

胡扯
48 楼 alfusen 2012-10-16  
学习了,学习了!
47 楼 在世界的中心呼喚愛 2012-10-15  
ls几位都说的很好。
我赞成你同事的写法,虽然多开销的一个list,但是action可读性变强了。何况你list=null的情况,返回一个null的list并不好,这样造成action都要去判断。
初期写代码尽量可读性好些,这样别人容易看懂,也容易给你一些意见。效率问题可以在后期慢慢调。
46 楼 michael8335 2012-10-15  
finallygo 写道
第一点我支持博主的,因为本来定义的接口就是可能多种实现的,虽然有的人说什么团队规范,但是一个团队规范不能保证所有人都认真的执行,其次还增加了一种潜规则在里面,所以,我采用不信任编程,另外第二个,我觉得还是尽量不要返回null

谢谢了,其实我写这个只是表达一下我的想法,同事想看看大家的看法,不过根据大家的评论来看,有两点是明显的,1、上层要做非空判断;2、下层不应该返回null。感谢大家的评论,从中学习不少。
45 楼 finallygo 2012-10-15  
第一点我支持博主的,因为本来定义的接口就是可能多种实现的,虽然有的人说什么团队规范,但是一个团队规范不能保证所有人都认真的执行,其次还增加了一种潜规则在里面,所以,我采用不信任编程,另外第二个,我觉得还是尽量不要返回null
44 楼 tobylxy 2012-10-15  
收获不少,谢lz和各位讨论!
43 楼 lycccxzt 2012-10-15  
虽不能提什么意见,但在各位的讨论中获得不少收获。
42 楼 formice 2012-10-15  
支持返回非null的集合,这里的问题关注点不在浪费那点内存上,而应该是程序的健壮性,不要太纠结理论,实际的应用应该灵活,不要把自己局限在条条框框上
41 楼 skzr.org 2012-10-15  
这个要顶-|-==============>
kidneyball 写道
谈谈返回null值的问题。在2009年,“null”的发明人就发表了一次演讲,说引入null这个概念是他犯下的一个巨大错误,几十年来造成了软件行业超过十亿美元的损失。(http://www.infoq.com/presentations/Null-References-The-Billion-Dollar-Mistake-Tony-Hoare)。因此我个人的倾向是,除非不得已,不要返回null。

以博主的场景为例,如果我在团队里规定,返回容器类型的方法不能返回null。那么对使用者来说,这个方法总共就三种合法情况:返回非空容器,返回空容器,抛出异常。如果发现返回值为null,那就说明是这个方法本身出错了,使用者可以直接反馈给编写者排错。

反之,如果团队里规定,“返回容器类型的方法里可能返回null”,那么对使用者来说,这个方法有以下几种可能情况:
1. 返回非空容器
2. 返回空容器
3. 抛出异常
4. 返回null,其意义相当于空容器
5. 返回null,其意义相当于抛异常
6. 返回null,有特殊的业务含义
7. 编写者没处理好,不小心返回了null(毕竟空指针是最常见的运行期错误,没有之一)。

使用者拿到一个这样的方法,就必须要在4、5、6、7之间做出选择。如果编写者没有写Javadoc,必须要看具体实现才能确认4和5,要加上一定推理才能确认6。无论选4,5,6,都无法排除可能是7。

40 楼 kidneyball 2012-10-15  
谈谈返回null值的问题。在2009年,“null”的发明人就发表了一次演讲,说引入null这个概念是他犯下的一个巨大错误,几十年来造成了软件行业超过十亿美元的损失。(http://www.infoq.com/presentations/Null-References-The-Billion-Dollar-Mistake-Tony-Hoare)。因此我个人的倾向是,除非不得已,不要返回null。

以博主的场景为例,如果我在团队里规定,返回容器类型的方法不能返回null。那么对使用者来说,这个方法总共就三种合法情况:返回非空容器,返回空容器,抛出异常。如果发现返回值为null,那就说明是这个方法本身出错了,使用者可以直接反馈给编写者排错。

反之,如果团队里规定,“返回容器类型的方法里可能返回null”,那么对使用者来说,这个方法有以下几种可能情况:
1. 返回非空容器
2. 返回空容器
3. 抛出异常
4. 返回null,其意义相当于空容器
5. 返回null,其意义相当于抛异常
6. 返回null,有特殊的业务含义
7. 编写者没处理好,不小心返回了null(毕竟空指针是最常见的运行期错误,没有之一)。

使用者拿到一个这样的方法,就必须要在4、5、6、7之间做出选择。如果编写者没有写Javadoc,必须要看具体实现才能确认4和5,要加上一定推理才能确认6。无论选4,5,6,都无法排除可能是7。
39 楼 lg_asus 2012-10-15  
原来我也是赞同楼主的做法,看样子我要看看effective java了。 thx

相关推荐

    软著申请:源代码文件模板,每页五十行,一共六十页

    本资源是一个基于Java语言的源代码文件模板,总共六十页,每页五十行,主要涉及到嵌入式校园网网络质量监测系统的开发。以下是相关知识点的总结: 1. Java语言基础:该源代码文件模板使用Java语言编写,涉及到Java...

    G代码解释程序

    G代码由一系列字母、数字和符号组成,每个代码或指令代表一个特定的动作,如移动刀具、设定速度、开始切割等。 C++ Builder是一款由Embarcadero Technologies开发的集成开发环境(IDE),它基于C++语言并提供了RAD...

    软著源代码整理工具

    标题中的“软著源代码整理工具”指的是一个专门用于整理软件著作权申请所需的源代码的工具。在IT行业中,软件著作权是保护开发者权益的重要法律手段,它要求申请者提交源代码以证明其原创性。源代码整理是这个过程中...

    软著代码整理工具,可以实现一键提取文件中的代码并且自动删去空行和注释,便于进行软著申请

    为了简化这一过程,"软著代码整理工具"应运而生,它能够一键提取代码、删除空行和注释,使代码更符合申请要求。 1. **一键提取文件中的代码**:此工具的核心功能之一是能够快速从各种类型的代码文件中提取出有效的...

    PACS代码查询表

    在描述中提到,很多科研工作者在发表文章时都需要查询PACS代码,这说明PACS代码在科研领域具有重要的应用价值。科研工作者在撰写文章时,根据研究的具体主题选择合适的PACS代码,可以方便读者根据代码快速检索到相关...

    android 经典代码例子

    在Android开发领域,经典代码例子是开发者学习和提升技能的重要资源。这些例子涵盖了各种关键功能和组件的实现,有助于深入理解Android应用的工作原理。在这个压缩包中,我们可能找到了多个有关Android编程的示例...

    editplus如何实现整段缩进

    EditPlus是一款功能强大的文本编辑器,尤其在编程领域广受欢迎,因为它提供了许多便捷的代码编辑功能,其中之一就是整段缩进。整段缩进在编程中扮演着至关重要的角色,它能够极大地提升代码的可读性和维护性。下面将...

    美食网站源代码

    【美食网站源代码】是一个专为展示美食相关资讯和功能的网站开发的源代码集合,是互联网上搜集并整理的资源,供开发者下载和使用。这个源代码可能包含了一个完整的美食主题网站的所有必要组成部分,例如前端用户界面...

    igh ethercat 台达电机控制代码

    本文将详细解析"igh ethercat 台达电机控制代码"中的核心概念和技术要点,以及如何利用这段代码来驱动符合DS402协议的台达电机。 EtherCAT 主站是EtherCAT网络中的核心部分,它负责协调整个网络的操作,向从站设备...

    编译原理中间代码生成实验报告——完整版

    在编译原理中,中间代码生成是编译过程的关键步骤之一,它位于词法分析、语法分析之后,目标代码生成之前。本实验报告主要围绕算术表达式的中间代码生成展开,旨在让学生深入理解算术表达式的语法分析原理,并能实现...

    恶意代码分析实战

    本书是一本内容全面的恶意代码分析技术指南,其内容兼顾理论,重在实践,从不同方面为读者讲解恶意代码分析的实用技术方法。, 本书分为21章,覆盖恶意代码行为、恶意代码静态分析方法、恶意代码动态分析方法、恶意...

    微信小程序游戏源代码

    7. **社交分享与邀请机制**:微信小程序的一大优势是社交功能,"跳一跳"的源代码可能会包含如何实现分享到朋友圈、微信群,以及邀请好友等功能,这是增强用户粘性和游戏传播的关键。 8. **性能优化**:源代码还会...

    vc++游戏(源代码)

    在本资源包中,我们收集了一系列使用VC++编写的经典游戏源代码,这些源代码提供了深入理解C++编程语言以及游戏开发基础的宝贵机会。通过分析和学习这些源码,开发者可以了解到如何利用C++来实现游戏逻辑、图形界面、...

    电子签章系统源代码

    "电子签章系统源代码" 电子签章系统源代码是指实现电子签章功能的源代码,它可以和WORD、EXCEL等办公软件集成,实现电子签章的功能。 1. 标题:电子签章系统源代码 电子签章系统源代码是指实现电子签章功能的源...

    8 位 CPU vhdl实现(含全部源代码)

    你可以写出一段17位的指令代码,并放入ROM区,该CPU即可自动运行出结果。压缩包里是源代码和我们当时的设计要求。本源代码的最后调试时在地址0--17是放入的斐波纳契数字(Fibonacci Numbers)指令。通过modelsim仿真...

    Android第二行代码(郭霖全书源代码)

    《Android第二行代码》是郭霖撰写的一本深入浅出的Android编程入门书籍,它旨在帮助初学者快速理解和掌握Android应用开发。这本书的全书源代码提供了丰富的实践示例,帮助读者通过动手实践来巩固理论知识。以下是...

    我的世界简易版(C++编写,源代码+程序)

    该项目的核心是一段用C++编写的源代码,它能够让开发者深入理解2D游戏设计与编程。游戏的源代码MC.cpp文件,包含了游戏的核心逻辑,例如游戏循环、用户输入处理、游戏状态更新、渲染逻辑等。在C++中,游戏循环是游戏...

    完整的matlab遗传算法代码

    遗传算法是一种基于生物进化原理的优化方法,广泛应用于解决复杂问题的全局寻优。MATLAB作为一款强大的数学计算...通过学习和实践这段代码,不仅可以掌握遗传算法的基本原理,还能提升在MATLAB中解决实际问题的能力。

    代码注释率统计工具下载

    代码注释是编程实践中至关重要的一个环节,它有助于提高代码的可读性和可维护性。注释率是指源代码中的注释行数占总代码行数的比例,通常用来衡量代码的文档质量。`linecount`是一款用于统计代码注释率的工具,其...

    用C语言实现的画线代码(画任意曲线)

    4. **代码实现**: 一个简单的例子是创建一个窗口,然后在窗口的Paint消息处理程序中绘制线条。首先,创建一个消息循环,注册窗口类,然后在窗口回调函数中处理WM_PAINT消息,调用GDI函数来画线。 5. **优化与扩展...

Global site tag (gtag.js) - Google Analytics