Android: оператор Switch, используемый с OnItemClickListener (), всегда выбирает последний случай независимо от выбранного элемента - PullRequest
Новая
       12

Android: оператор Switch, используемый с OnItemClickListener (), всегда выбирает последний случай независимо от выбранного элемента

1 голос
/ 11 марта 2011

Трудно придумать лучший способ сделать это.То, что у меня происходит, это представление галереи, загруженное изображениями, с текстовым представлением прямо под ним.Я хотел бы заполнить содержимое текстового представления в зависимости от того, какое изображение было нажато.Я следовал стандартному руководству GalleryView, в котором вы создали собственный класс ImageAdapter, который расширяет класс BaseAdapter.При этом я создал OnItemClickListener для галереи и предположил, что следующим логическим шагом будет создание оператора switch, чтобы выяснить, что поместить в текстовое представление.

Так что в конце концов я понял, что явероятно, следует выполнять итерацию по элементам галереи, в отличие от параметров, передаваемых в метод onitemclicklistener ().Проблема заключается в том, что независимо от того, по какому элементу щелкнули, вывод в нужное текстовое представление всегда выполняется так, как если бы последний элемент был нажат.Я закомментировал «случай 2», а затем он начал всегда принимать случай 1. Что могло бы привести к тому, что оператор switch переходит в «default» для последнего закодированного случая, даже если я не определил случай по умолчанию?

Кроме того, я вижу противоречивые замечания относительно оператора switch с точки зрения зацикливания.Он должен зацикливаться сам по себе, да?Если так, то я думаю, что это, вероятно, так, но, поскольку он всегда выбирает случай 2, я не понимаю, что вывод текста должен меняться?Мое заявление переключателя все испорчено?мне нужно включить какой-то цикл?Что я здесь не так делаю?

Пользовательский класс ImageAdapter: 

 public class ImageAdapter extends BaseAdapter
{
    private Context context;
    private int itemBackground;

    public ImageAdapter(Context c) 
    {
        context = c;
        //---setting the style---
        TypedArray a = obtainStyledAttributes(R.styleable.Gallery1);
        itemBackground = a.getResourceId(
         R.styleable.Gallery1_android_galleryItemBackground, 0);
        a.recycle();                    
        }
 //---returns the number of images---
 public int getCount() {
     return imageIDs.length;
 }
 //---returns the ID of an item--- 
 public Object getItem(int position) {
     return position;
 }            

 public long getItemId(int position) {
     return position;
 }

 //---returns an ImageView view---
 public View getView(int position, View convertView, ViewGroup parent) {
     ImageView imageView = new ImageView(context);
     imageView.setImageResource(imageIDs[position]);
     imageView.setScaleType(ImageView.ScaleType.FIT_XY);
     imageView.setLayoutParams(new Gallery.LayoutParams(150, 120));
     imageView.setBackgroundResource(itemBackground);
     return imageView;
 }
}
}

А вот код, который я использую для создания просмотра галереи и onitemclicklistener: 

Gallery gallery = (Gallery) findViewById(R.id.top_gallery);
    gallery.setAdapter(new ImageAdapter(this));
    gallery.setOnItemClickListener(new OnItemClickListener()
    {
        public void onItemClick(AdapterView parent, 
                View v, int position, long id) 
                {
                    TextView tView = (TextView) findViewById(R.id.cat_desc);
                    switch(gallery.getSelectedItemPosition())
                    {   
                    case 0:
                        tView.setText("Option1");

                    case 1:
                        tView.setText("Option2");

                    case 2:
                        tView.setText("Option3");
}
    }

Ответы [ 3 ]

5 голосов
/ 11 марта 2011

Вам не хватает оператора break в операторе case, и код выполняется линейно.Вот почему последний всегда выбирается.

1 голос
/ 11 марта 2011

Вам нужен перерыв;в твоем случае иначе оно провалится.Попробуйте: 

case 0 : tView.setText("Option1"); break;
0 голосов
/ 11 марта 2011

Вам не хватает breaks;

switch(gallery.getSelectedItemPosition())
{   
    case 0:
        tView.setText("Option1");
        break;
    case 1:
        tView.setText("Option2");
        break;
     case 2:
        tView.setText("Option3");
        break;
}
...