郁闷的两天

sumtec 发布于 2014年03月04日
tinyfool yuzhouwww 等2人欣赏。

我曾经说过,做“架构”最重要的就是平衡一事,比如要注意团队的理解能力和技术的先进性之间的平衡。

昨天老板回滚了我的代码,因为Flex里面有一个Bug,而他也读不懂我的代码。简单讲就是说有很多个DataGrid,这些列有些时候是完全没有任何值的。而这些DataGrid可能会有派生类,这些列有的直接用DataField绑定,有的用labelFunction,还有的自己写了各种Renderer。(没错,Flex……)这么多写得各种奇葩的类(DataGrid和Renderer),我都不想碰,主要是老板说Don't touch it,那好吧算啦那就。

我的解决方案是弄了一个辅助类,所有需要这种自动隐藏无内容列的DataGrid,只需要在onCreation事件上绑这个类的一个公开A方法就好。A方法会自动监听DataGrid的事件,一旦数据发生改变就自动遍历所有的列,自动判断是否应该显示。你看这样挺好吧,只要是个DataGrid就能应用,还不用改那些DataGrid和Renderer,简直就是即插即用。

问题是Flex里面有Bug,导致某些情况下有些元素会挂在上面不消失……昨天正好有事情没时间改,老板就给回滚了。大概是写得太好太容易回滚了吧……

现在你们知道了吧,如果你的团队成员里面有人达不到那个水平,就会各种的退化。(没错,这个Flex代码里面好多的Copy Paste,两个项目里面大量完全相似的代码。还有各种的if else来判断不同品牌下面逻辑如何,简直就是一锅粥。)

然后今天呢,老板站身后问我在干啥,我就耐心解释了一下现在遇到的问题是什么。可惜呢,老板改东西的思路属于标准的散弹枪,根本没时间听你解释:你有问题A啊,那就直接试试这个吧。什么?还有问题B?先不要管他,这个改好了再说。不凑巧今天的问题是A->B->C->A这样的问题,改好一个就会导致另一个出问题(代码奇葩不许笑)。说了半天老板以为我没有听明白,于是就开始给我pair coding了。“呐,你应该这样这样,如此这般这般……你看好了吧!” “嗯嗯(卖萌无耻)” “好,现在我们来改问题B……咦?”

就这样,我坐在边上看他折腾了大概3个小时。从A改到B,然后改到C,然后又回来折腾A。中间还改错了几次,以为改好了一测发现还是错了。后来到1点多了,老板说你去吃饭吧,然后就剩下他默默的坐在我的电脑前面噼里啪啦。

吃完中午饭回来一看,老板说改好咯,我一跑还是错的。然后老板摊手,“我要接女儿了。”跑了。我很淡定的坐下来,花了大概10分钟,改好了。

其实本来如果给我3小时,说不定都足够我想好一个很好的解决方案并重构好了。结果这么一折腾,去你奶奶的,老子随便了。于是各种违反单一职责、高内聚原则的代码piapia一贴,收工。

这也不能怪我啊,老板这三个小时里面代码一顿眼花缭乱的腾挪闪移,比如在一个Data ValueObject里面保存了UI的状态,我怎么能删掉这样的代码这么不给面子呢?那啥,顺着老板思路很重要的。你们晓得了吧?

好了,撒完气了。

共27条回复
晓书生 回复于 2014年03月04日

老板给你钱,你要允许他浪费时间。如果你是老板,可以上来抱怨花钱请来的员工水平不够,但只要Boss给你钱,你不能抱怨Boss的水平和理解力不够。

我现在手上的一个项目,Boss每次来听技术汇报,就是听两句然后说:“我没听懂,这样,我把我的商业模式再给你讲一遍,叭啦叭啦,你看你做的技术跟我这商业模式一样么?一样,那OK,下次我们什么时候再开会汇报?“

yangjie6020 回复于 2014年03月04日

先解决问题 后优化代码

saharabear 回复于 2014年03月04日

1楼 @晓书生 对,就是这么回事。

sumtec 回复于 2014年03月04日

2楼 @yangjie6020 本来想先回1楼的,后来觉得还是你这个比较简单。

