ktlint项目中的ArgumentListWrappingRule异常问题分析

问题背景

在ktlint代码格式化工具中，ArgumentListWrappingRule规则负责处理函数参数列表的换行格式化。该规则在1.1.1版本中出现了一个异常情况，当遇到特定格式的代码时会抛出IllegalArgumentException异常。

问题复现

问题出现在以下特定代码结构中：

fun Foo.map(): List<String> = listOf(
    1,
) + listOf(2).map { it }

当ktlint尝试格式化这段代码时，ArgumentListWrappingRule规则会抛出异常，提示"Count 'n' must be non-negative, but was -1"。这表明在计算缩进时出现了负数的情况。

技术分析

从堆栈跟踪可以看出，异常发生在ArgumentListWrappingRule.intendedIndent方法中。该方法尝试使用String.repeat函数生成缩进字符串，但传入的重复次数为-1，这违反了String.repeat函数的参数要求（必须为非负数）。

深入分析问题原因：

1. 该规则在处理参数列表时，会计算每个参数应有的缩进级别
2. 对于上述代码中的复合表达式（包含+操作符和.map调用），缩进计算逻辑出现了错误
3. 在特定情况下，计算出的缩进级别变成了负数
4. 当尝试用这个负数调用String.repeat时，触发了异常

解决方案

ktlint团队已经确认：

1. 这不是1.1.x版本引入的回归问题
2. 该问题将在1.2版本中修复

对于开发者而言，临时解决方案可以是：

1. 暂时禁用argument-list-wrapping规则
2. 或者重构代码以避免触发该规则的异常情况

最佳实践

为避免类似问题，建议：

1. 在升级ktlint版本前，先在测试环境中验证代码格式化的兼容性
2. 对于复杂的链式调用或复合表达式，考虑适当拆分以提高代码可读性
3. 关注ktlint的发布说明，了解已知问题和修复情况

总结

代码格式化工具在处理复杂表达式时可能会遇到边界情况。ktlint团队已经识别并修复了ArgumentListWrappingRule规则中的这个问题。开发者应保持工具更新，同时注意代码结构对格式化工具的影响，以保持代码库的整洁和一致性。