Ivy Wallet 项目中多行描述文本重复问题的分析与修复

问题背景

在Ivy Wallet财务管理应用的最新版本中，用户报告了一个影响使用体验的文本输入问题。当用户在交易记录（支出、收入或转账）的描述字段中输入多行文本时，按下回车键创建新行会导致前一行文本被自动复制到新行中。这种非预期的行为严重影响了用户输入多行描述的体验。

问题重现

该问题在Android 13系统的三星Galaxy S20FE设备上被确认，影响版本包括GitHub发布的4.6.3(163)版本以及Google Play商店的4.6.3版本。问题重现步骤如下：

1. 打开任意交易记录（支出、收入或转账）的编辑界面
2. 在描述字段中输入文本
3. 按下回车键创建新行
4. 观察到前一行文本被自动复制到新行

技术分析

通过审查项目代码，发现问题可能源于UI组件中的点击事件处理逻辑。具体来说，在Description.kt文件中，DescriptionText可组合函数存在潜在的点击事件处理冗余。

原始实现中，DescriptionText组件被包裹在PrimaryAttributeColumn容器内，该容器本身已经具有点击处理功能。然而，内部的Text组件又额外添加了.clickable修饰符，这可能导致事件处理冲突，特别是在文本输入和多行处理时产生异常行为。

解决方案

修复方案是移除Text组件中冗余的clickable修饰符，保持点击事件处理的单一性。修改后的代码如下：

private fun DescriptionText(  
    description: String,  
    onClick: () -> Unit,  
) {  
    PrimaryAttributeColumn(  
        icon = R.drawable.ic_description,  
        title = stringResource(R.string.description),  
        onClick = onClick  
    ) {  
        Spacer(Modifier.height(12.dp))  
        Text(  
            modifier = Modifier
                .padding(horizontal = 24.dp)  
                .testTag("trn_description"),  
            text = description,  
            style = UI.typo.nB2.style(  
                textAlign = TextAlign.Left  
            ),  
        )  
        Spacer(Modifier.height(20.dp))  
    }  
}

修复原理

在Jetpack Compose中，当多个可组合组件都设置了点击处理时，可能会导致事件冒泡或处理冲突。特别是对于文本输入场景，冗余的点击处理可能会干扰系统的文本输入管理，导致文本内容异常。通过简化点击事件处理层级，可以确保文本输入功能的正常运作。

经验总结

这个案例提醒开发者：

1. 在Jetpack Compose中应谨慎处理点击事件，避免多层嵌套的点击处理
2. 对于文本输入组件，应保持其修饰符尽可能简洁，避免干扰系统默认的输入处理
3. 组件设计时应遵循单一职责原则，确保每个组件只处理自己职责范围内的事件

该修复已被合并到主分支，解决了用户在多行描述输入时遇到的文本重复问题，提升了应用的整体使用体验。