你这句话没错,但不是在这个场景下面的,而且还少了最重要的几个字:正确的。

比如说,你会接受很迅速的把原来的三层架构搞成一锅粥,里面还错错得恰好对了的改正方式?或者Copy Paste代码?或者说你允许UI层的状态放到数据访问层里面解决,尽管看起来好像解决了问题?这些看起来快的方式,回头就是巨额负债。现在代码长这样,就是这种思路的结果。

只要代码改完之后“能够跑”,没事回过头来重新Review并改正的几率永远小于100%。通常抱有这种想法的队伍,通常几率不超过千分之一。既有纪律问题,也有猪队友问题。总之你要期望他不这么烂下去,是很难的一件事情。

我以前带队伍的时候,对于胡乱改对总是采取高压政策,根本就不会给你什么耐心——你丫这是在搞破坏知道不?而对于认真改错了,反而会尽可能讲解里面的细节,一起分析。(可惜是,大多数人的改Bug方式就是上面那种典型的散弹枪,随便轰一把中了就走人。)

好多次了,我都能预见有些做法后面会有问题,就不听,后来果然就问题一堆。这算什么呢?这种事情和过度优化、或早优化根本就不是一件事情。

你们看看Linus是如何处理和评价类似的事情的。

补充说明:严格来说,回滚我的代码没有任何问题。因为这只不过把之前解决的问题“不解决了”,但至少并没有产生新的问题。然而我要吐槽的重点并不是这个,这纯属被带歪楼了。我吐槽的点是:你不要以为一个好的设计就是好的,好的设计可能并不会被理解,然后因为不被理解而回滚,拆掉。这一次老板显然没明白这个东西是如何工作的,他认为那是我的代码里面的Bug,然后blahblahblah说了一堆为什么。我一听根本就不是那么一回事,解释了一下发现听不懂。于是算了就当他是对的好了。

清醒疯子 回复于 2014年03月04日

就像@sumtec 说的,凑合乱改就是找死。因为后面花多太多的时间来平衡以前犯下的错误。这个时候,知道选择的代价和优势很重要。不是说去平衡两者,而是知道偷鸡就是得不了便宜,不是现在死就是以后死。然后毫无例外地选择,现在就排清问题。

virushuo 回复于 2014年03月04日

你们老板还挺喜欢写代码的啊,哈哈。这是他的指责还是他的爱好还是他逞能?求细节,可以做下一期程序员漫画的题材了。

sumtec 回复于 2014年03月04日

1楼 @晓书生 你说的没错,所以我就卖萌啊。你那个老板说:我按我的说一遍,你听懂了吗?听懂了,好你实现吧。这跟我的故事稍微是有点不同的。

我说的第一个故事是:来雕个镂空玉器吧。好,我雕。雕完了那块玉内里有块瑕疵,老板内心:卧槽这么复杂我不会唉,算了砸了做个玉坠吧。

我说的第二个故事是:老板说了一遍他的解决思路,可是按他的来肯定不对啊。他亲自要上,那我就只好让他来了。

嗯,这第二个故事让我想起了以前亲历的一次项目经理和开发人员之间的争执:程序员A和产品人员B正在就一个问题进行讨论,产品经理C突然跑过来吼了一句说这里就应该怎样怎样这有什么好讨论的,你要做得了就做,做不了就换人。结果A也怒了:这TM不就是已经按照你的要求实现了所以现在才出了问题吗,不就是正在和你们部门的人讨论怎么擦屁股么?当时你说要这么做我也告诉你不行会有这样那样的问题,你不听那我有什么办法,就按你的来做咯。偏偏你的文档又没有对这些问题该怎么解决的说明,你现在还不让讨论了,你找别人做吧我不做了。关于产品经理的问题,以后(待会儿?)有时间我还会吐吐槽。

其实这种问题本身就是一个艺术活,根本就不是什么技术问题:明明是错误的想法,要么选择花点时间解释清楚然后用正确的方法做;或者如果领导只管结或者不验证过程,那就表面答应暗地里按正确的办法做;或者领导直接动手搞细节却不搞清楚、搞不清楚,那你只好等着他撞冰山,然后再按照正确的办法做。当然了,你可以选择明摆着发飚正面冲突,或者选择领导英明神武。

