RecyclerView - некорректно работает клик по изображению

@serSeventin · Регистрация: 09.11.2015

Студворк — интернет-сервис помощи студентам

Здравствуйте уважаемые форумчане!
Имеется RecyclerView, там загружаются изображения, теоретически для загрузки каждого изображения выделяется новый поток и не мешает главному, но дело в том, что когда загрузилось первое изображение, при нажатии на него ничего не происходит, а когда загрузились все изображения, то отрабатывается клик по первому, в чем проблема?

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
public class RVAdapterAllItems extends RecyclerView.Adapter<RVAdapterAllItems.ItemViewHolder> {
    String find;
    String code_catalog;
    int count_items;
 
    public interface setOnClickListener {
        void onClick(AllItemsBuyer allItemsBuyer, int position);
    }
 
    setOnClickListener setOnClick;
 
    public void setOnClickListenerRVAdapterAllItems(setOnClickListener setOnClick) {
        this.setOnClick = setOnClick;
    }
 
    public RVAdapterAllItems(String find, String code_catalog, int count_items) {
        this.find = find;
        this.code_catalog = code_catalog;
        this.count_items = count_items;
    }
 
    public static class ItemViewHolder extends RecyclerView.ViewHolder {
        PentagonImageView ivIcon;
        TextView  tvNameAllItems;
        TextView  tvPriceItem;
        ImageView start1;
        ImageView start2;
        ImageView start3;
        ImageView start4;
        ImageView start5;
        ProgressBar pbImage;
 
        ItemViewHolder(View itemView) {
            super(itemView);
 
            ivIcon         = (PentagonImageView) itemView.findViewById(R.id.ivIcon);
            tvNameAllItems = (TextView)          itemView.findViewById(R.id.tvNameAllItems);
            tvPriceItem       = (TextView)          itemView.findViewById(R.id.tvPriceItem);
            pbImage           = (ProgressBar)       itemView.findViewById(R.id.pbImage);
            start1         = (ImageView)         itemView.findViewById(R.id.star1);
            start2         = (ImageView)         itemView.findViewById(R.id.star2);
            start3         = (ImageView)         itemView.findViewById(R.id.star3);
            start4         = (ImageView)         itemView.findViewById(R.id.star4);
            start5         = (ImageView)         itemView.findViewById(R.id.star5);
        }
    }
 
    public RVAdapterAllItems.ItemViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
        View v = LayoutInflater.from(parent.getContext()).inflate(R.layout.item_all_items, parent, false);
 
        RVAdapterAllItems.ItemViewHolder ivh = new RVAdapterAllItems.ItemViewHolder(v);
 
