open62541中UA_Variant初始化/销毁时机疑问及空指针求助

代码写法问题 #

你给出的代码存在语法错误：UA_Variant_delete(&p) { 写法错误，正确调用应为UA_Variant_delete(&p);（移除多余的大括号）。

初始化/销毁的正确逻辑 #

不需要在每次读取时都初始化和销毁UA_Variant，初始化一次、循环结束后销毁一次是合理的，但必须在每次调用UA_Client_readValueAttribute前，调用UA_Variant_clear(&p)清理上一次读取的内容。

原因在于：UA_Client_readValueAttribute成功读取时会为UA_Variant分配内存存储数据，若不提前清理，旧数据的内存未释放会造成泄漏，且新数据可能无法正确写入。

data字段为空的排查方向 #

1. 检查读取返回值：先确认retval是否等于UA_STATUSCODE_GOOD。如果读取失败（比如NodeId错误、客户端未连接、权限不足等），UA_Variant的data字段自然为空。可通过UA_StatusCode_name(retval)打印具体错误原因。
2. 未清理旧数据：如果没调用UA_Variant_clear，旧数据占用的内存可能导致新读取无法正确赋值，表现为data为空。
3. 服务器节点无值：目标NodeId对应的服务器节点本身没有值，或者值类型与预期不匹配，也会出现data为空的情况。

修正后的示例代码 #

UA_Variant p;
UA_Variant_init(&p);

while (1) {
    // 每次读取前清理variant，释放上一次的内存
    UA_Variant_clear(&p);
    retval = UA_Client_readValueAttribute(client, NodeId, &p);
    
    if(retval == UA_STATUSCODE_GOOD) {
        if(p.data != NULL) {
            // 在这里处理读取到的数据
        }
    } else {
        UA_LOG_ERROR(UA_Log_Stdout, UA_LOGCATEGORY_CLIENT, 
                     "Read failed: %s", UA_StatusCode_name(retval));
    }
    
    // 避免频繁读取，添加延时
    UA_sleep(1000);
}

UA_Variant_delete(&p);

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