资讯频道 → 编程语言
阅读更多

23顶
29踩

编程语言

原创新闻 四个有害的java习惯

2008-07-29 10:11 by 见习记者 jolestar 评论(76) 有38825人浏览
Java DAO

声明：ITeye资讯文章的版权属于ITeye网站所有，严禁任何网站转载本文，否则必将追究法律责任！

John O'Hanley 的这篇文章列举了四个有害的java编码习惯，并提出了改正方案。

这四个习惯普遍存在于java程序员中，并且已经被广泛接受，大家都习以为常，流毒甚远。

 

1.对局部变量(local variables)，字段(fields)，参数(method arguments)这三种变量的命名没有区分，造成了代码的阅读困难，增加了代码的维护成本。

 

作者举了个例子:

public boolean equals (Object arg) {
  if (! (arg instanceof Range)) return false;
  Range other = (Range) arg;
  return start.equals(other.start) && end.equals(other.end);
}

 

在这个方法中，arg直接用argument的缩写，虽然大家一看就知道这是参数了，但这种命名方式却丢失了参数代表的对象本身的含义。大家知道这是参数，却不知道这是什么参数。如果方法的参数多一点，都按照arg1，arg2这样的方式命名，阅读代码的时候很头疼。另外两个字段变量，start和end，突然凭空而出，想一下才知道这应该是字段。当然，这个方法很短，造成的困难还不大，如果这个方法比较长的话，突然看到start和end两个变量，一般会先在前面找一下是不是局部变量，然后才能确定是类的字段变量。

 

这个问题貌似微不足道，但为什么要让代码阅读者花费额外时间在这些琐碎的问题上呢？如果有个方案能让代码阅读者一目了然的明白变量是那种变量，为什么不采用呢？就如同Steve McConnell在 《代码大全》中说的:"让人费神去琢磨神秘杀人凶手这没有问题，但你不需要琢磨程序代码，代码是用来阅读的。"


作者提出了解决方案:

  • 方法参数用前缀a开始
  • 类字段变量用前缀f开始
  • 局部变量不用前缀

修正后的代码样式应该是:

public boolean equals (Object aOther) {
  if (! (aOther instanceof Range)) return false;
  Range other = (Range) aOther;
  return fStart.equals(other.fStart) && fEnd.equals(other.fEnd);
}

 

这样的代码看起来一目了然，如果你没有一目了然，说明还是习惯问题，习惯养成了就好了。

 

不过作者的方案里，给类字段变量前面加 f 前缀，如果用代码生成工具生成get，set方法是会比较麻烦。get，set方法中我们不希望出现个f。不过这个问题可以用修改代码生成工具的方式解决。如果这个习惯普遍被java界接受，这些应该都不成问题了。

 

作者引用了一句名言:

 

"By relieving the brain of all unnecessary work, a good notation sets it free to concentrate on more advanced problems, and in effect increases the mental power of the race. Before the introduction of the Arabic notation, multiplication was difficult, and the division even of integers called into play the highest mathematical faculties. Probably nothing in the modern world would have more astonished a Greek mathematician than to learn that ... a large proportion of the population of Western Europe could perform the operation of division for the largest numbers. This fact would have seemed to him a sheer impossibility ... Our modern power of easy reckoning with decimal fractions is the almost miraculous result of the gradual discovery of a perfect notation."
-- Alfred North Whitehead, An Introduction To Mathematics

 

个人理解大意为:

把大脑从不必需的工作中解放出来，一个好的记号法可以让大脑去关注更高级的问题，提升民族的整体智力。在阿拉伯数字记号法被介绍进来以前，乘法是非常困难的，计算整除也需要非常高的数学才能。一个古希腊的数学家可能非常诧异于西欧的大多数人口可以计算很大数字的除法。这个事实对他来说似乎是不可能的。我们现代人很容易用小数算帐的能力几乎可以说是由逐渐探索出的一套完美的计数法而造成的奇迹。

 

2.包(package)的命名和划分按照行为和层次划分(package-by-layer)而不是根据特征和功能划分(package-by-feature)

 

这个问题在我刚学java的时候就遇到了，在看了众多的网上开源程序后，我也慢慢习惯了按层次命名包。

作者举了个例子:

 
  • com.blah.action
  • com.blah.dao
  • com.blah.model
  • com.blah.util

