嘿，我一眼就看穿问题所在了——你直接修改了全局/静态颜色数组的原始数据，而不是操作它的副本，这才导致后续所有调用都拿到被篡改过的颜色值！

问题根源拆解 #

你的colorWhite、colorRed这类变量应该是全局定义的float数组（比如float colorWhite[] = {1.0f,1.0f,1.0f,1.0f};）。当你把它们的指针赋值给col后，col其实就是指向这些全局数组的内存地址。这时候你修改col[3]，本质上是直接改掉了全局数组里的Alpha值——哪怕后续调用getColor(0)没触发if(alpha)判断，拿到的也是已经被改坏的原始数据！

至于你后来试的指针操作a = (float *)(&col + 3);，这完全是个错误：&col是指向指针的指针，&col +3会偏移3个指针大小的内存，根本碰不到颜色数组的第4个元素，所以当然解决不了问题。

正确的修复思路（附修正代码） #

你想到“复制到新变量”的方向是对的，但要注意必须为新颜色分配独立内存——你之前的修改代码里col是未初始化的指针，直接memcpy会导致崩溃或者未定义行为！

这里给你一个严谨的修复版本：

#include <stdlib.h>
#include <string.h>

// 注意：原代码里漏传了val参数，这里补上
float * getColor(int val, float alpha) { 
    // 分配4个float的内存空间，供新颜色使用
    float *col = malloc(sizeof(float) * 4);
    if (!col) return NULL; // 处理内存分配失败的情况

    float *tmp;
    if (val == 1) 
        tmp = colorWhite;
    else if (val == 2) 
        tmp = colorRed;
    // 补充其他颜色的分支逻辑
    else
        tmp = colEtc;

    // 把原始颜色的所有分量复制到新内存里
    memcpy(col, tmp, sizeof(float)*4);
    // 用浮点数的方式判断是否需要修改Alpha，避免精度误差
    if (alpha != 0.0f) { 
        col[3] = alpha;
    }
    return col;
}

额外提醒 #

• 调用这个函数后，一定要记得在不需要该颜色时用free(col)释放内存，否则会造成内存泄漏。
• 如果你的场景里颜色种类不多，也可以考虑预先为每种颜色创建固定的副本，或者用线程安全的局部存储方式，但malloc是最通用、不容易出问题的方案。

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