获得徽章 0
- code review 究竟如何是好？
1、容量。
贺师俊在这个回答里
www.zhihu.com 提到一点：每个 PR 强制要求改动行数小于100行，新人要求小于60行，以保证 code review 的实际可操作性。因为 PR 太大，review 和 reject 的成本都比较高，就容易倾向于放水。
2、时机。
不能赶在上线前提 PR，否则提出的问题改还是不改呢？改了要不要重新测呢？reviewer 一看要上线，也更容易倾向于“通情达理”直接 approve。所以至少应该在提测前提一次 PR。
3、匿名？
并非所有人都能完全做到公私分明、就事论事。你要不要指出比你职级更高的人的问题？某个问题改良的成本很高，不改的话尽管不好但也能用，那还要不要提？
4、范围。
为控制时间成本，并非所有代码都需要 review，就前端而言，通常只需要 review js，那么具体 review js 的什么呢？你并不知道背后的业务逻辑，那到底 review 啥？一般来说需要关注：函数的设计、有没有重复的轮子、某处逻辑完全不知所云是否增强语义或补充注释等。
5、开放？
如果公司有一定规模的话，除了组内的人 review，要不要开放给组外的人？code review 并不只是找问题，也是技术交流、学习提高的一种方式。我个人感觉这个很大程度上取决于团队整体的技术氛围，国内应该没有几家能做到吧
。 展开评论19 - 不知不觉带团队已经快两年了，很多之前工作的思路也在渐渐发生转变。先分享几条个人觉得比较有意思的给大家体会下：
之前：
1. 技术牛逼就最牛逼；
2. 所以一定要做团队里技术最牛逼的人；
3. 少说话，多做事，付出总会有回报；
4. 今年我要学 XXX，然后做点牛逼的开源项目！写点牛逼的博客！
5. 我写的这个东西大家都觉得很好用！
6. 这次重构太牛逼了，我自己加班也要把他弄好！
7. 怎么最近业务越来越多了，没时间学技术了啊，听说头条技术氛围不错，去看看？
之后：
1. 技术牛逼还是牛逼，但是还能在业务中落地就更牛逼了；
2. 怎么样才能让小伙伴们技术更牛逼？让他们做的牛逼的东西有更多人用？对业务产生的价值最大化？
3. 产品你们的 PRD 写得也太烂了，这样的需求我们没法做啊！不好意思测试小姐姐，我们后面一定加强自测！
4. XXX 不错，让大家有时间都学学，然后一起搞点事情！
5. 你写的这个东西真的很不错啊！不过一定要先把日志打起来，晋升的时候就有数据了！
6. 你这次重构要花多少人日，有没有更高优先级的技术任务？
7. 怎么最近业务越来越多了，小伙伴们好像都有点情绪，得想办法给他们争取点时间做技术。展开20108 - 淘宝用户增长前端团队招人呀，简历直接投递主管邮箱！！！
团队概要：
负责淘宝用户增长核心渠道的建设以及对不同渠道效果的评估，以实验的方式快速验证想法，达成最有效的渠道拉新。
同时我们也在建设用户增长基础设施，通过对数据分析工具来提升运行效率
岗位职责描述：
• 负责淘宝主要用户拉新与留存渠道的建设。
• 负责分析不同拉新渠道的效果，设计实验，通过实验的方式进行改进。
• 优化各个拉新渠道的开发建设效率，不断优化产品C端的性能和用户体验。
• 负责建设用户增长数据洞察相关平台与基础设施（包含运营平台、开发工具等）
• 用户增长相关图表组件平台研发
岗位要求：
• 熟练运用 React (Rax Weex) / Vue 等技术栈，有实战经验(基础)
• 有 数据分析工具/Canvas图表/Electron/TypeScript/Node.js/小程序 等开发使用经验可作为加分项
• 积极主动，能扛事儿，沟通能力好，能够与各种背景的同学合作。
工作地点：
杭州西溪园区
简历投递:
qiaofu@taobao.com
备注下来自掘金哈~展开1819 - 10 门数据结构，算法和编程精品课程，足以破解任何编码面试224
www.zhihu.com