我们已经习惯了按照层次分类或者叫按照行为分类，model一个包，dao一个包，service一个包，action一个包。这样就把具有同样特征或者功能的类划分到了不同的包里。这样的习惯，把java的包内私有(package-private)这个作用域给完全扔掉了，而包内私有是java的默认作用域。（ps:我学java来好像很少用过java的包内私有这个作用域，汗一个）

这中包的划分习惯也违反了面向对象编程的核心原则之--尽量保持私有以减少影响,因为这种习惯强迫你必须扩大类的作用域.

 

下面的包命名方式是按照特征划分命名:

  • com.blah.painting
  • com.blah.buyer
  • com.blah.seller
  • com.blah.auction
  • com.blah.webmaster
  • com.blah.useraccess
  • com.blah.util

举个例子，在一个web应用中，com.blah.painting包可能包含下面的成员:

  • Painting.java: model对象
  • PaintingDAO.java: dao对对象
  • PaintingAction.java:controller or action 对象
  • statements.sql: SQL文件
  • view.jsp: JSP文件

 

值得注意的是这种情况下，包里包含的不仅仅是java源码文件，同时也包含其他与该特征相关的文件。这点上好像违反大多数java程序员的习惯，并且如果要打包为jar好像也不方便，真实环境中如何应用，有没有别的麻烦，还要待实践一下。

 

作者列举了这种包划分方式的优点:

  • 包是高内聚的，并且模块化，包与包之间的耦合性被降到最低。

  • 代码的自文档性(或自描述性 self-documenting)增强. 读者只需看包的名字就对程序有些什么功能或特征有了大概的印象。在《代码大全》中, Steve McConnell 将自文档化(self-documenting)的代码比作 "the Holy Grail of legibility."(不知道怎么翻译)

  • 把类按照每个特征和功能区分开可以很容易实现分层设计。

  • 相关的成员在同一个位置。不需要为了编辑一个相关的成员而去浏览整个源码树。

  • 成员的作用域默认是包内私有。只有当另外的包需要访问某个成员的时候，才把它修改为public. (需要注意的是修改一个类为public，并不意味着它的所有类成员都应该改为public。public成员和包内私有(package-private)成员是可以在同一个类里共存的。)

  • 删除一个功能或特征只需要简单的删除一个文件夹。
  • 每个包内一般只有很少的成员，这样包可以很自然的按照进化式发展。如果包慢慢变的太大，就可以再进行细分，把它重构为两个或者更多新的包，类似于物种进化。而按照层次划分的方式，就没办法进化式发展，重构也不容易。

作者引用了一句Effective Java中的名言:

"The single most important factor that distinguishes a well-designed module from a poorly designed one is the degree to which the module hides its internal data and other implementation details from other modules."
-- Joshua Bloch, Effective Java

 

 

3.习惯用JavaBeans而不是不可变对象

 

按照javabeans的说明书(JavaBeans specification )，javabeans是用来解决特殊领域的问题:在图形界面程序的设计中充当小部件。但现在通常用javabean来做数据库记录的映射。作者反对javabean的这种用法。他提出了一个问题:

假如你要从数据库记录集映射一行为对象，不考虑现有的持久化方案和框架，你会将这个对象设计成什么样子？跟javabean相似呢还是完全不一样？

 

作者倾向于设计一个完全不一样的，他列举他的设计的几个特点:

 

  • 它不包含一个无参数构造方法（这一特征是javabean必备的。）。作者认为一个数据库记录的对象如果不包含任何数据是没有意义的。一个数据库表的所有字段都是可选的情况有多少？

  • It would likely not have anything to say about events and listeners.(不太明白作者的意思)

  • 它不强迫你用可变的对象。

  • 它内部有一个数据验证机制。这样一个验证机制对大多数数据库应用非常重要。(记住对象的第一原则：一个对象应该同时封装数据和对数据的操作。在这种情况下，操作就是验证数据。)

  • 数据验证机制可以给最终用户(end user)报错。

 

作者不满于现有的数据库持久化框架滥用javabean，提出了几个观点，但没有具体的代码演示。据我的理解，作者的意思和前一段时间javaeye上争论的充血模型和贫血模型有点类似。

 

4.类成员的排序没有按照成员的作用域(scope) 的大小从大到小排列，而是喜欢把private放在前面。

 

作者举了一个常见的类样式：

public class OilWell implements EnergySource {
   private Long id;
   private String name;
   private String location;
   private Date discoveryDate;
   private Long totalReserves;
   private Long productionToDate;
   
   public Long getId() {
      return id;
   }
   public void setId(Long id) {
      this.id = id;
   }
   
