Question

这篇文章基于here给出的解决方案，SO上的帖子

我编写了一个函数来获取给定的文件扩展名（字符串类型），如：

void GetFileExtension(string filename, char* extension)
{
    vector<string> rec;

    StrDelimit(filename, '.', rec);

    //cout << rec[rec.size()-2].c_str()[1] << endl;

    //extension = rec[rec.size()-2].c_str()[0];

    //extension = &rec[rec.size()-2].c_str()[0];

    string str = rec[rec.size()-2];

    // conversion from string to char*
    vector<char> writable(str.size() + 1);

    std::copy(str.begin(), str.end(), writable.begin());

    //cout << writable << endl;

    extension = &writable[0];
}

StrDelimit已经正常工作，它接收一个字符串并通过给定的分隔符分隔到子字符串的向量

我使用“//从字符串转换为char *”，如图所示，将结果返回给主程序

没有编译错误，但结果是一串无意义的字符。这是什么问题？

任何帮助将不胜感激！

谢谢！

Answer 1

char * extension是字符串地址的临时副本。要实际将新数据放入其中，您必须将数据复制到它所拥有的地址。

这有帮助吗？

#include <stdio.h>
#include <string.h>

void getStringInoExistingBufer(char* existingBufferOut)
{
    strcpy(existingBufferOut, "someData");
}

void getStringIntoNonExistingBuffer(char **newBufferOut)
{
    *newBufferOut = strdup("someOtherData");
}

int main()
{
    char buffer1[100] = {};
    char *buffer2 = NULL;

    printf("buffer1 (before): '%s'\n", buffer1);
    getStringInoExistingBufer(buffer1);
    printf("buffer1 (after): '%s'\n", buffer1);

    printf("\n\n");
    printf("buffer2 (before): '%s'\n", buffer2);
    getStringIntoNonExistingBuffer(&buffer2);
    printf("buffer2 (after): '%s'\n", buffer2);
}

<强>输出：

buffer1 (before): ''
buffer1 (after): 'someData'


buffer2 (before): '(null)'
buffer2 (after): 'someOtherData'

Answer 2

你的问题是你在堆栈上创建数据成员writable，然后用extension指向它但是可写范围是函数意味着当函数返回writable时已被清除，extension没有任何指示。

你可以为char *分配一个动态缓冲区，但这比C ++更多的是C方法。如果可能的话，我建议更改函数类型以返回字符串或向量，这将使其更容易，并且您可以在需要时将这些类型更改为char *。

Answer 3

为什么不直接将扩展名作为std::string？

例如：

std::string GetFileExtension(std::string filename)
{
    std::vector<string> rec;

    StrDelimit(filename, '.', rec);

    std::string extension = rec[rec.size() - 2];
    return extension;
}

Answer 4

首先，你并没有真正归还任何东西;

void GetFileExtension(string filename, char* extension)
{
    ...
    extension = &writable[0];  // Assignment to local variable
}

如果您打算更新extension中传递的字符*，则需要将其作为char**（即指向char*的指针）传递。

其次，即使你修复了这个问题，你也试图返回一个指向局部变量中数据的指针;

void GetFileExtension(string filename, char* extension)
{
    ...
    vector<char> writable(str.size() + 1);   // Local variable
    ...
    extension = &writable[0];       // writable disappears after this line
                                    // and extension will point to invalid memory
}

只需返回std::string即可简化您的代码。

从字符串转换为char * - 传递函数指针

4 个答案: