Спрошу всё-таки Как тут можно улучшить код? package com.hexakosioihexekontahex.ion.authentication import android.os.Bundle import android.support.v7.app.AppCompatActivity import android.widget.Button import

android.widget.EditText
import android.widget.ImageView

class MainActivity : AppCompatActivity() {
var img: ImageView? = null
var btn: Button? = null
var email: EditText? = null
var pass: EditText? = null

val email_txt: String = "123@gmail.com"
val pass_txt: String = "123"


override fun onCreate(savedInstanceState: Bundle?) {
super.onCreate(savedInstanceState)
setContentView(R.layout.activity_main)
login()
}

fun login(){
img = findViewById(R.id.imageView2) as ImageView?
btn = findViewById(R.id.button) as Button?
email = findViewById(R.id.editText) as EditText?
pass = findViewById(R.id.editText2) as EditText?

btn!!.setOnClickListener {
when {
((email!!.text.toString()).equals(email_txt) && (pass!!.text.toString()).equals(pass_txt)) ->
img!!.setImageResource(R.drawable.unlocked)
else ->
img!!.setImageResource(R.drawable.wrong)
}
}
}
}

Залить его на pastebin для начала 😏

байндинги по айди

Хотя бы замени все: var img: ImageView? = null на: lateinit var img: ImageView что бы глаза твои !! не мозолили

я считаю что тут 2 необходимо вещи сделать: 1. Использовать kotlin extensions 2. !! заменить на ?

Каст в, например, Button? подразумевает, что button может не быть.

стоит использовать lateinit там, где уместно