사용하지 않는 변수에 대한 경고를 가장 잘 끄는 방법은 무엇입니까?

크로스 플랫폼 응용 프로그램이 있으며 일부 함수에서 함수에 전달 된 모든 값이 사용되는 것은 아닙니다. 따라서 GCC에서 사용되지 않은 변수가 있음을 알리는 경고가 표시됩니다.

경고를 코딩하는 가장 좋은 방법은 무엇입니까?

함수 주위의 #ifdef?

#ifdef _MSC_VER
void ProcessOps::sendToExternalApp(QString sAppName, QString sImagePath, qreal qrLeft, qreal qrTop, qreal qrWidth, qreal qrHeight)
#else
void ProcessOps::sendToExternalApp(QString sAppName, QString sImagePath, qreal /*qrLeft*/, qreal /*qrTop*/, qreal /*qrWidth*/, qreal /*qrHeight*/)
#endif
{

이것은 추악하지만 컴파일러가 선호하는 방식 인 것 같습니다.

또는 함수 끝에서 변수에 0을 할당합니까? (컴파일러 경고를 침묵시키기 위해 프로그램 흐름에서 무언가를 변경하기 때문에 싫어합니다).

올바른 방법이 있습니까?

컴파일러가 사용하는 것을 볼 수 있도록 " (void)var;"표현식 (아무것도하지 않음) 에 넣을 수 있습니다 . 이것은 컴파일러간에 이식 가능합니다.

예 :

void foo(int param1, int param2)
{
    (void)param2;
    bar(param1);
}

또는,

#define UNUSED(expr) do { (void)(expr); } while (0)
...

void foo(int param1, int param2)
{
    UNUSED(param2);
    bar(param1);
}

GCC와 Clang에서는 __attribute__((unused))전 처리기 지시문을 사용 하여 목표를 달성 할 수 있습니다.
예를 들면 다음과 같습니다.

int foo (__attribute__((unused)) int bar) {
   return 0;
}

C ++ 17은 이제 [[maybe_unused]]속성을 제공 합니다.

http://en.cppreference.com/w/cpp/language/attributes

꽤 좋고 표준입니다.

현재 솔루션이 가장 좋습니다-매개 변수 이름을 사용하지 않으면 주석 처리하십시오. 이는 모든 컴파일러에 적용되므로 GCC를 위해 특별히 사전 처리기를 사용할 필요가 없습니다.

C ++ 17 업데이트

C ++ 17에서는 [dcl.attr.unused] 에서 다루는 [[maybe_unused]] 속성을 얻 습니다.

속성 토큰 maybe_unused는 이름 또는 엔티티가 의도적으로 사용되지 않았 음을 나타냅니다. 각 속성 목록에 최대 한 번 나타나야하며 속성 인수 절이 없어야합니다. ...

예:

 [[maybe_unused]] void f([[maybe_unused]] bool thing1,
                        [[maybe_unused]] bool thing2) {
  [[maybe_unused]] bool b = thing1 && thing2;
    assert(b);
 }

구현은 NDEBUG의 정의 여부에 관계없이 b가 사용되지 않음을 경고해서는 안됩니다. — 끝 예제]

다음 예의 경우 :

int foo ( int bar) {
    bool unused_bool ;
    return 0;
}

clang과 gcc는 모두 bar 및 uned_bool에 대해 -Wall -Wextra 를 사용하여 진단을 생성합니다 ( 실제 참조 ).

[[maybe_unused]]를 추가 하면 진단 기능이 꺼집니다 .

int foo ([[maybe_unused]] int bar) {
    [[maybe_unused]] bool unused_bool ;
    return 0;
}

그것을 참조하십시오 .

C ++ 17 이전

C ++ 11 에서는 사용되지 않는 변수를 캡처 UNUSED하여 람다 식 ( Ben Deane을 통해 )을 사용하여 대체 형식의 매크로를 만들 수 있습니다 .

#define UNUSED(x) [&x]{}()

다음 예제에서 람다 식의 즉각적인 호출을 최적화해야합니다.

int foo (int bar) {
    UNUSED(bar) ;
    return 0;
}

Godbolt 에서 호출이 최적화되었음을 알 수 있습니다.

foo(int):
xorl    %eax, %eax
ret

