对所有成员函数和属性使用“this”是不好的做法吗？ [关闭]

Question

最近我在这里发布了一段我的代码并得到了一条评论（与原始问题无关）使用this 处理类的所有成员函数和属性“不是只是个人编码风格的问题，这是不好的做法”。不幸的是，该人拒绝详细说明并告诉我自己查找。

我用过很多谷歌（但很难用“this”作为关键字查找任何东西），并在这里四处寻找，但我只发现someexamplesthis有被使用。

我知道在某些情况下使用this 是不可避免的（同名的参数/变量、模板继承等），但随着时间的推移我开始尽可能使用this，因为我可以找到解决方法我的代码更容易更快。我的理由包括：

• 快速检查函数f是否应该是成员函数：如果代码中没有this，则可以将其从类中取出
• 快速检查f 是否可以是const 函数：如果左侧没有this，很可能可以设为const（并非总是如此，但我发现它在略读时很有用）
• 快速检查对象是否以f 中的“预定义”方式“更改”自身，或者它是否是复合成员函数（使用this 调用的成员方法与在没有这个的对象）
• 调试；即，如果成员属性在任何时候被分配了错误的值，我必须专注于包含 this 的行来查找问题，因为其他行不会更改对象

坦率地说，关于这是“不良做法”的评论让我有些不安。但是，一条评论本身并没有多大意义，所以我想问一下对所有成员函数和属性始终使用this 有什么不好的地方吗？如果那么，是什么主要缺点使它超越了（可能是笨拙、不受欢迎或不普遍）的个人风格，并将其归入“不良做法”类别？

Answer 1

这个答案是基于意见的（正如其他人所指出的那样）。

我认为这是一种不好的做法，因为：

• 它不必要地使代码变大（最容易维护的代码是您不必编写的代码）。
• 这是出乎意料的（虽然您可能会预料到，但其他人不会 - 因此您的代码中的 WTF/SLOC 比率会增加）
• 它增加了维护成本。
• 它需要额外的努力来保持代码的一致性（几乎没有或没有额外的好处）。
• 虽然看起来一致，但它是多余的（类似于使用语法 class <class-name> var; 而非 <class-name> var; 声明所有对象实例，并忽略“零规则”）。
• 它会形成不适合大多数开发团队和编码标准的编码习惯。
• 重命名变量和函数以避免名称冲突比使用this-> 更好的做法（因为您用于类、函数和变量的名称形成了您用来理解代码结构的心理模型)。
• 在不遵循/接受这种做法的代码库中工作几个月后，您可能会发现自己的代码难以阅读/维护（换句话说，一年左右可能会变成纯杂货）。

Answer 2

没有技术原因不能在任何地方使用它。

如果您只对技术原因感兴趣，那就是您的答案。但是，我恳请您考虑非技术原因。意见的形成是有原因的，其中一些原因可能是好的。例如，我建议在任何地方使用它会降低代码的可维护性，而重新考虑命名方案会更好地为您服务。

请考虑通常this 用于需要的地方，而不是其他地方。正如您所说，可能需要this 是有原因的，当大多数程序员遇到this 时，他们会想，“这里一定需要它，因为一个不明显的原因。我想知道那个原因是什么。 "

一致性是可维护代码的一个重要属性。在任何地方使用this 的主要问题之一是它与大多数其他程序员的做法不一致。由于大多数其他程序员不会在任何地方使用this，当你确实在任何地方使用它时，他们将更难维护你的 代码。