Question

I am working on a slide-based application in C++.
Each slide has a slide-items collection which can
include items like caption, button, rectangle, etc.

Only some of these items support fill, while others
don’t.

What is the best way to implement the fill for the slide items in this case?
Here are two ways that I thought of:

1. Create an interface Fillable and implement this interface for slide items
   which support fill, keeping all the properties related to fill in the interface. When iterating over the list of slide items, dynamic_cast them
   into Fillable, and if successful, do the operation related to fill.

2. Make a fill class. Make a fill pointer a part of slide item class, assign the
   fill object to the fill pointer for those objects which support fill, and for rest of them keep it null. Give a function GetFill, which will return the fill for the items if it exists otherwise returns NULL.

What’s the best approach for this? I’m interested in performance and maintainability.

Answer 1

I would do a combination of the two. Make your Fillable interface and have it be the return type for your GetFill method. This is better than the dynamic cast approach. Using dynamic cast to query for the interface requires that the actual slide item object implement the interface if it is to support it. With an accessor method like GetFill however, you have the option of providing a reference/pointer to some other object that implements the interface. You can also just return this if the interface is in fact implemented by this object. This flexibility can help avoid class bloat and promote the creation of re-usable component objects that can be shared by multiple classes.

Edit:
This approach also works nicely with the null object pattern. Instead of returning a null pointer for the objects that don’t support Fillable, you can return a simple no-op object that implements the interface. Then you don’t have to worry about always checking for null pointers in the client code.