Omnet++ 5.7运行LEACH仿真突发崩溃退出，求代码修复建议

核心问题定位 #

• 最高优先级bug：gName1和gName2两个字符数组声明后未初始化就直接使用，数组内是随机内存垃圾值，后续的门存在校验、删除、新增操作全部基于随机名称执行，直接触发指针越界访问崩溃
• 遍历逻辑错误：OMNeT++中门（cGate）本身属于仿真组件，你在遍历全量组件的循环内动态新增门，会持续增大simulation->getLastComponentId()的返回值，导致for循环无限执行，最终内存耗尽或非法访问崩溃
• 类型转换无校验：直接将getModule(i)的返回值强制转为cModule*和Node*，如果遍历到的组件ID对应的不是Node模块，访问myId、xpos等成员变量时会触发非法内存访问
• 通道传参不规范：connectTo方法传入NULL通道参数不符合OMNeT++ API要求，部分版本会直接触发空指针异常
• 无效冗余代码：g->setDisplayString(g->getDisplayString())本身无任何实际作用，属于无效代码

修复步骤 #

1. 给门名称变量赋唯一有效值，保证每个节点对之间的门名称可识别：

// 放在门逻辑处理前，给两个名称变量赋值
sprintf(gName1, "out_to_node_%d", id);
sprintf(gName2, "in_from_node_%d", this->myId);

2. 调整遍历逻辑，先收集全量节点再处理门操作，避免遍历过程中组件ID变化导致无限循环：

auto simulation = getSimulation();
std::vector<Node*> allNodes;
// 先一次性收集所有节点，锁死遍历上限
int maxCompId = simulation->getLastComponentId();
for (int i = 1; i <= maxCompId; i++) {
    cComponent* comp = simulation->getComponent(i);
    // 先做类型校验再转换
    if (comp->isModule() && strcmp(comp->getName(), "node") == 0) {
        allNodes.push_back(static_cast<Node*>(comp));
    }
}
// 再遍历节点列表处理门的增删逻辑
for (auto* mod : allNodes) {
    int id = mod->myId;
    nodePtr[id] = mod;
    if (id == this->myId) continue;
    // 原有门增删逻辑放在这里
}

3. 实例化合法的通道对象再传入连接方法：

cIdealChannel* ch = cIdealChannel::create();
this->gate(gName1)->connectTo(mod->gate(gName2), ch);

4. 直接删除无效的g->setDisplayString(g->getDisplayString())代码即可

优化建议 #

如果这段代码是为了动态更新节点连通性，不需要反复删除、重建门，初始化阶段一次性创建所有可能用到的门，后续只需修改门的启用状态或者信道参数即可，性能更高也不会触发OMNeT++内部状态异常。

内容的提问来源于stack exchange，提问作者ali