如果有这样一个队友……

2014-11-28 15:07:32 +08:00
 wog
python 里面用 id,list,set做变量名,
敢在flask 的before requset函数里循环查询数据库几百次
随手就能再在首页为了取评论数在查几百次
有memcached从来不用,一用就全乱
提交的代码有语法错误
经常用未定义的变量名
你查他的bug,他去刷知乎
50行代码改10多次,共计60多个bug
peewee的查询一定异常处理说过无数次就是不加
还敢把一个应该出现在后台的链接,添加个后台的路径的样子伪装成后台代码
……
出了无数问题你终于忍无可忍跟他说你怎么能这样,他就给你摆脸色,新的代码提交了不告诉你直接上,还又出了问题
而你们公司有没专门的测试,你正好补缺负责这个代码的审查和测试,你的生活一定会很美好 ^_^
11875 次点击
所在节点    程序员
35 条回复
imn1
2014-11-28 15:11:32 +08:00
记录你检查到的错误,把态度问题用中性语句也写上,工作报告时一并提交
victor
2014-11-28 15:15:44 +08:00
经历过类似的问题之后,我们把 git 的管理方式改成了 pull request & merge 的方式。每个开发人员都从自己 fork 的 repo 上面开发。之后 pull request 到 origin repo。由专人负责 review 和 merge。如果有问题,就打回去,记得出问题的地方写好原因。写功能的人,要自行写好测试,多少有一些帮助。
frankzeng
2014-11-28 15:16:38 +08:00
所以说态度决定一切,态度不好写出来的代码更不好了,跟领导反映呗,这种事你一个人扛不下来的。以前写代码的时候也挺烦测试的,我自己运行好好的,到了他那就这个问题那个问题,但人家找出来的bug确实是可以重现的,再说测试不通过就不准上线,还是要老实改bug,这个就是相互制衡的道理。
wog
2014-11-28 15:16:51 +08:00
@imn1 错误太多,天天报告会像打小报告吧,而且我觉得老板可能会烦……
mengzhuo
2014-11-28 15:24:07 +08:00
╮(╯▽╰)╭
在前公司实现了一整套gitlab + gitlab CI 每个master commit跑一次UT,出问题了就直接发邮件到邮箱
我觉得楼主可以实现一套,然后出错邮件直接CC老板
--------------
p.s. 据说我离职后UT再也没用过……
ophunter
2014-11-28 15:51:42 +08:00
其实我每次碰到奇葩的队友的时候就告诉自己,因为自己不行,才会沦落到此,加油
imn1
2014-11-28 16:04:09 +08:00
@wog
如果错误太多可以写成统计数据,例如多少代码,错误多少,占比多少,修改耗时多少
我觉得报告写得尽可能中性,不要加入自己的判断,例如别说什么不行啊之类
如果你是小主管,把其他人的相似统计也写上,我觉得对下属工作评估很正常,不算打小报告
如果你不是主管,就把这些记录拿去给主管看,跟他说自己不再帮别人擦屁股,有错让他自己改
mahone3297
2014-11-28 16:37:56 +08:00
@mengzhuo 哈哈哈,看来大家都不想用。
66beta
2014-11-28 17:00:10 +08:00
@ophunter 大彻大悟!
em70
2014-11-28 17:10:13 +08:00
完全不可忍受,查BUG时候居然看知乎,为什么不看豆瓣
EPr2hh6LADQWqRVH
2014-11-28 17:17:34 +08:00
这是在黑知乎?
yuankui
2014-11-28 17:37:25 +08:00
找主管寻求帮助
qdcanyun
2014-11-28 17:59:32 +08:00
目测楼主没有单元测试和CI。。。。
改用 github flow 的开发方式吧
提 PR 你来做 Code Review,直到没问题了再合并。
Lycnir
2014-11-28 18:04:06 +08:00
放开他 让我来
anjianshi
2014-11-28 18:04:57 +08:00
专业坑队友二十年不动摇的我表示,终于碰到比我还坑的了!
Livid
2014-11-28 18:09:14 +08:00
6 楼的回复让我想给 V2EX 加个赞的功能。
mengzhuo
2014-11-28 18:13:01 +08:00
jakwings
2014-11-28 18:13:28 +08:00
@Livid 其实一直以来都把感谢和赞混合来用……倒不如把感谢改成赞…… -_-;
ffts
2014-11-28 18:20:57 +08:00
柴刀拿好不谢,嗯
nealfeng
2014-11-28 18:22:11 +08:00
@Livid 直接把感谢拿来用就好了。

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

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

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

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

© 2021 V2EX