  //..elided
}

 这种方式将private变量放在最前面。作者认为应该倒过来，把private变量的申明放在最后面。

 

因为人们认识一个事物的通常过程都是从一般到特殊，从抽象层次来说，是从高到底的认识过程。如果你倒过来的话，就不能从整体上把握事物，也不能抓住事物的本质，只能在一堆具体的片段事实中迷失。

 

整体的抽象允许你忽略细节。抽象的层次越高，你可以忽略越多的细节。读者阅读一个类时可以忽略的细节越多他会越高兴。脑袋里填充太多的细节是痛苦的，所以细节越少越好。因此，将private成员方在最后会显得更富有同情心，因为这样阻止了不必要的细节显露给读者。

 

原来C++程序的习惯也是把private成员放在最开始。然而，C++社区迅速的认识到这是一个有害的规范，这个规范现在已经被修正。(参看 a typical C++ style guide )

注意:public 接口应该放在class的最开始，其次是protected成员，最后是private成员。原因是:

  • 程序员应该更关心接口而不是具体实现。
  • 当程序员需要用一个类的时候，他们需要的是接口而不是实现。

把接口放在开始是非常有意义的。把实现部分，private 片段，放在开始是一个历史遗留问题。最后还是要反复强调一下，一个类的接口的重要性超过实现细节。

 

将私有成员放在类的开始是一个等待打破的怀习惯，它似乎是sun早期的编码规范造成的。

 

将代码按照javadoc的顺序编排是非常好的：首先是构造方法，然后是非私有方法，最后是私有字段和方法。这样读者阅读的时候很自然的从抽象层次的高向低运动。

 

作者的这个观点确实很有道理，以前也没仔细思考过。可以考虑给IDE做个重构插件，专门按照这种排序格式化代码。

 

 

本文不算严格意义上的翻译，算是该文章的摘要以及读后感。

 

文章原始地址:http://www.javaworld.com/javaworld/jw-07-2008/jw-07-harmful-idioms.html

 

作者介绍:

John O'Hanley 是 javapractices.com 的专栏作者， 同时也是 WEB4J framework  的创建者。他拥有10年编程经验，现在居住于Prince Edward Island, Canada.

 

参考资料:

23
顶
29
踩
评论 共 76 条 请登录后发表评论
16 楼 phpxer 2008-07-31 15:42
作者说的有道理。
15 楼 xiaoxizhen 2008-07-31 14:10
他是java程序员吗？
14 楼 andyao 2008-07-31 14:06
除了包命名有点意思, 其他的无稽之谈
13 楼 miaomiao0307 2008-07-31 13:49
包结构很是同意啊，有他的优点，但是private就不是很同意了
12 楼 sword721 2008-07-31 13:47
不敢苟同
11 楼 manysysy 2008-07-31 13:41
有点道理，但是......
10 楼 paranoid945 2008-07-31 13:16
我只看接口
接口是否清晰？使用是否方便？至于你怎么实现怎么瞎搞胡搞都是你自己的事儿，我不管。

一个人也许人品很差，情商不高，长的也很吓人，穿着也很不靠谱，但是的的确确能做得出很好的程序，做得出非常优秀的项目，那作为软件公司的我就愿意雇这样的人工作。反过来我要雇程序员做项目，不一定要条条框框，英语又要好，为人又要谦虚，学习成绩又要好，表达能力特别强，然后，技术要好，有必要么？
9 楼 nwangwei 2008-07-31 13:05
第一点严重不同意，除非像rails那样，使用特殊字符只是表明作用域，不作为标识符的一部分。
第四点也是不太同意，主要是getter setter的field虽然是private的，但还是喜欢放在前面，因为更少去看getter和setter。
:D
8 楼 x_root 2008-07-31 13:04
我觉得说的挺有道理的，
7 楼 Relucent 2008-07-31 12:30
其实方法和 字段很容易区分的
6 楼 fkpwolf 2008-07-31 12:30
除四害；苍蝇。麻雀。蚊子，
5 楼 sphenx 2008-07-31 12:28
太教条了. 基本上说的都没什么意义
4 楼 huangcs27 2008-07-31 12:08
public boolean equals (Object aOther) {  
  if (! (aOther instanceof Range)) return false;  
  Range other = (Range) aOther;  
  return fStart.equals(other.fStart) && fEnd.equals(other.fEnd);  
} 
这样的代码看起来一目了然，如果你没有一目了然，说明还是习惯问题，习惯养成了就好了

