在公司share技能吗？ - 未名空间MITBBS历史存档

L*o2013-03-19 07:03

1 楼

俺新到一家公司，才几个月，写机器控制方面的程序，最近总算是把产品比较熟悉了，
目标平台也建立起来了。
从一个在公司工作了7年的老工程师那里拿到了产品的源码，这一看源码，真是让俺大
吃一惊，这也太水了，大学里认真学习的学生也写得比这代码要好啊。
俺就看了几个文件，就找到了2个有可能发生X/0的错误，至于其他的在函数中定义了局
部变量，调用其他函数后，或者表达式运算后，却没有任何地方用到该变量，等等，就
很多了。
俺稍微做了些性能优化，就节省了1K多字节的Rom size,几十Byte的Ram Size。
俺现在犹豫，要不要教教这个老工程师呢，抑或是他也太不认真了？这个老工程师是
东欧人。话说俺因为他的代码问题太多，虽然事先和他讲了一些代码的问题，俺考虑后
，还是把这个情况向经理婉转的解释了一下，否则，俺担心这个产品俺写了代码以后，
就打上俺的烙印了。
大家遇到类似的情况，会Share一些基本的技能给这样的同事吗？他还比俺高好几个
LEVEL，真是要晕倒了。

c*y2013-03-19 07:03

2 楼

这种就算了，你给他讲他也许反而恨你。

【在 L******o 的大作中提到】

: 俺新到一家公司，才几个月，写机器控制方面的程序，最近总算是把产品比较熟悉了，
: 目标平台也建立起来了。
: 从一个在公司工作了7年的老工程师那里拿到了产品的源码，这一看源码，真是让俺大
: 吃一惊，这也太水了，大学里认真学习的学生也写得比这代码要好啊。
: 俺就看了几个文件，就找到了2个有可能发生X/0的错误，至于其他的在函数中定义了局
: 部变量，调用其他函数后，或者表达式运算后，却没有任何地方用到该变量，等等，就
: 很多了。
: 俺稍微做了些性能优化，就节省了1K多字节的Rom size,几十Byte的Ram Size。
: 俺现在犹豫，要不要教教这个老工程师呢，抑或是他也太不认真了？这个老工程师是
: 东欧人。话说俺因为他的代码问题太多，虽然事先和他讲了一些代码的问题，俺考虑后

L*o2013-03-19 07:03

3 楼

不会闻过则喜的吗？毕竟多知道了一些只知识点啊。
感觉他也是在是太不认真了，自己的代码感觉都从来不推敲演算似的，看代码注释，06
年初就到公司做了，真不知道他怎么混下来的。

【在 c**y 的大作中提到】

: 这种就算了，你给他讲他也许反而恨你。

t*i2013-03-19 07:03

4 楼

这个代码运行的时候出问题吗？
代码漂亮不漂亮是一回事，work 不 work 是另外一回事。

【在 L******o 的大作中提到】

: 俺新到一家公司，才几个月，写机器控制方面的程序，最近总算是把产品比较熟悉了，
: 目标平台也建立起来了。
: 从一个在公司工作了7年的老工程师那里拿到了产品的源码，这一看源码，真是让俺大
: 吃一惊，这也太水了，大学里认真学习的学生也写得比这代码要好啊。
: 俺就看了几个文件，就找到了2个有可能发生X/0的错误，至于其他的在函数中定义了局
: 部变量，调用其他函数后，或者表达式运算后，却没有任何地方用到该变量，等等，就
: 很多了。
: 俺稍微做了些性能优化，就节省了1K多字节的Rom size,几十Byte的Ram Size。
: 俺现在犹豫，要不要教教这个老工程师呢，抑或是他也太不认真了？这个老工程师是
: 东欧人。话说俺因为他的代码问题太多，虽然事先和他讲了一些代码的问题，俺考虑后

L*o2013-03-19 07:03

5 楼

偶尔会出问题，机器运行中，一年会遇到一两次，突然停机，自动重启。这是比较严重
的，还有其他的显示等问题。
都是偶尔出现，估计和该人的代码质量，不认真是有关系的。

【在 t*******i 的大作中提到】

: 这个代码运行的时候出问题吗？
: 代码漂亮不漂亮是一回事，work 不 work 是另外一回事。

l*t2013-03-19 07:03

6 楼

说了这么多老板知道么？

【在 L******o 的大作中提到】

: 俺新到一家公司，才几个月，写机器控制方面的程序，最近总算是把产品比较熟悉了，
: 目标平台也建立起来了。
: 从一个在公司工作了7年的老工程师那里拿到了产品的源码，这一看源码，真是让俺大
: 吃一惊，这也太水了，大学里认真学习的学生也写得比这代码要好啊。
: 俺就看了几个文件，就找到了2个有可能发生X/0的错误，至于其他的在函数中定义了局
: 部变量，调用其他函数后，或者表达式运算后，却没有任何地方用到该变量，等等，就
: 很多了。
: 俺稍微做了些性能优化，就节省了1K多字节的Rom size,几十Byte的Ram Size。
: 俺现在犹豫，要不要教教这个老工程师呢，抑或是他也太不认真了？这个老工程师是
: 东欧人。话说俺因为他的代码问题太多，虽然事先和他讲了一些代码的问题，俺考虑后

