From 948abae5f8ca733d6ba5187de41a55a48b37533b Mon Sep 17 00:00:00 2001 From: Stanisalv Date: Thu, 27 Aug 2020 12:00:04 +0300 Subject: [PATCH] reformat GroupItemDecoration reformat GroupItemDecoration --- .../decorators/GroupItemDecoration.kt | 31 +++++++------------ 1 file changed, 11 insertions(+), 20 deletions(-) diff --git a/recyclerview-adapters/src/main/java/ru/touchin/roboswag/recyclerview_adapters/decorators/GroupItemDecoration.kt b/recyclerview-adapters/src/main/java/ru/touchin/roboswag/recyclerview_adapters/decorators/GroupItemDecoration.kt index 67470b0..8cc38bf 100644 --- a/recyclerview-adapters/src/main/java/ru/touchin/roboswag/recyclerview_adapters/decorators/GroupItemDecoration.kt +++ b/recyclerview-adapters/src/main/java/ru/touchin/roboswag/recyclerview_adapters/decorators/GroupItemDecoration.kt @@ -31,28 +31,19 @@ class GroupItemDecoration( } @Suppress("detekt.NestedBlockDepth") - override fun onDraw(canvas: Canvas, parent: RecyclerView, state: RecyclerView.State) { - var isInvalidated = false - loop@ for (child in parent.children) { - val adapterPosition = parent.getChildAdapterPosition(child) - val groupView = viewHoldersPool[adapterPosition]?.view - when { - predicate(adapterPosition) -> { - if (groupView != null) { - onDrawGroupView(groupView, parent, child, canvas) - } else { - isInvalidated = true - break@loop - } - } - groupView != null -> { - isInvalidated = true - break@loop - } + override fun onDraw(canvas: Canvas, parent: RecyclerView, state: RecyclerView.State) = parent.children.forEach { child -> + val adapterPosition = parent.getChildAdapterPosition(child) + val groupView = viewHoldersPool[adapterPosition]?.view + if (predicate(adapterPosition)) { + if (groupView != null) { + onDrawGroupView(groupView, parent, child, canvas) + } else { + parent.invalidateItemDecorations() + return } - } - if (isInvalidated) { + } else if (groupView != null) { parent.invalidateItemDecorations() + return } }