|
锁定老帖子 主题：让人头疼的新手
精华帖 (0) :: 良好帖 (0) :: 隐藏帖 (0)
|
|
|---|---|
| 作者 | 正文 |
|
发表时间：2008-05-20
daoger 写道 刚进公司没多久时，领导让我带两个新人(07年7月份毕业的)。他们两个是试用期3个月都过了之后才参与到我们现在的项目中来的，算起来他们也工作了快一年了，可是今天无意中修改一个他们写过的一个类文件时，看到他们写的一些代码，让我又好气又好笑。可能也是我以前只注意给他们测试功能了，没怎么注意代码方面的问题吧！
晒晒他们的部分代码！ 我现在也体会到当初我找工作的时候为什么那么难了，同时也提醒一下即将毕业的朋友，想找到不错的工作，你的工作态度确实得端正！ 以前没带过新人，大家也介绍一下，带新人的经验吧！ 我需要说明一下，我发贴的目的是想提醒一下临近毕业的朋友，不是公司不需要应届毕业生，应届毕业生有优点也有缺点。但是你进入公司以后，自己的发展和进步还是要靠自己的努力。工作了半年还能写出这样令人佩服的代码，那就只能是你自己的问题了，当然也有我自己的责任，毕竟是我带他们。 有则改之，无则加勉，别无他意，仅此而已！ 只能说明两点： 1. 你们没有能力培养起程序员 2. 你们的项目管理混乱 在我们项目组，新人的招收是很严格的，但是招收进来， 就会给予很多的培养和指导。 一个新人，前几个月的试用，会有专人给予全方位的指导，如果有代码风格，代码质量的问题，很快就能够发现，并给他指出来， 新人会很快能够改进。 不至于会有人说， 某某写的代码有问题一年以后才发现。 难道你们没有code review 机制？ 我们的项目中，每一行代码的提交，都会有code review， 这种可笑的代码质量问题很难在cvs 代码中找到。 每个project的build 出去以后会做design 的review， 提出design的改进方法， review 的过程， 也是对整个团队提高的过程。 建议楼主也从自己项目组的管理上找找问题。 |
| 返回顶楼 | |
|
发表时间：2008-05-20
唉.看了下,自己代码也没新手高多少
只不过不会出现逻辑错误而已 |
| 返回顶楼 | |
|
发表时间：2008-05-23
我们TEAM里面也这样的同事, 受不了, 尽做些不必要的判断.
<代码大全2>一本介绍CODE STYLE的书, 对新手很有帮助 |
| 返回顶楼 | |
|
发表时间：2008-05-27
我也经常写这样的代码。
工作两年多了，最近才发现，这么写法真的是害人害己。 |
| 返回顶楼 | |
|
发表时间：2008-05-30
终于看完了。受益颇深。
07年毕业的人还不少哦。呵呵。 |
| 返回顶楼 | |
|
发表时间：2008-06-07
我觉得那两个人应该是受了误导，可能在试用期间看了公司的“规范代码”，他也许是想验证传入的参数，但又没加异常处理，呵呵，这事只能怨公司
|
| 返回顶楼 | |
|
发表时间：2008-06-25
mylifestyle1225 写道 armorking 写道 mylifestyle1225 写道 不过还是谢谢指点,我会改成英文的.然后把名字改得尽量意义明确些.
如果不是对美外包的项目的话，为什么非要用英语写注释 恩，是不是对美外包的,这是个金融报文的解析和生成,所以自然没什么SQL,JDBC,SPRING等东东了. 有道理,注释的作用本来就是为了让人看懂. 你觉得以下怎么样? // وقع الكتاب المقدس |
| 返回顶楼 | |
|
发表时间：2008-06-25
我们也遇到过这样的问题，想想解决的办法：
一，起步的时候，一定要有模板，当然如果能采用如继承的方式更好，像SS3那样，SCUD都继承。当然，于他们的提高有限，但可以提高代码质量，也有利于开发。 二，这也是经验问题，比如“魔术数字”问题，新手不太明白，得让他知道好处。LJ的例子里也有，为什么不写成'xxx='+id的方式，因为有种可能性被SQL注入。得让新手明白为什么。当然这个工作应该新手自己完成，但本着带新人的原则，应该适当指导一下。 三，工具，人都不可靠，机器好些。我最近在研究checkstyle，由机器保证代码。 四，代码生成，也见过类似的东东，虽然有些灵活性上的问题，但一般普通地方还是可以的，新手就做点简单的工作吧。 当然，我碰到过超XX的，让她看一下别人的代码写自己的，结果连包名类名都一样，把别人的给覆盖了，她自己的过了，别人不灵了还不知为什么。 |
| 返回顶楼 | |
|
发表时间：2008-06-25
我们也遇到过这样的问题，想想解决的办法：
一，起步的时候，一定要有模板，当然如果能采用如继承的方式更好，像SS3那样，SCUD都继承。当然，于他们的提高有限，但可以提高代码质量，也有利于开发。 二，这也是经验问题，比如“魔术数字”问题，新手不太明白，得让他知道好处。LJ的例子里也有，为什么不写成'xxx='+id的方式，因为有种可能性被SQL注入。得让新手明白为什么。当然这个工作应该新手自己完成，但本着带新人的原则，应该适当指导一下。 三，工具，人都不可靠，机器好些。我最近在研究checkstyle，由机器保证代码。 四，代码生成，也见过类似的东东，虽然有些灵活性上的问题，但一般普通地方还是可以的，新手就做点简单的工作吧。 当然，我碰到过超XX的，让她看一下别人的代码写自己的，结果连包名类名都一样，把别人的给覆盖了，她自己的过了，别人不灵了还不知为什么。 |
| 返回顶楼 | |
|
发表时间：2008-07-09
JAVA基础太差了!
|
| 返回顶楼 | |