        return ivh;
    }
 
    public void onBindViewHolder(final ItemViewHolder holder, final int position) {
        new BaseModel().loadXmlFile(new BaseModel.LoadUserCallback() {
            @Override
            public void onLoad(String result) {
                Log.d("RVAdapterAllItem", "OnClickImage = True");
 
                try {
                    final ArrayList<AllItemsBuyer> arrayList;
                    arrayList = new Parsers().getAllItemsBuyer(result);
 
                    if(arrayList.size() == 0)
                        return;
 
                    holder.tvNameAllItems.setText(arrayList.get(0).getNameItem());
                    holder.tvPriceItem.setText(arrayList.get(0).getPrice() + " руб.");
                    getRatingItem(arrayList.get(0).getRating(), holder);
 
                    holder.ivIcon.setOnClickListener(new View.OnClickListener() {
                        @Override
                        public void onClick(View v) {
                            if(Global.click) {
                                Log.d("RVAdapterAllItem", "click = " + arrayList.get(0).getNameItem());
                                setOnClick.onClick(arrayList.get(0), position);
                                Global.click = false;
                            }
                        }
                    });
 
                    DownloadPhoto(holder, arrayList.get(0).getIdItem(),
                            arrayList.get(0).getNameItem(), arrayList.get(0).getPrice());
                } catch (XmlPullParserException e) {
                    e.printStackTrace();
                } catch (IOException e) {
                    e.printStackTrace();
                }
            }
        }, "items.allitems",
            "in_find="       + new BaseModel().convertURL(find),
            "in_iditemtype=" + new BaseModel().convertURL(code_catalog),
            "in_position="   + new BaseModel().convertURL(String.valueOf(position)));
    }
 
    private void DownloadPhoto(final ItemViewHolder holder, final String IdItem, final String NameItem, final String Price) {
        new DownloadPhoto().downloadPhoto(IdItem, new BaseModel.LoadImageCallback() {
            @Override
            public void onLoad(ByteArrayOutputStream result) {
                holder.pbImage.setVisibility(View.GONE);
                holder.ivIcon.setImageBitmap(Global.decodeSampledBitmapFromByteArray(result,
                holder.ivIcon.getWidth(), holder.ivIcon.getHeight()));
            }
        });
    }
 
    public int getItemCount() {
        return count_items;
    }
 
    public void getRatingItem(String rating, final ItemViewHolder holder) {
        float ratingFloat = Float.parseFloat(rating);
        int   ratingInt   = (int) ratingFloat;
 
        switch (ratingInt) {
            case 1:
                holder.start1.setImageResource(R.drawable.star_true);
                break;
            case 2:
                holder.start1.setImageResource(R.drawable.star_true);
                holder.start2.setImageResource(R.drawable.star_true);
                break;
            case 3:
                holder.start1.setImageResource(R.drawable.star_true);
                holder.start2.setImageResource(R.drawable.star_true);
                holder.start3.setImageResource(R.drawable.star_true);
                break;
            case 4:
                holder.start1.setImageResource(R.drawable.star_true);
                holder.start2.setImageResource(R.drawable.star_true);
                holder.start3.setImageResource(R.drawable.star_true);
                holder.start4.setImageResource(R.drawable.star_true);
                break;
            case 5:
                holder.start1.setImageResource(R.drawable.star_true);
                holder.start2.setImageResource(R.drawable.star_true);
                holder.start3.setImageResource(R.drawable.star_true);
                holder.start4.setImageResource(R.drawable.star_true);
                holder.start5.setImageResource(R.drawable.star_true);
                break;
        }
    }
}

@Hank · 26.07.2017, 15:03

Мля.. ну и код... Кто так интерфейсы называет setOnClickListener?

Сообщение от serSeventin

то отрабатывается клик по первому, в чем проблема

Сообщение от serSeventin

setOnClick.onClick(arrayList.get(0), position);

Видимо причина в этой строке. Вы всегда передаете 0, а нужно position.

@serSeventin · 27.07.2017, 12:53 **[ТС]**

Нет, не в этой строке. Потому что для каждого объекта всегда новый лист. С этим разобрался, оказывается все работало. Теперь новый вопрос. Есть несколько фрагментов, на двух из них есть SearchView, если на первом фрагменте начать вводить текст, то сразу срабатывает переход на другой фрагмент, где есть SearchView. SearchView сразу становится в фокусе и туда передается текст. Все хорошо работает, но есть одна проблема, при переходе назад из фрагмента, появляются две лупы на первом, никак не могу понять что делать. Посоветовали использовать SearchView в активити, которое содержит фрагменты, но при переходе теряется фокус и текст. Что делать в этом случае?

Добавлено через 14 часов 44 минуты
Ну так что на счет SearchView?

Новые блоги и статьи Все статьи Все блоги /
Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Programma_Boinc 28.12.2025 Советы по крайней бережливости. Внимание, это ОЧЕНЬ длинный пост. Налог на собак: https:/ / ********/ gallery/ V06K53e Финансовый отчет в Excel: https:/ / ********/ gallery/ bKBkQFf Пост отсюда. . .	Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Нашел на реддите интересную статью под названием Anyone know where to get a free Desktop or Laptop? Ниже её машинный перевод. После долгих разбирательств я наконец-то вернула себе. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .

@serSeventin 1 / 1 / 0 Регистрация: 09.11.2015 Сообщений: 155
	27.07.2017, 12:53 [ТС]
	Нет, не в этой строке. Потому что для каждого объекта всегда новый лист. С этим разобрался, оказывается все работало. Теперь новый вопрос. Есть несколько фрагментов, на двух из них есть SearchView, если на первом фрагменте начать вводить текст, то сразу срабатывает переход на другой фрагмент, где есть SearchView. SearchView сразу становится в фокусе и туда передается текст. Все хорошо работает, но есть одна проблема, при переходе назад из фрагмента, появляются две лупы на первом, никак не могу понять что делать. Посоветовали использовать SearchView в активити, которое содержит фрагменты, но при переходе теряется фокус и текст. Что делать в этом случае? Добавлено через 14 часов 44 минуты Ну так что на счет SearchView? 0