V2EX › momuou 的所有回复 › 第 5 页 / 共 7 页

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

For Existing Member Sign In

1 2 3 4 5 6 7

❮

❯

2019 年 1 月 16 日

回复了 qiayue 创建的主题 › 互联网 › A 产品去 B 产品里发广告拉用户，什么情况下 B 可以封禁 A 的链接？什么时候不能？

现在谁敢动 pony。。？

2019 年 1 月 7 日

回复了 kingandyoga 创建的主题 › 分享创造 › [小程序]写了个 Leetcode 刷题小程序...

感谢。。。

2019 年 1 月 3 日

回复了 NicholasYX 创建的主题 › 问与答 › 最近开始申报个税了，只有我不满足条件干瞪眼吗？

--对于 5K 月薪的人来说。有一个租房减免是不是一年不用交租了？

2018 年 12 月 25 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

@zhaogaz 嗯，楼上多次提及 vaild 校验，这个更多我在前端接触过，服务端的真要学习一下

2018 年 12 月 25 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

@wuchenl 嗯，这位老哥的方法跟 24L 的有点相似，有空余时间我也要学习一下这种利用注解的方式.谢谢

2018 年 12 月 25 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

@icop 这不是执行问题，是优化问题。

2018 年 12 月 25 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

@tingfang 谢谢，不过这种写法我觉得太繁琐了，可以看看我 20L 写的最新的。也是感觉写的最方便的写法了

2018 年 12 月 25 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

@iceheart 谢谢，抛异常这种考虑过，不过也是感觉太繁琐

2018 年 12 月 25 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

@devilyaos 其实判断不是重点。。重点就是怎么优化这种多个判断的，不过还是很感谢

2018 年 12 月 25 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

@lihongjie0209 谢谢

2018 年 12 月 25 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

@pipicat 谢谢

2018 年 12 月 24 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

不会贴图，所以没什么贴上来没什么格式。。感谢提建议的各位。

2018 年 12 月 24 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

@james2013 这种写一两个还好，写多了容易烦躁

2018 年 12 月 24 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

好吧，大背景就是本来写了一段自我感觉良好的代码，可是硬盘丢失了，代码只能重写，却写出这种后人难以维护的代码，感觉不太好，所以发上来请教一下各位。。刚刚上了个厕所，好像又记起来之前怎么写的了。
if (StringUtils.isBlank(countTypeId) || "null".equals(countTypeId)) {
// 业务类型
result.put("msg", "业务类型不明，请联系管理员");
}
else if(StringUtils.isBlank(dealDate) || "null".equals(dealDate)){
// 日期
result.put("msg", "请选择确切日期进行查询后再进行审核.");
}else if(StringUtils.isBlank(regionCode) || "null".equals(regionCode)){
//地市编码
result.put("msg", "请选择确切地市进行查询后再进行审核");
}else if(StringUtils.isBlank(payType) || "null".equals(payType)){
//-支付类型（ 0 待确认 1 通过 2 不通过
result.put("msg", "选择的支付类型有误，请稍后重试");
}else{
paramMap.put("countTypeId", countTypeId.trim());
paramMap.put("dealDate", dealDate.trim());
paramMap.put("regionCode", regionCode.trim());
paramMap.put("payType", payType.trim());
int i = dataRecountService.updateDataPayType(paramMap);
if (i > 0) {
result.put("status", "1");
result.put("msg", "提交成功");
}
}
return result;

2018 年 12 月 24 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

@jorneyr 谢谢，我去查询一下。

2018 年 12 月 24 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

@kyuuseiryuu 就是本着我写着爽，维护的人想砍我的心态。。所以才放上来的。

2018 年 12 月 24 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

@misaka19000 这本身就是一段烂代码，就是想请教一下你怎么重构，瞎子都知道这种代码难维护，可是你要怎么写才能符合这种写法又优雅呢？

2018 年 12 月 24 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

@jcsmallming 这种写法肯定是不符合预期的，其实一直都是写 if()else{return}这种的，可是参数多了就觉得麻烦，而且最后还是会走到事务里面去.

2018 年 12 月 24 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

@wsyzzz 不是报错--是想优化一下这种写法。。。

2018 年 12 月 24 日

回复了 momuou 创建的主题 › 程序员 › 请问这段代码有什么问题？完全看不出有什么异常。。

// 业务类型
if (StringUtils.isNotBlank(countTypeId) && !"null".equals(countTypeId)) {
paramMap.put("countTypeId", countTypeId.trim());
// 日期
if (StringUtils.isNotBlank(dealDate) && !"null".equals(dealDate)) {
paramMap.put("dealDate", dealDate.trim());
//地市编码
if (StringUtils.isNotBlank(regionCode) && !"null".equals(regionCode)) {
paramMap.put("regionCode", regionCode.trim());
if (StringUtils.isNotBlank(payType) && !"null".equals(payType)) {
//-支付类型（ 0 待确认 1 通过 2 不通过
paramMap.put("payType", payType.trim());
try {
int i = dataRecountService.updateDataPayType(paramMap);
if (i > 0) {
result.put("status", "1");
result.put("msg", "提交成功");
}
} catch (Exception e) {
logger.error(e.getMessage(), e);
result.put("msg", "网络异常，请稍后再尝试。");
}
} else {
result.put("msg", "选择的支付类型有误，请稍后重试");
}
} else {
result.put("msg", "请选择确切地市进行查询后再进行审核");
}
} else {
result.put("msg", "请选择确切日期进行查询后再进行审核.");
}
} else {
result.put("msg", "业务类型不明，请联系管理员");
}

1 2 3 4 5 6 7

❮

❯