1

私は2つのフラグメントを持っていますが、どちらもそのカテゴリを表示するために配列アダプタで動作します。最初のフラグメントは表示アイテムカテゴリ、2番目のフラグメントは表示ジョブカテゴリです。ただし、両方の部分にジョブカテゴリが表示されます。どの部分が間違っているのか分かりません。Android ArrayAdapter getViewは正しい位置を取得できません

これは、ユーザが対応するタブを選択したときにフラグメントを表示するための、私のViewPagerです。 setCategory()は、データベースからデータを受け取ったときに呼び出されます。

public class ViewPagerAdapter extends FragmentPagerAdapter { 

private List<Category> categoryList = new ArrayList(); 
private DisplayItemFragment itemFragment; 
private DisplayJobFragment jobFragment; 

public ViewPagerAdapter(FragmentManager fm) { 
    super(fm); 
} 

@Override 
public Fragment getItem(int position) { 
    Log.d("categoryList", categoryList.size()+" "); 
    if (position ==0) { 
     itemFragment = new DisplayItemFragment(categoryList); 
     return itemFragment; 
    }else{ 
     jobFragment = new DisplayJobFragment(categoryList); 
     return jobFragment; 
    } 

} 

@Override 
public int getCount() { 
    return 2; 
} 

@Override 
public CharSequence getPageTitle(int position) { 
    switch (position){ 
     // 
     //Your tab titles 
     // 
     case 0:return "Item"; 
     case 1:return "Job"; 
     default:return null; 
    } 
} 

public void setCategory(List<Category> list){ 
    List<Category> tempList = new ArrayList(); 
    for(int i=0;i<list.size();i++){ 
     if(list.get(i).getType().equals("item")){ 
      tempList.add(list.get(i)); 
     } 
    } 
    itemFragment.update(tempList); 
    Log.d("this.categoryListitem",tempList.size()+""); 
    tempList.clear(); 
    for(int i=0;i<list.size();i++){ 
     if(list.get(i).getType().equals("job")){ 
      tempList.add(list.get(i)); 
     } 
    } 
    jobFragment.update(tempList); 
    Log.d("this.categoryListjob",tempList.size()+""); 
} 
} 

この

は、フラグメントは、これはどちらのフラグメントはGridViewの上のカテゴリを表示するCategoryGridAdapterと呼びます

public class DisplayJobFragment extends Fragment { 

private View rootView; 
private GridView gridView; 
private List<Category> categoryList; 
private DisplayFragmentPresenter presenter; 
private TextView text; 
private CategoryGridAdapter categoryAdapter; 

public DisplayJobFragment(List<Category> categoryList) { 
    this.categoryList=categoryList; 
} 



@Override 
public void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
} 

@Override 
public View onCreateView(LayoutInflater inflater, ViewGroup container, 
         Bundle savedInstanceState) { 

    rootView = inflater.inflate(R.layout.fragment_display, container, false); 
    text = (TextView) rootView.findViewById(R.id.test); 
    categoryAdapter = new CategoryGridAdapter(getActivity(), R.layout.fragment_display, categoryList); 
    gridView = (GridView) rootView.findViewById(R.id.gridview); 
    gridView.setAdapter(categoryAdapter); 
    gridView.setTextFilterEnabled(true); 

    //Log.d("category",categoryList.get(0).getName()); 
    return rootView; 
} 

public void update(List<Category> list) { 
    this.categoryList.clear(); 
    for(int i=0;i<list.size();i++){ 
     if(list.get(i).getType().equals("job")){ 
      this.categoryList.add(list.get(i)); 
     } 
    } 
    text.setText("job refresh"); 
    //categoryAdapter = new CategoryGridAdapter(getActivity(), R.layout.fragment_display, this.categoryList); 
    for(int i=0;i<this.categoryList.size();i++){ 
     Log.d("categoryListjob",categoryList.get(i).getName()); 
    } 

} 
} 

職種表示するためのフラグメントである項目のカテゴリに

public class DisplayItemFragment extends Fragment { 

private View rootView; 
private GridView gridView; 
private List<Category> categoryList; 
private DisplayFragmentPresenter presenter; 
private CategoryGridAdapter categoryAdapter; 
private TextView text; 

public DisplayItemFragment(List<Category> categoryList) { 
    this.categoryList=categoryList; 
} 

@Override 
public void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
} 

