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
CursorはMCPを使った方が良いぞ
taigakono
1
250
設計やレビューに悩んでいるPHPerに贈る、クリーンなオブジェクト設計の指針たち
panda_program
6
2k
Azure AI Foundryではじめてのマルチエージェントワークフロー
seosoft
0
160
効率的な開発手段として VRTを活用する
ishkawa
0
130
Goで作る、開発・CI環境
sin392
0
230
GraphRAGの仕組みまるわかり
tosuri13
8
530
Flutterで備える!Accessibility Nutrition Labels完全ガイド
yuukiw00w
0
160
コードの90%をAIが書く世界で何が待っているのか / What awaits us in a world where 90% of the code is written by AI
rkaga
52
33k
チームで開発し事業を加速するための"良い"設計の考え方 @ サポーターズCoLab 2025-07-08
agatan
1
330
GitHub Copilot and GitHub Codespaces Hands-on
ymd65536
2
150
VS Code Update for GitHub Copilot
74th
2
630
Webの外へ飛び出せ NativePHPが切り拓くPHPの未来
takuyakatsusa
2
540
Featured
See All Featured
Let's Do A Bunch of Simple Stuff to Make Websites Faster
chriscoyier
507
140k
I Don’t Have Time: Getting Over the Fear to Launch Your Podcast
jcasabona
32
2.4k
Stop Working from a Prison Cell
hatefulcrawdad
270
21k
Gamification - CAS2011
davidbonilla
81
5.4k
No one is an island. Learnings from fostering a developers community.
thoeni
21
3.4k
Reflections from 52 weeks, 52 projects
jeffersonlam
351
20k
Bash Introduction
62gerente
614
210k
Fireside Chat
paigeccino
37
3.5k
How to train your dragon (web standard)
notwaldorf
94
6.1k
Design and Strategy: How to Deal with People Who Don’t "Get" Design
morganepeng
130
19k
Unsuck your backbone
ammeep
671
58k
Building a Scalable Design System with Sketch
lauravandoore
462
33k
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]