还是不上截图了……就语言描述下吧
背景是这样的,一个创业团队中途走了一个技术人员,我就被临时工了……(因为觉得用的技术还是挺新的,于是才愿意尝试下的)
然后进去后看了几天遗留的代码,大概总结就是:
托管在 bitbucket 的,然后后台代码的 commits log 几乎清一色的 "fix bug"
Android 的项目代码结构及命名就更让我抓狂了, myActivity.java/myActivity_1.java/myActivity_2.java/myActivity_old.java/...
git 感情被用来当网盘备份使了……
然后,因为大家都有各自的事,于是不在一起办公,讨论的话非要我加什么 qq/weixin 群,协作糟糕的很, 很多事情讨论不清,那么多协作交流工具不用……
不知道这种情况,你们周边遇到的多不多,很纳闷……
![]() | 1 Septembers 2015-05-07 20:55:23 +08:00 via Android 建议尽早离开 |
![]() | 2 int64ago OP @Septembers 我只是临时工,学生赚点生活费而已 |
![]() | 3 Septembers 2015-05-07 20:57:51 +08:00 via Android |
![]() | 4 Septembers 2015-05-07 20:58:10 +0800 via Android @int64ago 不能要求使用邮件沟通吗? |
![]() | 5 wy315700 2015-05-07 20:58:26 +08:00 估计那个技术人员就是这么走的。。。 |
![]() | 6 int64ago OP @Septembers 我都是主动发邮件的,但是我不能要求别人啊,毕竟临时工…… |
![]() | 7 evlos 2015-05-07 20:59:12 +08:00 工资 = 精神损失费,如果什么时候你得到的精神损失费无法补偿你的精神损失了,就离开吧。 |
![]() | 8 lifanxi 2015-05-07 20:59:53 +08:00 ![]() > commits log 几乎清一色的 "fix bug" 这种太常见了,我所在的某大厂,照样有“XX研发专家”是这样写commit log的,而且比"fix bug"还不像话,他统一都写“modified by xx”,xx是他的名字。 |
![]() | 9 coney 2015-05-07 20:59:54 +08:00 一般这样的就是自己都发现程序没法往下写了。。赶紧走人。坐等接盘侠接盘。。 |
![]() | 11 int64ago OP @lifanxi 这我在另外的地方也见过,吓鸟,关键是 user.name/gmail 也不设置,都是 root@localhost |
12 jiang42 2015-05-07 21:03:15 +08:00 @int64ago 根据我这几天面试的经验来看,上海地区10家创业公司里大概有两家是靠谱的。。。敏捷开发是这类公司最好的借口。 |
![]() | 15 SeanChense 2015-05-07 21:14:54 +08:00 ![]() 给别人写外包 commit 几乎都是 update 待过的一家创业公司很严谨,commit 写的清清楚楚; 各个 branch 的生存周期; ticket 的任务追踪,每个 dev 的 branch 按照 ticket 的编号来; code review 完毕 branch#xx push 到 testing |
![]() | 16 ryanking8215 2015-05-07 21:17:59 +08:00 我都是写在ChangeLog里,commit只写比较重要的修改。有这样做的吗? |
![]() | 17 remotejob 2015-05-07 21:19:20 +08:00 ![]() 这个说明你们公司面试有问题? |
![]() | 18 viator42 2015-05-07 21:20:10 +08:00 项目一旦急起来有时候规范什么的就顾不上了.git log有时候不写,但取个有意义的变量名这是必须的吧. |
19 em70 2015-05-07 21:28:19 +08:00 @int64ago 你不能拿大公司的管理水平来要求创业公司,创业公司首要目标是生存,开发规范肯定不会是公司特别重视的地方,既然你看到了问题,跟老大提出来,尽量能帮团队规范化就行了. |
![]() | 20 killpanda 2015-05-07 21:31:14 +08:00 via iPhone ![]() 之前的接口 removeDiary 是取消收藏,myDynamic 是我的动态… |
21 siteshen 2015-05-07 21:32:01 +08:00 Slack 要翻墙,微信总体挺好,不过电脑上每天都要登录太麻烦。 |
![]() | 22 staticor 2015-05-07 21:32:16 +08:00 正常的现象, 既然来吐槽 还是应该选对节点 |
![]() | 23 michaelye1988 2015-05-07 21:32:38 +08:00 myActivity.java/myActivity_1.java/myActivity_2.java/myActivity_old.java/... 这样的写法我觉得一个大学生,稍微有点项目经验也不至于这样写吧~ |
![]() | 24 uniquecolesmith 2015-05-07 21:35:14 +08:00 我想知道,如何赚外快,业余贡献代码~~ |
![]() | 25 kxxoling 2015-05-07 21:48:10 +08:00 见过写 fox bug 的。 |
![]() | 26 yangqi 2015-05-07 21:56:30 +08:00 创业公司很正常,这些技术细节开始根本不重要,创业公司开始主要的是客户和市场,为了生存下去 |
![]() | 28 int64ago OP @SeanChense 是你写 "update" 还是看到别人写啊? |
![]() | 30 FradSer 2015-05-07 22:26:40 +08:00 多,太多了…… |
31 kimmykuang 2015-05-07 22:26:55 +08:00 wiki+commit 进行issue track会比较好 |
32 yanjunjie 2015-05-07 22:29:51 +08:00 创业公司,有的确实不规范! |
![]() | 33 shippo7 2015-05-07 23:21:59 +08:00 很多创业团队都是找临时工拼凑,为了便宜,少发点工资。大家来了就是混工资,反正应付完今天的工作就行,实在干不下去就走找下一家。这种团队钱花完了也就散了 |
![]() | 34 zoosucker 2015-05-07 23:30:01 +08:00 这不算什么,我见过写if (1==1) {// 一堆操作的}, 最后出了问题很久才发现,小公司没有人检查你的代码写的好不好,实现功能就行了,boss只要能run,才不管你的代码好不好维护... |
35 jun4rui 2015-05-07 23:32:12 +08:00 我见过电信内部有个代码循环n次就把代码复制n份的…… 我还见过中国某大型数据提供商给了个怎么都不会有结果的SQL语句(WHERE i.ic_num='90011941' AND i.trn_dt>'0001-12-31' AND i.trn_dt<'0001-12-31' ) 见过有到现在还在用SQLServer2000的Oracle认证DBA 见过还在用几年前的tomcat绿色版直接解压从来不关注补丁漏洞也从来不升级的架构师 。 。 。 。 。 |
![]() | 36 SeanChense 2015-05-07 23:35:58 +08:00 via iPhone @int64ago 我自己写的 update |
![]() | 39 donghouhe 2015-05-08 00:07:19 +08:00 via Android ![]() 好久以前我就是用复制粘贴代替循环,写了一个5M的源码文件,我骄傲地告诉自己我写了50万行代码了,我是大牛了! |
![]() | 40 guoer 2015-05-08 00:09:10 +08:00 命名中英混合怎么破 |
![]() | 41 kukat 2015-05-08 00:29:14 +08:00 我见过git commit log是这样的 backup 2015-05-01 backup 2015-05-02 ... backup 2015-05-08 尼玛daily backup做的挺好啊 |
![]() | 43 adspe 2015-05-08 08:00:41 +08:00 via iPad 像是我高中时候的风格。 |
44 xunfeng 2015-05-08 08:25:54 +08:00 哎,更离谱的我都见过。碰到有人把图片命名为 1.png 2.png ..... 初始化控件直接就一个for循环,把循环的index ( 其实他写的是 i,你懂得),格式化成图片的名字 1.png,2.png ....,本来我以为这特效已经很duang辣,没想到有个二级菜单 居然用两层for循环。。。 |
![]() | 45 lucifer9 2015-05-08 08:38:34 +08:00 见过变量命名用a to z,然后aa to zz的么 |
![]() | 46 joshualeung 2015-05-08 09:17:24 +08:00 这是得有多懒才会这样命名! |
![]() | 47 caiya21 2015-05-08 09:24:50 +08:00 via iPhone 做技术时要有追求,看来我司的代码管理还是相当合理的 |
![]() | 48 Phoinikas 2015-05-08 09:37:43 +08:00 一个函数,上半部分使用参数,中间把参数赋给一个局部变量,下半部分又用局部变量了。 一个函数,在堆里分配了内存,初始化,然后赋值给局部变量,然后下面又用局部变量。。。函数结束还没把分配的内存释放。 需要相同或者类似的功能直接把大段源码拷过去。。两个几千行代码的类,至少一半以上的功能相同或相似。 我只想说,不规范的团队接手别人的项目就是个巨大的坑。 |
![]() | 49 init 2015-05-08 09:39:09 +08:00 ![]() |
![]() | 50 recall704 2015-05-08 10:06:17 +08:00 @ryanking8215 有,我就是专门有个文件来存放修改记录,svn 或 git 只简单写写大概信息. |
51 simapple 2015-05-08 10:21:54 +08:00 好吧 我也经常把不知道怎么描述的 commit 写个fixbug |
![]() | 52 yangxiongwei 2015-05-08 10:37:26 +08:00 ![]() ![]() 是时候上这张图了 |
![]() | 53 v3u3i87 2015-05-08 10:52:17 +08:00 坑爹,现在很多的。。 |
![]() | 54 fxxkgw 2015-05-08 11:30:35 +08:00 受不了你们这些矫情仔。。 在不仅仅是世界500强公司,一个函数几千行,茫茫多的ifdef endif来区分几十种不同产品,十多年的代码,经过成百上千的不同水平和习惯的程序员之手,你还敢设想会好成什么样? |
![]() | 55 zythum 2015-05-08 11:34:13 +08:00 commits log 其实我感觉并不重要。代码逻辑清晰。注释或者文档详细就可以了。 |
56 civet 算好了,我的队友commits log根本没作用,全打1 只有我在用sourceTree,其他全在用乌龟git 至于代码质量就别提了 |
![]() | 58 mN71eOOprFyMsnPx 2015-05-08 12:52:25 +08:00 @Phoinikas 哈哈!我也做了别人的接盘侠。然后,发现各种复制的代码,各种功能重复,也就是只要实现功能就是好的。彻底无语。 一眼看懂代码?不要想了,变量名基本都是f、a、b、q等,很多时候再来一个“美丽”的循环,我就疯了。 |
![]() | 59 mN71eOOprFyMsnPx 2015-05-08 12:54:42 +08:00 我也在创业公司。至少,我手上的事情对得起拿的工资和良心。 1天的工作量,我会报3天。老板催,我就说这个必须这么久。一步步搞定,自己掌握进度。 |
![]() | 60 pepsin 2015-05-08 13:08:41 +08:00 说正常的估计都是来自即将倒闭的公司 |
![]() | 61 tension2012 2015-05-08 13:12:29 +08:00 我见过,用mysql,从来不建索引的, 不会用ConcurrentHashMap,只知道synchronized |
![]() | 62 zichen0422 2015-05-08 13:39:43 +08:00 话说我也是这么提交代码的 : 前几行注明是怎么修改 比如修改了什么地方 后两行 就是fix bug 和 modify by xxxx |
![]() | 63 cougar 2015-05-08 13:42:47 +08:00 几年前,很正经的写log,现在在小公司根本连fix bugs都不写的。。。。因为某种程度上根本没想到过有人会接手的,哎,坏毛病 |
![]() | 64 ttph1oc 2015-05-08 14:27:47 +08:00 90%创业单位都是这样,我觉得这也是90%创业单位活不下来的次要原因之一。 |
![]() | 65 dingyaguang117 2015-05-08 14:46:10 +08:00 via iPhone @yangmls 应该是海龟吧 |
![]() | 66 est 2015-05-08 14:50:16 +08:00 ![]() 每当看到有人吐槽各自厂里git用得混乱,我就回一句: 你们见过4kw用户的生产环境系统是在正式服上用root直接开发的10多个人用同一个git帐号提交是“update by xxx”所有修改只有一个master分支最新提交还是1个月前的直到有一天一个逗比用了一个中文branch名字把git web显示工具搞挂了 这种团队吗? |
![]() | 69 cdxem713 2015-05-08 15:32:52 +08:00 刚招来的前端,说是代码写了5年了。变量命名各种拼音缩写,一个class的CSS全写在一行里面,做页面就是Develop tools里面慢慢调,调到可以正常显示就算完成了 |
![]() | 70 jiangzhuo 2015-05-08 16:57:33 +08:00 不要跟它浪生命了 快撤 |
![]() | 71 saybye 2015-05-08 17:04:05 +08:00 MARK 另外问下协作交流工具有哪些呢? |
![]() | 73 kawaiiushio 2015-05-09 00:49:23 +08:00 @saybye trello slack 微信 qq |
![]() | 74 gateswong 2015-05-09 03:17:45 +08:00 via Android 感觉进一个好公司真的重要啊,最重要的是还要有一个好leader 我的实习就教会了我很多好习惯,老大说你commit的时候要make sense,用尽量简短的话语写清楚修改了什么 还有各种coding style要保持一致啊什么的 |
![]() | 76 SeanChense 2015-05-11 13:07:16 +08:00 via iPhone 233333 我今天谈了个外包,目测跟你是同一个公司 |
![]() | 77 int64ago OP @SeanChense 好的 欢迎来,我最经要开始重构代码了,你倒好 IOS还没人动,你可以安心的规范写代码了 |
![]() | 78 SeanChense 2015-05-11 19:41:00 +08:00 @int64ago 还没答应下来,还没谈好呢。主要是不知道要多少钱合适 |
![]() | OP @SeanChense 按月算还是按项目结? |