Mâu thuẫn giữa tên thành viên và tên tham số constructor

Question

thể trùng lặp:
Members vs method arguments access in C++

Tôi có một lớp học có một số thành viên, giống như x, y, width và height. Trong constructor của nó, tôi sẽ không làm điều này:

A::A(int x, int y, int width, int height) 
{ 
    x = x; 
    y = y; 
    width = width; 
    height = height; 
} 

này không thực sự có ý nghĩa và khi biên soạn với g ++ x, y, width và height trở thành giá trị kỳ lạ (ví dụ -1405737648).

Cách tối ưu để giải quyết những xung đột đặt tên này là gì?

Answer 1

Bạn có thể sử dụng danh sách khởi tạo chỉ tốt với những cái tên giống nhau:

A::A(int x, int y, int width, int height) : 
    x(x), 
    y(y), 
    width(width), 
    height(height) 
{ 
} 

Một cách khác là sử dụng tên gọi khác nhau, nếu bạn không muốn có cùng tên. Một số biến thể ký hiệu Hung-ga-ri đến với tâm trí (tôi có thể bị ghét vì điều này):

//data members 
int x_; 
int y_; 
int width_; 
int height_; 
//constructor 
A::A(int x, int y, int width, int height) : 
    x_(x), 
    y_(y), 
    width_(width), 
    height_(height) 
{ 
} 

Nhưng không có gì sai với đề xuất đầu tiên.

Answer 2

Mặc dù bạn có thể tránh sự cố bằng cách sử dụng danh sách khởi tạo của hàm tạo, tôi đề nghị theo quy ước đặt tên thành viên dữ liệu, ví dụ: _ hoặc m_ hàng đầu. Nếu không, bạn rất có khả năng có xung đột tên, đặc biệt nếu bạn có các thành viên có tên như x và y.

class A 
{ 
    public: 

    A(int x, int y, int width, int height) : x_(x), y_(y), with_(width), height_(height) {} 

    int x_; 
    int y_; 
    int width_; 
    int height_; 
}; 

Answer 3

Nếu bạn phải sử dụng bài tập trong các nhà xây dựng (như trái ngược với cách sử dụng một danh sách các initializers, được ưa thích) mô hình cụ thể để giải quyết vấn đề này là sử dụng this con trỏ, như sau:

this->a = a; 

Answer 4

Bạn chỉ có thể thay đổi tên của các đối số hàm tạo. Khi bạn viết

A::A(int x, int y, int width, int height) 
{ 
    x = x; 
    y = y; 
    width = width; 
    height = height; 
} 

sau đó bạn gán lập luận của các nhà xây dựng cho riêng mình, rời khỏi biến ví dụ thực tế uninitialized, đó là lý do tại sao bạn đang nhận được giá trị không có thật.

Các giải pháp chung tôi đề nghị (và rộng rãi sử dụng) là thay đổi tên của các đối số của phương thức khởi tạo:

A::A(int x_initial, int y_initial, int width_initial, int height_initial) 
{ 
    x = x_initial; 
    y = y_initial; 
    width = width_initial; 
    height = height_initial; 
} 

Answer 5

Nếu có thể, nó tốt hơn để thiết lập dữ liệu các thành viên thông qua danh sách initializer, trong trường hợp đó không có vấn đề với các đối số có tên thành viên bóng tối. Một cách khác là sử dụng this->foo = foo; trong phần thân của hàm tạo. Một vấn đề tương tự tồn tại đối với người định cư, nhưng bây giờ bạn không thể sử dụng giải pháp danh sách khởi tạo. Bạn đang mắc kẹt với this->foo = foo; - hoặc chỉ sử dụng các tên khác nhau cho các đối số và thành viên.

Một số người thực sự ghét các đối số làm bóng các thành viên dữ liệu; nhiều tiêu chuẩn mã hóa một cách rõ ràng nói không bao giờ làm điều này. Những người khác nghĩ rằng loại bóng tối này, ít nhất là cho các nhà xây dựng và người định cư, là meow của mèo. Tôi nhớ lại việc đọc một hoặc hai tiêu chuẩn mã hóa (nhưng tôi không nhớ cái nào) đã chỉ định loại bóng tối này là một thực hành "nên" (nhưng không phải "phải").

Một tùy chọn cuối cùng là sử dụng tính năng đánh bóng trong khai báo hàm để cung cấp cho người đọc một gợi ý về chức năng, nhưng sử dụng các tên riêng biệt trong quá trình triển khai.

Cập nhật: "shadowing" là gì?

#include <iostream> 

void printi (int i) { std::cout << "i=" << i << "\n"; } 

int i = 21; 

int main() { 
    printi (i); 
    int i = 42; 
    printi (i); 
    for (int i = 0; i < 3; ++i) { 
     printi (i); 
     int i = 10; 
     printi (i); 
    } 
    printi (i); 
} 

Việc kê khai trong cùng của i, int i=10, bóng tối biến i tuyên bố trong báo cáo for, do đó bóng tối biến i tuyên bố tại phạm vi chức năng, do đó bóng tối biến toàn cầu i.

Trong vấn đề ở bàn tay, các đối số x, y, width và height để các nhà xây dựng không mặc định cho lớp A bóng các dữ liệu thành viên có tên giống như những đối số.

width=width; của bạn không làm gì vì đối số width bóng (ẩn) thành viên dữ liệu width. Khi bạn có hai hoặc nhiều biến có cùng tên được khai báo ở các phạm vi khác nhau, người chiến thắng luôn là tên có phạm vi trong cùng. Nói chung, nó luôn là tên có phạm vi trong cùng nhất thắng.