# V1101. Changing the default argument of a virtual function parameter in a derived class may result in unexpected behavior.

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

Рассмотрим пример:

    struct Base
    {
      virtual void foo(int i = 0) const noexcept
      {
        std::cout << "Base::foo() called, i = " << i << std::endl;
      }
    };

    struct Derived : Base
    {
      void foo(int i = 10) const noexcept override
      {
        std::cout << "Derived::foo() called, i = " << i << std::endl;
      }
    };

В базовом классе 'Base' определена виртуальная функция 'foo' с одним параметром 'i' имеющим значение по умолчанию '0'. В классе 'Derived', унаследованном от 'Base', переопределяется виртуальная функция 'foo' и изменяется значение по умолчанию параметра 'i' на '10'.

Давайте разберёмся, к каким проблемам может привести такое переопределение. Допустим, мы используем код следующим образом:

    int main()
    {
      Derived obj;
      Base *ptr = &obj;
      ptr->foo();
    }

В результате выполнения функции 'main' будет выведена неожиданная строка "Derived::foo() called, i = 0". При формировании вызова функции 'foo' компилятор берёт статический тип объекта под указателем 'ptr' - 'Base'. Поэтому в вызов функции будет подставлено значение по умолчанию '0' из базового класса. В то же время переменная 'ptr' в реальности указывает на объект типа 'Derived'. Поэтому будет выполнена виртуальная функция из производного класса.

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

- не использовать аргументы со значением по умолчанию в виртуальных функциях;
- определять значение по умолчанию параметра виртуальной функции только в самом базовом классе.

Исправленный пример:

    struct Base
    {
      virtual void foo(int i = 0) const noexcept
      {
        std::cout << "Base::foo() called, i = " << i << std::endl;
      }
    };

    struct Derived : Base
    {
      void foo(int i) const noexcept override
      {
        std::cout << "Derived::foo() called, i = " << i << std::endl;
      }
    };

Замечание. Анализатор не выдает предупреждений на следующий код:

    struct Base
    {
      virtual void foo(int i = 0) const noexcept
      {
        std::cout << "Base::foo() called, i = " << i << std::endl;
      }
    };

    struct Derived : Base
    {
      void foo(int i = 0) const noexcept override
      {
        std::cout << "Derived::foo() called, i = " << i << std::endl;
      }
    };

Однако мы не рекомендуем писать такой код, потому что его сложнее поддерживать.

Данная диагностика классифицируется как:

CWE-665