在'{'标记之前的预期表达式

时间:2012-11-07 18:21:30

标签: c expression token

我得到:“错误:我之前评论过的行'{'token'之前的预期表达式。如果结构已经定义,为什么在令牌之前需要“{”。感谢您提供的任何帮助。

struct sdram_timing {
    u32 wrdtr;
    u32 clktr;
};

int calibration(void);
unsigned char read_i2c_cal(void);
static unsigned int eepcal[15];

main() {
    DQS_autocalibration();
}

int calibration(void)
{
    struct sdram_timing scan_list[30];

    read_i2c_cal();
    if(eepcal[0] == 0){

       scan_list = {{eepcal[1], eepcal[2]}, {-1, -1}}; // <-- PROBLEM LINE

        }
        else {
            //foo
        }

    return 0;
}

unsigned char read_i2c_cal(void) {
    eepcal[0] = 0;
    eepcal[1] = 02;
    eepcal[2] = 03;
}

4 个答案:

答案 0 :(得分:17)

错误是因为您不能以这种方式分配数组,这只能用于初始化它。

int arr[4] = {0}; // this works
int arr2[4];

arr2 = {0};// this doesn't and will cause an error

arr2[0] = 0; // that's OK
memset(arr2, 0, 4*sizeof(int)); // that is too

所以将此应用于您的具体示例:

struct sdram_timing scan_list[30];
scan_list[0].wrdtr = 0;
scan_list[0].clktr = 0;

或者您可以以相同的方式使用memset,但是您需要结构的大小而不是sizeof(int)。这并不总是有效...但鉴于你的结构,它会。

答案 1 :(得分:3)

C语言中的数组不可分配。无论使用何种语法,都无法为整个数组分配任何内容。换句话说,这个

scan_list = { { eepcal[1], eepcal[2] }, {-1, -1} };

是不可能的。

在C89 / 90中,你必须逐行拼写你的作业

scan_list[0].wrdtr = eepcal[1];
scan_list[0].clktr = eepcal[2];
scan_list[1].wrdtr = -1;
scan_list[1].clktr = -1;

在现代C(后C99)中,您可以使用复合文字来指定整个结构

scan_list[0] = (struct sdram_timing) { eepcal[1], eepcal[2] };
scan_list[1] = (struct sdram_timing) { -1, -1 };

最后,在现代C中,您可以使用memcpy和复合文字将数据复制到数组

memcpy(scan_list, (struct sdram_timing[]) { { eepcal[1], eepcal[2] }, {-1, -1} },
  2 * sizeof *scan_list);

最后一个变体虽然不是很优雅,但是最接近“模拟”数组赋值的方式。

答案 2 :(得分:1)

您只能在变量声明中使用初始化列表,而不是在事实之后。

答案 3 :(得分:1)

初始化列表只能用于初始化数组。之后你不能使用它。

但是,如果您使用GCC,则可以使用Compound Literal扩展程序:

scan_list = (struct sdram_timing[30]){{eepcal[1], eepcal[2]}, {-1, -1}};

您可能需要将scan_list类型更改为struct sdram_timing *