Xung đột giữa không gian tên và xác định

Question

Tôi gặp vấn đề nghiêm trọng này. Tôi có một điều tra trong vòng 2 không gian tên như thế này:

namespace FANLib { 
namespace ERROR { 

    enum TYPE { 

     /// FSL error codes 
     FSL_PARSER_FILE_IERROR,... 

và ở một nơi khác trong mã của tôi, tôi sử dụng nó như thế này:

FANLib::Log::internalLog(FSLParser::FILE_IERROR, file_ierror, true, FANLib::ERROR::FSL_PARSER_FILE_IERROR); 

Tất cả các biên dịch tốt và tốt, nhưng nếu tôi xảy ra bao gồm " windows.h ", tôi gặp lỗi! Vấn đề nằm trong "WinGDI.h" có dòng này:

#define ERROR    0 

và làm cho trình biên dịch nghĩ rằng, sau FANLib :: ..., có một số không! Các lỗi tôi nhận được là:

Lỗi 1 lỗi C2589: 'liên tục': dấu hiệu bất hợp pháp trên bên phải của '::'

Lỗi 2 lỗi C2059: lỗi cú pháp: '::'

lỗi 3 lỗi C2039: 'FSL_PARSER_FILE_IERROR': không phải là thành viên của '`namespace toàn cầu''

có bất cứ điều gì tôi có thể làm về vấn đề này, mà không cần phải thay đổi không gian tên của tôi do một số suy nghĩ #define? Tôi đã đọc trong một bài đăng khác mà tôi có thể #undef LRI, nhưng làm thế nào an toàn là điều đó?

Answer 1

Nói chung, bạn nên tránh sử dụng mã định danh tất cả mũ khi chúng được sử dụng cho macro. Trong trường hợp này, tôi đổi tên vùng tên.

(Như một mặt lưu ý, <windows.h>#define s thứ khác như GetPrinter và thực sự nó được gây phiền nhiễu. Tôi thường đi với #undef sau đó. Nó cũng giúp để chỉ bao gồm <windows.h> trong các tập tin cpp và đảm bảo phạm vi bị ảnh hưởng bởi các tiêu đề càng nhỏ càng tốt.)

Answer 2

Tôi nghĩ rằng sẽ không thành vấn đề nếu bạn #undef. Nhưng, bạn sẽ phải làm điều đó ở khắp mọi nơi bạn sử dụng cả hai enum của bạn & windows.h. Điều tốt nhất sẽ đổi tên vùng tên của bạn.

Answer 3

Đổi tên vùng tên của bạn là giải pháp sạch nhất, an toàn nhất, có khả năng tương tác nhất.

Answer 4

Nhảy trên dải "đổi tên không gian tên" của bạn, chỉ đơn giản là vì ERROR là cách quá phổ biến và mơ hồ với tên. Tìm một cái gì đó mô tả hơn.

Answer 5

Bạn có thể muốn cấu trúc lại mã của mình để #include chỉ xuất hiện khi cần thiết. Cách "đúng" thậm chí có thể liên quan đến việc tạo các tệp và tiêu đề riêng biệt bao gồm các giao diện cho các hàm bạn sẽ gọi từ windows.h

Tuy nhiên, nếu bạn chỉ muốn sửa chữa đơn giản và lo ngại về tác dụng phụ từ #undef ERROR , chỉ cần xác định lại L ERI sau khi bạn đã hoàn tất khai báo của mình:

 
#undef ERROR 
namespace ERROR { 
#define ERROR 0 

Bạn sẽ phải thực hiện việc này mỗi khi ERROR được nhắc đến (chứ không phải trong một chuỗi).

Có nói rằng, bạn nên ok nếu bạn chỉ đơn giản là không xác định ERROR. Nó sẽ chỉ ảnh hưởng đến cách bộ xử lý tiền xử lý C xử lý ERROR (hay đúng hơn cách nó không) từ điểm đó trở đi.

Nhân tiện, tôi thường thấy tên trong tất cả các chữ in hoa chỉ được sử dụng để chỉ định các hằng số, không dùng cho các loại và không gian tên. Tôi muốn xem xét lại quy ước đặt tên của tôi nếu tôi là bạn ..

Answer 6

Giải pháp tốt nhất là thay đổi sơ đồ đặt tên của bạn để tránh tất cả mũ, như những người khác đã gợi ý:

namespace FANLib { 
    namespace Error { 
    enum Type { 
     /// FSL error codes 
     FSL_Parser_File_IError, ... 

Ngoài ra giới hạn: định danh đó bắt đầu bằng _ hoặc __. (Nó phức tạp hơn thế, nhưng cuộc sống của bạn sẽ đơn giản hơn nếu bạn chỉ cần tránh tất cả các tên bắt đầu bằng dấu gạch dưới.)

Answer 7

Nếu bạn không sử dụng GDI thì bạn có thể xác định NOGDI để chặn các macro xác định trong WinGDI.h. Here bạn có thể tìm thấy các tùy chọn hữu ích khác.

Answer 8

Số nhận dạng bắt đầu bằng E và chữ số hoặc E và chữ hoa nên được coi là đã đặt trước, vì các macro mới này có thể được thêm vào <errno.h>.