# 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