- 主题:review代码哭笑不得
String stauts = respJson.getString("stauts");
String token = respJson.getString("token");
if (stauts.equals("0")) {
return token;
}
就问开发人员A,A说后端服务就返回的是stauts,我问开发人员B,B说我是写错了,可是
A看到这种低级错误为啥不说出来呢。
两个服务发布时间还不一致,先让B同时加上"status":0,再A下个版本改过来,在让B再
下下个版本把stauts删掉,哈哈
--
FROM 27.38.197.*
将错就错也没啥问题吧,只要整个系统都用这个错误的单词,那就是没问题的
我这有个人一直把status变量名写为statue,崩溃念成奔溃,剪贴板说成剪切板
【 在 iwannabe 的大作中提到: 】
: String stauts = respJson.getString("stauts");
: String token = respJson.getString("token");
: if (stauts.equals("0")) {
: ...................
--
FROM 221.218.161.*
跑得好好的,你这么一改,这下大家都知道有问题了。
而且测试还要陪着加班。
只要没问题就不要动的原则
【 在 iwannabe 的大作中提到: 】
: String stauts = respJson.getString("stauts");
: String token = respJson.getString("token");
: if (stauts.equals("0")) {
: ...................
--
FROM 223.198.83.*
既然是不对了就改了吧。
【 在 iwannabe 的大作中提到: 】
: String stauts = respJson.getString("stauts");
: String token = respJson.getString("token");
: if (stauts.equals("0")) {
: ...................
--
FROM 114.252.221.*
如果是某个接口问题
就来一次“接口升级”,加个正确的接口
哈哈
--
FROM 120.244.216.*
没有啥吧,体现了control+c,control+v的好处,保持一致性。
【 在 iwannabe 的大作中提到: 】
: String stauts = respJson.getString("stauts");
: String token = respJson.getString("token");
: if (stauts.equals("0")) {
: ...................
--
FROM 14.19.22.*
所以得让大家有看ide里拼写错误提示的习惯
【 在 iwannabe 的大作中提到: 】
: String stauts = respJson.getString("stauts");
: String token = respJson.getString("token");
: if (stauts.equals("0")) {
: ...................
--
FROM 114.248.219.193
这个是好习惯
而且推广一下使用完整单词做变量名
可以省去很多简写带来的“错误提示”
【 在 PaoloMaldini 的大作中提到: 】
: 所以得让大家有看ide里拼写错误提示的习惯
:
--
FROM 120.244.216.*
简写也可以add to dict,团队内部可以有一些项目名、缩写之类的加入进去
【 在 hothail 的大作中提到: 】
: 这个是好习惯
: 而且推广一下使用完整单词做变量名
: 可以省去很多简写带来的“错误提示”
: ...................
--
FROM 114.248.219.193
这样容易堆成 sh*t 山代码,如果是我的话,我会增加一个额外的判断,那就是拼对的和拼错的都能通过。同时写个注释,说等一过段时间要记得删掉。
然后通知同事可以改成拼写对的。等过一段时间,我把错的代码删掉。
如果数据库里面有这个数据,我就让同事写个脚本改一下。实在不能容忍数据库里面有错的数据。
【 在 chaobill 的大作中提到: 】
: 跑得好好的,你这么一改,这下大家都知道有问题了。
: 而且测试还要陪着加班。
: 只要没问题就不要动的原则
: ...................
--
FROM 14.19.44.*