千万不要在深夜提交代码

腾讯技术工程 腾讯技术工程     2022-11-30     201

关键词:

开发工程师的生涯中总是要经历几次Code Review的洗礼

他二哥刚就职时,第一次提交上线代码CR,

几十行的代码提交了十几次才通过。

修改了一夜后,终于得到了一个“LGTM“评论

这真是二哥一段难忘的回忆

(LGTM=Looks Good to Me)

他二哥的同事们也分享了不少在

Code Review的难忘经历

一起来看看这些不同等级的“名场面”吧


其实CR是非常必要的一个环节,

仔细进行CR可以在开发阶段发现大部分问题,

好的代码风格和设计也需要通过CR展现出来。

前期 CR 可以发现很多不足和问题,

慢慢提高了代码质量,要改的 Bug 就少了。

最后温馨提醒一下,

CR别人的时候至少给一条正面评价,

欲扬先抑or先抑后扬都可以,

都是对提升团队融洽度极好的。

各位大佬还有哪些难忘或者有趣的经历

一起在评论区分享吧!

(内容素材来自码客)

1月12日锁定有料程序员直播

千万不要误用java中的hashcode方法

刚才debug追堆栈的时候发现一个很奇怪的问题我用IE8和Google的浏览器访问同一个地址Action的scope="session"也设置了而且两个浏览器提交的参数map也是互相独立的不过很奇怪的一个两个Action对象的hashmap是不同的但是它们的对象变量par... 查看详情

千万不要在方法上打断点!有坑!

...结论,也就是标题:在本地以Debug模式启动项目的时候,千万不要在方法上打断点 查看详情

千万不要再这样创建集合了!极容易内存泄露!(代码片段)

由于Java语言的集合框架中(collections,如list,map,set等)没有提供任何简便的语法结构,这使得在建立常量集合时的工作非常繁索。每次建立时我们都要做:1、定义一个空的集合类变量2、向这个结合... 查看详情

程序员千万不要做舔狗啊!(代码片段)

工作中,我是一个高贵的程序员,产品经理让我改个需求最起码孝敬一杯奶茶。但是在爱情中,我却有一个爱而不得的女神。程序员做久了,很擅长安慰自己,比如每逢情人节,我都会安慰自己:"... 查看详情

千万不要在生产环境使用这个版本的jdk,这不?内存又溢出了!快要裂开了!(建议收藏)(代码片段)

大家好,我是冰河~~最近,一名小伙伴跟我说:他写的程序在测试环境一点问题没有,但是发到生产环境却会频繁出现内存溢出的情况,这个问题都困扰他一周多了。于是乎,周末我便开始帮他排查各种问... 查看详情

千万不要在生产环境使用这个版本的jdk,这不?内存又溢出了!快要裂开了!(建议收藏)(代码片段)

大家好,我是冰河~~最近,一名小伙伴跟我说:他写的程序在测试环境一点问题没有,但是发到生产环境却会频繁出现内存溢出的情况,这个问题都困扰他一周多了。于是乎,周末我便开始帮他排查各种问... 查看详情

svn提交代码的原则

 [1]先更新在提交  [2]多提交 [3]不要提交不能通过编译的代码 [4]每次提交必须书写明晰的标注   [5]提交时注意不要提交本地自动生成的文件  [6]不要提交自己不明白的代码  [7]慎用锁定... 查看详情

react-native修改xcode项目名(代码片段)

...名,改为新的工程名2.依次选择黄色文件夹,修改名字,千万不要在Xcode外修改!!!3.点击Find,点击FindandReplaceinProject...4.此时,还剩下一些需要手动修改,再次查找5.进入Tests和UITests文件夹,修改文件名,千万不要在Xcode外修改... 查看详情

程序员你为何喜欢在深夜写代码编程?

如果你问我为什么喜欢在深夜写代码?我会说安静,效率高,没人打扰。是的写代码的过程最害怕别人打扰自己。所以有人宁愿晚上熬夜写代码。往往而言确实夜晚比白天的效率高很多。为何如此?其实大脑疲倦的时候更能集中... 查看详情

房贷没放款前千万不要做的事

...信用卡或其他贷款,我们在房贷还没有正式放款之前千万不要随意申请其他贷款或大额信用卡,因为现在很多银行对于房贷限制很多,如果发现你申请了其他的贷款,那么可能会导致你房贷审核不通过。    2... 查看详情

程序员千万不要做舔狗啊!(代码片段)

????????关注后回复 “进群” ,拉你进程序员交流群????????作者丨阿光来源丨码农小说家(ID:mnxsj0011)工作中,我是一个高贵的程序员,产品经理让我改个需求最起码孝敬一杯奶茶。但是在爱情中,我... 查看详情

java示例代码_数据是如何存储在';歌手';表中的时间为';不要为该类的对象进行提交

java示例代码_数据是如何存储在';歌手';表中的时间为';不要为该类的对象进行提交 查看详情

hashmap正确遍历方式,千万不要再forforfor啦!!!(代码片段)

for(inti=0;i<map.size();i++)if(map.get(i)/n>=0.01)num++;上述代码报空指针异常。为什么?循环变量i取值的范围是从0到map.size(),但是这里的map.size()指的是HashMap中键值对的数量,而不是HashMap中键的集合的大小。因此,使用map.get(i)来访问... 查看详情

xise菜刀千万不要随便下载!(警惕!)

xise菜刀千万不要随便下载!(警惕!)最近小编在众多群里,看到有人善意伪装的提供免费xise下载地址!1:小心后门,小编发现众多免费下载xise的里面都隐藏后门也就是说,只要你使用带后门的xise菜刀,别人就可以获取到你... 查看详情

js数组遍历千万不要使用for...in...

昨天做个下拉框扩充了一下数组的方法Array.prototype.remove=function(val){varindex=this.indexOf(val);if(index>-1){this.splice(index,1);}};结果今天就遇到问题了for(iininitMarker){initMarker[i].setMap(null);}用forin循环数组总是在最后多出一个rom 查看详情

idea提交有一行对比不要

idea提交有一行对比不要操作如下:1、那么在Commit对话框中,选择需要提交的文件,双击鼠标左键盘就可以了。2、这个时候你的IntelliJIDEA将会为你打开一个对比的对话框。3、在这个对比的对话框中将会把你的本地文件版本和Git... 查看详情

项目管理最忌的5件事,千万不要忽视!

项目管理最忌的5件事,千万不要忽视!当下,虽然项目管理理论已经相当成熟,然而成熟的只是框架和方法论,真正的项目管理还远不止这些。在实际的项目管理中,各路项目经理仁者见仁,智者见智。 有的项目经理能把... 查看详情

[编程]成都控制器开发:千万注意,不要轻易把变量定义和赋值搞在一起

最近开发编写一个测试设备的程序(C程序),在定义一个全局变量的时候,之前我喜欢在定义的时候直接赋值,如这样:unsignedchara=0;//这里直接给变量a定义并赋值voidmain_function()//在这里使用a...不过这样做的时候却有时可能要出问... 查看详情