C 程序中的 Union 问题

Question

我正在开发一个使用 Union 的 C 程序。联合定义位于 FILE_A 头文件中，如下所示...

// FILE_A.h****************************************************
xdata union  
{
long position;
char bytes[4];
}CurrentPosition;

如果我在 FILE_A.c 中设置 CurrentPosition.position 的值，然后调用 FILE_B.c 中使用联合的函数，联合中的数据将返回到零。下面对此进行了演示。

// FILE_A.c****************************************************
int main.c(void)
{
    CurrentPosition.position = 12345;
    SomeFunctionInFileB();
}

// FILE_B.c****************************************************
void SomeFunctionInFileB(void)
{
    // After the following lines execute I see all zeros in the flash memory.
    WriteByteToFlash(CurrentPosition.bytes[0];
    WriteByteToFlash(CurrentPosition.bytes[1];
    WriteByteToFlash(CurrentPosition.bytes[2];
    WriteByteToFlash(CurrentPosition.bytes[3];
}

现在，如果我将 long 传递给 SomeFunctionInFileB(long temp) ，然后将其存储到该函数内的 CurrentPosition.bytes 中，最后调用 WriteBytesToFlash(CurrentPosition.bytes[n]... 它工作得很好。

看起来好像 CurrentPosition Union 不是全局的，所以我尝试更改头文件中的 union 定义以包含这样的 extern 关键字...

extern xdata union  
{
long position;
char bytes[4];
}CurrentPosition;

然后将其放入源（.c）文件中...

xdata union  
{
    long position;
    char bytes[4];
}CurrentPosition;

但这会导致编译错误：

C:\SiLabs\Optec Programs\AgosRot\MotionControl.c:76：错误 91：“CurrentPosition”的外部定义与声明不匹配。 C:\SiLabs\Optec Programs\AgosRot\/MotionControl.h:48: 错误 177: 先前在此处定义

那么我做错了什么？如何使工会全球化？

Answer 1

FILE_A.h 真的是 MotionControl.h 吗？如果是这样，我认为解决方法是在标头中定义联合类型：

typedef
union xdata
{
    long position;
    char bytes[4];
} xdata;

并在头文件中的其他位置声明该类型的全局变量（可能是同一个）：

extern xdata CurrentPosition;   // in a header file

最后在 C 文件中定义全局变量恰好一次。也许在 file_a.c 中：

xdata CurrentPosition;

当然，更好的解决方法可能是将要写入闪存的 xdata 变量传递给 SomeFunctionInFileB()因此您不必依赖全局变量，众所周知，如果不非常非常小心地使用全局变量，就会出现问题。似乎没有充分的理由不将数据作为参数传递：

// in a header file
void SomeFunctionInFileB( xdata const* pPosition);


void SomeFunctionInFileB( xdata const* pPosition)
{
    // After the following lines execute I see all zeros in the flash memory.
    WriteByteToFlash(pPosition->bytes[0];
    WriteByteToFlash(pPosition->bytes[1];
    WriteByteToFlash(pPosition->bytes[2];
    WriteByteToFlash(pPosition->bytes[3];
}

并像这样调用它：

int main.c(void)
{
    CurrentPosition.position = 12345;
    SomeFunctionInFileB( &CurrentPosition);
}

Answer 2

理想情况下，您需要联合的 typedef 和 FILE_A.h 中的外部声明以及 FILE_A.c 中联合的实际定义。

-

// FILE_A.h

typedef union  
{
    long position;
    char bytes[4];
} Position;

extern Position CurrentPosition; // declaration

-

// FILE_A.c

#include "FILE_A.h"

Position CurrentPosition; // definition

int main(void)
{
    CurrentPosition.position = 12345;
    SomeFunctionInFileB();
    return 0;
}

-

// FILE_B.c

#include "FILE_A.h"

void SomeFunctionInFileB(void)
{
    // now there will be valid data in the flash memory.
    WriteByteToFlash(cp.bytes[0];
    WriteByteToFlash(cp.bytes[1];
    WriteByteToFlash(cp.bytes[2];
    WriteByteToFlash(cp.bytes[3];
}

-

Answer 3

您尚未实例化联合。
你需要：

// FILE_A.c****************************************************

#include "File_a.h"
CurrentPosition cp;
int main(void)
{
    cp.position = 12345;
    SomeFunctionInFileB();
}

// FILE_B.c****************************************************
#include "File_a.h"
extern CurrentPosition cp;
void SomeFunctionInFileB(void)
{
    // now there will be valid data in the flash memory.
    WriteByteToFlash(cp.bytes[0];
    WriteByteToFlash(cp.bytes[1];
    WriteByteToFlash(cp.bytes[2];
    WriteByteToFlash(cp.bytes[3];
}

Answer 4

如果 sizeof(long) 不是 4，那么字节序就会发挥作用......

考虑

union{
   long position
   char bytes[sizeof long];
}