Rob*_*lls 6 language-agnostic feedback
天儿真好,
这关系到我的明星开发商的问题,并就告诉别人,他们正在编写恶意代码的这个问题,但我看这种情况是更具体的.
也就是说,我如何告诉一个"明星"他们对我编写的程序的更改很差并且执行不一致而不仅仅是因为我对某人"玩我的东西"感到恼火?
添加的新功能被明确地遗漏在这个shell脚本的原始版本之外,以使它尽可能简单,直到我们知道我们将在负载下看到的错误.
基本上,我认为尝试第二次猜测所有错误情况是不可能的,事实上可能会让我们在做了大量工作后走上了一条完全错误的道路.
在看到需要添加的内容之后,有人潜入并进行了添加但不幸的是:
我总是尝试从Damien Conway的角度来看待编码"始终编码,好像你的系统将由一个知道你住在哪里的精神病患者维护." 也就是说,我试图让它变得容易,而不是作为我自己才华的广告."这段代码做了什么?" 练习很有趣,最好留给混淆比赛恕我直言.
任何建议都很受欢迎.
干杯,
Jon*_*eet 10
我会诚实地对待它.你不一定需要指出每一个错误的小细节,但是值得你做一些通用点的几个例子.您可能想要在第一次简短反馈中记下您未提及的其他示例,以防他们质疑您的推理.
尽量确保反馈完全是关于代码而不是人.例如:
好:参数验证foo()
似乎与中的不一致bar()
.在foo()
,NullPointerException
如果调用者进入null
,则bar()
抛出a ,而抛出IllegalArgumentException
.
不好:你的论证验证到处都是.你扔NullPointerException
在foo()
,但IllegalArgumentException
在bar()
.请尽量保持一致.
即使有"请",第二种形式是谈论开发人员而不是代码.
当然,在许多情况下你不必担心如此小心,但如果你认为他们会非常敏感,那就值得付出努力.(仔细阅读你所写的内容,如果有书面反馈意见:我在第一个版本中偶然包含了一个"你":)
我发现大多数开发人员(超级明星与否)都非常合理地接受,"不,我没有实现该功能,因为它有问题X." 虽然我很幸运.
从另一个角度来看,我鼓励你在他们的鞋子里考虑一下.我将描述一种"假设的"体验.
要注意的一些事项:
请牢记这一点并与他们交谈.教他们.不需要大喊大叫或小便比赛.请记住,他们并没有故意让你的生活变得困难.