关于代码可能出现的问题
前几天,在CodeReview的时候,看了一个同事代码有许多不好的地方,
虽然业务运行是OK的,但是能为了规范或是后续维护迭代,都应该注意一下部分问题。
注:因为保密原则,文章贴出来的代码都是我按照格式仿写的,并非公司源码。
1.命名随意
代码:
List<UserDTO> list = new ArrayList<>();
问题:
直接将变量命名为 list,并没有按照其功能命名
2.缺少空格
if (userDO.getName() != null) { codeName= updateUser(userDO); } else { codeName= addUser(userDO); }
问题:
codeName后面应与=号之间有一个空格
3.使用反逻辑运算符
5.方法参数过多
private void collectUsers(Map<Long, UserVO> userMap, List<String> firstCodeList, List<String> secondCodeList, Map<String, String> nameCodeMap, Map<Long, String> idNameMap, Map<Long, String> personCodeMap)
List<UserAndPersonRelationDO> relateUserAndPersonList = userAndPersonConvertor.requestToDOList(userAndPersonRelationDO, nameAndCode);
总结:规则是死的,人是活的,有些实在无法避免的是不需要绞尽脑汁在这里纠结的,比如使用“!”,
文章只是指出一些注意点,尽可能注意就好了,为的只是更规范好看的代码。
【推荐】编程新体验,更懂你的AI,立即体验豆包MarsCode编程助手
【推荐】凌霞软件回馈社区,博客园 & 1Panel & Halo 联合会员上线
【推荐】抖音旗下AI助手豆包,你的智能百科全书,全免费不限次数
【推荐】博客园社区专享云产品让利特惠,阿里云新客6.5折上折
【推荐】轻量又高性能的 SSH 工具 IShell:AI 加持,快人一步