$ whois private lateinit var aakira: User data class User(val name: String, val twitterId: String, val githubId: String, val company: String) println("Name : ${aakira.name}") println("Twitter Id : ${aakira.twitterId}") println("Github Id : ${aakira.githubId}") println("Company : ${aakira.company}")
@_a_akira AAkira CyberAgent, Inc. Akira Aratani private lateinit var aakira : User data class User(val name: String, val twitterId: String, val githubId: String, val company: String) print("Name : ${aakira.name}”) println("Github Id : ${aakira.githubId}") print("Twitter Id : ${aakira.twitterId}") println("Company : ${aakira.company}") $ whois
定義 • bad • 非推奨 • not good • 書き方として問題はない、FRESHとしてのコード規約に沿ってはいない • good • FRESHとして推奨している書き方 ※ bad, not goodいずれも言語としては正しい書き方でコンパイルは通る コーディング規約上のbad, not good, good
IDEのSuggestionに従う class FeedFragment : Fragment() { override fun onCreate(savedInstanceState: Bundle?) { super.onCreate(savedInstanceState) // bad getActivity().finish() // good activity.finish() } } val array = ArrayList() // bad array.get(0) // good array[0]
型推論をなるべく使う val hoge = 0 // Int val foo = 10L // Long val bar = 100f // Float fun Context.isConnectToWifi() = (getSystemService(Context.CONNECTIVITY_SERVICE) as ConnectivityManager) .activeNetworkInfo?.type == ConnectivityManager.TYPE_WIFI
for文は書かない // not good for (i in 0..9) { } // good (0..9).forEach { } // good (index͕ཉ͍࣌͠) (0..9).forEachIndexed { index, value -> } Collections パッケージにあるforEachがあるのでfor文を書く必要はない
Rangeを利用 val char = 'K' // bad if (char >= 'A' && 'c' <= 'Z') print("Hit!") // good if (char in 'A'..'Z') print("Hit!") when (char) { in 'A'..'Z' -> print("Hit!") else -> return }
when val hoge: Hoge = Hoge() when (hoge) { is Hoge -> { } else -> { } } val hoge = 10 when (hoge) { in 0..4 -> print("0..4") in 5..10 -> print("5..10") } is range
Property // non-null & val private val hoge: Hoge = Hoge() private val drawablePadding: Int by lazy { activity.resources.getDimensionPixelSize(R.dimen.drawable_padding) } // non-null & var private lateinit var hoge: Hoge private var hoge: Hoge = Delegates.notNull() // nullable & var private var hoge: Hoge? = null
スコープ関数をなるべく使う class Hoge { fun fun1() {} fun fun2() {} } // bad val hoge = Hoge() hoge.fun1() hoge.fun2() // good val hoge = Hoge().apply { fun1() fun2() }
スコープ関数をなるべく使う class Hoge { fun fun1() {} fun fun2() {} } // bad val hoge = Hoge() hoge.fun1() hoge.fun2() // good val hoge = Hoge().apply { fun1() fun2() } ※ withは他で代用出来るので使わない run, let, apply, also
一度しか使わない変数は宣言しない class Foo class Bar(val foo: Foo) class Hoge { fun fun1() {} fun fun2(bar: Bar) {} fun fun3(foo: Foo) {} } // bad val hoge = Hoge() val foo = Foo() val bar = Bar(foo) hoge.fun1() hoge.fun2(bar) hoge.fun3(foo) // good Hoge().run { fun1() Foo().let { fun2(Bar(it)) fun3(it) } } スコープ関数を使って回避可能だが、過度にやりすぎると読みづらくなるデメリットもある 思考しないと読めない場合は変数宣言しても良い
aakira
coe401_
j5ik2o
kubode
blackbracken
aanrii
10xinc
masayaaoyama
yuuu
kentaroutakeda
yuya4
mpyw
jnunemaker
shpigford
stephaniewalter
brettharned
lara
bluesmoon
bermonpainter
chrislema
hatefulcrawdad
brianwarren
paulrobertlloyd
akmur