史上最坑爹的代碼！個個讓人崩潰！

Java 就是這麼一門神奇的語言，任何水平的人都能寫出能夠運行的代碼，可是一看代碼便知水平高低。做爲一個程序員，你看過哪些坑爹代碼，你又寫過多少坑爹代碼，還有多少你不知道的坑爹代碼？

有意思的是碼雲上建了一個代碼倉庫：bullshit-codes，倉庫的目的就是爲了收集這些坑爹代碼，可讓別人不掉坑或者少掉坑，能夠避免本身掉坑，或許哈哈一樂！

上邊匯聚了各類編程語言的，倉庫地址以下：

gitee.com/oschina/bul…

本文小編給你們整理了幾個比較坑爹的代碼，整理了幾個以後，實在整理不下去了，僅供你們參考，看看能不能崩潰掉！

1、幾個坑爹代碼的目錄

一、這樣使用 StringBuffer 的方法有什麼坑？

二、你寫過的最長的一行代碼有多長？？？

三、循環+條件判斷，你最多能嵌套幾層？

四、爲了後期優化查詢速度 ~ 很有商業頭腦！

五、你是如何被異常玩而後變成玩異常的？

六、Stream 玩得最 6 的代碼，看過的人都驚呆了！

2、坑爹代碼 | 這樣使用 StringBuffer 的方法有什麼坑？

你是否曾經這樣使用過 Java 的 StringBuffer 類？

/**
 * Create Time 2019/5/24
 * StringBuffer追加 如癡如醉的寫法
 * @author cailong
 **/
public class Append {
    public static void main(String[] ares){

        StringBuffer sb = new StringBuffer();
        //這裏都能理解
        sb.append("<?xml version=\"1.0\" encoding=\"UTF-8\"?><ROOT>");
        for (int i = 0; i < 10; i++) {
            //爲何到這裏就要這樣寫？？？既然都用StringBuffer了    （這裏省略集合遍歷用i代替 意思能懂就行）
            sb.append("<NSRXX>" +
                    "<NSRSBH>"+i+"</NSRSBH>" +
                    "<NSRMC>"+i+"</NSRMC>" +
                    "<DJXH>"+i+"</DJXH>" +
                    "<ZGSWJ_DM>"+i+"</ZGSWJ_DM>" +
                    "<ZGSWJ_MC>"+i+"</ZGSWJ_MC>" +
                    "<SJLY>sjzs</SJLY>" +
                    "<YWSX_DM>"+i+"</YWSX_DM>" +
                    "</NSRXX>");
        }
        sb.append("</ROOT>");
        System.out.println(sb.toString());
    }
}
複製代碼

3、坑爹代碼 | 你寫過的最長的一行代碼有多長？？？

你寫過的最長的一行代碼有多長嗎？你爲何要寫這麼長？是看着帥呢，仍是想減小代碼行數？

List<OperationPurchaseInfo> purchaseInfoList
                = sheet.getPurchaseInfoList()
                .stream()
                .filter(purchaseInfo -> purchaseInfo.getExteriorOperation()
                        .getExteriorPart()
                        .getExteriorOperationList()
                        .stream()
                        .filter(exteriorOperation -> exteriorOperation
                                .getProcessState()
                                .equals(ExteriorOperation.ProcessState.PROCESSING))
                        .count() != 0
                        // 訂單明細中工序對應的工件下的其餘工序存在加工中，
                        // 且已發給供應商且供應商不是當前訂單供應商時，須要判斷
                        && (purchaseInfo.getExteriorOperation()
                        .getExteriorPart()
                        .getTeamwork() == null || !purchaseInfo.getExteriorOperation()
                        .getExteriorPart().getTeamwork().equals(sheet.getTeamwork()))
                ).collect(Collectors.toList());
複製代碼

上面這段代碼雖然被拆開多行顯示，但本質上是一行，一個極其複雜的賦值語句！

這種代碼是否是爲了讓別人看不懂來彰顯本身的編碼水平呢？

小編以爲 Java Stream API 以及各類函數式編程方法，以及各類語法糖在某種程度讓這種糟糕代碼愈來愈多！

那麼一塊兒來批判一下這個代碼，或者你有什麼好的解決方案呢？

4、坑爹代碼 | 循環+條件判斷，你最多能嵌套幾層？

for 循環和 if 條件判斷語句，必不可少吧。可是你見過最多嵌套的循環和條件判斷有幾層呢？或者說，你最多能容忍多少層的嵌套呢？

