Y U NO CRAFTSMAN

Presentation we gave at Droidcon Stockholm "Y U NO CRAFTSMAN".

By Paul Blundell and Xavi Rigau

Transcript

1. Y U NO CRAFTSMAN Xavi Rigau & Paul Blundell
2. Who we are @xrigau @blundell_apps
3. What is craftsmanship? craftsmanship ˈkrɑːf(t)smənʃɪp/ the quality of design and work shown in something
4. Coder vs Craftsman “Make a chair”
5. We’re gonna talk about... - Improving day to day craftsmanship - Creating a positive working environment - The final few steps to awesomeness - Opinionated Bonus material
6. Improving day to day craftsmanship
7. Whiteboard ideation
8. Brainstorming sessions
9. Helping non dev team members WTF??
10. Pair programming
11. Pairing tennis
12. Driver & Navigator Mistakes Driver Would make Mistakes Navigator Would make Actual Mistakes
13. Best coding practices
14. Code reviews
15. Selfies & Gifs https://github.com/thieman/github-selfies
16. Caring about the CI
17. Static analysis reports
18. Testing is caring
19. CI Game LOL
20. Using the latest tools
21. Be “agile” build features the way that makes sense
22. YAGNI Overengineering You Ain’t Gonna Need It GSD
23. Pragmatism
24. Creating a positive working environment
25. Continuous communication
26. Morning standups
27. Daily news
28. Hack & Tells
29. Dojos
30. Zero walls office
31. 2 keyboards 2 mice per desk
32. Standing desks
33. Remote working
34. Remote working
35. Hal9000/Jukebox music
36. CI Alarm
37. Xbox downtime
38. PUB!
39. Hire the best (for you)
40. The final steps to awesomeness
41. That extra 5%
42. Optimise & leave the main thread alone
43. Strict Mode private void initializeStrictMode() { if (BuildConfig.DEBUG) { ThreadPolicy threadPolicy = new ThreadPolicy.Builder() .detectAll() .penaltyLog() .penaltyDeath() .build(); StrictMode.setThreadPolicy(threadPolicy); VmPolicy vmPolicy = new VmPolicy.Builder() .detectAll() .penaltyLog() .penaltyDeath() .build(); StrictMode.setVmPolicy(vmPolicy); } }
44. GPU Profiling
45. Show Overdraw
46. Polish the app
47. Animate all the things
48. User features - Second screen / Chromecast - Widget - Wear - Daydream - LiveWallpaper
49. Behind the scenes - Content provider - Sync Adapter - Deep linking / Web search deep linking
50. Listen for feedback
51. Measure the data
52. Measuring Tools Splunk MINT (a.k.a. Bugsense) Crashlytics
53. Follow the guidelines
54. What happens when you don’t follow the guidelines
55. Ensure your app listing is legit
56. Ensure your app content is legit
57. Debug screens
58. Gradle all the things
59. Build types buildTypes { debug { versionName "${VERSION_NAME}_${GIT_SHA}" runProguard false signingConfig signingConfigs.debug buildConfigField "String", "BUGSENSE_KEY", 'BuildConfig.INVALID' buildConfigField "boolean", "AB_TEST", 'false' } qa { runProguard false signingConfig signingConfigs.debug buildConfigField "String", "BUGSENSE_KEY", '"disSecret"' } release { runProguard true signingConfig signingConfigs.release buildConfigField "String", "BUGSENSE_KEY", '"lolNotTellingYou"' } }
60. Versioning
61. Proper versioning ext { GIT_SHA = gitSha() CI_BUILD_NUMBER = jenkinsBuildNumber() VERSION_CODE = 19101 // scheme: MINSDK-VERSION dd-ddd VERSION_NAME = "1.0.1" } def jenkinsBuildNumber() { // Local builds will always trump jenkins return System.getenv().BUILD_NUMBER?.toInteger() ?: 9999 } def gitSha() { return 'git rev-parse --short HEAD'.execute().text.trim() } // ... versionCode "${VERSION_CODE}${CI_BUILD_NUMBER}" as Integer versionName "${VERSION_NAME}_${GIT_SHA}"
62. Opinionated Bonus
63. The dark side of AOSP try { mWallpaper = getCurrentWallpaperLocked(context); } catch (OutOfMemoryError e) { Log.w(TAG, "No memory load current wallpaper", e); } try { BitmapFactory.Options options =new BitmapFactory.Options(); return BitmapFactory.decodeStream(is, null, options); } catch (OutOfMemoryError e) { Log.w(TAG, "Can't decode stream", e); } https://android.googlesource.com in WallpaperManager.java - line 263
64. Follow the examples - but not too closely
65. Patterns that work well for us
66. minSdkVersion 15 For more info: https://developer.android.com/about/dashboards/index.html
67. Activity lifecycle callbacks public interface ActivityLifecycleCallbacks { void onActivityCreated(Activity activity, Bundle savedInstanceState); void onActivityStarted(Activity activity); void onActivityResumed(Activity activity); void onActivityPaused(Activity activity); void onActivityStopped(Activity activity); void onActivityDestroyed(Activity activity); void onActivitySaveInstanceState(Activity activity, Bundle outState); } public class MyActivityLifecycleCallbacks implements Application.ActivityLifecycleCallbacks { … } public class MyApplication extends Application { @Override public void onCreate() { super.onCreate(); registerActivityLifecycleCallbacks(new MyActivityLifecycleCallbacks()); } }
68. Harden! Do not crash out there public class LoggingUncaughtExceptionHandler implements Thread.UncaughtExceptionHandler { private static final String TAG = "MyApp"; @Override public void uncaughtException(Thread thread, Throwable ex) { Log.wtf(TAG, "Swallowed an uncaught exception.", ex); } } private void swallowExceptionsInRelease() { if (!BuildConfig.DEBUG) { Thread.UncaughtExceptionHandler handler = new LoggingUncaughtExceptionHandler(); Thread.currentThread().setUncaughtExceptionHandler(handler); } }
69. newInstance all the things! public class WidgetImageLoader { private final Retriever memoryRetriever; private final Retriever fileRetriever; public static WidgetImageLoader newInstance(Context context) { Retriever memoryRetriever = MemoryRetriever.getInstance(); Retriever fileRetriever = FileRetriever.newInstance(context); return new WidgetImageLoader(memoryRetriever, fileRetriever); } WidgetImageLoader(Retriever memoryRetriever, Retriever fileRetriever) { this.memoryRetriever = memoryRetriever; this.fileRetriever = fileRetriever; } }
70. Hexagonal architecture cc. Alistair Cockburn
71. In summary
73. Questions?
74. We are hiring Liverpool, London, Berlin & New York info@novoda.com