Re: pure virtual function with unspecified parameters?
On Nov 27, 10:55 am, Markus Dehmann <markus.dehm...@gmail.com> wrote:
I have an abstract base class called Data. It has a pure virtual
function
virtual void write(std::ostream& out) =0;
which writes the internal data to a stream. Now the problem is that
this is not appropriate for some implementations of the class. Some
implementations have an internal representation that they should
rather write into several separate files. So for those, something like
this would be more appropriate:
void write(const std::string& outputDirectoryName);
What would be a good design for the abstract base class in this case?
It seems like it should require some kind of write function, but with
flexible parameter lists for different implementations.
Certainly we don't want to deal with void pointers:
write(void* toWhatever) = 0;
that the implementation classes would cast and use however they like
because void pointers are evil.
A template also doesn't work:
template<class T>
virtual void write(T& out) = 0;
because you can't template a virtual function. And it would seem like
overkill to template the whole class on this output parameter; after
all it's just a little write function within a much larger class.
What would be a good design in this case?
Thanks for all your answers and the lateness of this reply.
After reading up on Design Patterns, it seems to me that the Visitor
pattern would fit perfectly here and solve my problem. You all were
concerned about separation of responsibilities. So, just adding a
void accept(const Visitor& v){
v.visit(this);
}
should be fine and keep the class small. Then, different
implementations can define their own WriteVisitor version, like this:
class WriteVisitor : public Visitor {
void visit(const MyData& d){
// write by calling the appropriate functions specific to MyData
}
};
Markus
P.S.: Before getting the Visitor idea, I "solved" the problem by just
adding a
void write(const std::string& name);
function. Some implementations can write to a stream (e.g. to cout if
name=="-"), others can write to a directory of that name, etc.