查看: 4386| 回复: 9
跳转到指定楼层
上一主题 下一主题
收起左侧

[职场感言] 吐槽code review

全局:

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

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

x
本帖最后由 lcwyc 于 2018-1-15 01:56 编辑
.1point3acres
您好!
本帖隐藏的内容需要积分高于 110 才可浏览
您当前积分为 0。
使用VIP即刻解锁阅读权限或查看其他获取积分的方式
游客,您好!
本帖隐藏的内容需要积分高于 110 才可浏览
您当前积分为 0。
VIP即刻解锁阅读权限查看其他获取积分的方式
Unlock interview details and practice with AI
Curated Interview Questions from Top Companies


上一篇:被即将离职的同事利用
下一篇:上班族打卡
推荐
韩梅梅 2018-1-17 13:53:53 | 只看该作者
全局:
等你在大项目上做久了就会理解这些细节多重要了。随便throw exception,app动不动crash,保证你接blocker bug接到手软。你不认同可以去跟reviewer聊,只要人家讲道理基本是可以达成共识的。
回复

使用道具 举报

推荐
zhuangtuo 2018-1-15 06:17:16 | 只看该作者
全局:
鸡蛋挑骨头,刷存在感
回复

使用道具 举报

推荐
charleszhou 2018-1-15 06:28:15 | 只看该作者
全局:
其实各行各业都是这样你都有上级或者是客户,都要按照他们的想法设计。他们看到你的成品,或者是他们想一想又改变了主意,都有可能。这种Rework有时候是很恼人的。但楼主加油不要灰心。适当时候也要提出自己的意见想法,适时候的Push Back~

评分

参与人数 2大米 +8 收起 理由
Jiayueyoyo + 3 给你点个赞!
lcwyc + 5 给你点个赞!

查看全部评分

回复

使用道具 举报

🔗
gzxultra 2018-1-14 23:19:28 | 只看该作者
全局:
你这积分要求高于五百...一般人都看不到了
回复

使用道具 举报

🔗
William Zhang 2018-1-14 23:27:17 | 只看该作者
全局:
五百积分???这。。。。。。。。。

评分

参与人数 1大米 +1 收起 理由
Susan.Shen + 1 给你点个赞!

查看全部评分

回复

使用道具 举报

🔗
ivanyang 2018-1-15 00:20:38 | 只看该作者
全局:
刚道理…throw exception那里review的做法是对的…不要乱throw…
回复

使用道具 举报

🔗
joezhou 2018-1-15 04:14:11 | 只看该作者
全局:
1. 这点可以和你的reviewer讨论一下 review 的过程也是你学习的过程 软件开发到处都是trade off
2. 你们组没有coding standards吗?.1point3acres
3. 设计变动多也是软件业的特点所以很多公司都用agile
4. 这需要看个例
总之有什么你觉得不对的 不理解的可以和你的team讨论 或者你的reviewer 讨论有的时候可能是你对呢 你的reviewer没有想到某些情况 如果是你错了的话就是一个很好的学习机会

评分

参与人数 1大米 +5 收起 理由
lcwyc + 5 很有用的信息!

查看全部评分

回复

使用道具 举报

全局:
有人及时看就很好了,我两小时写完的代码被人拖了好几天才看,天天催也不管用
回复

使用道具 举报

🔗
hurtlocker 2018-2-19 19:44:16 | 只看该作者
全局:
这不是给自个儿挖坑么,到时候这个地方crash了你猜谁背锅?
回复

使用道具 举报

您需要登录后才可以回帖 登录 | 注册账号
职场达人
  • ↑ 本版用于讨论职场各种干货话题,闲聊请去🔗聊聊或者🔗匿名版
  • ❌ 本版严禁水贴,引战,发布广告,拉群,贴个人联系方式,扣分无警告
  • ☑ 求职、面经等去 🔗北美求职和 🔗回国求职大区,刷题和学习请去 🔗终身学习大区
  • ☑ 请去专版发布 🔗内推, 🔗招聘信息,和讨论 🔗创业内容
  • ☑ PIP / DevList/ Need Support 等话题也已开设 🔗专版

本版积分规则

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