더 깨끗한 방법은 변수 이름을 주석 처리하는 것입니다.

int main(int /* argc */, char const** /* argv */) {
  return 0;
}

동료는이 좋은 작은 매크로에 저를 지적 여기

쉽게하기 위해 아래 매크로를 포함하겠습니다.

#ifdef UNUSED
#elif defined(__GNUC__) 
# define UNUSED(x) UNUSED_ ## x __attribute__((unused)) 
#elif defined(__LCLINT__) 
# define UNUSED(x) /*@unused@*/ x 
#else 
# define UNUSED(x) x 
#endif

void dcc_mon_siginfo_handler(int UNUSED(whatsig))

gcc기본적으로 이러한 경고를 표시하지 않습니다. 이 경고는 명시 적으로 -Wunused-parameter컴파일러에 전달하거나 암시 적으로 전달 -Wall -Wextra(또는 플래그의 다른 조합)을 통해 설정해야합니다.

사용하지 않는 매개 변수 경고는 -Wno-unused-parameter컴파일러 에 전달 하여 간단히 억제 할 수 있지만,이 비활성화 플래그는 컴파일러 명령 행에서이 경고에 대한 가능한 활성화 플래그 뒤에 와야 적용 할 수 있습니다.

하나 이상의 매개 변수를 사용하지 않는 것으로 선언하는 매크로가 적고 이식 가능한 방법 :

template <typename... Args> inline void unused(Args&&...) {}

int main(int argc, char* argv[])
{
    unused(argc, argv);
    return 0;
}

전 처리기 지시문을 사용하는 것은 대부분 악으로 간주됩니다. 해충처럼 피하는 것이 이상적입니다. 컴파일러가 코드를 이해하도록하는 것은 쉽다는 점을 기억하십시오. 다른 프로그래머가 코드를 이해하는 것이 훨씬 어렵습니다. 여기에 이와 같은 수십 가지 사례가 있으며, 나중에 자신이나 다른 사람이 읽기 어려운 경우가 있습니다.

한 가지 방법은 매개 변수를 일종의 인수 클래스로 묶는 것입니다. 그런 다음 변수의 하위 집합 만 사용하거나 (실제로 0을 할당하는 것과 동일) 각 플랫폼에 대해 해당 인수 클래스의 다른 전문화를 가질 수 있습니다. 그러나 이것은 그만한 가치가 없을 수 있습니다. 적합한 지 분석해야합니다.

불가능한 템플리트를 읽을 수있는 경우 "예외 C ++"책에서 고급 팁을 찾을 수 있습니다. 코드를 읽는 사람들이 그 책에서 가르치는 미친 것들을 포괄하는 스킬 셋을 얻을 수 있다면, 쉽게 읽을 수있는 아름다운 코드를 갖게 될 것입니다. 컴파일러는 또한 전처리로 모든 것을 숨기지 않고 수행중인 작업을 잘 알고 있습니다.

먼저 경고는 헤더 파일이 아닌 소스 파일의 변수 정의에 의해 생성됩니다. API 문서를 생성하기 위해 doxygen과 같은 것을 사용하고 있기 때문에 헤더는 깨끗하게 유지 될 수 있습니다.

소스 파일에서 구현이 완전히 다르다고 가정합니다. 이 경우 문제가되는 매개 변수를 주석 처리하거나 매개 변수를 작성할 수 있습니다.

예:

func(int a, int b)
{
    b;
    foo(a);
}

이것은 암호처럼 보일 수 있으므로 UNUSED와 같은 매크로를 정의했습니다. MFC가 한 방식은 다음과 같습니다.

#ifdef _DEBUG
#define UNUSED(x)
#else
#define UNUSED(x) x
#endif

이와 같이 디버그 빌드에서 여전히 경고가 표시되는 것이 도움이 될 수 있습니다.

항상 매개 변수 이름을 주석 처리하는 것이 안전하지 않습니까? 그렇지 않은 경우 다음과 같은 작업을 수행 할 수 있습니다

#ifdef _MSC_VER
# define P_(n) n
#else
# define P_(n)
#endif

void ProcessOps::sendToExternalApp(
    QString sAppName, QString sImagePath,
    qreal P_(qrLeft), qreal P_(qrTop), qreal P_(qrWidth), qreal P_(qrHeight))

