Приложение cra sh для последнего элемента, удаленного в RecyclerView - PullRequest
1 голос
/ 26 января 2020

Я использую RecyclerView и заполняю данные из базы данных и сохраняю их в arrayLists. я также добавил onLongClickListener () в представление переработчика, которое удаляет строку, по которой долго нажимали. Все работает нормально, но когда я удаляю последний, приложение падает, и я получаю Java .Lang.IndexOutOfBoundsException index: 0 size: 0.

вот мой код

public class MainActivity extends AppCompatActivity implements AlarmRecyclerViewInterface {

    FloatingActionButton mAlarmAddButton;
    RecyclerView mRecyclerView;
    SQLiteDatabase db;
    AlarmsDBhelperClass mAlarmsDBhelperClass;
    ArrayList<String> nameArrayList = new ArrayList<>();
    ArrayList <String> modeArrayList = new ArrayList<>();
    ArrayList<String> repeatArrayList = new ArrayList<>();
    ArrayList<String> hoursArrayList = new ArrayList<>();
    ArrayList<String> minArrayList = new ArrayList<>();
    Switch mSwitch;
    AlarmAdapter alarmAdapter = new AlarmAdapter(hoursArrayList,minArrayList,modeArrayList,repeatArrayList,nameArrayList,this);
    ImageView emptyImageView;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        setRequestedOrientation(ActivityInfo.SCREEN_ORIENTATION_PORTRAIT);

        //Initializing RecyclerView, DatabaseHelperClass, FAB button, The ON OFF switch & the empty ImageView
        mAlarmsDBhelperClass = new AlarmsDBhelperClass(getApplicationContext());
        mAlarmAddButton = findViewById(R.id.btnAlarmADD);
        mRecyclerView = findViewById(R.id.alarmList);
        mSwitch = findViewById(R.id.switchONOFF);
        emptyImageView = findViewById(R.id.empty_view);

        //DividerItemDecoration class is used for getting a vertical line between rows of RecyclerView
        DividerItemDecoration itemDecoration = new DividerItemDecoration(this,DividerItemDecoration.VERTICAL);
        mRecyclerView.addItemDecoration(itemDecoration);

        //Getting a writable reference of the Database.
        db = mAlarmsDBhelperClass.getWritableDatabase();

        //Retrieving values from the database and storing them in custom ArrayLists
        boolean isDataEmpty = getAlarm(db);

        //Checking if our arrayList is empty? if yes then display some empty list text or an image
        if (!isDataEmpty){
            mRecyclerView.setVisibility(View.GONE);
            emptyImageView.setImageResource(R.drawable.no_alarm_black_white);
            emptyImageView.setVisibility(View.VISIBLE);
        }
        else {
            mRecyclerView.setVisibility(View.VISIBLE);
            emptyImageView.setVisibility(View.GONE);
        }

        //FAB Event handling
        mAlarmAddButton.setImageResource(R.drawable.addalarm);
        mAlarmAddButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                Intent mIntent = new Intent(getApplicationContext(), AddAlarm.class);
                startActivity(mIntent);
            }
        });

       //Warping up with the recyclerView
        mRecyclerView.setLayoutManager(new LinearLayoutManager(getApplicationContext()));
        mRecyclerView.setAdapter(alarmAdapter);
        mRecyclerView.setHasFixedSize(true);
    }

    public boolean getAlarm(SQLiteDatabase db) {
        Cursor cursor = db.rawQuery("SELECT * FROM alarms", new String[]{});
        boolean rowExists;
        if (cursor.moveToFirst()) {
            do {
                nameArrayList.add(cursor.getString(2));
                modeArrayList.add(cursor.getString(3));
                repeatArrayList.add(cursor.getString(4));
                hoursArrayList.add(Integer.toString(cursor.getInt(5)));
                minArrayList.add(Integer.toString(cursor.getInt(6)));
                rowExists = true;
            } while (cursor.moveToNext());
        }else {
            rowExists = false;
        }
        cursor.close();
        return rowExists;
    }

    //RecyclerView's onClick()
    @Override
    public void onItemClick(int position) {
        Toast.makeText(this, "Alarm Clicked !", Toast.LENGTH_SHORT).show();
    }

    //RecyclerView's onLongClick()
    @Override
    public void onLongItemClick(int position) {
        hoursArrayList.remove(position);
        //Updating the recyclerView
        alarmAdapter.notifyItemRemoved(position);
        //Deleting the row from the database
        db.delete("alarms","hours=?",new String[]{hoursArrayList.get(position)});
        Toast.makeText(this,"Alarm Deleted !", Toast.LENGTH_SHORT).show();
    }
}

вот мой RecyclerView Адаптер

