DEV Community

loading...

Discussion on: Why bother using Polymorphism?

Collapse
frontendphil profile image
Philipp Giese

However, now you've mixed all concerns into one big Uploader and coupled them together. This means if one concern changes you need to check all of them again. While I get the point I think this post might be misleading :) I've recently blogged about this topic. WDYT? philgiese.com/post/what-is-cohesio...

Collapse
chrisza4 profile image
Chakrit Likitkhajorn Author

I don't understand what do you mean by mixed all concern into one big Uploader. IMO, the Uploader classes are responsible for uploading to corresponding file storage and Uploader, as a static class, responsible for selecting appropriate uploader implementation. So I don't see how it is mixed together.

Can you clarify it a little bit more?

Collapse
frontendphil profile image
Philipp Giese

I have to apologize. I must have misread something. My comment really doesn't make much sense if I read your article again. Excuse my mumbling :D