그건 좀 덜 추한.

나는 (void)param2경고를 침묵시키는 대신에 이것을 보았습니다 .

void foo(int param1, int param2)
{
    std::ignore = param2;
    bar(param1);
}

C ++ 11에서 추가 된 것 같습니다

사용할 UNREFERENCED_PARAMETER(p)수 있습니다. Windows 시스템의 WinNT.h에 정의되어 있으며 gcc에 대해 쉽게 정의 할 수 있음을 알고 있습니다 (아직없는 경우).

UNREFERENCED PARAMETER(p) 로 정의된다

#define UNREFERENCED_PARAMETER(P)          (P)

WinNT.h에서.

컴파일러의 플래그를 사용하십시오 (예 : GCC의 경우 플래그). -Wno-unused-variable

당신이 사용할 수있는 __unused변수가 사용되지 않을 수 있음을 컴파일러에게.

- (void)myMethod:(__unused NSObject *)theObject    
{
    // there will be no warning about `theObject`, because you wrote `__unused`

    __unused int theInt = 0;
    // there will be no warning, but you are still able to use `theInt` in the future
}

C ++ 11에서는 이것이 사용중인 솔루션입니다.

template<typename... Ts> inline void Unreferenced(Ts&&...) {}

int Foo(int bar) 
{
    Unreferenced(bar);
    return 0;
}

int Foo2(int bar1, int bar2) 
{
    Unreferenced(bar1, bar2);
    return 0;
}

휴대용 (적어도 최신 msvc, clang 및 gcc에서) 휴대 가능하고 최적화가 활성화 된 경우 추가 코드를 생성하지 않는 것으로 확인되었습니다. 최적화가 없으면 추가 함수 호출이 수행되고 매개 변수에 대한 참조가 스택에 복사되지만 관련된 매크로는 없습니다.

추가 코드가 문제가되면이 선언을 대신 사용할 수 있습니다.

(decltype(Unreferenced(bar1, bar2)))0;

그러나 그 시점에서 매크로는 더 나은 가독성을 제공합니다.

#define UNREFERENCED(...) { (decltype(Unreferenced(__VA_ARGS__)))0; }

이것은 잘 작동하지만 C ++ 11이 필요합니다.

template <typename ...Args>
void unused(Args&& ...args)
{
  (void)(sizeof...(args));
}

제시된 답변의 대부분은 로컬 미사용 변수에 대해서만 작동하며 미사용 정적 전역 변수에 대해 컴파일 오류가 발생합니다.

또 다른 매크로는 사용되지 않은 정적 전역 변수의 경고를 억제해야했습니다.

template <typename T>
const T* UNUSED_VARIABLE(const T& dummy) { 
    return &dummy;
}
#define UNUSED_GLOBAL_VARIABLE(x) namespace {\
    const auto dummy = UNUSED_VARIABLE(x);\
}

static int a = 0;
UNUSED_GLOBAL_VARIABLE(a);

int main ()
{
    int b = 3;
    UNUSED_VARIABLE(b);
    return 0;
}

익명 네임 스페이스의 비 정적 전역 변수에 대해서는 경고가보고되지 않기 때문에 작동합니다.

그래도 C ++ 11이 필요합니다

 g++  -Wall -O3  -std=c++11 test.cpp

롤! 나는 이것에 대해 카오스에 의해 손상된 모든 이단자들을 드러내는 SO에 대한 또 다른 질문이 있다고 생각하지 않습니다!

C ++ 17과 관련하여 C ++ 핵심 가이드 라인에 명확한 지침이 있습니다. AFAIR, 2009 년에이 옵션은 현재뿐만 아니라 현재 사용 가능합니다. 누군가가 그것이 Doxygen의 버그로 간주되면 Doxygen에 버그가 있다고 말합니다

경고와 관련된 문제가 보이지 않습니다. 컴파일러 xy가 여기에 (올바른) 경고를 발행하지만 메소드 z가 함수 z에 필요하다는 것을 메소드 / 함수 헤더에 문서화하십시오.

경고는 정확하므로 끄지 않아도됩니다. 프로그램을 무효화하지는 않지만 이유가 있음을 문서화해야합니다.