[bisq-network/bisq] Remove / refactor BSFormatter (tracking issue) (#3240)

[Justin Carter]

Reposting the design principles that led me to this refactoring:

- Minimize dependencies on all levels. ie. small public interfaces for classes and libraries.
- Prefer static functions for stateless helpers that don't contain domain logic
- Prefer composition over inheritance for code reuse
- Prefer interfaces over inheritance for polymorphism
- Don't depend on global state in regular classes, better to inject it via constructor

I assume these are not contentious and am omitting justification. Happy to discuss these points if there is dissent.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/issues/3240#issuecomment-530308217
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20190911/717697e1/attachment.html>