咱們仍是先來看看極端的坑爹代碼吧：

// 這個無限循環嵌套，只是總循環的一部分。。。我已經繞暈在黃桷灣立交
        if (recordList.size() > start) {
            for (int i = start; i < end; i++) {
                Map<String, Object> map = recordList.get(i);
                Map<String, Object> field11 = (Map<String, Object>) map.get("field"); //name -> code
                Map<String, Object> record11 = (Map<String, Object>) map.get("record"); // code -> value
                String catagory1 = map.get("categoryId").toString();
                //  查詢第一種類型對應的其餘類型
                SalaryDataVo ss = JSON.parseObject(JSON.toJSONString(map), SalaryDataVo.class);
                Page page3 = salaryManagerService.getAllRecordsByCondition(ss);
                if (page3.getRecords().size() > 0) {
                    List<Map<String, Object>> salaryDataVos = page3.getRecords();
                    salaryDataVos = this.reSetMap(salaryDataVos, null, null);
                    for (Map<String, Object> map2 : salaryDataVos) {
                        Map<String, Object> field2 = (Map<String, Object>) map2.get("field");
                        Map<String, Object> record2 = (Map<String, Object>) map2.get("record");
                        String catagory2 = map2.get("categoryId").toString();
                        List<SalaryGroupVO> groupList2 = salaryGroupService.getSalaryGroupsItems(this.getUserCorpId(), catagory2);
                        for (SalaryGroupVO cc : groupList2) {
                            cc.setCode(cc.getParentId() + cc.getCode());
                        }
                        //計算
                        for (Map.Entry<String, Object> entity : field2.entrySet()) {
                            String keyName = entity.getKey();
                            for (SalaryGroupVO s2 : groupList2) {
                                if ("bigDecimal".equals(s2.getItemType()) && s2.getCode().equals(field2.get(keyName).toString()) && ("部門" != keyName) && ("姓名" != keyName) && StringUtils.isNotEmpty(s2.getItemType())) {
                                    if (field11.containsKey(keyName)) {
                                        if (field11.containsKey(keyName)) {
                                            String code1 = field11.get(keyName).toString();
                                            Double newValue = 0.0;
                                            Double oldValue = 0.0;
                                            if (!record11.get(code1).toString().matches("^[0-9]*$")) {
                                                oldValue = Double.parseDouble(record11.get(code1).toString());
                                                if (record2.containsKey(entity.getValue().toString()) && (!record2.get(entity.getValue().toString()).toString().matches("^[0-9]*$"))) {
                                                    String value2 = record2.get(entity.getValue().toString()).toString();
                                                    newValue = Double.parseDouble(value2);
                                                }
                                                record11.remove(field11.get(keyName).toString());
                                            }
                                            if (code1.startsWith(catagory1) || code1.startsWith(catagory2)) {
                                                String co = code1.replace(catagory1, "hz");
                                                field11.put(keyName, co);
                                                record11.put(co, oldValue + newValue);
                                            }
                                        }
                                    } else {
                                        String code = entity.getValue().toString();
                                        String str = entity.getValue().toString();
                                        Object value2 = record2.get(entity.getValue().toString());
                                        if (str.startsWith(catagory1) && str.replace(catagory1, "").startsWith("hz")) {
                                            code = str.replace(catagory1, "");
                                        } else if (str.startsWith(catagory2) && str.replace(catagory2, "").startsWith("hz")) {
                                            code = str.replace(catagory2, "");
                                        }
                                        field11.put(keyName, code);
                                        record11.put(code, value2);
                                    }
                                }
                            }
                        }
                    }
                }
                List<SalaryGroupVO> sList = salaryGroupService.getSalaryGroupItemsByParentId(catagory1);
                for (SalaryGroupVO s : sList) {
                    if (field11.containsKey(s.getName()) && s.getCode().startsWith("hz")) {
                        String k = field11.get(s.getName()).toString();

                        field11.put(s.getName(), s.getCode());
                        String value = null;
                        if (record11.containsKey(k)) {
                            value = record11.get(k).toString();
                        }
                        record11.put(s.getCode(), value);
                    }
                }
                resultList.add(map);
                pageInfo.setRecords(resultList);
            }
        }
複製代碼

我數了數，一共有 11 層的嵌套！！！

吐槽歸吐槽，這樣的代碼邏輯有什麼重構的好方法嗎？

