问题标签 [code-complete]

我想要一个进入我的 TFS 的程序，每周对不同的指标（如代码复杂性、代码行、代码覆盖率和其他内容）进行快照，并将它们显示在图表中。

看到个别程序员的东西也很酷，尽管我并不担心这一点，因为我认为在这个级别进行测量实际上对团队有害。

我不可能是第一个想到这一点的人。这种事情存在吗？

我将首先提到似乎具有历史 C# 代码指标的 Sonar。应该还有更多？

所以，我的老板有充分的理由喜欢 Steve McConnell 的书 Code Complete。我想知道是否有人曾经遇到过一种方法来测试是否有人读过这本书。我发誓必须在某个地方进行测验/测试。唉，我找一个人的努力是徒劳的。我们正在启动开发人员资历的游戏化过程。我们提供了可以带来奖金和福利的代码成就列表。其中一项成就是阅读 Code Complete。我个人的成就之一是找到一种确定的方法来测试是否有人读过它。

这似乎模棱两可，但有一个明确的答案。

TLDR：有人知道关于 Code Complete 的测验或测试吗？如果你不这样做，你会问别人哪个问题来确定他们是否阅读了 Code Complete？

我的一些问题：（“我喜欢这些，但我正在寻找一个明确的问题”）

为了澄清，我不是在寻找总结甚至答案。我只是想要一个预制的测验或一个问题来问某人。非常感谢你们。

我创建了一行，以下列方式将对象附加到列表中

线

会给 pylint W1060 表达式分配任何内容，但是由于我已经在使用 foo 列表来附加值，因此我不需要将列表理解行分配给某些东西。

我的问题更多是关于编程正确性的问题

我应该放弃列表理解而只使用简单的 for 表达式吗？

还是有正确的方法来使用列表理解和分配给任何东西？

回答

谢谢@user2387370、@kindall 和@Martijn Pieters。对于其余的注释，我使用 append 是因为我没有使用 list()，我没有使用 i+x，因为这只是一个简化的示例。

我离开它如下：

我发现代码完整对于未保存的文件不起作用（通过 libclang-c）（但对于保存的文件非常有用）。

代码（不是全部）：

源文件（保存的文件，完全等于未保存的文件内容）：

输出：

有什么想法有什么问题吗？

我希望使用 Aptana Studio/Eclipse 使用 Sencha Touch 2.3 编写应用程序。是否可以在不借助 Sencha Touch Eclipse 插件的官方（付费）版本的情况下以某种方式向 Eclipse 添加代码完成？诸如此类的一些方法建议安装 Spket IDE，然后添加 jsb3 文件。但是 Sencha touch 2.3 不包含 jsb3 文件。在 Sencha cmd 4.0 中，使用命令sencha create jsb创建 jsb3 文件是不可能的。关于如何创建 jsb3 文件或以另一种方式添加代码完成的任何建议？

在最新的代码审查中，有人问我，为什么我将方法名称从 更改GetHDRFrame为GetHdrFrame，而HDR是一个缩写。我很确定 Code Complete 中有这样的建议：在 CamelCase 名称中使用缩写时，将它们视为常规单词。但是我找不到写它的地方。有人可以在 Code Complete 中给我一个确切的短语吗？

与 MS 规则的有用链接有一个类似的问题，但我正在寻找 Code Complete 报价。

我与另一位程序员在如何编写具有大量错误检查的方法方面存在分歧：

-

对我来说， if语句的深度嵌套使得第一个示例难以阅读。
第二个，尽管有三个return ，但更具可读性。

我好奇地检查了 Code Complete 是怎么说的，这让我不太确定如何处理这个问题：

嵌套底部的错误条件堆栈是编写良好的错误处理代码的标志。

但是之后

在四个 if 语句中缩进例程的主体在美学上是丑陋的，尤其是在最里面的 if 语句中有很多代码的情况下。

并考虑使用保护子句，如第二个示例

尽量减少每个例程中的返回次数。当您在底部阅读例程时，您没有意识到它可能会返回到上面的某个位置时，就更难理解例程了。

您如何编写方法的错误检查部分以使其可读且容易出错？

所以我已经确定 for 循环中的 12 应该是一个常量，而不是仅仅使用一个整数，并且 montlysales 应该作为参数传递给函数，以便可以运行检查以查看销售的长度等于月份的整数值，也是十二。我不确定这些是否都是违反原则的原因。我觉得最后一行

是错误的，它真的困扰着我，因为我似乎无法弄清楚为什么它实际上困扰着我，我想我可能已经跳过了别的东西。

谁能帮我验证一下？

How could I implement high-quality routines (mentioned by Steve McConnell, on Code Complete, chapter 7) on some Javascript code? For example, in this case:

This is a very common snippet, but it is passing a function as parameter to another function. In my point of view, are not self-documented (it is not readable) and does not maintain the program abstraction as the book indicates; but is very commmon to see.

tl;dr -- What does the below quoted paragraph mean?

Be suspicious of classes of which there is only one instance. A single instance might indicate that the design confuses objects with classes. Consider whether you could just create an object instead of a new class. Can the variation of the derived class be represented in data rather than as a distinct class? The Singleton pattern is one notable exception to this guideline.

McConnell, Steve (2004-06-09). Code Complete (2nd Edition)

Extended version:

I'm currently reading Code Complete, and I'm having trouble understanding the above mentioned paragraph. For context, it's from Chapter 6 under guidelines for inheritance. At first I thought this was advice against using Singletons, but I was obviously proven wrong by the time I got to the end of the paragraph.

I simply can't grasp what the author is trying to get through my thick skull. For example, I don't know what he means by design confusing objects with classes, nor what that means in the context of having a class with only one instance. Help!