Java :如何处理空指针？

[sandman511]

Integer code = getCode();
String codeMeaning = code == null ? null :
                    code == 1 ? "你好" :
                    code == 2 ? "你好好" :
                    code == 3 ? "你好好好";

请各大佬帮忙修改这段代码
code == null ? null 看起来有点怪怪的

[oaix]

@earther01 不直观。
https://dzone.com/articles/using-optional-correctly-is-not-optional 第 12 条

[wutiantong]

从 append 的代码来看，你应该把 Integer unit + String transferredUnit 这两样东西 wrap 成一个简单的 class，不妨就还叫 unit 好了，数字和字符串作为 unit 这个 class 的两个成员属性。

你的 getUnit(type) 应该直接返回这个 unit class。

[palmers]

public enum UnitOfWeight {
KG("kg") {
@Override
public boolean match(int unit) {
return false;
}
},
GRAM("g") {
@Override
public boolean match(int unit) {
return false;
}
},
TON("t") {
@Override
public boolean match(int unit) {
return false;
}
},
UNKNOWN("x-0"){
@Override
public boolean match(int unit) {
return false;
}

@Override
public boolean unknown() {
return true;
}
};


/**
* e.g. kg, g, t
*/
@Getter
private String note;

static UnitOfWeight[] units = UnitOfWeight.values();

/**
* kg's enums
*/
static int[] kgs = {1, -1, 4};
/**
* gram's enums
*/
static int[] grams = {2, 0, 5};
/**
* ton's enums
*/
static int[] tons = {3, 1, 6};

UnitOfWeight(String note) {
this.note = note;
}


public static UnitOfWeight valueOf(Integer unit) {
for (UnitOfWeight ofWeight : units) {
if (unit == null) {
return UNKNOWN;
}
if (ofWeight.match(unit)) {
return ofWeight;
}
}
return UNKNOWN;
}


/**
* 获取对应枚举
* @param unit
* @return
*/
public abstract boolean match(int unit);

/**
* 错误的值
* @return
*/
public boolean unknown() {
return false;
}


public static void main(String[] args) {
Integer unit = null;
//getUnit (type2);
UnitOfWeight ofWeight = UnitOfWeight.valueOf(unit);
if (!ofWeight.unknown()) {
String transferredUnit = ofWeight.getNote();
//...
}
}
}

根据你 append 的代码用了一个枚举来做， 根据你的描述我想有几个点需要扩展，
1. 从 pdf 解析 type 值；
2. 有很多情况（可能因为历史原因很多地方使用了不同的值代表某一个单位）；
所以我建议使用枚举， 原因是:
1. 我觉得后面随着业务需要或者其他原因重构， 可能慢慢的这些代表某一个单位的值会改变或者统一为一个，所以你需要修改 通过枚举可以统一在一处，便于修改；
2. 再者也方便返回的字符值变更 总之对于变更这个枚举类基本上可以应对 90%以上的情况;
3. match 方法可以做针对行的扩展和优化；

[boileryao]

```java
// Put this map somewhere
Map<Integer, String> codeMeaningInfo = new HashMap<>();
codeMeaningInfo.put(1, "你好");
codeMeaningInfo.put(2, "你好好");
codeMeaningInfo.put(3, "你好好好");

// Using this map
String codeMeaning = codeMeaningInfo.get(code);
```
一般情况下 Map 应该全局唯一，另外推荐 Kotlin 的 `when` 表达式。

[loryyang]

可以的话，尽量不要用 null，容易在各种传递运算中出现空指针异常。比如你这个 code 和 codeMeaning，code=-1，codeMeaning=“”表示异常。甚至可以用 Enum 来包装这些内容
如果硬是要用 null，我建议把 null 的处理和其他处理分开。尽量不要用三元表达式，这种东西比较反人类，写起来很爽，看起来一头包。
你这种情况可以写个函数 getCodeMeaning(Integer code) {
if (code == null) {
return null;
} else if (code == 1) {
******
}
代码在可读可维护性面前，简洁性可以要求低一些

[luozic]

如果 java 有 pattern match 就可以玩优雅；但是 Java 没有，所以，理智一点，用表驱动去糊弄算了。

[redford42]

前几天看到一个 java8 的 optional
但其实我没看太明白...

[loryyang]

@loryyang #45 另外还有一个避免魔法值，前面也好多人提了。对于这一点，我是觉得，你考虑一下是否有必要做。你这种 case，直接用一个 ImmutableMap 好了，这样可以变成：
ImmutableMap.<Integer, String>builder().put(1, "").put(2, "").build()
然后 getCodeMeaning 函数里面变成
if (code == null) {
return null;
}
if (code in immutableMap) {
return immutableMap.get(code);
}
else {
return ***;
}

回头看一下，还是 null 太多了，null 这东西还是不要在正常流程中出现吧

[nekoneko]

@redford42 #47 感觉 optional 就是婊子立牌坊

[godoway]

@nekoneko
@redford42
这其实是语法上提醒你检测空安全，例如有个接口入参 optional，你就知道这里可能为空，入参一个对象有可能你就忘了检测非空

[godloveplay]

你写的代码是给别人看的。 请你站在别人的角度思考一下，看到这个代码的心情

[oneisall8955]

@nekoneko 深层嵌套获取值，一直判断非空很啰嗦，Optional 可以解决这个问题

[winiex]

永远不要嵌套多层三元表达式，三元表达式当且仅当只有一层逻辑判断时才可以使用。其它情况一律用 if else 老实地写。我甚至连一层逻辑判断都想强制只能用 if else。

[brucefu]

@micean 能让别人一秒就看懂的东西，就不要搞成两秒

[micean]

@brucefu

code == null ? null :
code == 1 ? "你好" :
code == 2 ? "你好好" :
code == 3 ? "你好好好" :
"其他";

这不就是一个简易秒懂的 switch，楼上其他代码也做不到这么工整

[micean]

另外嵌套的三元还可以轻轻松松给 final 修饰的赋值

final String name =

code == null ? "" :
code == 1 ? "你好" :
code == 2 ? "你好好" :
code == 3 ? "你好好好" :
"其他";

[secondwtq]

这不就是一个 Maybe Integer 解决的问题 ... 搞这么复杂