Избегайте предупреждения «Формальный параметр без ссылки»

Question

У меня супер класс такой:

class Parent
{
public:
    virtual void Function(int param);
};

void Parent::Function(int param)
{
    std::cout << param << std::endl;
}

.. и такой подкласс:

class Child : public Parent
{
public:
    void Function(int param);
};

void Child::Function(int param)
{
    ;//Do nothing
}

Когда я компилирую файл .cpp подкласса, я получаю эту ошибку

warning C4100: 'param' : unreferenced formal parameter

На практике мы привыкли рассматривать предупреждения как ошибки. Как избежать вышеуказанного предупреждения?

Спасибо.

Answer 1

В C ++ вам не нужно указывать параметр, для которого вы не используете имя, поэтому вы можете просто сделать это:

void Child::Function(int)
{
    //Do nothing
}

Возможно, вы захотите сохранить имя параметра в объявлении взаголовочный файл в порядке документации, хотя.Пустой оператор (;) также не нужен.

Answer 2

Я предпочитаю использовать макрос, так как он сообщает не только компилятору о моих намерениях, но и другим сопровождающим кода, и он доступен для поиска позже.

Метод комментирования имени аргумента может быть легко пропущен людьми, незнакомыми с кодом (или мной 6 месяцев спустя).

Однако это проблема стиля, ни один из методов не является «лучшим» или более оптимальным с точки зрения генерируемого кода, производительности или надежности. Для меня решающим фактором является информирование других о моем намерении через стандартизированную систему. Пропуск имени параметра и добавление комментария будет работать одинаково хорошо:

void CFooBar::OnLvnItemchanged(NMHDR *pNMHDR, LRESULT *pResult)
{
    UNREFERENCED_PARAMETER(pNMHDR);

В качестве альтернативы:

void CFooBar::OnLvnItemchanged(NMHDR* /* pNMHDR */, LRESULT *pResult)
{
    // Not using: pNMHDR

Я бы сказал, что худшим решением является подавление предупреждающего сообщения; это повлияет на весь ваш файл или проект, и вы потеряете знание о том, что, возможно, вы что-то упустили. По крайней мере, добавив макрос или закомментировав имя аргумента, вы сказали другим, что приняли сознательное решение не использовать этот аргумент и что это не ошибка.

Windows SDK в WinNT.h определяет UNREFERENCED_PARAMETER() вместе с DBG_UNREFERENCED_PARAMETER() и DBG_UNREFERENCED_LOCAL_VARIABLE(). Все они оценивают одно и то же, но разница в том, что DBG_UNREFERENCED_PARAMETER () используется, когда вы начинаете, и ожидаете использовать параметр, когда код будет более полным. Если вы уверены, что никогда не будете использовать этот параметр, используйте версию UNREFERENCED_PARAMETER ().

У базовых классов Microsoft (MFC) есть аналогичное соглашение с более короткими макросами UNUSED() и UNUSED_ALWAYS().

Выберите стиль и придерживайтесь его. Таким образом, позже вы можете искать «DBG_UNREFERENCED_PARAMETER» в своем коде и находить любые случаи, когда вы ожидали использовать аргумент, но не сделали этого. Приняв непротиворечивый стиль и привычно используя его, вы потом упростите его для других и для себя.

Answer 3

Другой метод, который вы можете использовать, если хотите сохранить имя параметра, - привести к void:

void Child::Function(int param)
{
    (void)param;   //Do nothing
}

Answer 4

Как упомянул @Charles Bailey, вы можете пропустить имя параметра.

Однако в некоторых случаях вам нужно имя параметра, так как в отладочных сборках вы вызываете ASSERT() для него, но дляРитейл строит это nop.Для этих сценариев есть удобные макросы (по крайней мере, в VC ++ :-)) UNREFERENCED_PARAMETER(), которые определены следующим образом:

#define UNREFERENCED_PARAMETER(x) x

Обратите внимание, что простой приведенный @R Сэмюэль Клатчко, опубликованный также, работает, но ялично я нахожу это более читабельным, если в коде явно указано, что это параметр без ссылки, а не просто необъяснимое приведение типа.

Answer 5

Прагма тоже хорошо работает, так как ясно, что вы используете VS.Это предупреждение имеет очень высокое отношение шума к выгоде, учитывая, что параметры без ссылок очень распространены в интерфейсах обратного вызова и производных методах.Даже команды в Microsoft Windows, которые используют W4, устали от его бессмысленности (было бы более подходящим для / Wall) и просто добавили в свой проект:

#pragma warning(disable: 4100)

Если вы хотите снять предупреждение только для блокакода, окружите его:

#pragma warning(push)
#pragma warning(disable: 4100)
void SomeCallbackOrOverride(int x, float y) { }
#pragma warning(pop)

Практика исключения имени параметра имеет недостаток в отладчике, который вы не можете легко проверить по имени или добавить в часы (если выиметь более одного параметра, на который нет ссылок), и хотя конкретная реализация метода может не использовать этот параметр, знание его значения может помочь вам выяснить, на какой стадии процесса вы находитесь, особенно если у вас нет всего стека вызовов вышевы.

Answer 6

Я бы использовал макрос для подавления необязательного предупреждения о формальном параметре:

#define UNUSED( x ) ( &reinterpret_cast< const int& >( x ) )

Это имеет следующие преимущества:

• В отличие от #define UNUSED (x) (void)x, он не вводит необходимость полного определения типа параметра, который можно увидеть там, где такой необходимости раньше не было.
• В отличие от #define UNUSED (x) & x, его можно безопасно использовать спараметры, типы которых перегружают унарный оператор &.