@Override 
public View onCreateView(LayoutInflater inflater, ViewGroup container, 
         Bundle savedInstanceState) { 

    rootView = inflater.inflate(R.layout.fragment_display, container, false); 
    text = (TextView) rootView.findViewById(R.id.test); 
    categoryAdapter = new CategoryGridAdapter(getActivity(), R.layout.fragment_display, categoryList); 
    gridView = (GridView) rootView.findViewById(R.id.gridview); 
    gridView.setAdapter(categoryAdapter); 
    gridView.setTextFilterEnabled(true); 

    //Log.d("category",categoryList.get(0).getName()); 
    return rootView; 
} 

public void update(List<Category> list) { 
    this.categoryList.clear(); 
    for(int i=0;i<list.size();i++){ 
     if(list.get(i).getType().equals("item")){ 
      this.categoryList.add(list.get(i)); 
     } 
    } 
    text.setText("item refresh"); 
    for(int i=0;i<this.categoryList.size();i++){ 
     Log.d("categoryListitem",categoryList.get(i).getName()); 
    } 
    } 

を表示することです

public class CategoryGridAdapter extends ArrayAdapter<Category> { 

private List<Category> categoryList; 
private Context context; 

public CategoryGridAdapter(Context context, int resource, List<Category> categoryList) { 
    super(context, resource, categoryList); 
    this.context = context; 
    this.categoryList = categoryList; 
} 

@Override 
public boolean isEnabled(int position) { 
    return true; 
} 

@Override 
public int getCount() { 
    return ((null != categoryList) ? 
      categoryList.size() : 0); 
} 

@Override 
public Category getItem(int position) { 
    return ((null != categoryList) ? 
      categoryList.get(position) : null); 
} 

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

@Override 
public View getView(int position, View convertView, ViewGroup parent) { 
    View view = convertView; 
    LayoutInflater layoutInflater = (LayoutInflater) context 
      .getSystemService(Context.LAYOUT_INFLATER_SERVICE); 

    if (null == view) { 
     view = layoutInflater.inflate(R.layout.grid_category, null); 
    } 

    final Category category = categoryList.get(position); 
    Log.d("categorygridlist",category.getName()+" "+position+ " " + categoryList.size()); 

    if (category != null) { 
     final CardView categoryGridLayout = (CardView) view.findViewById(R.id.category_gridlayout); 
     final TextView categoryName = (TextView) view.findViewById(R.id.category_name); 
     final ImageView categoryIcon = (ImageView) view.findViewById(R.id.category_icon); 

     categoryName.setText(category.getName()); 
     categoryName.setSelected(true); 
     try { 
      byte[] decodedString = Base64.decode(category.getImage(), Base64.DEFAULT); 
      BitmapFactory.Options options=new BitmapFactory.Options();// Create object of bitmapfactory's option method for further option use 
      options.inPurgeable = true; // inPurgeable is used to free up memory while required 
      Bitmap decodedByte1 = BitmapFactory.decodeByteArray(decodedString, 0, decodedString.length);//Decode image, "thumbnail" is the object of image file 
      Bitmap decodedByte = Bitmap.createScaledBitmap(decodedByte1, 50 , 50 , true);// convert decoded bitmap into well scalled Bitmap format. 

      categoryIcon.setImageBitmap(decodedByte); 
     } catch (Exception e) { 
      e.printStackTrace(); 
     } 

     categoryGridLayout.setOnClickListener(new View.OnClickListener() { 
      @Override 
      public void onClick(View v) { 
       Toast.makeText(context, "worked", Toast.LENGTH_LONG).show(); 
      } 
     }); 
    } 

    return view; 
} 

私は、ビューのページャにアイテムカテゴリを表示することを期待します。ユーザーがジョブタブを選択したときにビューページャでアイテムタブとジョブカテゴリを選択したときにジョブカテゴリを表示するだけです。誰にでもこれに関する解決策がありますか? ViewPagerAdapterクラスのおかげで

+0

@Mike M. Log.dを使用してtempListを正しく更新しました。 – phoon

+0

最初にあなたの 'update()'メソッドが表示されなかったので、私はコメントを削除しました。 –

+0

ああ、私は今それを参照してください。両方の 'Fragment'は' ViewPagerAdapter'で定義した同じ 'List categoryList'を使用しており、それらのコンストラクタを渡しています。それをしないでください。それぞれに新しいArrayList を作成し、そのパラメータをコンストラクタから削除します(とにかくそこにあるべきではありません)。 –

答えて

3

は、あなたが自分のコンストラクタの両方Fragment秒に渡しているList<Category> categoryListを持っており、各Fragmentは、そのデータセットと同じリストを設定しています。 setCategory()update()メソッドのログプリントは正しいと思われます。これは、アップデートが順番に行われており、その間にリストをクリアしているからです。

を各Fragmentに渡す代わりに、それぞれ独自の新しいArrayList<Category>を作成し、各コンストラクタからそのパラメータを削除します。

関連する問題