Golang递归扁平化Map时connectedTo字段偶发为空问题

你的猜测完全正确，问题根源就是Go语言对map类型的遍历顺序是随机不固定的：

• 原有逻辑在同一个for range mp循环中并行处理type、properties、connected三类字段
• 当某次遍历先命中connected分支时，用于存储当前节点类型的key变量还没有被type分支赋值，此时给子节点设置connectedTo拿到的就是空字符串，就会出现偶发的属性为空问题
• 另外原有逻辑还存在类型断言缺失校验、多层嵌套connected时父节点传递断层的隐患，极端场景下会触发panic。

核心调整思路是固定字段处理顺序：不要在遍历map的流程中边取当前节点值边处理子节点，先完成当前节点的type、properties解析，拿到确定的当前节点标识后，再递归处理connected下的子资源，处理子节点时直接传入当前节点的类型作为父级标识，从根源上避免遍历顺序的影响。
修复后的可运行代码如下：

func keyValuePairs(m interface{}) map[string]interface{} {
	kvs := make(map[string]interface{})
	// 非map类型直接返回
	if reflect.ValueOf(m).Kind() != reflect.Map {
		return kvs
	}
	mp, ok := m.(map[string]interface{})
	if !ok {
		return kvs
	}

	var currentKey string
	var currentProps interface{}
	var connectedList []interface{}

	// 第一步：先拆分提取所有字段，不做递归处理，避免顺序影响
	for k, v := range mp {
		switch k {
		case "type":
			if typeStr, ok := v.(string); ok {
				currentKey = typeStr
			}
		case "properties":
			currentProps = v
		case "connected":
			if list, ok := v.([]interface{}); ok {
				connectedList = list
			}
		default:
			// 处理其他自定义字段
			for nk, nv := range keyValuePairs(v) {
				kvs[nk] = nv
			}
		}
	}

	// 第二步：先把当前节点存入结果集
	if currentKey != "" {
		kvs[currentKey] = currentProps
	}

	// 第三步：当前节点key已确定，再递归处理子连接节点，传入当前节点作为父级
	for _, child := range connectedList {
		childKvs := keyValuePairs(child)
		for childKey, childVal := range childKvs {
			childProps, ok := childVal.(map[string]interface{})
			if !ok {
				kvs[childKey] = childVal
				continue
			}
			// 没有设置过connectedTo才赋值，避免覆盖更上层的关联关系
			if _, exists := childProps["connectedTo"]; !exists && currentKey != "" {
				childProps["connectedTo"] = currentKey
			}
			kvs[childKey] = childProps
		}
	}

	return kvs
}

修复后运行可以稳定得到预期的输出结构，即使存在多层嵌套的connected节点，也能正确关联父节点标识，不会出现随机空值问题。

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