Konflikt między przestrzenią nazw a zdefiniuj

Question

Mam ten poważny problem. Mam wyliczenie zasięgu 2 nazw takiego:

namespace FANLib { 
namespace ERROR { 

    enum TYPE { 

     /// FSL error codes 
     FSL_PARSER_FILE_IERROR,... 

i gdzieś w moim kodu, używam go tak:

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

Wszystko kompiluje dobrze i dobrze, ale jeśli zdarza mi się to " windows.h ", dostaję błędy! Problem jest w „WinGDI.h”, która ma tę linię:

#define ERROR    0 

i sprawia, że ​​kompilator myśleć, że po FANLib :: ..., jest zerem! Błąd pojawia się:

Error 1 error C2589: 'stała': nielegalne znak po prawej stronie '::'

Błąd 2 error C2059: błąd składni: '::'

błąd 3 error C2039: „FSL_PARSER_FILE_IERROR”: nie jest członkiem globalnej przestrzeni nazw „`””

Czy mogę coś z tym zrobić, bez konieczności zmiany nazw ze względu na moje jakiegoś bezmyślnego #define? Czytałem w innym poście, że mogłem #undef ERROR, ale jak bezpieczne jest to?

Answer 1

Zasadniczo należy unikać używania identyfikatorów uniwersalnych, ponieważ są one używane w makrach. W tym przypadku zmieniłbym nazwę przestrzeni nazw.

(Na marginesie, <windows.h>#define S inne rzeczy jak GetPrinter i rzeczywiście robi się denerwujące. I zwykle z #undef wtedy. Pomaga również zawierać tylko <windows.h> w plikach .cpp i upewnij się, że zakres dotknięte nagłówku jest tak mały, jak to możliwe.)

Answer 2

Myślę, że nie będzie problemu, jeśli przekroczysz limit. Ale musisz to zrobić wszędzie, gdzie używasz obu enum & windows.h. Najlepszą rzeczą byłoby przemianowanie twojej przestrzeni nazw.

Answer 3

Zmiana nazwy przestrzeni nazw jest najczystszym, najbezpieczniejszym, najbardziej interoperacyjnym rozwiązaniem.

Answer 4

Przeskakiwanie po "zmianie nazwy przestrzeni adresowej", po prostu dlatego, że ERROR jest zbyt pospolite i niejasne. Znajdź coś bardziej opisowego.

Answer 5

Być może zechcesz zreorganizować swój kod, aby ten #include pojawiał się tylko w razie potrzeby. "Prawidłowy" sposób może nawet wymagać utworzenia osobnych plików i nagłówków, które będą zawierać interfejsy do funkcji, które wywołasz z Windows.h

Jeśli jednak potrzebujesz prostej poprawki i jesteś zaniepokojony efektami ubocznymi od #undef ERROR tylko przedefiniować ERROR po zakończeniu z deklaracją:

 
#undef ERROR 
namespace ERROR { 
#define ERROR 0 

trzeba by zrobić to za każdym razem ERROR określany jest (a nie w ciągu).

Powiedziawszy to, powinieneś być w porządku, jeśli po prostu nie zdefiniujesz BŁĘDU. Wpłynęłoby to tylko na to, jak procesor C obsługuje BŁĄD (a raczej jak nie) od tego momentu.

Nawiasem mówiąc, generalnie widziałem, że nazwy we wszystkich wielkich literach są używane tylko do określenia stałych, a nie do typów i przestrzeni nazw. Chciałbym zrewidować moją konwencję nazewnictwa gdybym cię ..

Answer 6

Najlepszym rozwiązaniem jest, aby zmienić schemat nazewnictwa, aby uniknąć wszystkich kapitalizacji, jak inni sugerują:

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

także poza granice: Identyfikatory zacznij od _ lub __. (To jest bardziej skomplikowane, ale twoje życie będzie prostsze, jeśli unikniesz wszystkich nazw zaczynających się od podkreślenia.)

Answer 7

Jeśli nie używasz GDI, możesz zdefiniować NOGDI, aby wyłączyć definiowanie makr w WinGDI.h. Here możesz znaleźć inne przydatne opcje.

Answer 8

Identyfikatory rozpoczynające się od E i cyfra lub E oraz wielka litera powinny być traktowane jako zastrzeżone, ponieważ takie nowe makra można dodać do <errno.h>.