Я новичок в программировании на Android, в своем приложении я использую recycleview для отображения содержимого базы данных SQLite, я читаю базу данных в Arraylist, который показываю в recycleview. Я успешно реализовал функцию смахивания влево (удаление элемента) и вправо (изменение элемента). У меня проблема при левом смахивании, значит, с функцией удаления я реализовал функцию отмены:

    void restoreItem( SequenceItem sequenceItem, int position) {
    sequenceItemList.add(position, sequenceItem);
    notifyItemInserted(position);
    notifyItemRangeChanged(position, getItemCount());
    }

(SequenceItem - это мой пользовательский класс элементов, а sequenceItemList - мой массив)

Эта функция хорошо работает, если я удаляю элемент списка между первым или последним элементом и отменяю их. Если я удаляю первый или последний элемент в корзине после отмены, я вижу пустое пространство в этой позиции, макет элемента списка абсолютно не виден, он просто белый. Затем я добавил "recycleview.scrollToPosition" после вызова метода восстановления:

    recycleAdapter.restoreItem(deletedItem, deletedIndex);
    recyclerView.scrollToPosition(deletedIndex);

После этого изменения, если я удалю первый элемент из корзины, чем нажму «Отменить», ранее удаленный элемент будет правильно восстановлен, но если я удалю последний элемент из корзины и нажму «Отменить», я все равно вижу пустой (белый) пункт.

Не могли бы вы помочь мне, в чем может быть проблема? Если вам нужна дополнительная информация, я могу опубликовать дополнительный код или рассказать больше о приложении.

ИЗМЕНИТЬ 06.10.2018 Я прикрепил класс RecyclerItemTouchHelper, потому что, на мой взгляд, это вызывает проблему:

public class RecyclerItemTouchHelper extends ItemTouchHelper.SimpleCallback {
    private RecyclerItemTouchHelperListener listener;
    private Context mContext;

    public RecyclerItemTouchHelper(Context mContext, int dragDirs, int swipeDirs, RecyclerItemTouchHelperListener listener) {
        super(dragDirs, swipeDirs);
        this.listener = listener;
        this.mContext = mContext;
    }

    //This is not used
    @Override
    public boolean onMove(@NonNull RecyclerView recyclerView, @NonNull RecyclerView.ViewHolder viewHolder, @NonNull RecyclerView.ViewHolder viewHolder1) {
        return false;
    }

    @Override
    public void onSwiped(@NonNull RecyclerView.ViewHolder viewHolder, int direction) {
        if (listener != null) {
            listener.onSwiped(viewHolder, direction, viewHolder.getAdapterPosition());
        }
    }

    @Override
    public void clearView(RecyclerView recyclerView, RecyclerView.ViewHolder viewHolder) {
        View list_foreground = ((RecycleAdapter.mViewHolder)viewHolder).list_foreground;
        getDefaultUIUtil().clearView(list_foreground);
    }

    @Override
    public int convertToAbsoluteDirection(int flags, int layoutDirection) {
        return super.convertToAbsoluteDirection(flags, layoutDirection);
    }

    @Override
    public void onSelectedChanged(RecyclerView.ViewHolder viewHolder, int actionState) {
        if (viewHolder != null) {
            View list_foreground = ((RecycleAdapter.mViewHolder)viewHolder).list_foreground;
            getDefaultUIUtil().onSelected(list_foreground);
        }
    }

    @Override
    public void onChildDraw(Canvas c, RecyclerView recyclerView, RecyclerView.ViewHolder viewHolder, float dX, float dY, int actionState, boolean isCurrentlyActive) {

        float mdX = dX / 1;

        if(actionState == ItemTouchHelper.ACTION_STATE_SWIPE){

            Bitmap icon;
            Paint p = new Paint();

            Resources resources = mContext.getResources();

            View itemView = viewHolder.itemView;
            float height = (float) itemView.getBottom() - (float) itemView.getTop();
            float width = height / 3;

            if(dX > 0){ //Swipe right
                p.setColor(Color.parseColor("#D32F2F"));
                RectF background = new RectF((float) itemView.getLeft() , (float) itemView.getTop(), mdX, (float) itemView.getBottom());
                c.drawRect(background,p);
                icon = BitmapFactory.decodeResource(resources, R.drawable.ic_delete_white);
                RectF icon_dest = new RectF((float) itemView.getLeft() + width ,(float) itemView.getTop() + width,(float) itemView.getLeft()+ 2*width,(float)itemView.getBottom() - width);
                c.drawBitmap(icon,null,icon_dest,p);
            } else { //Swipe left
                p.setColor(Color.parseColor("#388E3C"));
                RectF background = new RectF((float) itemView.getRight() + mdX, (float) itemView.getTop(), (float) itemView.getRight(), (float) itemView.getBottom());
                c.drawRect(background,p);
                icon = BitmapFactory.decodeResource(resources, R.drawable.ic_edit_white);
                RectF icon_dest = new RectF((float) itemView.getRight() - 2*width ,(float) itemView.getTop() + width,(float) itemView.getRight() - width,(float)itemView.getBottom() - width);
                c.drawBitmap(icon,null,icon_dest,p);
            }
        }
        super.onChildDraw(c, recyclerView, viewHolder, mdX, dY, actionState, isCurrentlyActive);
    }

