Как выборочно украшать элементы RecyclerView

Я создаю подкласс ItemDecoration из этого ItemDecoration: https://gist.github.com/alexfu/0f464fc3742f134ccd1e

Как сделать его только украшением предметов с определенным условием? Например, только украшайте предметы определенными позициями, типом ViewHolder и т.д.

Я изменил вышеупомянутый gist (плюс некоторые изменения в устаревшем Android API) с этим кодом, но все элементы оформлены в любом случае:

public boolean isDecorated(View view, RecyclerView parent) {
    RecyclerView.ViewHolder holder = parent.getChildViewHolder(view);
    return holder instanceof MenuIconViewHolder || holder instanceof MenuDetailViewHolder;
}

@Override
public void getItemOffsets(Rect outRect, View view, RecyclerView parent, RecyclerView.State state) {
    if (isDecorated(view, parent)) {
        if (mOrientation == VERTICAL_LIST) {
            outRect.set(0, 0, 0, mDivider.getIntrinsicHeight());
        } else {
            outRect.set(0, 0, mDivider.getIntrinsicWidth(), 0);
        }
    } else {
        super.getItemOffsets(outRect, view, parent, state);
    }
}

Что не так с вышеуказанным кодом? Кстати, можно ли считать лучшей практикой (в отношении разделения проблем) поместить такой код в класс ItemDecoration?

ОТВЕТЫ

Ответ 1

Вам также нужно вызвать isDecorated на методе draw, так как на данный момент вы не ставите смещение на эти элементы, но вы все равно нарисовываете его.

Метод перебирает все дочерние представления, находящиеся в текущем виджете RecyclerView, видимые на экране.

public void drawVertical(Canvas c, RecyclerView parent) {
    final int left = parent.getPaddingLeft();
    final int right = parent.getWidth() - parent.getPaddingRight();

    final int childCount = parent.getChildCount();
    for (int i = 0; i < childCount; i++) {
        final View child = parent.getChildAt(i);
        if(isDecorated(child, parent))
        {
            final RecyclerView.LayoutParams params = (RecyclerView.LayoutParams) child
                .getLayoutParams();
            final int top = child.getBottom() + params.bottomMargin;
            final int bottom = top + mDivider.getIntrinsicHeight();
            mDivider.setBounds(left, top, right, bottom);
            mDivider.draw(c);
        }
    }
}

public void drawHorizontal(Canvas c, RecyclerView parent) {
    final int top = parent.getPaddingTop();
    final int bottom = parent.getHeight() - parent.getPaddingBottom();

    final int childCount = parent.getChildCount();
    for (int i = 0; i < childCount; i++) {
        final View child = parent.getChildAt(i);
        if(isDecorated(child, parent))
        {
            final RecyclerView.LayoutParams params = (RecyclerView.LayoutParams) child
                .getLayoutParams();
            final int left = child.getRight() + params.rightMargin;
            final int right = left + mDivider.getIntrinsicHeight();
            mDivider.setBounds(left, top, right, bottom);
            mDivider.draw(c);
        }
    }
}