169 похожих чатов

Всем привет, и всех с праздником! С помощью интента запускаю новую

активность. В ресайклере у ImageView у всех одинаковый OnClick, тк запускается анимация нажатия. В джава-коде Onclick обрабатываю, и там же хочу запустить новую активность. Активность зависит от нажатой кнопки. Единственный вариант проверки, какая кнопка нажата, придумал получать имя background-а ImageView и через свитч запускать активность. Похоже на лютый говнокод, как лучше сделать?

21 ответов

12 просмотров

это не похоже, это оно и есть

Вы смешиваете визуальную часть с бизнес-логикой. Если в activity нужно грузить картинку по названию, то передавайте в функцию название картинки, а не какое-то view из которого как-то нужно выковыривать название.

Andrey-Karchevsky Автор вопроса

Мне нужно по нажатию на ImageView просто запускать активность. Проблема была в том, что ImageView генерились из кода, и я только сейчас нагуглил, как там задать у них ID

Если нужно "просто запускать", то какая разница какой там id?

Andrey-Karchevsky Автор вопроса

Так от ID зависит, какая активность будет запущена

Ну, можно и так...

Andrey-Karchevsky Автор вопроса
Andrey-Karchevsky Автор вопроса

Что-то я не так сделал: В onBindViewHolder в цикле через 'img.setId()' задаю id от 0 до 5. В MainActivity следующий код (скрин). При нажатии на кнопку, приложение вылетает, трейс (скрин)

Во-первых, у вас startActivity вызывается в каждом case и ещё один раз на выходе. Оставьте вызов на выходе.

Andrey-Karchevsky Автор вопроса

Наоборот же, только на выходе вызывается, в кейсах только интент получаю

Я на скриншоте вижу - много их.

Andrey-Karchevsky Автор вопроса

А, да, у первых двух забыл убрать, извиняюсь

Почему onClick падает - я 🤷‍♂, наверное назначен не там и не так. Подозреваю, что в лейауте.

ты в xml устанавливаешь онклик чтоли?

Andrey-Karchevsky Автор вопроса

Падает, если тут начинаю задавать setId()

зачем?

id не для этого Если нужно привязать свои данные, то используй tag. А вообще, если приходится извращаться, то что-то однозначно делается не так.

если я правильно понял что ты хочешь сделать, то решается все своими листенерами которые вешаешь на кнопки

Andrey-Karchevsky Автор вопроса

Даааа, понял наконец-то, tag помог. Да, я понимаю, что сделал криво, сейчас попробую нормально

Там типа заранее неизвестно число кнопок.

Andrey-Karchevsky Автор вопроса

Сейчас попробую, спасибо за помощь!

Похожие вопросы

Обсуждают сегодня

Какой-то там пердун в 90-х решил, что есть какая-то разная типизация. Кого вообще это волнует?
КТ315
49
void terminal_scroll() { memmove(terminal_buffer, terminal_buffer + VGA_WIDTH, buffer_size - VGA_WIDTH); memset(terminal_buffer + buffer_size - VGA_WIDTH, 0, VGA_WIDTH); ...
Егор
47
Всем привет! Подскажите, пожалуйста, в чем ошибка? Настраиваю подключение к MySQL. Либы лежат рядом с exe. Все как по "учебнику"
Евгений
16
А можете как-то проверить меня по знаниям по ассемблеру?
A A
132
Здравствуйте! У меня появилась возможность купить книгу "Изучай Haskell во имя добра!". Но я где-то слышал, что эта книга устарела. Насколько это правда??
E
22
Здравствуйте! Я вот на stepic решаю задачи на хаскеле https://stepik.org/lesson/8443/step/8?unit=1578 мой код import Data.List (isInfixOf) removing :: String -> [String] ->...
E
10
Камрады, кто тесно работал с vtv, хотел уточнить. Ширина column задаётся жёстко на этапе создания дерева или можно в рантайме ее менять программно (не мышкой)?
Ed Doc
10
да ладно ... что там неочевидного ? глянуть в исх-ки датасета и/или кверика чтобы понять в каком месте и как выполняется обращения к св-вам blablaSQL - минутное дело, даже е...
Сергей
7
Здесь для arm кто-нибудь кодит ?
Nothing
52
Всем привет, у меня есть сервер принимающий входящие HTTP подключения, как проверить, что подключение было через прокси или нет, есть какие то поля в заголовках по которым мо...
Кибер Бомж
8
Карта сайта