1

В MainActivity кнопка запускает код из другого файла

val btnExit: Button = findViewById<Button>(R.id.btnExit) as Button
btnExit.setOnClickListener {
   exitSid()
}

код из другого файла, имя пакета одинаковое

https://pastebin.com/embed/NaExjTDB

fun exitSid() {
    val alertDialogBuilder = AlertDialog.Builder(this)
 
    alertDialogBuilder.setTitle("Закрыть программу")
    alertDialogBuilder.setIcon(R.drawable.appicon)
    alertDialogBuilder.setMessage("Выйти из программы?")
    alertDialogBuilder.setCancelable(false)
    alertDialogBuilder.setPositiveButton("Выйти"){_,_ ->
        Toast.makeText(this,"exiting!",Toast.LENGTH_SHORT).show()
    }
    alertDialogBuilder.setNegativeButton("Нет"){_,_ ->
        Toast.makeText(this,"Вы не вышли из программы!",Toast.LENGTH_SHORT).show()
    }
    alertDialogBuilder.setNeutralButton("Cancel"){_,_ ->
        Toast.makeText(this,"Вы отменили выход из программы!",Toast.LENGTH_SHORT).show()
    }
    val alertDialog = alertDialogBuilder.create()
    alertDialog.show()
}

Проблема: программа ругается на this

1
  • 1
    Сделайте из функции exitSid экстеншен-функцию: fun Context.exitSid(). 2 авг 2021 в 9:24

1 ответ 1

5

В языке Kotlin, так же как и в Java, this — это ссылка на текущий объект. Находясь внутри глобальной функции, никакого объекта нет, ссылаться на this не имеет смысла (кроме случая с экстеншен-функцией, см. ниже). Поэтому вы и получаете ошибку компиляции.

Многим сущностям в андроиде нужен контекст. Почитайте, какие они бывают.

Как исправить ошибку? Два варианта:

  1. Передать контекст как аргумент:

    fun exitSid(context: Context) {
        Toast.makeText(context, "exiting!", LENGTH_SHORT).show()
    }
    

    Из активити можно вызвать так (активити является контекстом):

    exitSid(this)
    
  2. Либо передать контекст как ресивер, то есть сделать функцию экстеншен-функцией, тогда this снова обретет смысл:

    fun Context.exitSid() {
        Toast.makeText(this, "exiting!", LENGTH_SHORT).show()
    }
    

    В этом случае из активити эта функция вызывается так:

    this.exitSid()
    

    Или просто:

    exitSid()
    
5
  • в статье написано "Неправильное использование Context может легко привести к утечкам памяти в Android приложении." Поэтому какой из двух вариантов на ваш взгляд лучше? (и на всякий случай - у меня теперь алерт, а не тост)
    – jurvrn
    2 авг 2021 в 9:46
  • 1
    Любой из способов, они отличаются только синтаксисом, как вам кажется выглядит более красиво, так и пишите. К утечкам не приведет, так как контекст никуда не сохраняем. Экстеншен-функции это фича языка котлин. В процессе компиляции ресивер по сути станет первым аргументом этой функции. Если в IDE выполнить "decompile kotlin to java", то в этом легко можно убедиться. 2 авг 2021 в 9:56
  • Для меня первый вариант более понятен - когда в качестве аргумента функции exitSid передаем (content: this). Но ваш второй пример как раз является extensions. А раз котлин надо значит его использовать ) А данном моем случае что расширяется класс или функция?
    – jurvrn
    2 авг 2021 в 9:59
  • хочу все свои функции вынести в отдельный файл, поэтому и нужно выбрать правильный вариант, может там и переменные будут... которые будут оставаться в памяти.
    – jurvrn
    2 авг 2021 в 10:08
  • Почитал также kotlinlang.ru/docs/reference/extensions.html, но не понял как это применить к "В большинстве случаев используйте контекст, доступный непосредственно из компонента, в котором вы работаете в данный момент. Вы можете безопасно хранить ссылку на него, если она не выходит за пределы жизненного цикла этого компонента. Как только вам нужно сохранить ссылку на контекст в объекте, который живет за пределами вашей Activity или другого компонента, даже временно, используйте ссылку на контекст приложения."
    – jurvrn
    2 авг 2021 в 10:10

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.