7

ビューホルダー/アダプターパターンに従って、「ドキュメント」のグリッドビューを作成しています。アクティビティでは、ネットワーク クラスからコールバックを取得するため、各グリッド セルを異なる時間に更新できるようにする必要があります。

私がこれを行っている方法は、オブジェクト要素 (ドキュメント) から対応するビュー ホルダーへの Map マッピングを持つことです。アダプターがセルをリサイクルしているため、これを行う必要があるため、表示されていないセルを更新するためのコールバックを受信する場合があり、その場合、コールバック情報は無視されます。

アダプターのメソッドがこのように位置 0 に対して何度も呼び出されていることがわかりgetViewます。ただし、これは正常であることを読んでいます。

inflating position 0  ***  progressBar 1
recycling position 0
recycling position 0
inflating position 0  ***  progressBar 2
recycling position 0
recycling position 0
recycling position 0
recycling position 0
recycling position 0
recycling position 0

コールバックから更新する必要がある要素の 1 つはプログレス バーで、ドキュメントに関連付けられたマップに をUI追加して追跡します。viewHolder

このアプローチ全体は、最初のポジションを除くすべてのポジションでうまく機能します。私が識別できた唯一の違いは、これらの複数の呼び出しであったgetViewため、そこからデバッグを開始しました。progressBar私が更新しているのは " 2" です。progressBarこれは最新の膨張ですが、実際には(View.VISIBLE)geViewに応答しません。次に、「 1」setVisibleを更新するためにコードにいくつかの変更を加えたところ、動作しました。progressBar

これは、getViewが同じ位置で 2 回膨張していることを意味しますが、2 回目は使用されていないか、表示されていません。

私は何が間違っているのでしょうか?なぜプログレスバー1は機能するのにプログレスバー2は機能しないのですか? getView位置に対応する最新のビューが得られると思います。

ありがとう。

アダプタ:

public class DocumentPickerGridViewAdapter extends ArrayAdapter<Document> implements Filterable {
  private final DocumentPickerGridViewController picker;
  private ArrayList values;
  private ArrayList filtered;
  private LayoutInflater inflater;

  public DocumentPickerGridViewAdapter(Context context, ArrayList values) {
    super(context, R.id.documentPickerGridView, values);
    this.picker = (DocumentPickerGridViewController) context;
    this.filtered = values;
    this.values = (ArrayList)values.clone();
    inflater = (LayoutInflater) picker.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
  }

  @Override
  public View getView(int position, View convertView, ViewGroup parent) {
    View v = convertView;
    ViewHolder oldHolder;
    ViewHolder holder;
    Document doc = (Document) filtered.get(position);

    if (v == null) {
      v = inflater.inflate(R.layout.documentpickergriditem, parent, false);

      holder = new ViewHolder();
      holder.actionView = (Button) v.findViewById(R.id.document_action_button);
      holder.coverView = (ImageView) v.findViewById(R.id.documentCoverImage);
      holder.archiveButton = (Button) v.findViewById(R.id.document_archive_button);
      holder.progressView = (ProgressView) v.findViewById(R.id.documentProgress);
      holder.progressBar = (ProgressBar) v.findViewById(R.id.documentCoverImageProgressBar);


      picker.allProgressViews.add(holder);
      Log.d("MARIANO","adding to allProgressViews "+holder.progressView);
      Log.d("MARIANO","inflating position "+position);

      v.setTag(holder);
    }
    else{
      // here we are recycling, we should clean old stuff.
      holder = (ViewHolder)v.getTag();



      synchronized (picker){
        oldHolder = picker.documentHoldersMap.get(holder.doc);
      }

      if(oldHolder != null){
        oldHolder.progressView.setVisibility(View.GONE);
        oldHolder.progressBar.setVisibility(View.GONE);
      }

      // Document associated with previous holder it's now out of the adapter visible window.
      // we have to remove it from the map so if picker receives any call back we won't update anything.
      if(! holder.doc.uuid().equals(doc)){
        synchronized (picker){
          picker.documentHoldersMap.remove(holder.doc);
        }
      }
    }

    holder.doc = doc;
    holder.position = position;
    synchronized (picker){
      picker.documentHoldersMap.put(doc,holder);
    }

    /*
       TODO: check why view extendedGrid starts to measure the size of the grid after gridpicker view has disapear. This causes a ArrayOutofIndexes
       exception. DocumentGridPicker view does setDocuments(null); which clears all arrays, but the adapter stays setting as many cells as before.
       https://kaldorgroup.jira.com/browse/NEWSWEEK-49
    */
    if(picker.documents() != null && picker.documents().size() > 0 ){
      picker.setCoverView(holder.coverView, doc);
      picker.setActionButton(holder.actionView, doc);
      picker.setArchiveButton(holder.archiveButton, doc);
      picker.refreshButton(doc);
    }

    TextView textView = (TextView) v.findViewById(R.id.documentName);
    textView.setText(doc.name());

    if(position == 0)
      Log.d("MARIANO", "progressView: "+holder.progressView);

    return v;
  }

  @Override
  public Filter getFilter() {
    return new Filter() {
      @Override
      protected FilterResults performFiltering(CharSequence constraint) {
        FilterResults result = new FilterResults();

        if (constraint.equals(DocumentPickerGridViewController.FILTER_DOWNLOADED)) {
          ArrayList items = new ArrayList();

          synchronized (this) {
            items.addAll(values);
          }

          for (int i = items.size() - 1; i >= 0; i--) {
            if (((Document) items.get(i)).state() != DocumentStates.Downloaded)
              items.remove(i);
          }

          result.count = items.size();
          result.values = items;
        } else {  // ALL ITEMS
          synchronized (this) {
            result.count = values.size();
            result.values = values;
          }
        }
        return result;
      }

      @SuppressWarnings("unchecked")
      @Override
      protected void publishResults(CharSequence constraint, FilterResults results) {
        filtered = (ArrayList)results.values;
        DocumentPickerGridViewAdapter.this.notifyDataSetChanged();
        clear();
        for(int i = 0, l = filtered.size(); i < l; i++)
          add((Document)filtered.get(i));
        notifyDataSetInvalidated();
      }
    };
  }

  static class ViewHolder{
    ImageView coverView;
    Button actionView;
    Button archiveButton;
    ProgressView progressView;
    ProgressBar progressBar;
    int position;
    Document doc;
  }

}

グリッドビュー :

<com.kaldorgroup.newsweek.ExpandableGridView
                    android:id="@+id/documentPickerGridView"
                    android:layout_width="match_parent"
                    android:layout_height="wrap_content"
                    android:columnWidth="150dp"
                    android:numColumns="auto_fit"
                    android:horizontalSpacing="25dp"
                    android:stretchMode="spacingWidth"
                    android:verticalSpacing="20dp"
                    android:layout_marginTop="10dp"
                    android:layout_marginBottom="5dp"
                    android:layout_marginLeft="15dp"
                    android:layout_marginRight="15dp"
                    android:isScrollContainer="false"
                    android:paddingTop="350dp" />
4

1 に答える 1