Tôi có nên sử dụng cùng một tên cho biến thành viên và tham số hàm trong C++

Question

Tôi tự hỏi nếu thực hành tốt là sử dụng cùng tên cho cả biến thành viên và tham số hàm trong C++. Tôi đến từ một nền Java, nơi mà điều này là phổ biến. Tôi tự hỏi nếu trong C++ có nhược điểm cách làm như sau (mã công trình):

class Player 
{ 
    public: 
    void setState(PlayerState *state) 
    { 
     this->state = state; 
    } 

    private: 
     PlayerState *state; 
} 

Cảm ơn bạn đã trả lời. Theo tôi được biết trong khi nó hoạt động, một thực hành tốt hơn là nên đặt một số loại đánh dấu để phân biệt biến thành viên từ các thông số chức năng như:

_ or m_ 

Trong một số biên tập viên (như Qt Designer), các biến thành viên là các chương trình trong một khác nhau màu. Đây là lý do tại sao nó không có vẻ cần thiết để thêm bất kỳ tiền tố.

Answer 1

Điều đó đúng và được phép theo tiêu chuẩn. Nhưng cách tiếp cận tốt hơn là sử dụng một số quy ước đặt tên cho các biến thành viên. Ví dụ: bạn có thể sử dụng tiền tố m_ cho tất cả các biến thành viên, sau đó bất kỳ ai cũng có thể suy ra những gì m_state là. Nó làm tăng khả năng đọc mã và tránh những lỗi phổ biến.

Ngoài ra, nếu m_state là thành viên, thì bạn không phải viết this->m_state = state trong chức năng thành viên, bạn chỉ có thể viết m_state = state. Trong mã hiện tại của bạn, phần this-> trở nên cần thiết, mà không cần state = state sẽ tự chuyển nhượng.

Answer 2

Tôi khuyên bạn nên tuân theo một số quy ước kiểu mã hóa. Cá nhân tôi sử dụng:

class Player 
{ 
    public: 
    void setState(PlayerState *state) 
    { 
     _state = state; 
    } 

    private: 
     PlayerState* _state; 
} 

Answer 3

Thông thường mọi người chỉ đặt dấu gạch dưới sau biến hoặc sử dụng tên biến đổi ít mô tả ngắn hơn cho tham số hàm.

Cá nhân tôi không thích điều trùng tên vì khi đọc nó, rất dễ mắc lỗi.

Answer 4

Thật tốt, trên thực tế, nó thậm chí có thể là hình thức tốt, miễn là nó chỉ có trong nhà xây dựng của bạn.

Answer 5

Không có bất kỳ sự khác biệt nào giữa C++ và java, nhược điểm duy nhất là bạn phải gõ this-> state = state thay vì state = arg. Nhưng mã của bạn là hoàn toàn chấp nhận được, nó có nhiều kiểu dáng hơn bất cứ thứ gì khác.

Answer 6

Hãy làm theo cách này:

class Player 
{ 
    public: 
    void setState(PlayerState *state) 
    { 
     this->m_state = state; 
    } 

    private: 
     PlayerState *m_state; 
} 

Bạn sẽ cảm ơn tôi một thời gian sau này. Haha .. (:

Các "m_" ("Thành viên") tiền tố phân biệt thành viên từ các chức năng và các công cụ khác Rất hữu ích với các công cụ như IntelliSense (hoặc bất kỳ IDE tự động gợi ý khác)

Ngoài ra, dấu ấn.. .. m_state như const nếu bạn không có ý định thay đổi nó sau Chỉ trong trường hợp

Answer 7

Đây là thêm một vấn đề phong cách hơn bất cứ điều gì khác Hầu hết thời gian, không có vấn đề:. state là một cái tên rất nghèo cho một biến hoặc giá trị, dưới dạng biến và giá trị phải là danh từ đủ điều kiện, ví dụ:

void setState(PlayerState* newState) 
{ 
    currentState = newState; 
} 

Về lý thuyết, dù sao đi nữa.Trong thực tế, tôi đã tìm thấy nó hữu ích để sử dụng các tiền tố, dọc theo dòng:

class Player 
{ 
    PlayerState* myState; 
public: 
    void setState(PlayerState* newState) 
    { 
     myState = newState; 
    } 
}; 

Khi đọc mã, nếu tên bắt đầu bằng my, nó rõ ràng là một biến thành viên (và nếu nó bắt đầu với our, nó là một thành viên tĩnh biến).

Note quá rằng trong các nhà xây dựng, bạn có thể làm những việc như:

Player::Player(PlayerState* state) 
    : state(state) 
{ 
} 

Tôi không chắc chắn điều này không cho dễ đọc, tuy nhiên:

Player::Player(PlayerState* initialState) 
    : myState(initialState) 
{ 
} 

trông rõ ràng hơn rất nhiều (nhưng đối với các chủ sở hữu dữ liệu đơn giản, sự khác biệt có thể là không quá quan trọng).

Answer 8

Tôi tìm thấy nó là một lựa chọn tốt để cung cấp cho các biến thành viên cùng tên với tham số khởi tạo hàm tạo. Dưới đây là lý do của tôi:

• giảm số lượng các định danh = giảm sự phức tạp
• bạn không cần phải phát minh ra rất nhiều định danh
• điều tương tự nên có cùng tên nếu có thể, đó là nói một cách logic , Tôi biết tham số! = Thành viên.
  • bối cảnh và các chỉ số có thể cho phép để cung cấp cho cùng tên để điều tương tự
• bạn dễ dàng tìm thấy tài liệu tham khảo (định danh) để điều hợp lý bằng cách tìm kiếm, nếu tất cả các tài liệu tham khảo có cùng tên

Answer 9

Xin lưu ý rằng một số trình biên dịch (vs 2015) có thể tạo ra một cảnh báo nếu một biến đổ bóng khác. Tất nhiên, người ta có thể vô hiệu hóa các loại cảnh báo này. Nhưng tôi nghĩ rằng đó là thực hành tốt để có các kiểm tra được kích hoạt.