关于可读性与高级技巧之间的折衷

昨天维护了这位同事的代码(刚离职)，脑壳疼，简单贴些片段吧：

• 这是一段查询分页数据，这里前端传的是id数组，这里使用 getFlag() 转换还可以理解

alarmInfoService.getSimpleInfoPage(param.getPerPageCount(), param.getCurPage(), param.getKeyWord(), getFlag(param.getScenesList()), cameraIds);

• 由于他把逻辑都写在了 mapper层，就不贴 ServiceImpl 的内容了，这里是 mapper 层的逻辑...

default IPage<AlarmInfoDO> selectPage(int count, int cur, String keyword, int flag, List<String> cameraIds) {
    ....
    if (flag != 0) {
        List<Integer> flags = getAllFlags(flag);
        for (int i = 0; i < flags.size(); i++) {
            if (i != 0) {
                sql.append(" or ");
            } else {
                sql.append(" and (");
            }
            sql.append(String.format("((type & %d) > 0)", flags.get(i)));
        }
        sql.append(")");
    }
    ....
}

default List<Integer> getAllFlags(int flag) {
    List<Integer> flags = new ArrayList<>();
    for (int i = 0; i < 32; i++) {
        if (((flag >>> i) & 1) == 1) {
            flags.add(1 << i);
        }
    }
    return flags;
}

唉，无奈