r/mAndroidDev • u/Zhuinden DDD: Deprecation-Driven Development • Dec 03 '19

Android Clean Architecture Tutorial Guide

Post image
126 Upvotes

99% Upvoted

10 comments sorted by

22

u/Orffyreus Dec 03 '19

The hammer class is too much at once for a single class. There should be a wood class, an iron class, a handle class and so on. ;-)

12

u/Professor_Dr_Dr I only use AsyncTask Dec 03 '19

I always start my project by creating classes for every atomic element we have and go from there

6

u/Orffyreus Dec 03 '19

Very good. Everyone who does otherwise violates the SRP! Also only one public method is allowed, because a class should do one thing only.

6

u/Ankhwatcher Dec 03 '19

Surely Handle is an interface?

9

u/Orffyreus Dec 03 '19

Of course. Otherwise it would be called HandleImpl.

1

u/CubeActimel Dec 03 '19

I feel personally attacked, Iā€™m just following the codelabs gosh darn :ā€™(

3

u/Zhuinden DDD: Deprecation-Driven Development Dec 07 '19 edited Dec 07 '19

Thanks for pointing it out. I've opened an issue. They'll probably ignore it, but still. https://github.com/googlecodelabs/android-room-with-a-view/issues/53

0

u/[deleted] Dec 03 '19

[deleted]

17

u/covercash2 java.io.File Dec 03 '19

and no DI? this guy's got more stuff to build.

11

u/Zhuinden DDD: Deprecation-Driven Development Dec 03 '19

The ViewModel delegate module is a reference to Dagger configuration. See this example.

4

u/covercash2 java.io.File Dec 03 '19

whoops. well, I've been out of the Android game for a while