Kotlinhomework2 #13
Kotlinhomework2 #13
Conversation
| <intent-filter> | ||
| <action android:name="android.intent.action.MAIN"/> | ||
| <category android:name="android.intent.category.LAUNCHER"/> | ||
| </intent-filter> |
There was a problem hiding this comment.
а зачем тебе второй интент фильтр LAUNCHER?) У нас только одна стартовая активити - это MainActivity
| private var greetings: String? = null | ||
| private var name: String? = null | ||
| private var textView: TextView? = null |
There was a problem hiding this comment.
если нам известно, что при создании активности эти поля в любом случае будут иметь значение, то можно возпользоваться отложенной инициализацией:
private lateinit var textView: TextView
это позволит нам не указывать начальное значение и обозначать поле non-nullable типом, что в дальнейшем не будет заставлять нас применять !! или ?. для его обработки
| } else { | ||
| getString(R.string.anon) | ||
| } | ||
| textView = findViewById(R.id.textViewHello) |
There was a problem hiding this comment.
Если в котлине появляется IF с условием что-то ==/!= null - то скорее всего игнорируется одна из главных фичей котлина)
можно записать это намного лакончинее:
name = savedInstanceState?.getString(NAME_KEY) ?: getString(R.string.anon)
| } | ||
| textView = findViewById(R.id.textViewHello) | ||
|
|
||
| val button: Button = findViewById(R.id.buttonNameYourSelf) |
There was a problem hiding this comment.
о findViewById нужно знать, но в следующий раз лучше сразу использовать View Binding, ну или Kotlin Synthetic
| if (requestCode == SecondActivity.GET_NAME_REQUEST_CODE && resultCode == RESULT_OK && data != null) { | ||
| val nameFromData: String? = data.getStringExtra(SecondActivity.NAME_KEY) | ||
| if (nameFromData != null) { | ||
| name = nameFromData | ||
| } | ||
| } |
There was a problem hiding this comment.
опять же, безопасные вызовы позволят нам сделать код лаконичнее и уменьшить обрамляющее условие:
if (requestCode == SecondActivity.GET_NAME_REQUEST_CODE && resultCode == RESULT_OK)
data?.getStringExtra(SecondActivity.NAME_KEY)?.let { name = it }
| @SuppressLint("SetTextI18n") | ||
| override fun onResume() { | ||
| super.onResume() | ||
| textView?.text = "$greetings, $name!" |
There was a problem hiding this comment.
а если бы textView был определён как lateinit, то можно было бы избежать безопасного вызова
| } | ||
|
|
||
| override fun afterTextChanged(editable: Editable) { | ||
| button.isEnabled = !TextUtils.isEmpty(editable) |
There was a problem hiding this comment.
у библиотеки kotlin есть удобные extension методы для работы со строками:
button.isEnabled = editable.isNotEmpty()
2 participants
Исправленный перевод на kotlin