Question

我正在将以下代码从C＃转换为C，我不认为我这样做是正确的。

Entry3D和entry_3d只是在两种情况下都包含int lookup, length, extra的结构。这是C＃片段（它完全正常）：

    BinaryReader reader = new BinaryReader(File.Open("artidx.mul", FileMode.Open));
    BinaryReader art = new BinaryReader(File.Open("art.mul", FileMode.Open));
    int count = (int)(reader.BaseStream.Length / 12);
    int length = 0x10000;
    Entry3D[] index = new Entry3D[length];
    Console.WriteLine("Loading indexes {0}", count);
    for (int i = 0; i < count && i < length; i++)
    {
        index[i].lookup = reader.ReadInt32();
        index[i].length = reader.ReadInt32();
        index[i].extra = reader.ReadInt32();
    }
    for (int i = count; i < length; ++i)
    {
        index[i].lookup = -1; 
        index[i].length = -1;
        index[i].extra = -1;
    }

这是C中我遇到问题的部分。

/* Init everything else */
printf( "Init maps & gfx" );
mapFile = fopen( "sd:/map0.mul",   "rb" ), 
 artIdx = fopen( "sd:/artidx.mul", "rb" ),
    art = fopen( "sd:/art.mul",    "rb" );

if ( mapFile == NULL || artIdx == NULL || art == NULL )
{
  printf( "D'oh, file loading failed: [%d][%d][%d]", mapFile == NULL, artIdx == NULL, art == NULL  );
  return(1);
}

/* Find artidx count and set length */
printf( "Init art index" );
fseek( artIdx, 0, SEEK_END ); 
int count = (ftell(artIdx) / 12), length = 0x10000; // 0x10000 entries availible
entry_3d entries[length];
fseek( artIdx, 0, SEEK_SET );

printf( "Reading art-idx, count=%d", count );
int i;
for( i = 0; i < count && i < length; i++ )
{
  //printf( "loc: %u i: %u", ftell( artIdx ), i );
  //fread( &entries[i], 12, 1, artIdx );
  fread( &(entries[i].lookup), 4, 1, artIdx ); // read 12 bytes total
  fread( &(entries[i].length), 4, 1, artIdx );
  fread( &(entries[i].extra), 4, 1, artIdx );
}

printf( "Setting aidx remainders" );
for ( i = count; i < length; i++ )
{
  entries[i].lookup = -1; 
  entries[i].length = -1;
  entries[i].extra = -1;
}

我有点不可能调试C部分，但我可以告诉它在第一个for循环中的某个地方“死”。我猜它可能读过流的末尾（即使它不应该）？

Answer 1

我在这里看不出太多错误。代码可以在linux / windows框上正常工作。

可以改进错误处理。

当artidx文件不是12字节的倍数时，它不会正常失败。

我认为这段代码很容易在Wii游戏机上耗尽内存。尝试动态分配entry_3d数组（使用calloc / malloc）。另外，尝试缩短长度以查看是否内存不足。当堆栈重叠堆时，Wii上的libc实现并不是不可能警告你的;嵌入式系统有时会假设程序员知道设备的所有复杂性和限制（带有许多处理芯片的Wii在很多方面都是非常好的例子，所以我真的考虑过这个。）

我很确定你知道如何找到自制软件开发者所潜伏的论坛。他们或许可以在这里说清楚。

Answer 2

如果您无权访问调试器，则可以在循环内添加一些printf语句，以查看它在死亡之前的距离。在每个循环迭代的顶部添加printf("i == %i\n", i);之类的内容。此外，将fread的返回值保存到变量并将其打印出来（对于每个fread调用）。当程序死亡时，这应该至少留下一些你可以遵循的面包屑。

另外，请检查fopen和fseek来电的返回值，并确保它们已成功返回。

在C中需要有关文件IO的帮助

2 个答案: