2

我正在尝试使用 GCC 编译以下 C 程序,但在第 7 行出现错误,因为该类型taus88_t*无法从调用的初始化函数调用中正确返回make_taus88(seed);

error: incompatible types when initializing type 'struct taus88_t *' using type 'taus88_t'|

我尝试过使用taus88_t TAUS88 = make_taus88(6346456);,但这会产生更多错误/警告。

taus88main.c||In function 'main':|
taus88main.c|8|error: incompatible type for argument 1 of 'taus88u32'|
taus88.h|21|note: expected 'struct taus88_t *' but argument is of type 'taus88_t'|
taus88main.c|9|error: incompatible type for argument 1 of 'taus88f32'|
taus88.h|22|note: expected 'struct taus88_t *' but argument is of type 'taus88_t'|
taus_88_cpp\taus88main.c|9|warning: unused variable 'numberf32'|
taus_88_cpp\taus88main.c|8|warning: unused variable 'numberu32'|

该项目位于以下 3 个文件中。

taus88main.c

#include "taus88.h"
#include <stdint.h>

int main()
{

    taus88_t* TAUS88 = make_taus88(6346456);
    u32 numberu32 = taus88u32(TAUS88);
    f32 numberf32 = taus88f32(TAUS88);

    return 0;
}

taus88.h

#ifndef _COMMON_TAUS88_H
#define _COMMON_TAUS88_H

#include <stdint.h>
typedef int8_t    i8;
typedef int16_t   i16;
typedef int32_t   i32;
typedef int64_t   i64;

typedef uint8_t   u8;
typedef uint16_t  u16;
typedef uint32_t  u32;
typedef uint64_t  u64;
typedef float  f32;
typedef double f64;

typedef struct {u32 s1, s2, s3;} taus88_t;

taus88_t make_taus88(u32 seed);
u32 taus88u32(taus88_t *t);
f32 taus88f32(taus88_t *t);

#endif

taus88.c

#include <stdint.h>
#include "taus88.h"

taus88_t make_taus88(u32 seed)
{
  taus88_t t;
  t.s1 = 1243598713U ^ seed; if (t.s1 <  2) t.s1 = 1243598713U;
  t.s2 = 3093459404U ^ seed; if (t.s2 <  8) t.s2 = 3093459404U;
  t.s3 = 1821928721U ^ seed; if (t.s3 < 16) t.s3 = 1821928721U;
  return t;
}

u32 taus88u32(taus88_t *t)
{
  t->s1 = ((t->s1 &  -2) << 12) ^ (((t->s1 << 13) ^  t->s1) >> 19);
  t->s2 = ((t->s2 &  -8) <<  4) ^ (((t->s2 <<  2) ^  t->s2) >> 25);
  t->s3 = ((t->s3 & -16) << 17) ^ (((t->s3 <<  3) ^  t->s3) >> 11);
  return t->s1 ^ t->s2 ^ t->s3;
}

f32 taus88f32(taus88_t *t)
{
  union {u32 i ; f32 f ;} u;
  u.i = 0x3F800000 | (taus88u32(t) >> 9);
  return u.f - 1.0;
}
4

3 回答 3

4

所以这里的主要问题是你正在返回 ataus8_t并试图将它分配给一个taus88_t *无效的,如果你需要使用指针的原因从代码中不明显,那么修复如下:

taus88_t* TAUS88 = malloc(sizeof(taus88_t)) ;

*TAUS88 = make_taus88(6346456);

完成后,您必须记住调用free指针。一种更简单的方法是跳过使用指针并执行以下操作:

taus88_t TAUS88 ;

TAUS88 = make_taus88(6346456);
u32 numberu32 = taus88u32(&TAUS88);
f32 numberf32 = taus88f32(&TAUS88);

现在您不必再担心打电话free了。

我指出的另一个问题是最有可能taus88f32违反strict aliasing规则。

于 2013-07-27T03:16:56.393 回答
0

taus88_t* TAUS88 = make_taus88(6346456);分成两行怎么样

taus88_t* TAUS88;
*TAUS88 = make_taus88(6346456);

http://ideone.com/KVbf79

于 2013-07-27T03:02:41.123 回答
0

看起来make_taus88返回了一段taus88_t时间,您尝试将结果分配给taus88_t *(pointer to taus88_t)

因此,第一个大问题是在本地make_taus88定义t,然后尝试返回它。你不能这样做,因为当make_taus88终止时,t将超出范围。

第二个是您返回的是 ataus88_t而不是 a taus88_t *

您可以通过声明ttaus88_t *in来解决这两个问题make_taus88,并使用类似的东西动态初始化它malloc。然后,您可以返回t,它将是正确的类型,并在堆上分配,这样当make_taus88超出范围时,内存仍然可用。

请注意,make_taus88您还必须将表单的所有引用更改t.s1t->s1.

于 2013-07-27T03:11:23.617 回答