Chống lại sự nguy hiểm của tên lớp ẩn tên chức năng

Question

#include <iostream> 
using namespace std; 

#include "other_library.h" 

struct Foo 
{ 
    Foo() { cout << "class\n"; } 
}; 

int main() 
{ 
    Foo(); // or Foo() in any expression 
} 

Kết quả đầu ra này class, hoặc vì vậy chúng tôi nghĩ. Vấn đề là nếu other_library.h xảy ra để có một chức năng gọi là Foo mà trở lại loại phù hợp để xuất hiện trong bất cứ biểu chúng tôi sử dụng Foo trong sau đó nó âm thầm thay đổi hành vi, ví dụ:

int Foo() { cout << "func\n"; return 1; } 

nguyên nhân func là đầu ra mà không cần bất kỳ mã thay đổi trong main. Điều này là xấu vì khả năng cho các lỗi phát hiện và khó phát hiện; ngay cả khi nó không phải là ý định độc hại trên một phần của other_library, một cuộc đụng độ tên có thể không bị phát hiện.

---

Cách tốt nhất để giải quyết vấn đề này là gì? Ban đầu nó được nuôi dưỡng bởi Dan Saks in 1997, và one suggested resolution là tất cả lớp nên được typedef'd:

typedef struct Foo Foo; 

như trình biên dịch không phải báo cáo một vụ đụng độ giữa một typedef tên và một tên hàm . Tuy nhiên điều này dường như không phổ biến - tại sao không?

Làm rõ: câu hỏi này là về các phương pháp hay để mã của chúng tôi ngừng thay đổi hành vi không bị phát hiện này xảy ra mà chúng tôi không nhận thấy. (Trái ngược với cách giải quyết nó khi chúng tôi nhận ra rằng điều đó đang xảy ra, điều này dễ dàng hơn - ví dụ: đổi tên lớp của chúng tôi).

Answer 1

typedef struct Foo Foo;

như trình biên dịch không phải báo cáo một vụ đụng độ giữa một typedef tên và một tên hàm. Tuy nhiên, điều này dường như không phổ biến - tại sao không?

Tôi giữ sự thật này là hiển nhiên: mã cồng kềnh rườm rà. Bạn thực sự có thể dễ dàng sửa lỗi này bằng cách làm việc trong một không gian tên của chính bạn (trong đó, giả sử tập tin tiêu đề là C++, cũng vậy,

Answer 2

Thật không may, bạn không thể tránh điều này. (3.3.10):

Tên lớp (9.1) hoặc tên liệt kê (7.2) có thể được ẩn bằng tên của biến, thành viên dữ liệu, hàm hoặc điều tra được khai báo trong cùng một phạm vi. hoặc tên điều tra và biến, thành viên dữ liệu, hàm hoặc điều tra được khai báo trong cùng một phạm vi (theo thứ tự bất kỳ) có cùng tên, tên lớp hoặc liệt kê được ẩn ở bất kỳ nơi nào, biến, thành viên dữ liệu, hàm hoặc tên điều tra hiển thị

Cách duy nhất để bảo vệ chống lại nó là mẹo typedef bạn đã chỉ ra (cảm ơn điều đó!), Sử dụng không gian tên hoặc tôn trọng quy ước đặt tên (không hỗ trợ khi xử lý mã bên thứ 3). Bạn cũng có thể thử bọc phần bao gồm trong một không gian tên (được hiển thị dưới đây), nhưng như các chú thích đã chỉ ra, điều này có thể gây ra các lỗi liên kết trong một số tình huống nhất định.

namespace otherlib { 
#include "other_library.h" 
} 

Sau đó:

otherlib::Foo(); 

Answer 3

thực hành tốt nhất hiện nay bảo vệ chống lại điều này đã

• ước đặt tên, chẳng hạn như
  void camelCase() vs class PascalCase
  

• namespace

• libs wrapper cho tất cả mọi thứ không hiện đại C++