Java 开发中如何正确踩坑

为什么说一个好的员工能顶 100 个普通员工

我们的做法是，要用最好的人。我一直都认为研发本身是很有创造性的，如果人不放松，或不够聪明，都很难做得好。你要找到最好的人，一个好的工程师不是顶10个，是顶100个。所以，在核心工程师上面，大家一定要不惜血本去找，千万不要想偷懒只用培养大学生的方法去做。最好的人本身有很强的驱动力，你只要把他放到他喜欢的事情上，让他自己有玩的心态，他才能真正做出一些事情，打动他自己，才能打动别人。所以你今天看到我们很多的工程师，他自己在边玩边创新。

所以，找最好的人，要给他做他喜欢和擅长的事情。研发人员千万不要去管太严，一管就“死”了。工程师很讨厌跟规章制度打交道，作汇报他都很烦，大家不要管他，让用户去管他。他做好了一个产品，用户表扬他，这个大神多牛逼。他做不好了，用户骂他，他自己赶紧去改。

再谈阿里巴巴 Java 开发手册

之前在这个手册刚发布的时候看过一遍，当时感觉真是每个开发者都应该必读的一本手册，期间还写过一篇关于日志规约的文章：《下一个项目为什么要用 SLF4J》，最近由于在总结一些我们日常开发中容易忽略的问题，可能是最低级的编码常见问题，往往这也是最最容易忽略的，所以，又重新看了一遍这个手册，好像最近它也更新到了 1.2 版本。

这个手册目的就是让我们尽可能少踩坑，杜绝踩重复的坑。我接下来就打算试着写一些“坑”出来，来看看我们如何一不留神踩坑的，以及如何用正确的姿势跳出坑。

随随便便写出 NPE

首先声明一个 User 对象，接下来所有代码可能都会用到这个对象做演示，在下面将不在赘述。很简单，不上代码，上图片：

1.自动解箱抛 NPE

代码只有一行，再简单不过了：int method() { return new User().getId(); }

踩坑姿势：包装类型为 null 时，进行自动转换为基本数据类型报错。 解决方案：返回之前进行判断与处理或者改为相同类型。

2.级联调用易产生 NPE

这段代码有点容易迷惑人，因为它进行了集合元素的 isEmpty 判断，按说不会出问题了吧。看代码：

static void method1() {
    List<User> list = new ArrayList<User>();
    list.add(new User());
    if (!CollectionUtils.isEmpty(list)) {
        for (User user : list) {
            System.out.println("userid:" + user.getId().toString());
        }
    }
}

不废话，看运行结果：

没错，还是报错了。

踩坑姿势：其实就是尽管你在之前做了对象不为空的判断，但你并不能保证对象中的值不为空，而且这时候去级联调用就会抛 NPE 。

手册中关于 NPE 的描述：

防止 NPE 是调用者的责任。即使被调用方法返回空集合或者空对象，对调用者来说，也并非高枕无忧，必须考虑到远程调用失败、序列化失败、运行时异常等场景返回 null 的情况。 集合里的元素即使 isEmpty，取出的数据元素也可能为 null。 级联调用 obj.getA().getB().getC()；一连串调用，易产生 NPE

3.关于 Equals

这是日常开发中用于相等比较使用最多的方法了吧，因为当年谁没被 == 坑过阿。现在一般我们都会这么写：user.getName().equals("mafly");

踩坑姿势： 一不小心使用了 null 值调用了 Equals 方法。 解决方案： 很简单咯，这么写："mafly".equals(user.getName());

equals 方法容易抛空指针异常，应使用常量或确定有值的对象来调用 equals。

4.Map 下的 NPE

Map 应该是我们开发中使用最频繁的了，最常用的可能有 HashMap、ConcurrentHashMap 这俩了，可能会一不留神写出这样的代码：

踩坑姿势： 可能我们知道 ConcurrentHashMap 的 K/V 都不能为空，但我们有时候并不知道传进来的值是否为空。 解决方案： 设置时做下检验，对它的特性正确理解及使用。

由于 HashMap 的干扰，很多人认为 ConcurrentHashMap 是可以置入 null 值，而事实上，存储 null 值时会抛出 NPE 异常

Map 类集合 K/V 能不能存储 null 值的情况，如下表格：

简单聊聊常用的集合

5.foreach 遍历集合删除元素

大家应该都知道，在遍历集合时对元素进行 add/remove 操作要使用 Iterator，使用 for 循环时会报错，一定会报错吗？看代码：

public static void main(String[] args) {
    List<String> a = new ArrayList<>();
    a.add("1"




    
);
    a.add("2");
    a.add("3");
    for (String temp : a) {
        if ("2".equals(temp)) {
            a.remove(temp);
        }
    }
    Iterator<String> it = a.iterator();
    while (it.hasNext()) {
        String temp = it.next();
        if ("2".equals(temp)) {
            it.remove();
        }
    }