这代码写的如何?是不是很烂?

曾经的十字镐 发布于 2015/12/10 22:13
阅读 2K+
收藏 0

这代码写的如何?是不是很烂?欢迎吐槽。

加载中
2
双城记
双城记

notice为null就扔出一个runtimeexception。。首先这个异常类型使用的不合理,应该用resourcenotfoundexception之类的异常。然后这个地方其实不适合抛异常,应该返回一个没有该id的资源的json信息数据。

其次,catch快去掉,加一个异常handler统一处理所有mvc的异常,简化代码

1
Moses_Fu
Moses_Fu

居然没人说这个问题。。

八个空格!!两个TAB。。

嗯。

中场三分
中场三分
+12306
maradona
maradona
显示器大就是任性啊
0
记住一座城
记住一座城
把try,catch里面的逻辑拆出去。
0
公孙二狗
公孙二狗

1. 异常里最好给出描述信息,不要等出问题后分析 stack 信息,定位代码和逻辑,当然都能找到,但是可以做得更好

2. 只有逻辑,没有日志,不利于出错是分析问题

3. 没有注释(不要人文简单的代码就不需要注释,即使很简单的代码,要是看代码来了解逻辑,谁都要花不少时间)

4. 代码格式还可以更好

0
王涛
王涛
这已经很不错了,大多数情况下,都是在赶工,谁去考虑代码格式、异常等问题
0
嘻哈开发者
嘻哈开发者
写得比我好
0
景愿
景愿
所以说你的注释呢?
0
ArrayListt
ArrayListt

我觉得可以吧抛异常的放在try上面。

然后把try,catch去掉,建立一个拦截器,统一去做try,catch

0
Leaybc
Leaybc
难道为空就要异常么,不能返回查不到数据的消息给前台么。
0
分流砥柱
分流砥柱

1.此处基本上不会出现异常错误,即使错误也是数据逻辑错误,做错误返回就好。

2.不知道你返回的response是否是ModelAndView的子类,从代码可以初判你可能少了@responseBody注解,或者你controller 应该有@restController.

3.在controller中实现dto不是很简洁,最好还是封装一下吧。

返回顶部
顶部