尝试向文件添加扩展名时，Rename() 函数不起作用

Question

我有一个无扩展名的文件，我想为其添加 .avi 扩展名。因此，我在通过打开文件对话框选择文件后调用此函数（我想在重命名后打开文件）：

int RenameVidAddExt(HWND hWnd, char oldname[MAX_PATH])
{
    int filenamesize=0, n=0;
    char extension [] = ".avi";
    char* newname;

    for (n=0 ; oldname[n]!=NULL ; n++)
    {
        filenamesize++;                                 
    }

    newname = new char[filenamesize+sizeof(extension)];

    newname = oldname;
    strcat (newname, extension);
    //SetDlgItemTextA(hWnd, IDC_EDIT1, szFile);     //test print of filenameof selected file
    //SetDlgItemTextA(hWnd, IDC_EDIT2, newname);    //test print of newname
    rename(oldname,newname);
    return 0;
}

文件名没有改变，我不知道为什么。测试打印显示 oldname 和 newname 都包含正确的字符串（旧的不带扩展名，新的带扩展名）。

Answer 1

由于您使用的是 C++，请考虑同时使用 std::string：

int RenameVidAddExt(HWND hWnd, char oldname[MAX_PATH])
{
    char extension [] = ".avi";
    std::string newname = oldname;

    newname += extension;

    rename(oldname, newname.c_str());

    return 0;
}

一些旁注：

1. 请记住，当您使用 new 或 new[] 时，您必须稍后使用 delete 或 delete[]（当使用 new 或 new[] 创建的对象不存在时）需要更长的时间）。
2. 当使用字符串时，尽可能尝试使用 std::string 。它将使您不必手动处理内存，并且可以将它们分配/复制到其他变量。
3. 通常建议使用 std::vector，而不是使用 new[]。将矢量大小调整为所需的大小。当您需要指向第一个元素的指针时，请使用 &vec[0]。使用 std::vector 的好处是可以为您完成内存管理。
4. 不要手动计算字符串的长度，使用像 strlen 这样的函数。或者，可以使用 str.length() 获取 std::string 的长度。

Answer 2

您无法通过分配指针值来复制 C 样式字符串。您的代码动态创建一个新的缓冲区，然后通过分配给该指针立即将其孤立。

我建议使用 std::string 代替。但是，如果您想继续使用 C 型琴弦，请尝试以下操作：

newname = new char[filenamesize + sizeof(extension)];
strcpy(newname, oldname);   // Copies the original string character by character
strcat(newname, extension);  

祝您好运！