does it make sense? or this is not the best approa...

# announcements

[standinga]

does it make sense? or this is not the best approach? Thanks!

[karelpeeters]

Why not just have separate nextMovie and nextImage?

[Kroppeb]

Or an enum ?

[standinga]

I have more assets I put two to simplify

[standinga]

I mean more asset types

[standinga]

I thought about that approach because this way I don't need to update enum when new types appears. I've started just from Movie and Image, but I now I have also Audio and Light

[standinga]

and probably more to come (depending on what my client comes with)

[standinga]

My initial approach was nextMovie and nextImage

[standinga]

The reason I want to make it generic is I want to abstract presenting them into AssetPresenter

[standinga]

if I keep them generic I will be able to keep internals of presenter private and my AssetPresenter only exposes a few methods and it just returns view (image or video) which I add to my container

[codeslubber]

In that case, the asset is not really a base class, but just an interface. Stated goal of interfaces is to make it possible to treat dissimilar things in a given way. Your main use is in a presenter, hence no real domain benefit of constraining everything to a single hierarchy inheritance scheme. Before you do that you have to have not just attributes but behavior, AFAIC. (Interface could just be PresentableItem, as there is no real use of the concept of an asset here, which implies value.)

[standinga]

hey good point will think about it! Thanks