0

这是我拥有的函数，“Sprite”是程序中的一个对象，“GetSpriteAtPosition”只是返回一个指向坐标处正确精灵的指针。

我的问题是我在每个精灵中存储了一个字母，以整数的形式。0 是 a，25 是 z，分别介于两者之间。我需要我的函数返回一个 char*，它给我一行精灵的字母，所以如果在程序中精灵拼出“abcdefgh”，那么这就是我需要这个函数打印出来的。有一个 8x8 的精灵网格，我得到了正确的坐标，但是我得到一个错误，我无法在标记的行中将 int 转换为 char*。我该怎么做才能让它发挥作用？

提前致谢！

char* RowLetters(int row)
{
    char* pointer;
    for( int i = 0; i < 8; i++)
    {
        Sprite* selectedSprite = SpriteAtPosition(row*50, i * 50);
        if(selectedSprite != NULL)
        {
            char* temp = (char)(selectedSprite->Frame() + 97); //error here
            pointer = strcat(pointer, temp);
        }
        else
        {
            pointer = strcat(pointer, "test");
        }
    }
    return pointer;
}
4

2 回答 2

3

试试这个：

        char temp = (char)(selectedSprite->Frame() + 97);
        pointer = strcat(pointer, &temp);

我已将变量更改为标准char而不是指针，然后将引用传递给strcat()运算&符。

编辑：

正如评论中指出的那样，这不起作用，因为&temp不是 NULL 终止的。当我通过执行以下操作编写更多 C 语言时，我曾经解决过这个问题。

        char temp[2];
        temp[0] = (char)(selectedSprite->Frame() + 97);
        temp[1] = '\0';
        pointer = strcat(pointer, temp);

当然，temp可以在for()循环之外声明数组以获得更好的性能（理论上）。

这些都没有解决代码的其他问题，比如pointer从未被声明过。我认为对调用函数的更广泛理解是为了确定是pointer应该在这个函数中分配还是由调用者传入。

于 2012-11-27T21:34:24.883 回答
2

您编写的代码将具有未定义的行为，因为pointer未初始化，并且未指向您分配的任何有效内存（以保存行中适当长度的字母。

如果这确实是 C++，正如您所说，那么您不想char*从该函数返回 a，因为这意味着您已经在该函数中分配了一个静态字符串（yuck），或者您将动态分配字符串该函数和调用者必须释放它（恶心）。

这些选项都不是理想的。

我建议一个非常简单的更改来返回 a std::string，如下所示：

std::string RowLetters(int row)
{
    std::string pointer;
    for( int i = 0; i < 8; i++)
    {
        Sprite* selectedSprite = SpriteAtPosition(row*50, i * 50);
        if(selectedSprite != NULL)
        {
            pointer.push_back((char)(selectedSprite->Frame() + 97));
        }
        else
        {
            // ???
            // pointer = strcat(pointer, "test");
        }
    }

    return pointer;
}
于 2012-11-27T21:41:06.857 回答