Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
Y U NO CRAFTSMAN
Search
Xavi Rigau
October 09, 2014
Programming
3
160
Y U NO CRAFTSMAN
Presentation we gave at Droidcon Stockholm "Y U NO CRAFTSMAN".
By Paul Blundell and Xavi Rigau
Xavi Rigau
October 09, 2014
Tweet
Share
More Decks by Xavi Rigau
See All by Xavi Rigau
Android UI Testing with Espresso
xrigau
5
1.3k
Other Decks in Programming
See All in Programming
エラーって何種類あるの?
kajitack
5
290
Beyond Portability: Live Migration for Evolving WebAssembly Workloads
chikuwait
0
390
来たるべき 8.0 に備えて React 19 新機能と React Router 固有機能の取捨選択とすり合わせを考える
oukayuka
2
840
都市をデータで見るってこういうこと PLATEAU属性情報入門
nokonoko1203
1
560
A comprehensive view of refactoring
marabesi
0
970
#kanrk08 / 公開版 PicoRubyとマイコンでの自作トレーニング計測装置を用いたワークアウトの理想と現実
bash0c7
1
290
C++20 射影変換
faithandbrave
0
520
Kotlin エンジニアへ送る:Swift 案件に参加させられる日に備えて~似てるけど色々違う Swift の仕様 / from Kotlin to Swift
lovee
1
250
生成AIで日々のエラー調査を進めたい
yuyaabo
0
640
A2A プロトコルを試してみる
azukiazusa1
2
1.1k
KotlinConf 2025 現地で感じたServer-Side Kotlin
n_takehata
1
230
関数型まつりレポート for JuliaTokai #22
antimon2
0
150
Featured
See All Featured
Making Projects Easy
brettharned
116
6.3k
What’s in a name? Adding method to the madness
productmarketing
PRO
23
3.5k
How STYLIGHT went responsive
nonsquared
100
5.6k
Building a Scalable Design System with Sketch
lauravandoore
462
33k
Fight the Zombie Pattern Library - RWD Summit 2016
marcelosomers
233
17k
Optimizing for Happiness
mojombo
379
70k
The Art of Delivering Value - GDevCon NA Keynote
reverentgeek
15
1.5k
A better future with KSS
kneath
239
17k
Making the Leap to Tech Lead
cromwellryan
134
9.3k
Templates, Plugins, & Blocks: Oh My! Creating the theme that thinks of everything
marktimemedia
31
2.4k
"I'm Feeling Lucky" - Building Great Search Experiences for Today's Users (#IAC19)
danielanewman
228
22k
Stop Working from a Prison Cell
hatefulcrawdad
270
20k
Transcript
Y U NO CRAFTSMAN Xavi Rigau & Paul Blundell
Who we are @xrigau @blundell_apps
What is craftsmanship? craftsmanship ˈkrɑːf(t)smənʃɪp/ the quality of design and
work shown in something
Coder vs Craftsman “Make a chair”
We’re gonna talk about... - Improving day to day craftsmanship
- Creating a positive working environment - The final few steps to awesomeness - Opinionated Bonus material
Improving day to day craftsmanship
Whiteboard ideation
Brainstorming sessions
Helping non dev team members WTF??
Pair programming
Pairing tennis
Driver & Navigator Mistakes Driver Would make Mistakes Navigator Would
make Actual Mistakes
Best coding practices
Code reviews
Selfies & Gifs https://github.com/thieman/github-selfies
Caring about the CI
Static analysis reports
Testing is caring
CI Game LOL
Using the latest tools
Be “agile” build features the way that makes sense
YAGNI Overengineering You Ain’t Gonna Need It GSD
Pragmatism
Creating a positive working environment
Continuous communication
Morning standups
Daily news
Hack & Tells
Dojos
Zero walls office
2 keyboards 2 mice per desk
Standing desks
Remote working
Remote working
Hal9000/Jukebox music
CI Alarm
Xbox downtime
PUB!
Hire the best (for you)
The final steps to awesomeness
That extra 5%
Optimise & leave the main thread alone
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); } }
GPU Profiling
Show Overdraw
Polish the app
Animate all the things
User features - Second screen / Chromecast - Widget -
Wear - Daydream - LiveWallpaper
Behind the scenes - Content provider - Sync Adapter -
Deep linking / Web search deep linking
Listen for feedback
Measure the data
Measuring Tools Splunk MINT (a.k.a. Bugsense) Crashlytics
Follow the guidelines
What happens when you don’t follow the guidelines
Ensure your app listing is legit
Ensure your app content is legit
Debug screens
Gradle all the things
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"' } }
Versioning
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}"
Opinionated Bonus
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
Follow the examples - but not too closely
Patterns that work well for us
minSdkVersion 15 For more info: https://developer.android.com/about/dashboards/index.html
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()); } }
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); } }
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; } }
Hexagonal architecture cc. Alistair Cockburn
In summary
None
Questions?
We are hiring Liverpool, London, Berlin & New York
[email protected]