Android Идентифицировать listView с помощью прослушивателя onItemClick

У меня есть два ListViews в моей деятельности, которые используют один и тот же OnItemClickListener. Есть ли способ определить, какой элемент ListViews я сейчас нажимаю? Я использовал этот код:

@Override
public void onItemClick(AdapterView<?> list, View view, int position,
        long id) {


    if (view.getId() == R.id.listDictionary) {
    // TODO Auto-generated method stub


    Intent intent = new Intent(MainActivity.this, WordActivity.class);
    DictionaryListElement ele = (DictionaryListElement) dictionaryList
            .getAdapter().getItem(position);
    intent.putExtra("word", ele.getWord());

    startActivity(intent);
    } else if (view.getId() == R.id.listFavourites) {
        Intent intent = new Intent(MainActivity.this, WordActivity.class);
        String ele = (String)favouritesList.getAdapter().getItem(position);
        intent.putExtra("word", ele);
        startActivity(intent);

    }
}

Но это не работает. Я думаю, что он получает идентификатор каждого нажатого элемента, а не ListViews


android listview onitemclicklistener onitemclick
person Mr.D    schedule 19.02.2014    source источник
comment
что не так в приведенном выше коде?   -  person Raghunandan    schedule 19.02.2014
comment
Попробуйте использовать идентификатор представления списка, а не view.getId()   -  person Abx    schedule 19.02.2014

Ответы (2)


arrow_upward
1
arrow_downward

Вы должны использовать идентификатор ListView (здесь ListView передается как AdapterView в onItemClick()), а не идентификатор View, так как этот View является элементом ListView.

if(list.getId() == R.id.listDictionary) {
    // item in dictionary list is clicked
} else if (list.getId() == R.id.listFavourites) {
   // item in favourite list is clicked
}
person Gopal Gopi    schedule 19.02.2014
comment
Я понимаю, что вопрос задавался, почему код не работает, и ваш ответ полностью правильный, но я не понимаю, почему можно продолжать использовать такую ​​запутанную логику. - person Alex K.; 19.02.2014
comment
@AlexanderGavrilov, почему ты считаешь этот код беспорядочным? но для меня это повторное использование кода и минимальный код, а правильный документированный код никогда не бывает грязным... - person Gopal Gopi; 19.02.2014
comment
Я заметил параметр списка только сейчас, LOL. - person Mr.D; 19.02.2014
comment
Потому что в идеале метод должен делать только одну вещь (есть реальные исключения, но не это). Кроме того, правильно документированный код - это самодокументированный код, что недостижимо при таких конструкциях. - person Alex K.; 19.02.2014

arrow_upward
1
arrow_downward

Зачем вам один и тот же слушатель, если вы различаете логику с если? Создайте отдельные слушатели для каждого представления. Это был бы более чистый код, и он также должен работать.

// dictionary listener
@Override
public void onItemClick(AdapterView<?> list, View view, int position,
        long id) {
    Intent intent = new Intent(MainActivity.this, WordActivity.class);
    DictionaryListElement ele = (DictionaryListElement) dictionaryList
            .getAdapter().getItem(position);
    intent.putExtra("word", ele.getWord());

    startActivity(intent);
}

// favorites listener
     @Override
        public void onItemClick(AdapterView<?> list, View view, int position,
                long id) {
            Intent intent = new Intent(MainActivity.this, WordActivity.class);
            String ele = (String)favouritesList.getAdapter().getItem(position);
            intent.putExtra("word", ele);
            startActivity(intent);
        }
person Alex K.    schedule 19.02.2014
comment
Спасибо за ответ. Мои мысли стали такими же сложными, как код Android - person Mr.D; 19.02.2014