我看你也是习惯了你自己的做法吧?我相信其他人看你的代码不会跟你一样一目了然!!!!!!

作者提出了解决方案:

方法参数用前缀a开始
类字段变量用前缀f开始
局部变量不用前缀

你这样的做法是普遍使用的吗?编程语法应该还没有这样的规定吧?
3 楼 liusong1111 2008-07-31 11:38
1. 方法参数也是局部变量，没觉出有区分两者的必要 。成员变量强制要求带this.就够了吧。其实区分他们，最直接实际的是把IDE的配色弄显眼点。

2. package的划分，他提出可以按两种维度：技术分层或业务模块。无论选哪种方案都有利有弊，看具体情况而定吧。对于通用框架，只体现了设计者偏好，像django和rails对于MVC文件存放位置的策略。

3. java语言的约束

4. IDE formatter可能有帮助。可是，java attribute的表达，要声明一个private的field和两个public的方法（setter,getter)，我个人习惯读field声明，怎么办泥？只能说它恶心。

整体来说原作者对不住标题“四个有害的java习惯”
2 楼 darkjune 2008-07-31 11:24
有些思想还是可以借鉴一下
1 楼 xly_971223 2008-07-31 11:10
引用
2.包(package)的命名和划分按照行为和层次划分(package-by-layer)而不是根据特征和功能划分(package-by-feature)

这一点在打包的时候确实有点不方便
引用
3.习惯用JavaBeans而不是不可变对象

要把面向对象思想强加到entity上 不靠谱，作者犯了本本主义的错误

通篇开来作者像是一个光理论没有实践的家伙

发表评论

您还没有登录,请您登录后再发表评论

