参数名称与成员变量名称相同是不是很糟糕？ [复制]

Question

例如，这是以下任何一种吗

• 不好的做法
• 不可读
• 效率低下（对this指针的调用）
• 这样做不好的任何其他原因

.

class Person {

    public:
        string name;
        Person(string name) {
            this->name = name;
        }

};

附言

Person(string name) : name(name) { }怎么样

Answer 1

不，我不认为这是一个不好的方法。有时我们甚至会面对来自不同库的相同方法名或属性名。这就是我们创建命名空间和类来解决命名冲突的原因。

只要不会造成混乱，就应该尽量简单。即使他们使用相同的名称。但是，您不应该混合使用它们，例如：

class Person {
public:
    Person(name) {
        this->name = name;
        name = clean(this->name);
        this->name = prefix + name;
    }

private:
    string name;
};

保持清洁：

class Person {
public:
    Person(name) {
        name = clean(name);
        name = prefix + name;

        this->name = name;
    }

private:
    string name;
};

Answer 2

我能想到的唯一问题（不是真正的问题）是您无法将member variable 与local variable 或function parameter 区分开来。这只是编码风格，与效率无关，但是当您谈论 Unreadable 时，对我来说是的。

对我来说，我通常使用尾随下划线命名类成员变量。它有助于代码的可读性并使其更易于维护。

class Person {    
    public:
        string name_;                // member variable with traling `_`
        string m_surname;            // some microsoft style declares member start with `m_`
        Person(const string& name)   // pass parameter by reference. 
        : name_(name)                // you know you are constructing member name_ with name variable
        {
        }

};