Определение макросов в Visual Studio - / D или #define?
Недавно при переносе некоторого кода STL на VS2008 я хотел отключить предупреждения, генерируемые std::copy, путем определения нового флага _SCL_SECURE_NO_WARNINGS. Сделать это можно двумя способами:
- С помощью переключателя компилятора / D, который можно указать в свойствах проекта. Вы должны убедиться, что он определен как для сборки выпуска, так и для сборки отладки, что я часто забываю сделать.
Определив его макростиль перед включением соответствующих заголовков STL или, для полного охвата, в stdafx.h:
#define _SCL_SECURE_NO_WARNINGS
Оба эти метода работают нормально, но мне было интересно, есть ли аргументы в пользу предпочтения одного перед другим?
Ответы 6
Если у вас есть заголовок, который включен во все остальные (например, stdafx.h), вы должны поместить его туда. Переключатель командной строки компилятора обычно используется для параметров сборки, которые не всегда устанавливаются, таких как NDEBUG, UNICODE и тому подобное. Хотя ваш макрос необходим всегда.
Это может показаться произвольным. И действительно, некоторые могут сказать другое. В конце концов, вы должны решить, что подходит вашей ситуации.
Параметр / D обычно используется, когда вы хотите определить его по-разному в разных сборках (чтобы его можно было изменить в make-файле)
Если вы «всегда» хотите, чтобы он устанавливался одинаково, используйте #define.
Помещая их в файл проекта, вы поддерживаете тесную связь между предупреждениями, относящимися к конкретной платформе, и платформой, что мне кажется правильным.
Если они в коде, они всегда в коде, независимо от того, подходит он для платформы или нет. Он вам не нужен для GCC или, возможно, будущих версий Visual C++. С другой стороны, если это есть в коде, более очевидно, что это вообще есть. Если вы переместите (скопируете) код, будет легче не забыть переместить это определение вместе с ним.
Плюсы и минусы в каждом случае. YMMV.
Если вы все же вставляете их в свой код, не забудьте правильно их определить:
#ifdef _MSC_VER
#define _SCL_SECURE_NO_WARNINGS
#endif
Это сохранит ваш код переносимым.
В этом случае определенно нет, но в общем случае я считаю разумным иметь определения, специфичные для компилятора, внутри соответствующих пар ifdef / endif.
В общем, я предпочитаю использовать #define в коде, а не использовать переключатель компилятора / D для большинства вещей, потому что кажется более интуитивно понятным искать #define, чем проверять настройки компилятора.
/ D не является допустимым флагом для msbuild.exe (по крайней мере, версия, которую я использую v2.0.50727).
Это делается следующим образом:
/p:DefineConstants = "MY_MACRO1;MY_MACRO2"
Результатом этого является:
Target CoreCompile:
C:\WINDOWS\Microsoft.NET\Framework\v2.0.50727\Csc.exe /define:MY_MACRO1;MY_MACRO2 ...
Определение символа обычно не повредит при перемещении на другую платформу, даже если он ничего не делает. Простой комментарий был бы столь же эффективным.