其实我也就是事后在这里发个牢骚,没想到还是……

还有,不是是个老板就是发工资的那个,找你那样想的话,以前那50来人大概都是我花钱雇的了。我真没这么多闲钱……

sumtec 回复于 2014年03月04日

6楼 @virushuo 老板嘛,广义上讲可以指你的直接上级。至于这个直接上级上面还有多少个屁股那就是另外一回事了对不?所以,我老板要写代码其实是很正常的事情。现在我小兵一枚,呵呵呵。有时候这种花15%的脑力干完活可以上上网的生活有多美好,你们是体会不到的。

thankwsx 回复于 2014年03月04日

求教如何做上老板?(不一定要发工资)

晓书生 回复于 2014年03月04日

7楼 @sumtec 哈哈,我也是刚开完项目碰头会,出来吐槽的。反正咱是小兵,就需要自娱自乐的心态。

minddriven 回复于 2014年03月04日

微软就是想让脑残的人也能用,结果大大降低了入门门槛,导致微软界比JAVA界程序猿水平还低。

看来,在微软的世界里头,遇到脑残码农的几率会很高。

sumtec 回复于 2014年03月04日

11楼 @minddriven 喂!我说的是flex啊…flex简直就是脑残体。

aminic 回复于 2014年03月04日

脑残csharp码农爬过...

damon 回复于 2014年03月05日

我在想,带下面的人是不是要高压一下,要不随意一大,就容易散弹?

huang9012 回复于 2014年03月05日

你们老板还主动敲代码,我们老板简直就是神码一通乱喷:就这样了,应该不难,其实有时候也不难。但他好像就没碰到他认为难的东西,但一到处理问题的时候,就歇菜了

tinyfool 回复于 2014年03月05日

15楼 @huang9012

8楼 @sumtec

所以我才是最好的,不懂的代码绝对不碰 @sycx 老师可以证明

terryso 回复于 2014年03月05日

11楼 @minddriven 这位同学,人家讲flex,你来黑微软?难道你的女朋友被.NET 工程师抢了?

tinyfool 回复于 2014年03月05日

17楼 @terryso 注意口气

minddriven 回复于 2014年03月05日

17楼 @terryso

12楼 @sumtec

哈,我记错了。以为是 sliverlight。

IMAGICE 回复于 2014年03月05日

19楼 @minddriven 这么说做c++的怎么看你。其实语言越简单越好,简单到感觉不到它的存在最好,这样会把更多精力转到业务方面去。

cnsoft 回复于 2014年03月05日

修bug 得治根 还是粉饰太平 区别很大的说... 有空我也开个帖子. 说说以往看到的修bug的事.

minddriven 回复于 2014年03月05日

20楼 @IMAGICE

  1. 在微软界遇到脑残的程序员的几率大,并不代表微软界每个程序员都是脑残。
  2. C++ 是微软界的么?

我的主要偏见是源于这个神文章

iamaflyingpig 回复于 2014年03月05日

20楼 @IMAGICE 那样程序员的价值越来越小了;不过语言就是这个趋势,越来越趋向于自然语言

IMAGICE 回复于 2014年03月05日

23楼 @iamaflyingpig 程序员的价值不在于使用某种语言,而是解决问题的能力,然后随便翻译成哪种语言都行,哪种语言合适就用哪种语言,只有初学者才有语言偏见,还属于比较低的层次。

sumtec 回复于 2014年03月06日

22楼 @minddriven 这种东西我都懒得反驳。典型贴标签思维。

sumtec 回复于 2014年03月06日

24楼 @IMAGICE 用什么语言的人会是什么样的是一种偏见,语言无差异其实也是一种偏见。

kxdd2002 回复于 2014年03月06日

很同情楼主,但合作者的能力确实也是架构的一部分,很多事确实折腾不起来。

本帖有27个回复,因为您没有注册或者登录本站,所以,只能看到本帖的10条回复。如果想看到全部回复,请注册或者登录本站。

登录 或者 注册
相关帖子

[顶 楼]
|
|
[底 楼]
|
|
[首 页]