Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

The article ends with a diagram showing a bunch of classes whose names end with "er". In my experience, that's usually a strong indicator of confusing design. Objects that have a single method and no real state should just be functions.


Conversely, objects should have a single responsibility [1] and so functions that are unrelated should go on different objects. An InvoiceFetcher is definitely unrelated to a PaymentApplier, unless one takes the argument that "Its about money", in which case you have to argue for a single giant class.

[1] http://c2.com/cgi/wiki?PrinciplesOfObjectOrientedDesign


I dunno, I always thought objects were about managing state. If an object does nothing except provide a method, why can't the method just be a free function instead?

    InvoiceFetcher.FetchInvoice()
    PaymentApplier.ApplyPayment()
just seems terrible.


Because Java. At least I can use an interface to define the function's specification, and then use that in mocks. On the one hand, it seems very different to what I learned about OOP twenty odd years ago, but on the other, it makes for extremely well designed and easy to understand code.



On Java and Java-like languages, objects are also a good option for providing namespaces.


In several languages coughJavacough you can't really have free functions.




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: