xp-robotaxi-map-boot 代码问题分析

本文档记录了对该模块代码实现中发现的问题。

问题列表

GaodeMapService.java

1. 安全问题 - 硬编码 API Key (line 23):
java private String key = "66ae577f63898dae3016d599fda57221";
- 高德地图 API Key 直接硬编码在源代码中
- API Key 应该通过外部配置 (application.yml/application.properties) 配置
- 注释掉的 @Value 表明本应该外部化配置，但实际上没有做

2. 性能问题 (line 68-78):
java CloseableHttpClient httpclient = HttpClients.createDefault(); ... httpclient.close();
- 每次请求都创建一个新的 CloseableHttpClient 然后关闭
- 这会毁掉连接池，创建过多的垃圾对象
- 应该重用共享的 HTTP 客户端

3. NPE 风险:
- line 78: 当 entity 为 null 时返回 null，但调用者可能没有检查 null 就使用
- line 84: responseJson 可以为 null，直接传给 objectMapper.readTree() 会抛出 NPE