a*****n
发帖数: 158 | 1 问题是这样的,,刚到一家新公司,参与一个旧项目的维护,,这个项目是组里另外的一些
CONSULTANT开发的,,现在我们还是一个组,,不过他们开发新的项目...在维护旧的项目
中,,发现有一些问题,,我提出来,,,感觉他们不是太高兴,,不过那些问题太弱质料,,比
如应该是开10个线程的线程池,,处理大概几百个请求,,结果写成生成几百个线程,,但是
只有十个能运行...现在我也参与到新的项目中,,发现CODE写的比较糟糕,,,有些应该设
LOCK,,保证THREAD-SAFE的地方都没有设，，，请问我还应该指出来吗？？ＴＥＡＭＬ
ＥＡＤＥＲ 是他们中的一个。。。。感觉ＭＡＮＡＧＥＲ 也不太上心一样。。。 | b******y
发帖数: 9224 | 2 可以指出来，但需要注重技巧，给人留面子。比如，直接email对方或者直接谈，不要
群发email,搞的对方下不来台。 | G****r
发帖数: 5579 | 3 Just fix problems in the code based on "bug report": if your change fixed
the bug and no regression, then no body can blame you
【在 a*****n 的大作中提到】
: 问题是这样的,,刚到一家新公司,参与一个旧项目的维护,,这个项目是组里另外的一些
: CONSULTANT开发的,,现在我们还是一个组,,不过他们开发新的项目...在维护旧的项目
: 中,,发现有一些问题,,我提出来,,,感觉他们不是太高兴,,不过那些问题太弱质料,,比
: 如应该是开10个线程的线程池,,处理大概几百个请求,,结果写成生成几百个线程,,但是
: 只有十个能运行...现在我也参与到新的项目中,,发现CODE写的比较糟糕,,,有些应该设
: LOCK,,保证THREAD-SAFE的地方都没有设，，，请问我还应该指出来吗？？ＴＥＡＭＬ
: ＥＡＤＥＲ 是他们中的一个。。。。感觉ＭＡＮＡＧＥＲ 也不太上心一样。。。
| m*****i
发帖数: 2325 | 4 能work 就别管了。还是弄新项目吧。 等你成熟人再说吧。
【在 a*****n 的大作中提到】
: 问题是这样的,,刚到一家新公司,参与一个旧项目的维护,,这个项目是组里另外的一些
: CONSULTANT开发的,,现在我们还是一个组,,不过他们开发新的项目...在维护旧的项目
: 中,,发现有一些问题,,我提出来,,,感觉他们不是太高兴,,不过那些问题太弱质料,,比
: 如应该是开10个线程的线程池,,处理大概几百个请求,,结果写成生成几百个线程,,但是
: 只有十个能运行...现在我也参与到新的项目中,,发现CODE写的比较糟糕,,,有些应该设
: LOCK,,保证THREAD-SAFE的地方都没有设，，，请问我还应该指出来吗？？ＴＥＡＭＬ
: ＥＡＤＥＲ 是他们中的一个。。。。感觉ＭＡＮＡＧＥＲ 也不太上心一样。。。
| C********g
发帖数: 1548 | 5 100% bad idea
【在 G****r 的大作中提到】
: Just fix problems in the code based on "bug report": if your change fixed
: the bug and no regression, then no body can blame you
| C********g
发帖数: 1548 | 6 100% bad idea
【在 G****r 的大作中提到】
: Just fix problems in the code based on "bug report": if your change fixed
: the bug and no regression, then no body can blame you
| k**n
发帖数: 3989 | 7 嗯...搞明白你的目的不应只是fix code. 而是要树立自己的权威. 写个doc 给
teamleader 与你的manager . 他们不懂没关系..code 没fix也没关系. 目的就要让他
们"不明觉利".
地位就慢慢稳固了..
【在 a*****n 的大作中提到】
: 问题是这样的,,刚到一家新公司,参与一个旧项目的维护,,这个项目是组里另外的一些
: CONSULTANT开发的,,现在我们还是一个组,,不过他们开发新的项目...在维护旧的项目
: 中,,发现有一些问题,,我提出来,,,感觉他们不是太高兴,,不过那些问题太弱质料,,比
: 如应该是开10个线程的线程池,,处理大概几百个请求,,结果写成生成几百个线程,,但是
: 只有十个能运行...现在我也参与到新的项目中,,发现CODE写的比较糟糕,,,有些应该设
: LOCK,,保证THREAD-SAFE的地方都没有设，，，请问我还应该指出来吗？？ＴＥＡＭＬ
: ＥＡＤＥＲ 是他们中的一个。。。。感觉ＭＡＮＡＧＥＲ 也不太上心一样。。。
| p**r
发帖数: 5853 | 8 说实话，楼主你刚出道没多久吧
＃1
你觉得不合理的地方可能是最合理的，
因为别人那么做可能是因为其他原因。
客户a和b要做一个一样的功能，
但是客户a付了1万，客户b付了10万。
那凭什么给客户a一样的功能，故意写个loop consuming performance都有可能。
＃2
很多时候项目赶进度，是不可能做到最完美的。
目标只有一个，能用，而不是最好。
＃3
政治斗争部分我就不说了。 | l*********u
发帖数: 19053 | 9 well said
【在 p**r 的大作中提到】
: 说实话，楼主你刚出道没多久吧
: ＃1
: 你觉得不合理的地方可能是最合理的，
: 因为别人那么做可能是因为其他原因。
: 客户a和b要做一个一样的功能，
: 但是客户a付了1万，客户b付了10万。
: 那凭什么给客户a一样的功能，故意写个loop consuming performance都有可能。
: ＃2
: 很多时候项目赶进度，是不可能做到最完美的。
: 目标只有一个，能用，而不是最好。
| a*****n
发帖数: 158 | 10 不是新手,,老油条,,但是这次是第一次换公司碰到这种事情.第一个项目我跟经理说料,
经理让我写个文档,,放在WIKI里面,,而且说得很委婉. 第二个问题是在ＦＩＸ 他们的
ＢＵＧ的时候发现的，，因为很类似，，所以就一起全部ＦＩＸ了，，现在这个新项目
，，我就没有说什么，，看情况吧．．．谢谢．． |
|
