10

Maybe this is a simple question, because I'm still new to C++. I would like to use some kind of factory to encapsulate the logging in my application. The idea is that only the factory knews which concrete class will handle the function calls. The application will always call abstract interfaces of the base logging class.

The factory method should look like:

std::unique_ptr<AbstractLoggingClass> Factory::getDefaultLogger(const std::string& param){
    return new ConcreteLoggingClass(param);
}

ConcreteLoggingClass is a subclass of AbstractLoggingClass.

But I get the following error:

Error: could not convert '(operator new(64ul), (<statement>,
((ConcreteLoggingClass*)<anonymous>)))' from 'ConcreteLoggingClass*'
to 'std::unique_ptr<AbstractLoggingClass>'

My problem is that I don't know how to instantiate ConcreteLoggingClass and return a unique_ptr to AbstractLoggingClass

I already found this post, but I still don't see the solution.

Community
  • 1
  • 1
little_planet
  • 1,005
  • 1
  • 16
  • 35

5 Answers5

14

The std::unique_ptr constructor you want is explicit, hence you need to be... well... explicit about it. Try

return std::unique_ptr<AbstractLoggingClass>(new ConcreteLoggingClass(param));
Community
  • 1
  • 1
Frerich Raabe
  • 90,689
  • 19
  • 115
  • 207
  • That did the job. Thank you very much! – little_planet Dec 21 '15 at 14:06
  • Correct answer as far C++ goes, but I don't know why C++ made this decision. A `ConcreteLoggingClass` is a `AbstractLoggingClass` using the "is a" in the sense of the Liskov Substitution Principle. – wcochran Jun 05 '20 at 23:13
13

If you can use C++14 you should use std::make_unique:

return std::make_unique<ConcreteLoggingClass>( param );

otherwise explicitly create std::unique_ptr:

return std::unique_ptr<AbstractLoggingClass>{ new ConcreteLoggingClass{param}};
Slava
  • 43,454
  • 1
  • 47
  • 90
1

The constructor of std::unique_ptr<T> is explicit. It won't implicitly convert from a pointer as doing so would mean a pointer is silently deleted.

You can return a std::unique_ptr<T> explicitly constructing it, e.g.:

return std::unique_ptr<AbstractLoggingClass>(new ConcreteLoggingClass(param));
Dietmar Kühl
  • 150,225
  • 13
  • 225
  • 380
1

You can't convert your pointer to a unique_ptr, you have to create it :

std::unique_ptr<AbstractLoggingClass> Factory::getDefaultLogger(const std::string& param){
    return std::unique_ptr<AbstractLoggingClass>(new ConcreteLoggingClass(param));
}
dkg
  • 1,775
  • 14
  • 34
1

if you have c++14 or better:

std::unique_ptr<AbstractLoggingClass> 
Factory::getDefaultLogger(const std::string& param)
{
    return std::make_unique<ConcreteLoggingClass>(param);
}
Richard Hodges
  • 68,278
  • 7
  • 90
  • 142