Trying to implement strategy pattern. Let me know if this is bad code

# announcements

[Mani]

Trying to implement strategy pattern. Let me know if this is bad code

[Pavlo Liapota]

What error?
Works fine for me
https://pl.kotl.in/gZPnK0byh
Just write object instead of Object.

[Mani]

Also, is this correct?

val labelByCategoryMap = mapOf(
            ShippingLabelCategoryConstants.FRESH to FreshLabelPrinter,
            ShippingLabelCategoryConstants.PHARMA to PharmaLabelPrinter
    )

where FreshLabelPrinter, PharmaLabelPrinter are objects

[Mani]

Can I have map of enum to objects?

[Pavlo Liapota]

Sure

[Pavlo Liapota]

Do they implement same interface? Otherwise this may not be very useful.

[Mani]

yeah they do. Let me share the exact scrensot

[Mani]

Screen Shot 2019-10-12 at 1.39.39 AM.png

[Pavlo Liapota]

FreshLabelPrinter does not implement any interface, so I can guess that your map labelByCategoryMap will have type Map<ShippingLabelCategoryConstants, Any>, that’s why method is not available.

[Mani]

Please suggest me a better alternative implementation

[Mani]

I don't want to duplicate the code and make it readable instead of bunch of functions together.

[Pavlo Liapota]

If I understood your use-case correctly, then this may help https://pl.kotl.in/HMdOjuGHP

[Mani]

Thank you so much! It looks the interface is missing. 🙂