From 5aae03a0e27f361ec4133ac1fda43a3d4e72061e Mon Sep 17 00:00:00 2001 From: hannibal2 <24389977+hannibal00212@users.noreply.github.com> Date: Wed, 17 Apr 2024 17:00:00 +0200 Subject: code cleanup --- .../features/misc/compacttablist/TabListReader.kt | 81 +++++++++++----------- 1 file changed, 39 insertions(+), 42 deletions(-) (limited to 'src/main/java/at/hannibal2/skyhanni/features/misc') diff --git a/src/main/java/at/hannibal2/skyhanni/features/misc/compacttablist/TabListReader.kt b/src/main/java/at/hannibal2/skyhanni/features/misc/compacttablist/TabListReader.kt index 5c037eecb..9d21c7421 100644 --- a/src/main/java/at/hannibal2/skyhanni/features/misc/compacttablist/TabListReader.kt +++ b/src/main/java/at/hannibal2/skyhanni/features/misc/compacttablist/TabListReader.kt @@ -57,7 +57,6 @@ object TabListReader { val renderColumns = mutableListOf() - private fun updateTablistData(tablist: List? = null) { if (!LorenzUtils.inSkyBlock) return if (!config.enabled.get()) return @@ -197,60 +196,58 @@ object TabListReader { var firstColumnCopy = firstColumn var lastTitle: String? = null - for (column in columns) { - for (section in column.sections) { - var sectionSize = section.size() + for (section in columns.flatMap { it.sections }) { + var sectionSize = section.size() + + var needsTitle = false + if (lastTitle != section.columnValue.columnTitle) { + needsTitle = true + sectionSize++ + } + + var currentCount = firstColumnCopy.size() - var needsTitle = false - if (lastTitle != section.columnValue.columnTitle) { - needsTitle = true - sectionSize++ + if (sectionSize >= TabListRenderer.MAX_LINES / 2) { + if (currentCount >= TabListRenderer.MAX_LINES) { + renderColumns.add(RenderColumn().also { firstColumnCopy = it }) + currentCount = 1 + } else { + if (firstColumnCopy.size() > 0) { + firstColumnCopy.addLine(AdvancedPlayerList.createTabLine("", TabStringType.TEXT)) + } } - var currentCount = firstColumnCopy.size() + if (needsTitle) { + lastTitle = section.columnValue.columnTitle + firstColumnCopy.addLine(AdvancedPlayerList.createTabLine(lastTitle, TabStringType.TITLE)) + currentCount++ + } - if (sectionSize >= TabListRenderer.MAX_LINES / 2) { + for (line in section.lines) { if (currentCount >= TabListRenderer.MAX_LINES) { renderColumns.add(RenderColumn().also { firstColumnCopy = it }) currentCount = 1 - } else { - if (firstColumnCopy.size() > 0) { - firstColumnCopy.addLine(AdvancedPlayerList.createTabLine("", TabStringType.TEXT)) - } - } - - if (needsTitle) { - lastTitle = section.columnValue.columnTitle - firstColumnCopy.addLine(AdvancedPlayerList.createTabLine(lastTitle, TabStringType.TITLE)) - currentCount++ } - for (line in section.lines) { - if (currentCount >= TabListRenderer.MAX_LINES) { - renderColumns.add(RenderColumn().also { firstColumnCopy = it }) - currentCount = 1 - } - - firstColumnCopy.addLine(AdvancedPlayerList.createTabLine(line, TabStringType.fromLine(line))) - currentCount++ - } + firstColumnCopy.addLine(AdvancedPlayerList.createTabLine(line, TabStringType.fromLine(line))) + currentCount++ + } + } else { + if (currentCount + sectionSize > TabListRenderer.MAX_LINES) { + renderColumns.add(RenderColumn().also { firstColumnCopy = it }) } else { - if (currentCount + sectionSize > TabListRenderer.MAX_LINES) { - renderColumns.add(RenderColumn().also { firstColumnCopy = it }) - } else { - if (firstColumnCopy.size() > 0) { - firstColumnCopy.addLine(AdvancedPlayerList.createTabLine("", TabStringType.TEXT)) - } + if (firstColumnCopy.size() > 0) { + firstColumnCopy.addLine(AdvancedPlayerList.createTabLine("", TabStringType.TEXT)) } + } - if (needsTitle) { - lastTitle = section.columnValue.columnTitle - firstColumnCopy.addLine(AdvancedPlayerList.createTabLine(lastTitle, TabStringType.TITLE)) - } + if (needsTitle) { + lastTitle = section.columnValue.columnTitle + firstColumnCopy.addLine(AdvancedPlayerList.createTabLine(lastTitle, TabStringType.TITLE)) + } - for (line in section.lines) { - firstColumnCopy.addLine(AdvancedPlayerList.createTabLine(line, TabStringType.fromLine(line))) - } + for (line in section.lines) { + firstColumnCopy.addLine(AdvancedPlayerList.createTabLine(line, TabStringType.fromLine(line))) } } } -- cgit