Блог

Мой код работает 9 999 из 10 000 раз. Но примерно раз в 10 000 раз приложение завершает UninitializedPropertyAccessException работу с ошибкой, это проблематично, поскольку в производстве находится около 20 000 ~ 30 000 устройств Android с моим кодом.

Похоже, что иногда фактическое присвоение lateinit переменной происходит недостаточно быстро (и, следовательно, вызывает исключение выше).

У кого-нибудь еще была подобная проблема? Каково было ваше решение?

TcpService.kt

класс TcpService: Service() {

     private lateinit var mTcpClient: TcpClient

    override fun onCreate() {
        registerReceiver(object: BroadcastReceiver() {
            override fun onReceive(context: Context, intent: Intent) {
                when(intent.action) {
                    // This will cause an Uninitialized Property Exception 1 in 10,000 times
                    ACTION_SEND_MESSAGE -> mTcpClient.sendMessageAsync(intent.getStringExtra(EXTRA_NEW_MESSAGE)
                }
            }
        }, IntentFilter(ACTION_SEND_MESSAGE)
        })
    }

    override fun onStart() {
        mTcpClient = mTcpClient()
    }

    // ...

}
  

TcpClient.kt

 class TcpClient() {

    init {
        // ...
        sendBroadcast(Intent(ACTION_SEND_MESSAGE).putExtra(EXTRA_NEW_MESSAGE, "Message Contents")
    }

    // ...

}
  

ах, это случилось со мной однажды, и я потратил целый день, а потом решил это (до сих пор не знаю, как), но почему бы вам не сделать mTcpClient null, а затем проверить null:

  `when(intent.action) {
 // This will cause an Uninitialized Property Exception 1 in 10,000 times
                ACTION_SEND_MESSAGE ->{
if(mTcpClient != null){
mTcpClient.sendMessageAsync(intent.getStringExtra(EXTRA_NEW_MESSAGE)  
}else{
mTcpClient=mTcpClient()
mTcpClient.sendMessageAsync(intent.getStringExtra(EXTRA_NEW_MESSAGE)
}
  }
    }`
  

что-то в этом роде, я не профессионал, извините

Вместо того, чтобы присваивать свойству значение null, а затем проверять значение null, что может быть опасно в некоторых сценариях, вы можете использовать isInitialized метод для своих lateInit свойств :

     when (intent.action) {
        if (::mTcpClient.isInitialized) { ACTION_SEND_MESSAGE ->
          mTcpClient.sendMessageAsync(intent.getStringExtra(EXTRA_NEW_MESSAGE)
        }
    }