    @Override
    public void onChildDrawOver(Canvas c, RecyclerView recyclerView, RecyclerView.ViewHolder viewHolder, float dX, float dY, int actionState, boolean isCurrentlyActive) {
        View list_foreground = ((RecycleAdapter.mViewHolder)viewHolder).list_foreground;
        getDefaultUIUtil().onDrawOver(c, recyclerView, list_foreground, dX, dY, actionState, isCurrentlyActive);
    }
}

А вот реализация активности свайпа:

    @Override
public void onSwiped(final RecyclerView.ViewHolder viewHolder, int direction, final int position) {
    //Swipe LEFT to DELETE
    if (direction == ItemTouchHelper.RIGHT) {
        Snackbar snackbar = Snackbar.make(rootLayout, "Phase " + (position + 1) + " is removed!", Snackbar.LENGTH_LONG);

        //Backup of removed item dor UNDO purpose
        final SequenceItem deletedItem = sequenceItemList.get(viewHolder.getAdapterPosition());
        final int deletedIndex = viewHolder.getAdapterPosition();

        //Remove the item from recyclerview
        recycleAdapter.removeItem(viewHolder.getAdapterPosition());

        snackbar.setAction(R.string.undo, new OnClickListener() {
            @Override
            public void onClick(View v) {
                //Restore deleted item
                recycleAdapter.restoreItem(deletedItem, deletedIndex);
                recyclerView.scrollToPosition(deletedIndex);

            }
        }).addCallback(new Snackbar.Callback() {
            @Override
            public void onDismissed(Snackbar snackbar, int dismissType) {
                super.onDismissed(snackbar, dismissType);

                if (dismissType != DISMISS_EVENT_ACTION) {
                    //Delete the item from the database
                    databaseHelper.removePhase(deletedItem);
                    Toast.makeText(getApplicationContext(), R.string.phase_successfully_deleted, Toast.LENGTH_LONG).show();
                }
            }
        });
        snackbar.setActionTextColor(getResources().getColor(R.color.warningColor));
        snackbar.show();
    }
    //Swipe RIGHT to EDIT
    else {
        Intent intent = new Intent(this, AddPhaseActivity.class);
        intent.putExtra("dbPosition", (long) viewHolder.itemView.getTag());
        intent.putExtra("rvPosition", position);
        recycleAdapter.notifyItemChanged(position);
        this.startActivity(intent);
    }
}
0
tikiss 5 Окт 2018 в 00:30

1 ответ

Лучший ответ

Ваш код в порядке. Единственное возможное место для этого странного исполнения - функция ниже.

@Override public void clearView(RecyclerView recyclerView, RecyclerView.ViewHolder viewHolder) { View list_foreground = ((RecycleAdapter.mViewHolder)viewHolder).list_foreground; getDefaultUIUtil().clearView(list_foreground); }

Вы очищаете неправильную точку зрения. Попробуйте очистить корневой вид вашего элемента, например

@Override public void clearView(RecyclerView recyclerView, RecyclerView.ViewHolder viewHolder) { View itemView= ((RecycleAdapter.mViewHolder)viewHolder).itemView; getDefaultUIUtil().clearView(itemView); }

ИЛИ

Измените свой метод public void onChildDraw(Canvas c, RecyclerView recyclerView, RecyclerView.ViewHolder viewHolder, float dX, float dY, int actionState, boolean isCurrentlyActive).

Ставить

View list_foreground = ((RecycleAdapter.mViewHolder)viewHolder).list_foreground; getDefaultUIUtil().onDraw(c, recyclerView, list_foreground, dX, dY, actionState, isCurrentlyActive);

Вместо того

super.onChildDraw(c, recyclerView, viewHolder, mdX, dY, actionState, isCurrentlyActive);

Это решит вашу проблему.

ОБЪЯСНЕНИЕ :

Источник проблемы в том, что вызовы метода onChildDraw

super.onChildDraw(c, recyclerView, viewHolder, mdX, dY, actionState, isCurrentlyActive);

И это зовет onDraw(Canvas c, RecyclerView recyclerView, View view, float dX, float dY, int actionState, boolean isCurrentlyActive) функция внутри передачи viewHolder.itemView в качестве аргумента для view и реализации перевода к нему, как

view.setTranslationX(dX); view.setTranslationY(dY);

Но четкое представление работает как

    @Override
    public void clearView(View view) {
        view.setTranslationX(0f);
        view.setTranslationY(0f);
    }

Итак, в вашем случае функции onChildDraw и clearView работают с разными представлениями.

0
Viktor Vasyliev 7 Окт 2018 в 00:04