18

考虑以下类：

class A
{
  A();
  int number;
  void setNumber(int number);
};

您可以通过 3 种方式实现“setNumber”：

方法1：使用'this'指针。

void A::setNumber(int number)
{
  this->number = number;
}

方法 2：使用范围解析运算符。

void A::setNumber(int number)
{
  A::number = number;
}

方法 3：相反，用 'm' 或 '_' 表示所有成员变量（这是我的首选方法）。

void A::setNumber(int number)
{
  mNumber = number;
}

这只是个人喜好，还是选择特定方法有好处？

4

6 回答 6

17

这主要是个人喜好，但让我从一家同时制作许多小游戏的公司内部分享我对这个问题的看法（因此我周围使用了许多编码风格）。

这个链接有几个很好的相关答案：Why use prefixes on member variables in C++ classes

您的选择1：

void A::setNumber(int number)
{
  this->number = number;
}

首先，许多程序员往往觉得这很麻烦，不断地输入'this->'。其次，更重要的是，如果您的任何变量与参数或局部变量共享名称，则旨在更改“数字”名称的查找替换可能会影响位于查找替换范围内的成员变量好。

您的选择2：

void A::setNumber(int number)
{
  A::number = number;
}

我遇到的问题是，在大型类或具有大型函数的类中（你看不到函数或类被意外命名），A::(thing) 的格式看起来非常像访问命名空间的一部分，因此可能会产生误导。另一个问题与前一个选项中的 #2 相同，如果您的名称与您正在使用的任何变量相似，有时可能会出现意外的混淆。

您的选择 3：

void A::setNumber(int number) 
{
  mNumber = number;
}

这是提供的三个选项中最好的。通过创建（并坚持！）包含明确且有意义的前缀的语法，您不仅可以创建本地（或全局）变量不会共享的唯一名称，而且可以立即明确该变量的声明位置，不管你在什么上下文中找到它。我已经看到它既像这样'mVariable'又像这样'm_variable'，它主要取决于你是否更喜欢下划线而不是大写连接。此外，如果您的样式倾向于添加诸如' p '用于指针或' g '用于全局，则这种样式将很好地融合并受到读者的期望。

于 2012-04-17T20:26:38.187 回答
8

这是一个风格问题，因此是个人偏好，或者是与您合作的团队或与您合作的团队的老板的偏好。

于 2012-04-17T19:46:33.723 回答
7

选项 4：

void A::setNumber(int n)
{
  number = n;
}

为什么对成员和参数使用相同的名称有好处。这没有什么好处。当然，现在很清楚了，但是当您的方法变大，并且原型不再适合屏幕，并且有其他开发人员编写代码时，他可能会忘记对成员进行限定。

于 2012-04-17T19:46:19.100 回答
2

这都是个人喜好

但是这里有一个很好的非语言级别的讨论

https://stackoverflow.com/questions/381098/what-naming-convention-do-you-use-for-member-variables

于 2012-04-17T19:49:29.923 回答
2

我想说方法 1 和方法 3 之间的选择是个人或组织风格的问题。

方法 2 较差，因为 Class::member 通常表示静态成员变量，因此如果用于消除参数和成员变量之间的歧义，会引起混淆。

于 2012-04-17T20:24:25.100 回答
0

我将通过提供更好的变体来同意 Luchian。您提供的其他示例过于繁琐或令人困惑。

选项 4：

void A::setNumber(int aNumber)
{
  theNumber = aNumber;
}

这是我们在我的雇主处使用的编码标准的一部分，您所描述的内容非常清楚。这不是匈牙利符号。

于 2012-04-17T20:40:29.800 回答