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

评论

精彩评论

丶Lion
10/08 15:00  
快来个人说说,哪里有毛病,楼主下不来台了
首席搬砖工程师
10/08 16:57  
单独抠出一段代码,看不出有什么问题,别人不理解你的业务,如果你觉得不好,贴出你认为好的来,跟多少年码农写的有什么关系,不是核心代码,能用就好
Java劝退师_Joyden_Lo
10/08 16:39  
1、RestTemplate 是否可以静态申明呢, 一定需要每个调用方法的时候都要去new吗 2、“http://。。。。” 这一串是否可以动态读取配置呢 3、如果"http://。" 地址不可用了,是不是这个方法就调用不成功了呢,假设支持多个地址,是否可以当第一个地址失效时,轮询第二个地址呢,从最大程度上保证可用。
淡淡的分
10/10 08:38  
任何脱离项目背景谈代码质量的行为都是甩流氓。我经历过来三个人,两个项目并行开发,同时要维护一百来台爬虫,那时候任何不影响实际业务的模块都是能跑就行。我记得有一个返回IP归属地固定返回中国,如果要让楼主看到了估计也得发一波图了
小星星开心
10/08 16:06  
没在问题里把业务需求描述出来,所以大家都认为是没有问题的。就我看第一眼也觉得没问题,因为这个工具方法给人的第一眼感觉是这样的:根据api接口获取本地ip地址。如果以这个角度理解的话,无非就是没有进行异常处理这一个问题。但是根据题主描述的业务需求来说,这个代码是完全不合格的。

最新评论(90

OSCHINA首席渣男
10/10 15:17  
其实没啥毛病,就是强迫症看见这种命名不太舒服
开源中国马桶盖
10/10 10:21  
@丶Lion @OSC首席小可爱 如果我是领导 我也这么说, 只是别让我接手 这种代码就好, 跟我有啥关系..... 遇到个8年的 命名都是3个大写字母加1-3位数字..... 代表全程无注释..... 人家老大让我接手, 我一看 我说我水平有限,接不了.....
OSC首席小可爱
10/10 10:14  
@开源中国马桶盖 怎么实现我不管,只要结果能运行就ojbk
开源中国马桶盖
10/10 10:13  
我感觉我写代码跟你们比 已经够严格了...可惜 我每次面试都喷不过面试官 这是啥原因.....人家现在 貌似就是比较流行不申明泛式,不用转换字符串方法,不写注释,不用工厂.....
淡淡的分
10/10 09:38  
@OSC首席小可爱 开玩笑,家国梦能肝到榜首的人会怕伤肝?
OSC首席小可爱
10/10 09:35  
@pj220 @淡淡的分 @老菜鸟0217 各位戾气别那么重好不好,我后面已经说明了这段代码只是不符合需求,没说任何代码质量和逻辑的问题。跟我同事一说客户还没发现大家都是哈哈一笑,觉得挺搞笑的才发上来的,包括当事人都在下面回复了。年轻人火气都别这么大,伤肝啊
淡淡的分
10/10 08:44  
@回首已千年 是吗?那百度怎么还没有倒闭
回首已千年
10/10 08:40  
@淡淡的分 现在爬虫不是犯法么
淡淡的分
10/10 08:38  
任何脱离项目背景谈代码质量的行为都是甩流氓。我经历过来三个人,两个项目并行开发,同时要维护一百来台爬虫,那时候任何不影响实际业务的模块都是能跑就行。我记得有一个返回IP归属地固定返回中国,如果要让楼主看到了估计也得发一波图了
zigzagroad
10/10 03:51  Android
看了弹主后面补充的需求描述,再结合代码来看,这段代码是完全不能满足需求的错误实现。这个代码返回的是运行此代码的机器(通常是服务器)的外网IP,而需求是获取客户端机器的访问IP,这个IP地址应该从请求头里面解析获取。
染墨若流云
10/10 00:03  
参数写死不太好
老菜鸟0217
10/09 22:37  Android
可以骂别人代码写的乱,但是搞到这里羞辱别人就是不道德的,都是码农何必呢,写的又不是高精尖的玩意。
紫风tianq
10/09 22:35  
就怕这种截图的
TGVvbmFyZA
10/09 19:08  
获取客户端ip 不需要第三方接口吧
酷酷的就
10/09 18:40  
就一个静态方法, 能死人么?
丶Lion
10/09 17:26  
@中华人民共和国程序员 这个外部接口,一看就是外部提供的,不是自己维护的。应该搞成可配置的
OSC知行合一
10/09 17:00  
打回去重写
OSC首席小可爱
10/09 16:49  
@Kit_lee 这个返回的永远是服务器的ip
Kit_lee
10/09 16:35  
应该是喷人家为什么通过一个HTTP外部调用来获取本机IP吧
OSC首席小可爱
10/09 16:08  
上热门了,解释一下发这个动弹,这个方法就是用来获取用户ip地址记录操作日志用的。了解了这个需求大家应该就知道问题出在哪里了此外,这个(辣鸡)代码运行是没有问题的,编译也不会报错,到现在还在正常运行(客户还没发现问题)各位大佬提出的各个建议都是非常好的点,感谢各位大佬。

没有更多内容

加载失败,请刷新页面

返回顶部
顶部