5、坑爹代碼 | 爲了後期優化查詢速度 ~ 很有商業頭腦！

什麼樣的程序員是一個好程序員呢？當咱們在給客戶開發系統時候，爲了後期的優化，預留一些埋點。

經過對這些埋點的優化，可讓客戶瞬間感受系統在運行速度上有質的飛躍，讓公司順利的簽署二期開發合同，收取鉅額開發費用。

從公司角度來看，這樣的程序員就是一個好程序員。 —— 這句話不是紅薯說的！

好比：

我想說的是：兇碟，你下手可否不那麼狠啊，建議對代碼進行優化，改爲 Thread.sleep(1000);  —— 這句話也不是紅薯說的。

或者你有什麼更好的建議呢？不要以爲駭人聽聞，真有這樣的人，這樣的代碼！！！

6、坑爹代碼 | 你是如何被異常玩而後變成玩異常的？

玩 Java 的人，剛開始會被各類異常虐，空指針應該是最多見的。多玩兩年就開始玩異常，各類奇葩異常玩法層出不窮。

你以爲下面這種異常的定義妥嗎？

/**
 * 處理業務的異常
 * 竟然有一堆靜態異常，準備好了隨時能夠拋？？
 * 錯誤碼是字符串
 */
public class CommandException extends BaseException {

  private static final long serialVersionUID = -6354513454371927970L;
  
  public static CommandException PARAM_NULL= new CommandException("Command_assemble_01", "Parameter is Needed But Empty");
  public static CommandException DEVID_NULL = new CommandException("Command_assemble_02", "DevId Cannot Be Null");
  public static CommandException MDCODE_NULL = new CommandException("Command_assemble_03", "Model Code Cannot Be Empty");
  public static CommandException ORDER_NULL = new CommandException("Command_assemble_04", "Order Cannot Be Empty");
  public static CommandException TYPE_NULL = new CommandException("Command_assemble_05", "Upstream / Downstream Type Cannot Be Empty");
  public static CommandException MENUID_NULL = new CommandException("Command_assemble_06", "Menu Id Cannot Be Null");
  public static CommandException CTRLTYPE_NOT_RANGE= new CommandException("Command_assemble_07", "Ctrltype Cannot Be Recognized, Which is not in Range");
  public static CommandException CMD_NULL = new CommandException("Command_analyze_01", "CMD Cannot Be Null");
  public static CommandException PAYLOAD_NULL = new CommandException("Command_analyze_02", "Payload Cannot Be Null");
  public static CommandException FRAMEWORK_FAILED= new CommandException("Command_analyze_03", "Framework Failed to be Checked");
  public static CommandException CHECK_FAILED= new CommandException("Command_analyze_04", "Command Failed to be Checked");
  public static CommandException CONFIGURE_NOT_EXIST = new CommandException("Command_error_1001", "Configure is not Exist");
  public static CommandException REDIS_ERROR = new CommandException("Command_error_1002", "Cache Command in Redis Error", true);
  
  //省略構造函數、get/set方法
}
複製代碼

若是不妥，有什麼問題呢？

7、坑爹代碼 | Stream 玩得最 6 的代碼，看過的人都驚呆了！

Stream 做爲 Java 8 的一大亮點，它與 java.io 包裏的 InputStream 和 OutputStream 是徹底不一樣的概念。Java 8 中的 Stream 是對集合（Collection）對象功能的加強，它專一於對集合對象進行各類很是便利、高效的聚合操做（aggregate operation），或者大批量數據操做 (bulk data operation)。Stream API 藉助於一樣新出現的 Lambda 表達式，極大的提升編程效率和程序可讀性。

那麼 Java 8 的 Stream 究竟是王者，仍是個巨坑，這徹底取決於你是怎麼玩的？

我不得不說，下面代碼是我從業 20 年（說誰呢，誰從業 20 年，我今年 19  歲！！！）看過最牛逼的 Stream 的用法：

