Kotlin实现按钮左右切换访问列表元素的优化方案咨询

我当前的布局是左右两个箭头按钮搭配中间的TextView，用户点击箭头可以前后切换查看列表中的日期。请问有没有更优的实现方案？

我的现有代码如下：

// 日期每4周重复一次
val d = LocalDate.of(2023, 2, 10)
val f = DateTimeFormatter.ofPattern("dd-MM-yyyy")
val futurePayDate = d.plusWeeks(4).format(f)

paydayTextView.text = futurePayDate.toString()

val arrowright = view.findViewById<ImageButton>(R.id.imageButtonArrowRight)
val arrowleft = view.findViewById<ImageButton>(R.id.imageButtonArrowLeft)

val g = listOf<String>(
    d.plusWeeks(4).format(f),
    d.plusWeeks(8).format(f),
    d.plusWeeks(12).format(f),
    d.plusWeeks(16).format(f),
    d.plusWeeks(20).format(f),
    d.plusWeeks(24).format(f),
    d.plusWeeks(28).format(f),
    d.plusWeeks(32).format(f),
    d.plusWeeks(36).format(f),
    d.plusWeeks(40).format(f),
    d.plusWeeks(44).format(f),
    d.plusWeeks(48).format(f)
)

var indexright = 0
arrowright.setOnClickListener {

    if (indexright == 12) {
        val end = "The End!"
        paydayTextView.text = end
    } else {
        paydayTextView.text = g[indexright]
        indexright += 1
    }
}

arrowleft.setOnClickListener {
       
    if (indexright == 12) {
        paydayTextView.text = g[g.lastIndex]
        indexright -= 1
    }
    else if (indexright == 1){
        val end = "The End!"
        paydayTextView.text = end
    } else {
        indexright -= 1
        paydayTextView.text = g[g.lastIndex - (g.size - indexright)]
    }
}

优化方向和实现方案 #

1. 取消预生成日期列表，按需计算日期 #

原代码提前生成12个日期字符串存在列表里完全没必要——只需维护当前的周期索引，每次点击时直接计算对应日期，既节省内存，逻辑也更灵活。

2. 简化索引逻辑，统一管理当前位置 #

原代码用indexright同时处理左右箭头逻辑，容易混淆出错。建议用currentIndex变量管理当前显示的日期位置，范围从0到总周期数-1，点击箭头直接增减索引，边界判断更清晰。

3. 优化视图操作和代码结构 #

用Kotlin的View Binding替代findViewById，代码更简洁；把日期更新逻辑封装成单独方法，提高可读性和复用性。

优化后的代码示例 #

// 基础日期：2023年2月10日，每4周一个周期
private val baseDate = LocalDate.of(2023, 2, 10)
private val dateFormatter = DateTimeFormatter.ofPattern("dd-MM-yyyy")
// 总周期数（对应原代码的12个日期）
private val totalPeriods = 12
// 当前显示的周期索引，从0开始
private var currentIndex = 0

// 初始化显示第一个日期
updatePaydayDisplay()

// 右箭头点击事件
arrowRight.setOnClickListener {
    if (currentIndex < totalPeriods - 1) {
        currentIndex++
        updatePaydayDisplay()
        // 若之前禁用了左箭头，重新启用
        arrowLeft.isEnabled = true
    } else {
        paydayTextView.text = "已到最后！"
        // 可选：到达边界后禁用按钮，避免重复点击
        arrowRight.isEnabled = false
    }
}

// 左箭头点击事件
arrowLeft.setOnClickListener {
    if (currentIndex > 0) {
        currentIndex--
        updatePaydayDisplay()
        // 若之前禁用了右箭头，重新启用
        arrowRight.isEnabled = true
    } else {
        paydayTextView.text = "已到最前！"
        // 可选：到达边界后禁用按钮
        arrowLeft.isEnabled = false
    }
}

// 封装日期更新方法
private fun updatePaydayDisplay() {
    // 计算当前日期：基础日期 + (当前索引+1)*4周（第一个日期是+4周）
    val currentDate = baseDate.plusWeeks(4L * (currentIndex + 1))
    paydayTextView.text = currentDate.format(dateFormatter)
}

额外扩展建议 #

• 如果需要循环切换（最后一个日期点击右箭头回到第一个），修改索引逻辑：
  右箭头：currentIndex = (currentIndex + 1) % totalPeriods
  左箭头：currentIndex = (currentIndex - 1 + totalPeriods) % totalPeriods
  这种情况无需提示边界，也不用禁用按钮。
• 把baseDate、totalPeriods这类配置项提取到常量或者res/values/strings.xml中，后续修改更方便。
• 保留LocalDate类型而非存储字符串，方便后续扩展（比如计算日期差、切换不同日期格式）。

内容的提问来源于stack exchange，提问作者Jonas.S.