Amaze文件管理器SFTP连接编辑崩溃问题分析

问题概述

在Amaze文件管理器3.10版本中，当用户尝试编辑已创建的SFTP连接时，应用程序会发生崩溃。崩溃日志显示这是一个NumberFormatException异常，具体原因是尝试将空字符串("")解析为整数时出错。

技术背景

SFTP(SSH文件传输协议)是一种安全的文件传输协议，通常运行在SSH协议之上，默认使用22端口。Amaze文件管理器通过SftpConnectDialog类来处理SFTP连接的创建和编辑操作。

问题根源分析

根据崩溃日志和代码审查，问题出现在SftpConnectDialog.kt文件的第793行，具体是在createConnectionSettings方法中。该方法尝试将端口号从字符串转换为整数时，传入了一个空字符串("")，导致Integer.parseInt()抛出异常。

在正常情况下，SFTP连接应该有一个有效的端口号(通常是22)，但当用户编辑现有连接时，端口号字段可能未被正确初始化或传递，导致出现空字符串的情况。

代码层面分析

问题核心代码位于createConnectionSettings方法中：

val port = if (portEditText.text.toString().isNotEmpty()) {
    Integer.parseInt(portEditText.text.toString())
} else {
    DEFAULT_PORT
}

这段代码的逻辑是：如果端口编辑框中有内容，则解析为整数；否则使用默认端口。然而，在实际执行时，portEditText.text.toString()返回了空字符串，而条件判断却未能正确捕获这种情况。

解决方案

正确的处理方式应该包括以下改进：

1. 在解析端口号前进行更严格的输入验证
2. 为空值或无效值提供合理的默认值
3. 添加异常处理机制

建议修改后的代码逻辑：

val portText = portEditText.text.toString()
val port = try {
    if (portText.isNotBlank()) portText.toInt() else DEFAULT_PORT
} catch (e: NumberFormatException) {
    DEFAULT_PORT
}

影响范围

此问题影响所有使用SFTP功能的用户，特别是在以下场景：

• 编辑已存在的SFTP连接
• 端口号字段为空或包含非数字字符时
• 使用IPv6地址的连接(虽然问题本身与IP版本无关)

预防措施

为避免类似问题，开发过程中可以采取以下措施：

1. 对所有用户输入进行严格的验证
2. 为数字输入字段设置合理的默认值
3. 添加全面的异常处理
4. 编写单元测试覆盖边界条件

总结

这个崩溃问题展示了在Android开发中处理用户输入时常见的一个陷阱。通过更健壮的输入验证和异常处理，可以显著提高应用的稳定性和用户体验。对于文件管理器这类工具软件，正确处理网络连接配置尤为重要，因为任何崩溃都可能导致用户数据访问中断。