//Stream 用的66的
final EventAction eventAction = redisObj(
                EventActionKey + distributionEventId,
                () -> Optional
                        .of(distributionEventId)
                        .map(eventId -> {
                            final EventActionExample example = new EventActionExample();
                            example.createCriteria()
                                    .andEventIdEqualTo(eventId)
                                    .andTriggerTypeEqualTo(EnumEventTriggerType.DISTRIBUTION_PURCHASE.getCode().byteValue());
                            return example;
                        })
                        .map(eventActionMapper::selectByExample)
                        .filter(StringUtil::isNotEmpty)
                        .map(e -> e.get(0)).orElseThrow(() -> ExceptionUtil.createParamException("事件觸發信息不存在"))
                , EventAction.class);
        final AwardConfig awardConfig = redisObj(EventConfigKey + eventAction.getId(),
                () -> Optional.ofNullable(eventAction.getId())
                        .map(actionId -> {
                            final AwardConfigExample example = new AwardConfigExample();
                            example.createCriteria()
                                    .andActionIdEqualTo(actionId);
                            return example;
                        })
                        .map(awardConfigMapper::selectByExample)
                        .filter(StringUtil::isNotEmpty)
                        .map(e -> e.get(0)).orElseThrow(() -> ExceptionUtil.createParamException("xxx")),
                AwardConfig.class
        );
        Optional.of(req)
                .map(e -> e.clueUid)
                .map(id -> {
                    final ClueExample example = new ClueExample();
                    example.createCriteria()
                            .andClueUidEqualTo(id)
                            .andDeletedEqualTo(false)
                            .andReceivedEqualTo(false)
                            .andCreateTimeGreaterThan(now - cluetime);
                    example.setOrderByClause("create_time asc");
                    return example;
                })  // 獲取該被邀請人全部未過時且未被領取的線索的線索
                .map(clueMapper::selectByExample)
                .filter(StringUtil::isNotEmpty)
                .ifPresent(clues -> {
                            final ClueResp clueResp = Optional.of(req)
                                    .filter(c -> {
                                        c.count = clues.size();
                                        return true;
                                    })
                                    .map(this::awardValue)
                                    .orElseThrow(() -> ExceptionUtil.createParamException("參數錯誤"));
                            final Integer specialId = req.getIsHead()
                                    ? clues.get(0).getId()
                                    : clues.get(clues.size() - 1).getId();
                            clues.stream()
                                    .peek(clue -> {
                                        final AwardConfig awardConfigclone = Optional.of(awardConfig)
                                                .map(JSONUtil::obj2Json)
                                                .map(json -> JSONUtil.json2Obj(json, AwardConfig.class))
                                                .orElseGet(AwardConfig::new);
                                        awardConfigclone.setMoney(
                                                Optional.of(clue.getId())
                                                        .filter(specialId::equals)
                                                        .map(e -> clueResp.specialReward.longValue())
                                                        .orElse(clueResp.otherAverageReward.longValue())
                                        );
                                        eventActionService.assembleAward(
                                                awardConfigclone,
                                                clue.getAdviserUid(),
                                                clue.getAdviserUid(),
                                                clue.getClueUid(),
                                                eventAction,
                                                new PasMessageParam(),
                                                clue.getId(),
                                                AwardRelationType.Clud.code()
                                        );
                                    })
                                    .forEach(clue -> {
                                        clue.setOrderNo(req.orderNo);
                                        clue.setCommodityName(req.commodityName);
                                        clue.setOrderAmount(req.orderAmount);
                                        clue.setReceived(true);
                                        clue.setModifyTime(now);
                                        clueMapper.updateByPrimaryKeySelective(clue);
                                    });
                        }
                );
複製代碼

Java 就是這麼一門神奇的語言，任何水平的人都能寫出能夠運行的代碼，可是一看代碼便知水平高低。可是這樣的 Stream 代碼你必定一口老談不吐不快！

8、小編已陣亡

好了，小編實在整理不下去了，你們有興趣的能夠去：

gitee.com/oschina/bul…

更進一步的吐槽！

熱門內容：

一、不一樣時重寫equals和hashCode又會怎樣？

二、千萬不要這樣使用Arrays.asList !

三、2019年程序員高考真題新鮮出爐！看看你能得幾分！

四、從開源小白到 Apache Member，個人成長之路！

五、GitHub 五萬星登頂，命令行的藝術！

六、19 個強大、有趣、好玩、又裝B的 Linux 命令！

七、如何正確訪問Redis中的海量數據？服務纔不會掛掉！

八、爲何有些大公司技術弱爆了？

【視頻福利】2T免費學習視頻，搜索或掃描上述二維碼關注微信公衆號：Java後端技術（ID: JavaITWork）,和20萬人一塊兒學Java！回覆：1024，便可免費獲取！內含SSM、Spring全家桶、微服務、MySQL、MyCat、集羣、分佈式、中間件、Linux、網絡、多線程，Jenkins、Nexus、Docker、ELK等等免費學習視頻，持續更新！