帮别人填坑到心累

[kuyuzhiqi]

11 月底换工作，Android 开发，另外一个小伙伴兼职(同时做两份工作)，我全职，进去后发现他的代码很乱，各种耦合，变量名定义不规范，签名文件的密码 123456，我的简历在 git 仓库里面(滑稽)，还一直丢自己的 bug 让我改，这个项目是他开发的，现在是各种小问题不断。

进去的时候耐着性子看完代码，并花了一个星期简单重构了下，想着以后代码结构什么的弄好点，但昨天发生了两件事让我彻底心累了：1.做的 app 没有适配 Android8.0，导致在有的手机上图标显示默认 logo，他和我说是第三方的 sdk logo 覆盖了系统的 logo，将 ic_launcher 换个名字，我的想法是为什么是第三方 sdk 覆盖 app 的呢？不想下这个问题就去换名字？我的做法是创建一个 image asset，最后证明我的是对的； 2.原来的项目中 mimap-xhdpi,mimap-xxhdpi...和 drawable-xhdpi,drawable-xxhdpi...两种文件夹同时在用，我想着这样很混乱，他还一直坚持要采用原来的结构，好不容易得到他的默认许可后，就全部将图片素材移动到 mimap-xxx 类型的文件夹，我是在 develop 分支上开发的，他说我不该提交这么大的改动，我在 develop 分支上提交，又不是 release 分支上提交的，而且这个素材改动在编译期间出现的问题就可以发现，然后在 qq 讨论组和我争论了半天。

以前的代码还出现过在启动的时候有几百次登录请求，在讨论组里面问后台为啥拒绝请求，想想也是挺搞笑的。哎，你代码那么乱，别人帮你重构，不感谢不说，还各种阻挠，公司的人也说让他拍板，我前面也有个 Android 开发在试用期间就走了，我现在还没过试用期。

[petelin]

@UIXX 其实这事都是通用的，看见一个烂摊子管不管的问题，社会上的烂摊子我一点能力没有，所以好不 care （就是无所谓爱国，真能量啥的） 但是公司内部，我还是希望用自己的影响力，去帮助别人（傻逼是不会帮的，我会想办法远离傻逼，找好一点的或者跳槽）。我希望我的精力不光用在提升自己，也可以用在提升帮助他人上。而且后一条是我司已经我对领导的定义。

[petelin]

我司以及我对一个合格的 leader 的定义

[kuyuzhiqi]

今天早上发生了更搞笑的事情，两个分支合并后，一个类他说找不到，然后因为这争论起来，在领导所在的讨论组里面骂我，我就受不了了直接回家准备辞了，结果他自己拉了下代码找到了那个类，在讨论组疯狂打脸。

[UIXX]

@petelin
不冲突。事实上这不是一件事情。我的观点是，在填坑的过程中你发现 bug，可以上报跟修复。但是对于一些正常运行的功能，如果仅仅是“我认为 /觉得”这样做不好而产生了修改的想法，这是不恰当的，缺乏谨慎的。当想去验证的时候，常常发现验证与测试成本是远超过所谓的提升成本...

这个东西其实没有什么管理上面的意义，纯属个人的经验，我做过很多年的协作开发，也做过很多年的独立开发，偶尔也做做甲方的外包，对于技术上的一些小执拗（是不是要坚持去修改别人的代码）我不认为是“对”或者“错”，但我更倾向于符合生活智慧的选择。

[UIXX]

@kuyuzhiqi 小老弟，别冲动。不妨跟你那个同事还有领导沟通一下，实在沟通不了再走也不迟。

[kuyuzhiqi]

@UIXX 人家把话说死了， 没法圆了呀

[petelin]

@UIXX 明白，你说的场景更多。我也不会轻易改别人的代码的。我说的更多的是我自己 owned 的代码，整个逻辑我最清楚那种，在这个基础上会对别人的代码就行 review，不需要额外的精力去阅读他们的代码。