相关推荐

  • 利用反射实现类的动态加载

    利用反射实现类的动态加载

  • Java 动态代理

    1、最简单的动态代理例子 使用动态代理的基本要素：首先需要有一个写好的动态代理工具类，然后要有个接口和对应的实现类， 1、动态代理工具类 package proxy; import java.lang.reflect.InvocationHandler; import java.lang.reflect.Method; import java.lang.reflect.Proxy; /** 代理类：帮助我们做一个被代理的业务对象返回。 java.lang.reflect.Proxy：这是 J

  • java合理估算线程池大小——Dark Magic

    具体见并发编程网的文章http://ifeve.com/how-to-calculate-threadpool-size/， 原代码乱序了，这里做一个排序后分享 github地址： https://github.com/sunshanpeng/dark_magic

  • Java中的动态类加载详解

    今天我们将深入探讨Java中的动态类加载，这是一种在运行时加载类文件的重要机制。动态类加载通过反射和自定义类加载器机制，使得程序在运行时可以根据需要动态加载和使用类，提高了应用程序的灵活性和可维护性。Java中的动态类加载能够使程序在运行过程中动态地装载、链接和初始化类，从而增强程序的灵活性和可扩展性。除了Java标准库提供的类加载器外，我们还可以自定义类加载器来实现更灵活的动态类加载需求。在Java中，实现动态类加载主要依赖于Java的反射机制和类加载器（ClassLoader）机制。

  • java工厂模式加策略模式实现动态选择类例子

    1.工厂模式加策略模式实现动态选择类 /** * 退款实现接口 */ public interface RefundIdentifyService { CommonNotify toRefund(RefundModel refundModel); } //支付宝退款实现 @AllArgsConstructor @Service public class AliIdentifyService implements RefundIdentifyService { private

  • Spring动态选择实现类

    一、需求描述 在实际工作中，我们经常会遇到一个接口及多个实现类的情况，并且在不同的条件下会使用不同的实现类，那么如何动态选择某一个实现类呢。 二、代码实现 1.根据ApplicationContext的getBeansOfType和自定义注解实现 spring上下文工具类ApplicationContextUtil import org.springframework.beans.BeansException; import org.springframework.context.Appli

  • java 动态指定方法_spring如何动态指定具体实现类

    在写接口实现时，有时会有多个实现类。这篇文章介绍在调用时通过传入字符串来指定具体的实现类。一.接口与实现类：// 接口public interface ServiceInterface {public void method();}// 具体两个实现类@Service("aService")public class AServiceImpl implements ServiceInterface ...

  • java 动态创建类_Java实现动态创建类操作示例

    本文实例讲述了Java实现动态创建类操作。分享给大家供大家参考，具体如下：Java可以创建动态类，学习看到，此处作为笔记。代码如下：import java.io.IOException;import java.lang.reflect.Method;import java.net.URI;import java.util.ArrayList;import java.util.Arrays;impo...

  • Java动态加载类

    今天上课讲了Java中的一些反射知识。 随便练习了一下。 学生类： public class Student { public static int number; static { number = 0; } public Student() { } private String studentName; public String getStudentName() { return student

  • 《Java核心技术面试精讲》24讲学习总结

    第24讲心得 该讲介绍了哪些方法可以在运行时动态生成一个 Java 类。 我们可以从常见的 Java 类来源分析，通常的开发过程是，开发者编写 Java 代码，调用 javac 编译成 class 文件，然后通过类加载机制载入 JVM，就成为应用运行时可以使用的 Java 类了。从上面过程得到启发，其中一个直接的方式是从源码入手，可以利用 Java 程序生成一段源码，然后保存到文件等，下面就只需要解决编译问题了。有一种笨办法，直接用 ProcessBuilder 之类启动 javac 进程，并指定上面生

  • java动态类加载_实现Java动态类载入机制

    作 为 充 分 利 用Java 的 动 态 类 载 入 机 制 的 最 好 例 子， 带 有Java 扩 展 的Web 浏 览 器 根 据 请 求 从 网 络 或 本 地 文 件 系 统 中 动 态 加 载Java applet( 遵 循 一 定 规 则 的Java 小 应 用 程 序 类)， 然 后 在 本 地 系 统 中 执 行 它， 大 大 增 强 了 主 页 的 功 能。---- 其 实，...

  • Java动态调用类的方法案例

    一、场景 需要根据编号动态给类设置值，属性名为“f+费用编号”，例如f01表示编号为01的费用 public class MonitorDayFee implements Serializable { private static final long serialVersionUID = 1L; private Long dayFeeId;//主键 privat...

  • Java动态类加载

    Java动态类加载 文章首发自: https://www.le1a.com/posts/9d41d3f8/ 前言 前面学习了反序列化，正准备趁热打铁去学cc3了，但是发现cc3需要用到动态类加载，就先来学一下。 利用URLClassLoader加载远程class文件 首先了解下什么是ClassLoader? ClassLoader是一个"加载器"，它会让Java虚拟机知道如何加载这个类。默认的ClassLoader是根据类名来加载类的，这个类名必须是类的完整路径(跟反射有点类似)，例如java.lang.R

  • Spring中动态选择实现类

    在spring中当一个接口有多个实现类的时候，通过创建简单工厂类，根据传入的不同的参数获取不同的接口实现类。 public interface ExecuteService { ExecuteEnum getCode(); // 业务方法 void execute(); } @Service public class FirstExecuteServiceImpl imp...

  • Java反射之一——动态加载机制

    Java程序的运行过程的一个简单的示意图如下： 首先，ClassLoader将xxx.class文件加载到code segment中，然后找到main()函数开始执行，在程序运行过程中，会有更多的class被load到内存。需要注意的是需要一个类时再加载一个类，而不是事先就加载所有的类。这叫做动态加载机制。 public class Reflection { public st...

  • java动态加载类_Java：类的动态加载

    package com.biao.util;import java.io.File;import java.net.URL;import java.net.URLClassLoader;import com.biao.ActionInterface;/*** 使用URLClassLoader动态的加载Class类和jar包中的类. 这样，可以把这些类路径放在配置文件中，动态的加载.*/public...

  • Java中实现动态加载类？

    一、案例引入： 假如某项目组要开发一款Office平台， 里面有Word模块和Excel模块， 你负责的部分是启动程序，即： 当键盘输入"Word"时，启动Word 当键盘输入"Excel"时，启动Excel 你写完代码，但是并不能运行， 因为， 你并没有拿到其他人所写的Word类和Excel类 ...

  • java 动态加载类

    一、什么是动态加载类动态加载：通过Class.forName的方式来得到一个Class类型的实例，然后通过这个Class类型的实例的newInstance来初始化，这种方法我们称之为动态加载程序在运行时调用相应方法，即使其他方法是错误的，程序依旧会执行。通过动态加载可以让程序的可延长性大大提升，对以后的维护和扩展有重要意义。Class c = Class.forName("com.mianshi.t

  • java 通过代码实现动态选择数据源

    类注解 import java.lang.annotation.Documented; import java.lang.annotation.ElementType; import java.lang.annotation.Inherited; import java.lang.annotation.Retention; import java.lang.annotation.Retentio

Global site tag (gtag.js) - Google Analytics