论添加一行代码需要付出多少努力

157 天前
 roundRobin

需求是需要在某一行计算费用总和的代码中除去某一项费用,就是类似加一行

totalfee -= excluded_fee

这个需求是公司的税务部门提出的,说这个费用必须免除。PM ,TPM ,SDM 们经过一周的开会后确定必须免除,于是让我开始写 LLD 和 LoE 。

然后我花了一天研究这个excluded_fee的是否在任何情况下都等于要求免除的费用,免除的费用是否能够同步到所有的数据库,中间是否存在数据不同步所导致的错误计算的可能。LLD 和 LoE 写好后开始分别找:

  1. 这个 calculator 的 up stream
  2. 这项费用的 down stream
  3. 这个 total fee 的 down stream
  4. 公司会计部门
  5. 公司的会计工程部门(会计相关的 service 开发)
  6. 公司税务部门
  7. 公司的税务工程部门(税务相关的 service 开发)
  8. 组内的 stake holder 们

在两周的时间里,开了无数会,写了无数个 meeting notes ,deployment plan 也因为 prime day 改了无数次,最终才得到所有的 approval 。

昨天一边吃止痛药一边花了一个小时的时间完成了代码修改和测试,接下来,这个 CR 需要再让他们 approva 一遍,才能部署。

这个漫长的过程,到底体现了流程的严谨,还是有太多的冗余步骤呢,过程中自己好像变成了 TPM ,跟各个部门扯皮较劲,心累不已。

9654 次点击
所在节点    程序员
66 条回复
zcf0508
157 天前
代码是为了业务服务的
Vtwoguest
157 天前
yyyy-MM-dd hh:mm:ss ⇨ yyyy-MM-dd 00:00:00
就改这半个变量 得提申请 得做十多个文档 经过 4.5 轮 review 部署也要提申请
noahlias
157 天前
越大的公司越是能发现牵一发动全身😄 流程复杂哈哈
kmyzzy
157 天前
楼主亚麻的吧?大公司都这样。
0xsui
157 天前
@Vtwoguest 这都是抠脚程序员留下的坑。
0xsui
157 天前
@Vtwoguest 这样才能养更多的文职人员
liprais
157 天前
十多年前在创业公司改类似的是快,半天就改完了
结果过了半年税务局来了让补交税说不能都扣除.....
estk
157 天前
楼主谨慎是对的
有些中国老板,看到代码量会觉得这是摸鱼,还好楼主的公司应该不是这种文化
opengps
157 天前
金融业务需要严谨,要不然一行代码毁掉一个公司不是问题,这种案例不是没有。。。
buxudashi
157 天前
下次再说要打个折。涉及到对账,涉及到小数点后两位的精度。接着搞
yhm2046
157 天前
建议改成“在大厂改一行代码需要走多少流程和时间”
powerman
157 天前
不搞这么多流程怎么摸鱼呢,其实国内的程序猿并不讨厌繁琐的流程,讨厌的是繁琐的流程需要消耗太多的时间,但是老板又不给这些时间买单,简而言之这些时间 浪费的都是自己的,当然厌恶的很
hez2010
157 天前
一切现有的制度都是前面的教训总结下来的。
如果没有这么多流程的话,你轻松一行改完 commit 进去出事故了,那恭喜你喜提权责。
之所以有这么多的流程,一方面是为了尽可能把更改提前通知到所有人,另一方面也是为了把责任分散到整个团队,这样出问题了责任不会全都落在你一个人头上。
hez2010
157 天前
@hez2010 这样万一真的出了问题,结论也是:“这个问题非常的隐蔽,尽管我们已经使用了一切手段来避免问题的发生,但是还是发生了,这是没有办法的事情”。
kneo
157 天前
这种东西,能不改最好不改。非要改,最好有界面能给用户加一个选项。没有选项,在展示结果的时候最好说明已扣除某费用。这条说明可以在几年后删掉。

要谁说改就改才是离了大谱。
murmur
157 天前
复用度越高的逻辑越不敢动,宁可加参数,复制一份,也别轻易动
1062740012
157 天前
这个是正确严谨的流程,改代码容易,但是影响的范围,会对业务造成的影响,这些都是要通过 meeting 后才能确定,并且这也是为了规避责任的一部分,否则出事了责任归属就是个问题了。
leighton
157 天前
CR 暴露了楼主蕉厂员工
kratosmy
157 天前
为什么你可以有五台显示器😂
orioleq
157 天前
你们的系统好搞笑,减个费用不是改个配置就行而是要加一代码…你们是个小工具级别的系统么?啥都是写死的?
我觉得增加一项费用减免,从业务角度上需要讨论两周是合理的,技术要在上面耗两周,是不是要先反思下自己系统不是可配置的。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/1057143

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX