楼主: desperatedman
跳转到指定楼层
上一主题 下一主题
收起左侧

[职场感言] flag的代码质量怎么样啊?

 
🔗
Jing666 2020-7-11 12:21:15 | 只看该作者
全局:
在g,逛过的codebase都比较规范,越多人用的那种越规范,如果是很specific product的话至少会写到组内人看得懂的地步。我读过的基本都是,public function有比较详细的comment,function内部不太有,除非要解释一些特殊case,但是并不会让人觉得读不懂,可能是naming比较好

感觉一般SWE新人进公司最开始的几个changelist都要被爆锤一下。之前我和我TLM给我们project刚来的一个L4小哥留了几十个comment。。。。我自己也是刚进去的时候每个CL都要被TL们爆锤。不过可能org/组的关系,comment的语气都很友好,不会让你觉得自己useless怎么这都写不好

评分

参与人数 1大米 +1 收起 理由
littledonkey15 + 1 赞一个

查看全部评分

回复

使用道具 举报

🔗
457368837 2020-7-11 12:31:34 | 只看该作者
全局:
flymonkey987 发表于 2020-7-11 00:09
All code is garbage - Tech Lead
anything we talked here is not a joke ~
回复

使用道具 举报

🔗
goldfish01 2020-7-11 12:34:43 | 只看该作者
全局:
狗家的代码质量一般来说是最好的,FB也不差。不过组和组的差异还是很大的。

评分

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

查看全部评分

回复

使用道具 举报

🔗
fatfatjoey 2020-7-11 12:54:14 | 只看该作者
全局:
悟空QLOR 发表于 2020-7-11 08:02
为什么不每年定期更新一下旧代码呢?
. 1point 3 acres
不坏不修啊,跑得好好的,去refactor一下然后坏了怎么办?别说refactor了,我连dependency版本都不敢动,动一动就得测试很久。一些开源项目都不严格遵守semantic versioning,改个小版本号都有可能出问题。
回复

使用道具 举报

🔗
fatfatjoey 2020-7-11 12:55:35 | 只看该作者
全局:
384john 发表于 2020-7-11 10:54
再加一手“(as a millionaire)”,你就能绝杀!

(as an ex-google, ex-facebook, multi-millionaire, techlead)
回复

使用道具 举报

全局:
悟空QLOR 发表于 2020-7-11 08:02
为什么不每年定期更新一下旧代码呢?
.--
就我目前这个组codebase来说吧(非flag),随着需要/客户/市场 改变,codebase有些core的东西已经不适用了。但是牵扯的东西太多,大家也求稳所以都是遇见问题再改(改了不是完全改掉core的东西,加上平常也着急写新东西,没有太多时间来全职做refactor)
直到某天发现legacy实在太重,就开始着手重新写一套系统。可是新系统不仅写需要时间,还得慢慢测慢慢rampup。所以没个一两年换不掉老系统。.--
对于组里其他人来说,既然目前的系统未来要被淘汰,大家更没激情去redesign and refactor了。
回复

使用道具 举报

🔗
gregregre 2020-7-11 14:18:13 | 只看该作者
全局:
ForrestGump 发表于 2020-7-11 09:17
我也天天用outlook 没发现啥bug。。

你可能比较幸运吧
回复

使用道具 举报

全局:
jordan23back 发表于 2020-07-10 10:14:41.google  и
会刷题只是知道这些算法实现,不代表代码就符合代码规范和容易维护。有些team leade的代码,一个函数几百行,函数名变量名喜欢用极短的名字,让你不看完所有的代码,不知道是干嘛的。维护非常困难
几百行算啥,我司有八千行的函数,17层的嵌套,🤣
回复

使用道具 举报

🔗
amgfan 2020-7-11 15:04:44 | 只看该作者
全局:
cutekitty 发表于 2020-7-11 09:49
哪家公司是金融版微软?

那肯定是蓬勃
回复

使用道具 举报

全局:
狗在这方面确实不错,每天都被同事在style和test方面锤到爆,严重降低throughput
回复

使用道具 举报

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

本版积分规则

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