获取卡片视图标签时出现空指针异常

Question

我创建了一个自定义列表，其中包含卡片视图、按钮、文本视图和开关。我已经为卡片视图设置了标签。

但是当我试图在 table 视图持有者中检索这个标签时，它抛出了一个空指针异常。

如果我在监听器中获取标签，它不会抛出任何异常。我不知道从哪里访问这个标签。

如果状态为 1，我想为开关执行 setChecked true，如果状态为 0，setChecked 应该为 false。

但我不知道该从哪里开始？

CustomAlertAdapter

public class TableListAdapter extends RecyclerView.Adapter<TableListAdapter.TableViewHolder> {


    public static TimeTableHelper db;
    public static TimeTableList timeTableList;

    public static int cardId,id,status=0;
    public static boolean editMode;
    public static List<TimeTable> tableList;
    public static TimeTable t = new TimeTable();
    public static TimeTable ci;

    public TableListAdapter(List<TimeTable> tableList,TimeTableList timeTableList) {
        this.tableList = tableList;
        this.timeTableList = timeTableList;
    }
    private Context context;

    public TableListAdapter(Context context) {
        this.context = context;
    }
    @Override
    public int getItemCount() {
          return tableList.size();
    }

    @Override
    public void onBindViewHolder(TableViewHolder tableViewHolder, int i) {


        ci = tableList.get(i);

        tableViewHolder.cv.setTag(i);

        tableViewHolder.aSwitch.setTag(i);

        Log.d("setId", String.valueOf(i));
        tableViewHolder.tableTitle.setText(ci.getTitle());

        ((GradientDrawable)tableViewHolder.color.getBackground()).setColor(ci.getTableColor());



    }

   @Override
   public TableViewHolder onCreateViewHolder(ViewGroup viewGroup, int i) {

        View  itemView = LayoutInflater.
                   from(viewGroup.getContext()).
                   inflate(R.layout.table_card, viewGroup, false);


       return new TableViewHolder(itemView);
   }

  public static class TableViewHolder extends RecyclerView.ViewHolder {

      protected TextView tableTitle;
      protected CardView cv;
      protected SwitchCompat aSwitch;
      protected Button color;


      public TableViewHolder(final View v) {
          super(v);
          tableTitle = (TextView) v.findViewById(R.id.tableTitle);
          cv = (CardView) v.findViewById(R.id.card_view);
          aSwitch = (SwitchCompat) v.findViewById(R.id.switch2);
          color = (Button) v.findViewById(R.id.selectColor);

          db = new TimeTableHelper(timeTableList);


          cv.setOnLongClickListener(new View.OnLongClickListener() {

              @Override
              public boolean onLongClick(final View v) {
                  // TODO Auto-generated method stub

                  final AlertDialog.Builder builder = new AlertDialog.Builder(timeTableList);

                  builder.setTitle("Delete Time Table")
                          .setMessage("Are you sure you want to Delete this Time Table?")
                          .setPositiveButton(android.R.string.yes, new DialogInterface.OnClickListener() {
                              public void onClick(DialogInterface dialog, int which) {

                                  cardId = (int)v.getTag();

                                  Log.d("cardId", String.valueOf(cardId));

                                  t = tableList.get(cardId);
                                  id = t.getId();

                                  t = db.getTable(id);
                                  db.deleteTable(t);

                                  Intent intent = new Intent(timeTableList,TimeTableList.class);
                                  timeTableList.finish();
                                  timeTableList.startActivity(intent);

                              }

                          })


                          .setNegativeButton(android.R.string.no, new DialogInterface.OnClickListener() {
                              public void onClick(DialogInterface dialog, int which) {
                                  // do nothing
                              }
                          })

                          .setIcon(R.drawable.ic_warning_black_36dp)
                          .show();
                  return true;
              }

          });


          cv.setOnClickListener(new View.OnClickListener() {
              @Override
              public void onClick(final View v) {

                  cardId = (int)v.getTag();

                  Log.d("cardId", String.valueOf(cardId));

                  t = tableList.get(cardId);
                  id = t.getId();

                  Log.d("Id",String.valueOf(id));

                  editMode = true;
                  Intent i = new Intent(timeTableList, NewTimeTable.class);
                  i.putExtra("editMode", editMode);
                  i.putExtra("tableId", id);
                  timeTableList.startActivity(i);
              }
          });


          cardId = (int) cv.getTag();

          Log.d("cardId", String.valueOf(cardId));

          t = tableList.get(cardId);
          id = t.getId();

          if(status == 1)
          {
              aSwitch.setChecked(true);
          }
          else
          {
              aSwitch.setChecked(false);
          }


          aSwitch.setOnCheckedChangeListener(new CompoundButton.OnCheckedChangeListener() {
              @Override
              public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {



                  if (isChecked) {

                      status = 1;

                      t = db.getTable(id);
                      t.setStatus(status);
                      db.updateStatus(t);

                      Log.d("status", String.valueOf(status));

                      final List<TimeTable> events = db.getAllTables();
                      for (TimeTable cn : events) {
                          String log = "Id: " + cn.getId() + " ,Title: " + cn.getTitle() +
                                  "Status: " + cn.getStatus() + ",color: " + cn.getTableColor();
                          Log.d("Data ", log);
                      }

                  } else {

                      status = 0;

                      t = db.getTable(id);
                      t.setStatus(status);
                      db.updateStatus(t);
                      final List<TimeTable> events = db.getAllTables();
                      for (TimeTable cn : events) {
                          String log = "Id: " + cn.getId() + " ,Title: " + cn.getTitle() +
                                  "Status: " + cn.getStatus() + ",color: " + cn.getTableColor();
                          Log.d("Data ", log);
                      }
                      Log.d("status", String.valueOf(status));
                  }

              }
          });

      }
  }

