如何告诉别人他应该提高自己的技能？

Question

您正在与某个项目中的某个人合作，而这个人会编写错误的代码。解雇他并不是那么糟糕你不能使用它， 但是

• 人不遵守编码惯例
• 代码并不总是包含单元测试（应该是）
• 代码很草率，没有评论它应该有它们
• 人们根本不了解设计模式/根本不知道
• 把衣服放在桌子上，就像他需要一些空间一样。

那么如何告诉他需要升级他的技能而不伤害他？我描述了一些问题。但想象一切都很好，除了一些真正伤害项目的事情。做什么让你还是朋友，但是家伙真的明白他需要改变什么？

Answer 1

这是一个很好的候选人，让整个公司进行一个月的代码审查。你不会把他单独出来（这很好，除非所有其他方法都失败了），而你是更好的工程师可以帮助他。

Answer 2

保持专业，并指出错误跟踪，代码审核或用于跟踪工作项和错误的任何方法中的问题。

该人员的经理和项目负责人有责任确定谁负责，并礼貌地通知该人。这是他们的工作，如果他们无法做到这一点，你的项目（可能是公司）可能不是一个很好的工作。

Answer 3

我建议“失败”任何不符合质量标准的代码。例如，如果缺少单元测试，则“失败”代码，以便它返回给开发人员进行纠正。这类似于使代码失败UAT或其他东西，除了它更多的是来自开发团队的团队领导者的内部审查。

举个例子，在我工作的一个地方，我领导着一个开发团队。我们有PHP生产代码，它们会在日志文件中写入不必要的警告，最终导致磁盘空间问题，并且难以调试真正的错误。

因此，为了解决这个问题，我制定了规则：如果您编写的任何代码都会向日志文件写入警告，则发布将无法通过测试，您需要在进一步测试之前修复它

我们几乎没有在下一个版本中记录任何警告。现在，它在技术上不是一个错误，只是一个代码质量问题。关键是有人需要成为在开发生命周期的某个阶段审查和实施质量控制的守门人。

更好的是，编写一些自动集成测试，检查单元测试是否存在，代码是否正确设置样式。有一些工具可以为您检查（不确定它们的名称，其他人可能能够启发我们）。这样，构建就无法进行自动化，可重复的测试。如果发生这种情况我会说，代码会很快被清理干净。

Answer 4

您可以在构建过程中实现代码分析/样式工具，以强制执行某些设计和样式指南。这种方法的好处是不会让任何人单独出来，它适用于每个人。

根据您的项目使用的语言，可能还有工具可以帮您完成。防爆。 C＃项目StyleCop和FxCop。

Answer 5

放松，只是邀请他喝一杯然后扔石头然后向他解释整个情况。第二天早上，如果他仍然能记住，他会理解的。如果不 这次他要脱掉所有的衣服，把它们放在你该死的桌子上：）

Answer 6

• 在他的桌面上匿名留下一些books，例如“代码完成”或“重构”。
• 假设你不是他的领导，请告诉他的主管你的担忧，并让那个人对这个问题负责。
• 编写一些涵盖其代码的单元测试。他可能不知道如何编写好的测试，所以他可以用你的例子。设置定期运行这些测试的自动脚本，并发送详细说明失败的邮件，这样他就不能简单地忽略这些测试的存在。

Answer 7

我们有一个人在工作，每周只洗一次（不，他不是法国人）。在本周末，他的BO非常糟糕。我们在他的桌子上留下了一堆卫生用品，让他试着给他一个线索。没工作。我认为他之后在洗澡之间开始了2周。唷。这是20多年前的事了。

编码惯例？像什么？在哪里放大括号？

缺少评论？好吧，有时没有评论比无偿或错误的评论更好。

邋code的代码？可能是吧。发布一两个例子。也许他是一个优秀的程序员，你并不那么聪明。毕竟，PM通常不那么明亮，他们只知道如何亲吻屁股比我们有些人更关心。

设计模式？废话宾果饲料。

但脱掉衣服脱下桌子很奇怪。 IMHO。