Блог

Безопасно ли создавать экземпляр класса внутри функции-члена этого класса? Например, допустим, у меня есть класс CMyClass с функцией-членом CMyClass::MemberFunc , и я хочу создать другой экземпляр CMyClass внутри CMyClass::MemberFunc .

 void CMyClass::MemberFunc( void )
{
    CMyClass * pMyClass = new CMyClass();
}
  

Это законно / безопасно? Я знаю, что это компилируется. Что меня беспокоит, так это рекурсия. Столкнусь ли я с ошибкой рекурсии при создании экземпляра CMyClass в первый раз из основного приложения?

 void main( void )
{
    static CMyClass * s_pMyClass = new CMyClass(); // Will this cause recursion?
}
  

Или рекурсия будет происходить только в том случае, если вызывается конкретная функция-член с дополнительным экземпляром класса?

 void CMyClass::MemberFunc( void )
{
    CMyClass * pMyClass = new CMyClass();
    pMyClass->MemberFunc(); // Pretty sure this will cause a recursive loop.
}
  

Другими словами, могу ли я безопасно создать экземпляр данного класса внутри функции-члена этого класса, при условии, что я не вызываю эту функцию-член второго экземпляра этого класса? Спасибо.

Это не более и не менее безопасно, чем создание экземпляра любого другого объекта. Обратите внимание, что в вашем примере внизу рекурсия строго основана на том факте, что метод вызывает сам себя; он будет повторяться бесконечно независимо.

В целом: все должно быть в порядке.

Функция (член), вызывающая саму себя, известна как рекурсия.

Это законно / безопасно?

 void CMyClass::MemberFunc( void )
{

    CMyClass * pMyClass = new CMyClass();

    delete pMyClass ;  // newly added.

}
  

Частично верно. Потому что каждая new операция должна сопровождаться delete операцией над конкретным экземпляром, чтобы вернуть ресурсы обратно в бесплатное хранилище. Кроме утечки памяти, приведенный выше фрагмент в порядке.

Или рекурсия будет происходить только в том случае, если вызывается конкретная функция-член с дополнительным экземпляром класса?

 void CMyClass::MemberFunc( void )
{
    CMyClass * pMyClass = new CMyClass();
    pMyClass->MemberFunc(); // Pretty sure this will cause a recursive loop.
}
  

Да, и через некоторое время у вас должна закончиться память, поскольку нет способа вызвать деструктор для конкретного экземпляра, поскольку CMyClass::MemberFunc вызывается рекурсивно. ( предполагая delete pMyClass; , что это оператор end в функции-члене)

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

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

Нет, это не вызовет рекурсию — ваш компилятор пометил бы это как ошибку, если бы это произошло. Я подозреваю, что вы действительно хотите создать статический экземпляр вашего класса (например, singleton). Можете ли вы опубликовать реальный пример использования, иллюстрирующий, что вы хотите сделать?