不要再寫這樣的神級代碼了！

JDK8提供的Stream雖然好用，Lambda雖然簡潔，但一定不能濫用，我舉一個實際遇到的例子（已做脫敏處理）：

Map<Long, List<Student>> studentMap = students.stream().collect(Collectors.groupingBy(Student::getStudentNumber)).entrySet().stream().sorted(Map.Entry.comparingByKey()).collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue, (oldValue, newValue) -> oldValue, LinkedHashMap::new));

試問誰能看得懂？難道是沒有換行格式化？

Map<Long, List<Student>> studentMap = students.stream().collect(Collectors.groupingBy(Student::getStudentNumber))      //這裡是要把students按studentNumber分組
                .entrySet().stream().sorted(Map.Entry.comparingByKey())     //分組後再把把key值拿來排序組成新的Stream流
                .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue,       //然後再組成Map？不好意思實在是讀不下去了
                        (oldValue, newValue) -> oldValue, LinkedHashMap::new));

換行格式化後，前面的流操作還能勉強讀懂，遇到最後的lambda表達式實在沒辦法讀下去了，根本不知道他想表達什麼意思。

但是，如果我們真正遇到這樣的“大神級”代碼怎麼辦？還好有IDEA這樣的神奇幫助我們，滑鼠移動到代碼處，點擊右鍵出現以下菜單：

點擊“Show Context Actions”，出現以下內容：

選擇“Replace Stream API chain with loop（利用迴圈代替Stream流操作）”。

當我把所有的Stream流操作以及簡寫的lambda表達式用“傳統”代碼取代後，代碼邏輯如下：

Map<Long, List<Student>> map = new HashMap<>();
//按step進行分組
for (Student student : students) {
  	//computeIfAbsent方法等同下列代碼
  	/*List<Student> list = map.get(student.getStudentNumber());
     if (list == null) {
         list = new ArrayList<>();
         map.put(list);
     }
     list.add(student)*/
    map.computeIfAbsent(student.getStudentNumber(), new Function<Long, List<Student>>() {
        @Override
        public List<Student> apply(Long k) {
            return new ArrayList<>();
        }
    }).add(student);
}
//把Map的Entry元素放入List中，併排序
List<Map.Entry<Long, List<Student>>> toSort = new ArrayList<>();
for (Map.Entry<Long, List<Student>> integerListEntry : map
        .entrySet()) {
    toSort.add(integerListEntry);
}
toSort.sort(Map.Entry.comparingByKey());
//再使用LinkedHashMap按插入順序排列
Map<Long, List<Student>> studentkMap = new LinkedHashMap<>();
for (Map.Entry<Long, List<Student>> integerListEntry : toSort) {
    studentkMap.putIfAbsent(integerListEntry.getKey(), integerListEntry.getValue());
}

這樣看代碼邏輯清晰了，實際上不就是把List按元素中的step分組，並按字典序排列麽？如果按照開始的Stream+Lambda表達式，別說優化，連看懂都是問題。當我們把代碼改為“傳統”代碼後，邏輯一下就清晰了。

Map<Long, List<Student>>> studentMap = new TreeMap<>();
for (Student student : students) {
    List<Student> list = map.get(student.getStudentNumber());
    if (list == null) {
        list = new ArrayList<>();
        map.put(list);
    }
    list.add(student)
}

適當是使用Stream和Lambda這樣是不是更好呢？

關註公眾號（CoderBuff）回覆“stream”獲取《Java8 Stream編碼實戰》PDF完整版。