public class AlarmAdapter extends RecyclerView.Adapter<AlarmAdapter.AlarmView> {
    //Variables for the main recycler view
    private ArrayList<String> hoursArrayList;
    private ArrayList<String> minArrayList;
    private ArrayList<String> modeArrayList;
    private ArrayList<String> repeatArrayList;
    private ArrayList<String> nameArrayList;
    private AlarmRecyclerViewInterface mInterface;

    public AlarmAdapter(ArrayList<String> hours,ArrayList<String> mins,ArrayList<String> mode,ArrayList<String> repeat,ArrayList<String> name,AlarmRecyclerViewInterface mInterface){
        this.hoursArrayList = hours;
        this.minArrayList = mins;
        this.modeArrayList = mode;
        this.nameArrayList = name;
        this.repeatArrayList = repeat;
        this.mInterface = mInterface;
    }

    @NonNull
    @Override
    public AlarmView onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
        LayoutInflater layoutInflater = LayoutInflater.from(parent.getContext());
        View view = layoutInflater.inflate(R.layout.alarm_profile,parent,false);
        return new AlarmView(view);
    }

    @Override
    public void onBindViewHolder(@NonNull AlarmView holder, int position) {
        if(Integer.parseInt(hoursArrayList.get(position)) < 10 ){
            holder.hours.setText("0"+hoursArrayList.get(position));
        }else {
            holder.hours.setText(hoursArrayList.get(position));
        }
        if (Integer.parseInt(minArrayList.get(position)) < 10){
            holder.mins.setText("0"+minArrayList.get(position));
        }else {
            holder.mins.setText(minArrayList.get(position));
        }
        holder.repeat.setText(repeatArrayList.get(position));
        holder.mode.setText(modeArrayList.get(position));
        holder.name.setText(nameArrayList.get(position));
    }

    @Override
    public int getItemCount() {
        return hoursArrayList.size();
    }

    /*ItemTouchHelper.SimpleCallback itemTouchHelperCallback = new ItemTouchHelper.SimpleCallback(0,ItemTouchHelper.RIGHT) {
        @Override
        public boolean onMove(@NonNull RecyclerView recyclerView, @NonNull RecyclerView.ViewHolder viewHolder, @NonNull RecyclerView.ViewHolder target) {
            return false;
        }

        @Override
        public void onSwiped(@NonNull RecyclerView.ViewHolder viewHolder, int direction) {
            nameArrayList.remove(viewHolder.getAdapterPosition());
            modeArrayList.remove(viewHolder.getAdapterPosition());
            repeatArrayList.remove(viewHolder.getAdapterPosition());
            hoursArrayList.remove(viewHolder.getAdapterPosition());
            minArrayList.remove(viewHolder.getAdapterPosition());
        }
    };*/

    public class AlarmView extends RecyclerView.ViewHolder{
        TextView hours,mins,repeat,name,mode;
        public AlarmView(@NonNull View itemView) {
            super(itemView);
            hours = itemView.findViewById(R.id.txtHOUR);
            mins = itemView.findViewById(R.id.txtMins);
            repeat = itemView.findViewById(R.id.txtRepeatDays);
            name = itemView.findViewById(R.id.txtName);
            mode = itemView.findViewById(R.id.txtMode);

            itemView.setOnClickListener(new View.OnClickListener() {
                @Override
                public void onClick(View view) {
                    mInterface.onItemClick(getAdapterPosition());
                }
            });

            itemView.setOnLongClickListener(new View.OnLongClickListener() {
                @Override
                public boolean onLongClick(View view) {
                    mInterface.onLongItemClick(getAdapterPosition());
                    return true;
                }
            });

        }
    }

}

Спасибо за помощь!

1 Ответ

3 голосов
/ 26 января 2020
  • Cra sh: - IndexOutOfBoundsException

    • Причина: - Вы удалили данные из списка, а затем снова извлекаете это значение из списка, чтобы удалить данные из базы данных в это время они будут sh, потому что они не имеют значения, поскольку вы удалили данные.

    • Решение: - Сначала вы можете удалить из базы данных, а затем удалить из вашего списка.

  • Пример: -

    Заменить ниже метод onLongClick () Override в вашем классе.
//RecyclerView's onLongClick()
@Override
public void onLongItemClick(int position) {

    // Best way to remove data from list
    if(hoursArrayList != null && hoursArrayList.size() > 0){

        //Deleting the row from the database
        db.delete("alarms","hours=?",new String[]{hoursArrayList.get(position)});

        //Delete from your list if you successfully removed data from your database.
        hoursArrayList.remove(position);

        //Updating the recyclerView
        if(alarmAdapter){
            alarmAdapter.notifyItemRemoved(position);
        }

        //Message to show User.
        Toast.makeText(this,"Alarm Deleted !", Toast.LENGTH_SHORT).show();
    }

}
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...