Что лучше: getActivity() или onAttach(Context) во фрагментах?

Question

[terminator-light]

Получить ссылку на activity во фрагментах можно этими способами: вызовом getActivity(), либо onAttach(Context) и приведением к Activity.
Почему рекомендуют использовать 2й способ вместо первого? Есть какая-то загвоздка?

Comments

[Денис Загаевский]

Кто рекомендует?

[terminator-light]

Денис Загаевский, в книге Android программирование для профессионалов. И ещё в некоторых статьях читал

[Денис Загаевский]

terminator-light, и что, с примерами кода прям?

[terminator-light]

Денис Загаевский, просто любительский интерес. Со вторым способом получается больше кода, но почему-то первый способ не использует. Книга 2017 года. А вы какой способ предпочитаете?

Answer 1

[Денис Загаевский]

terminator-light, ты думаешь, я из любопытства спрашиваю? Мне непонятно, как вообще можно сравнивать коллбек и метод для доступа. В коллбеке активити становится тебе доступна первый раз, если тебе надо при этом что-то сделать - делаешь в этом коллбеке. Если в дальнейшем тебе понадобится активити - получаешь её из getActivity. При этом если в коллбеке ты возьмешь и сохранишь ссылку, это будет неправильно, вплоть до утечек памяти при плохом раскладе(например, если ты сделаешь setRetainInstance(true)).

Comments

[AWE64]

Если обнулять ссылку в onDetach(), не должно быть утечки.

[Денис Загаевский]

AWE64, если, то да. А зачем это извращение, если эта ссылка будет абсолютно аналогична getActivity()?

[AWE64]

Денис Загаевский, я вижу плюс в том, что не нужно каждый раз дергать getActivity() и кастовать активити к интерфейсу, если она реализует интерфейс.

[Денис Загаевский]

AWE64, я уж позабыл, когда такое делал руками... А когда делал, то в базовом классе фрагмента обычно был метод requireActivity(Class), который возвращал нужный инстанс. Обнуления вот такие это прямой путь к ошибкам. Имхо.

[AWE64]

Денис Загаевский, какие конкретно ошибки?

[Денис Загаевский]

AWE64, появляется вероятность, что ты забудешь обнулить, как ни странно.

[AWE64]

Денис Загаевский, на мой взгляд слабый аргумент. Если мы говорим о ситуации, когда разработчик знает этот приём и использует его правильно, мне на ум не приходит каких-то страшных последствий, которые могут возникнуть. По крайней мере, у нас в production коде такое есть, и работает оно отлично.
А по забывчивости или невнимательности можно везде накосячить.

[Денис Загаевский]

AWE64, лучше писать код, который позволяет тебе ошибиться в меньшем количестве мест, чем в большем. Я не говорю, что это супер плохо, я говорю, что это довольно странный велосипед, который ещё и порождает лишний бойлерплейт. Твой аргумент принят. Но я так просто не делаю, при большом количестве фрагментов работать через активити тупо неудобно.