l*a2013-03-19 07:03

7 楼

你改写了code,万一有了更多问题怎么办？出了问题你扛得住吗？
要我说，保险起见，先原样用他的code，把该完成的任务完成，让整个code运行起来。
然后再跟你老板说你觉得某些code有提升空间，问要不要另立一project来调查。有的
时候code虽然有问题，但也许在你老板看来你有更重要的别的项目呢？

【在 L******o 的大作中提到】

: 偶尔会出问题，机器运行中，一年会遇到一两次，突然停机，自动重启。这是比较严重
: 的，还有其他的显示等问题。
: 都是偶尔出现，估计和该人的代码质量，不认真是有关系的。

t*n2013-03-19 07:03

8 楼

不用和这个人说，也不要教。你们有bug管理系统吗？如果你能预见bug，或者你复制
bug，直接给老板汇报，同时修复。汇报的时候只说程序有问题，不要提人的名字。

【在 L******o 的大作中提到】

: 俺新到一家公司，才几个月，写机器控制方面的程序，最近总算是把产品比较熟悉了，
: 目标平台也建立起来了。
: 从一个在公司工作了7年的老工程师那里拿到了产品的源码，这一看源码，真是让俺大
: 吃一惊，这也太水了，大学里认真学习的学生也写得比这代码要好啊。
: 俺就看了几个文件，就找到了2个有可能发生X/0的错误，至于其他的在函数中定义了局
: 部变量，调用其他函数后，或者表达式运算后，却没有任何地方用到该变量，等等，就
: 很多了。
: 俺稍微做了些性能优化，就节省了1K多字节的Rom size,几十Byte的Ram Size。
: 俺现在犹豫，要不要教教这个老工程师呢，抑或是他也太不认真了？这个老工程师是
: 东欧人。话说俺因为他的代码问题太多，虽然事先和他讲了一些代码的问题，俺考虑后

l*u2013-03-19 07:03

9 楼

当然不。别出声，等出事了，你当救火员。

【在 L******o 的大作中提到】

: 俺新到一家公司，才几个月，写机器控制方面的程序，最近总算是把产品比较熟悉了，
: 目标平台也建立起来了。
: 从一个在公司工作了7年的老工程师那里拿到了产品的源码，这一看源码，真是让俺大
: 吃一惊，这也太水了，大学里认真学习的学生也写得比这代码要好啊。
: 俺就看了几个文件，就找到了2个有可能发生X/0的错误，至于其他的在函数中定义了局
: 部变量，调用其他函数后，或者表达式运算后，却没有任何地方用到该变量，等等，就
: 很多了。
: 俺稍微做了些性能优化，就节省了1K多字节的Rom size,几十Byte的Ram Size。
: 俺现在犹豫，要不要教教这个老工程师呢，抑或是他也太不认真了？这个老工程师是
: 东欧人。话说俺因为他的代码问题太多，虽然事先和他讲了一些代码的问题，俺考虑后

L*o2013-03-19 07:03

10 楼

决定了，咱也不滥做好人了。尤其是这样的指出别人错误，和改进办法的好人。如果是
个中国人，还可以交流。现在咱只做好自己这块就行了。
只是本人有点洁癖似的，看到垃圾代码，垃圾文件结构，垃圾目录结构，就感到很不舒
服。

t*n2013-03-19 07:03

11 楼

楼主，我理解你。但是公司有个原则，就是东西不坏，一般不改。你看着不舒服，只能
跟老板煽风点火，直到老板同意为止。
让老板同意建个bug管理系统挺重要的。还有你们的代码是否版本控制系统，比如git？

【在 L******o 的大作中提到】

: 决定了，咱也不滥做好人了。尤其是这样的指出别人错误，和改进办法的好人。如果是
: 个中国人，还可以交流。现在咱只做好自己这块就行了。
: 只是本人有点洁癖似的，看到垃圾代码，垃圾文件结构，垃圾目录结构，就感到很不舒
: 服。

k*n2013-03-19 07:03

12 楼

你刚来, 可以问, 别直接改别人程序. 再过几个月你就会庆幸了

【在 t*****n 的大作中提到】

: 楼主，我理解你。但是公司有个原则，就是东西不坏，一般不改。你看着不舒服，只能
: 跟老板煽风点火，直到老板同意为止。
: 让老板同意建个bug管理系统挺重要的。还有你们的代码是否版本控制系统，比如git？