0

Создаю кнопки динамически: сколько их будет изначально я не знаю, это зависит от длины массива.

for (int i = 1; i < strWord.length; i++) {
    LinearLayout.LayoutParams lParams = new LinearLayout.LayoutParams(
            wrapContent, wrapContent);
    int btnGravity = Gravity.LEFT;
    lParams.gravity = btnGravity;
    Button btnNew = new Button(this);
    btnNew.setText(strWord[i]);
    llMain.addView(btnNew, lParams);
} 

Далее, мне необходимо обращаться к этим кнопкам по id. Как мне использовать метод .setId(), чтобы записать id в массив, и потом по номеру элемента обращаться к кнопке?

Код

public int[] btnId;
btnId[i] = btnNew.setId(i);

компилятор не принимает.

2
  • ну и костыль же вы придумали!
    – s_klepcha
    21 сен 2015 в 21:50
  • 1
    ну извините...я только осваиваю программирование 21 сен 2015 в 21:55

3 ответа 3

2

А почему бы не сложить кнопки в замечательный ArrayList и с id париться не надо. + Лучше сразу кнопкам назначить листенеры что бы они сами дергали код по нажатию, тогда если задача позволяет можно вообще их в массив не собирать.

2
  • Пишу игру на подобие виселицы. После вывода кнопок на экран, с буквами присутствующими в слове, при нажатии на кнопку необходимо окрашивать её в зеленый цвет и выводить в TextView, при положительном результате, и в красный при отрецательном 21 сен 2015 в 21:50
  • Тогда массив и правда не нужен. Можно каждой кнопке назначить OnClickListener который одним из параметров имеет view. Его можно привести к button и делать все что угодно с ним.
    – Kota1921
    22 сен 2015 в 9:04
-1

НИКОГДА, НИКОГДА не применяйте SetId()!!! Так вы можете случайно назначить уже существующий ID другого элемента и что тогда? 2 элемента с одинаковым ID, и тогда как понять телефону к какому элементу вы хотели обратиться, лучше доверьтесь телефону, когда создаёте ID он вам сам выдаст его, можете использовать метод getId() для View елемента

public int[] btnId;
btnId[i] = btnNew.getId();

Такой код будет более верным

-1

Я думаю вы используете не правильный подход. Но если очень надо то можно генерировать уникальный id при помощи View.generateViewId(), сгенерированный id не будет конфликтовать с R.id. Поэтому утверждение @BORSHEVIK не верно.

7
  • @Дмитрий Александров@ использовал SetId() метод установки ID своим View компонентам, по его методике может возникнуть конфликт при совпадении, что тут неверного? Чем плохи те ID, которые выдаются при создании View?
    – BORSHEVIK
    22 сен 2015 в 9:04
  • 1. При создании View из кода ей присваивается значение NO_ID. 2. При использовании View.generateViewId() он не будет конфликтовать с R.id.
    – antslava
    22 сен 2015 в 9:10
  • Смотри исходники grepcode.com/file/repository.grepcode.com/java/ext/…
    – antslava
    22 сен 2015 в 9:13
  • Да, тут вы правы, но данный метод всё равно плох, лучше использовать массив Button-ов длинной в strWord.length и сравнивать с View в OnClick()
    – BORSHEVIK
    22 сен 2015 в 10:10
  • @BORSHEVIK, человек спросил как ему установить ID. :) Я ответил как. Из его условия непонятно что конкретно он хочет делать. Если ему нужен текст из Button, то я бы установил один ClickListener на всех и в полученное при клике View доставал text.
    – antslava
    22 сен 2015 в 10:17

Ваш ответ

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

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