【AI-Review】【致命】【基础代码问题】【代码逻辑错误】delete *it 导致 MMKV 内部缓存悬空指针，引发 use-after-free

● 问题：delete *it; 直接释放了 MMKV 对象的内存，但未从 MMKV 的全局实例字典 g_instanceDic 中移除该条目。MMKV 库的 mmkvWithID() 方法在创建实例时会将其缓存到 g_instanceDic（见 MMKV/Core/MMKV_Android.cpp:147-151），后续调用 mmkvWithID 时会优先从字典中查找并返回已缓存的实例。正确的释放方式是调用 MMKV::close()（见 MMKV/Core/MMKV.cpp:335-345），该方法会先从 g_instanceDic 中移除条目，再执行 delete this。

● 影响：触发路径：setupMMKVInstance 被调用时 → delete *it 释放旧 MMKV 对象但 g_instanceDic 仍持有悬空指针 → 紧接着 MMKV::mmkvWithID(id, ...) 从 g_instanceDic 中找到并返回该悬空指针 → 返回的 kv 指向已释放内存 → 后续所有读写操作（setStringMMKV、getStringMMKV 等）均使用该悬空指针 → 导致 use-after-free，可能引发崩溃、数据损坏或未定义行为。

● 建议：将 delete *it; 替换为 (*it)->close();。close() 方法的实现（MMKV/Core/MMKV.cpp:335-345）会在 delete this 之前先从 g_instanceDic 中擦除该条目，确保后续 mmkvWithID 调用不会返回悬空指针。修改后代码：

if (it != mmkvInstances.end()) {
    (*it)->close();
    mmkvInstances.erase(it);
}

【AI-Review】【建议】【基础代码问题】【稳定性问题】nullptr 被推入 mmkvInstances 导致无效条目累积

● 问题：在 setupMMKVInstance 中，当 MMKV::mmkvWithID 抛出异常时（第 588-590 行的 catch 块），kv 保持为 nullptr，但第 592 行仍然执行 mmkvInstances.push_back(kv) 将 nullptr 推入向量。PR 新增的 null 检查（inst != nullptr &&）使得 find_if 在后续调用中跳过这些 nullptr 条目，因此它们永远不会被找到和清理，在向量中持续累积。

● 影响：每次 setupMMKVInstance 因异常失败都会在 mmkvInstances 中新增一个永久的 nullptr 条目，向量持续膨胀。虽然功能上不会崩溃（null 检查保护了 find_if），但属于资源浪费，且使调试更困难。

● 建议：在 push_back 前检查 kv 是否有效，仅当 kv 非 null 时才推入向量和设置索引：

if (kv) {
    mmkvInstances.push_back(kv);
    indexing_enabled[id] = NValueToBool(env, args[4]);
    initIndexForId(id);
}