一个7年老码农写出的代码,另一个7年的觉得写得很好
图片

评论

精彩评论

丶Lion
今天 15:00??
快来个人说说,哪里有毛病,楼主下不来台了
小星星开心
今天 16:06??
没在问题里把业务需求描述出来,所以大家都认为是没有问题的。就我看第一眼也觉得没问题,因为这个工具方法给人的第一眼感觉是这样的:根据api接口获取本地ip地址。如果以这个角度理解的话,无非就是没有进行异常处理这一个问题。但是根据题主描述的业务需求来说,这个代码是完全不合格的。
首席搬砖工程师
今天 16:57??
单独抠出一段代码,看不出有什么问题,别人不理解你的业务,如果你觉得不好,贴出你认为好的来,跟多少年码农写的有什么关系,不是核心代码,能用就好
粉色调佩奇
今天 17:39??
这个命名确实太水了 尼玛getIP恰到好处 还加个byapi 日尼玛 难道是makeIP?
红薯的爸爸
今天 16:39??
1、RestTemplate 是否可以静态申明呢, 一定需要每个调用方法的时候都要去new吗 2、“http://。。。。” 这一串是否可以动态读取配置呢 3、如果"http://。" 地址不可用了,是不是这个方法就调用不成功了呢,假设支持多个地址,是否可以当第一个地址失效时,轮询第二个地址呢,从最大程度上保证可用。

最新评论(46

锦年
今天 19:14??Android
这里没别人
老鼠盘根
今天 18:09??
如果是记日志同时调用这个方法问题就大了. 毕竟记日志要求吞吐量大, 写日志时还要先网络请求,效率可想而知.
粉色调佩奇
今天 17:39??
这个命名确实太水了 尼玛getIP恰到好处 还加个byapi 日尼玛 难道是makeIP?
诗词
今天 17:18??
单例
首席搬砖工程师
今天 16:57??
单独抠出一段代码,看不出有什么问题,别人不理解你的业务,如果你觉得不好,贴出你认为好的来,跟多少年码农写的有什么关系,不是核心代码,能用就好
红薯的爸爸
今天 16:39??
1、RestTemplate 是否可以静态申明呢, 一定需要每个调用方法的时候都要去new吗 2、“http://。。。。” 这一串是否可以动态读取配置呢 3、如果"http://。" 地址不可用了,是不是这个方法就调用不成功了呢,假设支持多个地址,是否可以当第一个地址失效时,轮询第二个地址呢,从最大程度上保证可用。
巴林的狗尾草
今天 16:34??
主要还是得看你们的业务是啥,这个代码能work就没啥问题,你要非说毛病,那换个场景,随便加个要求就挂了,问题是,真正把很多问题都考虑清楚了并且解决了的代码你们又维护不了,这可咋办
tousss
今天 16:31??
@小星星开心 有个明白人了
OSC首席小可爱
今天 16:08??
@小星星开心 怪我没说清楚需求
小星星开心
今天 16:06??
没在问题里把业务需求描述出来,所以大家都认为是没有问题的。就我看第一眼也觉得没问题,因为这个工具方法给人的第一眼感觉是这样的:根据api接口获取本地ip地址。如果以这个角度理解的话,无非就是没有进行异常处理这一个问题。但是根据题主描述的业务需求来说,这个代码是完全不合格的。
OSC首席小可爱
今天 16:04??
@丶Lion 你觉得这样真的能获取到客户端的ip么
帝婳
今天 15:54??
代码垃圾,老板会扣工资吗?企业会倒闭吗?HR会开除你吗?如果不会,有什么好逼逼的
丶Lion
今天 15:48??
@OSC首席小可爱 。这就是个获取ip工具类,保存肯定不在这写。让我挑毛病,顶多就是没考虑异常情况,没判空。其他,还真不好挑毛病。
OSC首席小可爱
今天 15:39??
@别以而过 这个家伙@tousss 是弯的,不好了,你被盯上了
别以而过
今天 15:38??
@tousss 我男的,不接受
OSC首席小可爱
今天 15:37??
@nice_so 需求:记录用户访问日志,保存用户的ip地址。实现:上图。还没看出来什么问题么,不是说这个代码辣不辣鸡,能实现这个需求么
OSC首席小可爱
今天 15:36??
@别以而过 @丶Lion @疏影横斜 需求:记录用户访问日志,保存用户的ip地址。实现:上图。还没看出来什么问题么
nice_so
今天 15:32??
就算 100 年经验的人写出的代码,还是会有人吐槽。如果你自认为没有写过垃圾代码,你回过头看看就知道了

没有更多内容

加载失败,请刷新页面

返回顶部
顶部