📣 独立日限时特惠: VIP通行证立减$68
查看: 2267| 回复: 3
跳转到指定楼层
上一主题 下一主题
收起左侧

新人一个CR发7,8个revision正常吗?

全局:

注册一亩三分地论坛,查看更多干货!

您需要 登录 才可以下载或查看附件。没有帐号?注册账号

x
香蕉厂,这cr一开始两个人在看,轮流隔天comment,我每次都是当天搞完新的revision然后第二天又要改,今天又来了第三个人,md就加了个无关痛痒的comment让我把两个if check顺序换一下,又得发个新revision,吐了真的

上一篇:小蓝鸟2022summer三番实习群
下一篇:Meta(FB) 加拿大同事群
全局:
我在焦厂的时候第一年,有一个cr 历时一个月搞了13个版本,写了一千行,二十来个file.没人带我,全在comment里面design实现,测试 。很多修改。唉。特烦那种后面问一些问题,明明之前也可以问的,却不一下子问完。一次二次就算了,搞是很多次。
回复

使用道具 举报

全局:
前几个cr需要学习一下组里的代码规范 多发几个rev也不要太在意。还有有时候reviewer一样的问题不会重复提,但是自己需要检查一下自己的code有没有出现他之前提的问题在下一个revision一起改了。但如果一直这样的话,建议可以直接share screen和reviewer探讨 这样也更效率 毕竟如果是face to face 也是可以拿电脑过去和对方探讨的
回复

使用道具 举报

全局:
🍌很多经理还看comments的数量呢。我在🍌的时候听组员吐槽升职被打回来了是因为code review给别人的comments少。
一般没有规范代码经验的intern或者新人进来,开始还很喜欢发一个大CR,这种CR基本随便每人都能留20-30条comments 导致 10来个revision。建议做下task break down, 然后那个cr小一点。
当然很多人一次不看完,每次提新的comments确实烦人。
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 注册账号
隐私提醒:
  • ☑ 禁止发布广告,拉群,贴个人联系方式:找人请去🔗同学同事飞友,拉群请去🔗拉群结伴,广告请去🔗跳蚤市场,和 🔗租房广告|找室友
  • ☑ 论坛内容在发帖 30 分钟内可以编辑,过后则不能删帖。为防止被骚扰甚至人肉,不要公开留微信等联系方式,如有需求请以论坛私信方式发送。
  • ☑ 干货版块可免费使用 🔗超级匿名:面经(美国面经、中国面经、数科面经、PM面经),抖包袱(美国、中国)和录取汇报、定位选校版
  • ☑ 查阅全站 🔗各种匿名方法

本版积分规则

>
快速回复 返回顶部 返回列表