    public void updateAdapaterList(List<TimeTable> newTimeTableList) {
        //Replace the current list with new list
        this.tableList = newTimeTableList;
        //notify the adapter that the data set has changed
        notifyDataSetChanged();
    }
}

谢谢。

Answer 1

您应该在您的视图持有者中使用 ID、标签或任何类型的标识符，因为您的视图持有者在回收者视图中被重复使用。我相信这就是给你带来麻烦的原因。

您应该使用 onBindViewHolder 来分配和链接每个视图持有者的数据。

为了在该方法中分配侦听器和 ID，您应该执行如下操作：

public void onBindViewHolder(TableViewHolder tableViewHolder, int i) {
    final TimeTable ci = tableList.get(i);

    //...

    tableViewHolder.setOnSomethingListener() {
    // You can use the ci reference here because you declared it as final
    // When this view is reused and references another position, the listener will be updated on the onBindViewHolder, to reference the new position that it will be being associated with
    }

}

此外，在旁注中，来自此 class 的所有静态变量可能都不应是静态的，因为如果您碰巧在其他视图中有更多适配器实例，您最终不需要它们互相干扰。

Answer 2

您应该在适配器的 onBindMethod 而非 ViewHolder.

中将内容分配给 ViewHolder 中引用的 UI 字段

您的代码示例：

public class TableListAdapter extends RecyclerView.Adapter<TableListAdapter.TableViewHolder> {
  private Context context;

  public TableListAdapter(Context context) {
      this.context = context;
  }
  @Override
  public int getItemCount() {
        return tableList.size();
  }

  @Override
  public void onBindViewHolder(TableViewHolder tableViewHolder, int i) {

    /* update referenced UI fields in view holder */

    tableViewHolder.cv.setOnClickListener(new View.OnClickListener() {

          @Override
          public void onClick(final View v) {

              cardId = (int)v.getTag();

              Log.d("cardId", String.valueOf(cardId));

              t = tableList.get(cardId);
              id = t.getId();

              Log.d("Id",String.valueOf(id));

              editMode = true;
              Intent i = new Intent(timeTableList, NewTimeTable.class);
              i.putExtra("editMode", editMode);
              i.putExtra("tableId", id);
              timeTableList.startActivity(i);
          }
      });    

      tableViewHolder.cv.setOnLongClickListener(new View.OnLongClickListener() {

            @Override
            public boolean onLongClick(final View v) {
                // TODO Auto-generated method stub

                final AlertDialog.Builder builder = new AlertDialog.Builder(timeTableList);

                builder.setTitle("Delete Time Table")
                        .setMessage("Are you sure you want to Delete this Time Table?")
                        .setPositiveButton(android.R.string.yes, new DialogInterface.OnClickListener() {
                            public void onClick(DialogInterface dialog, int which) {

                                cardId = (int)v.getTag();

                                Log.d("cardId", String.valueOf(cardId));

                                t = tableList.get(cardId);
                                id = t.getId();

                                t = db.getTable(id);
                                db.deleteTable(t);

                                Intent intent = new Intent(timeTableList,TimeTableList.class);
                                timeTableList.finish();
                                timeTableList.startActivity(intent);

                            }

                        })


                        .setNegativeButton(android.R.string.no, new DialogInterface.OnClickListener() {
                            public void onClick(DialogInterface dialog, int which) {
                                // do nothing
                            }
                        })

                        .setIcon(R.drawable.ic_warning_black_36dp)
                        .show();
                return true;
            }

        });
  }

 @Override
 public TableViewHolder onCreateViewHolder(ViewGroup viewGroup, int i) {

    /* Inflate view and pass it to view holder */
    View  itemView = LayoutInflater.from(viewGroup.getContext()).inflate(R.layout.table_card, viewGroup, false);

    return new TableViewHolder(itemView);
 }

public static class TableViewHolder extends RecyclerView.ViewHolder {

    protected TextView tableTitle;
    protected CardView cv;
    protected SwitchCompat aSwitch;
    protected Button color;


    public TableViewHolder(View v) {
        super(v);

        cv = (CardView) v.findViewById(R.id.card_view);   // Reference your layout fields.

    }    
}

}

获取卡片视图标签时出现空指针异常

Getting null pointer exception while getting tag of card view

java

android

nullpointerexception

android-adapter

android-cardview