C++ std::string 可以替代 strcpy 吗？

Question

我知道已经有一个类似标题的问题，但我想知道针对这个特定案例的选择。

MSVC 编译器给出有关 strcpy 的警告：

1>c:\something\mycontrol.cpp(65): warning C4996: 'strcpy': This function or
variable may be unsafe. Consider using strcpy_s instead. To disable
deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.

这是我的代码：

void MyControl::SetFontFace(const char *faceName)
{
    LOGFONT lf;

    CFont *currentFont = GetFont();
    currentFont->GetLogFont(&lf);
    strcpy(lf.lfFaceName, faceName); <--- offending line
    font_.DeleteObject();
    // Create the font.
    font_.CreateFontIndirect(&lf);

    // Use the font to paint a control.
    SetFont(&font_);
}

注意 font_ 是一个实例变量。 LOGFONT 是一个 Windows 结构，其中 lfFaceName 定义为 TCHAR lfFaceName[LF_FACESIZE]。

我想知道的是我可以做类似以下的事情（如果不是为什么不）：

void MyControl::SetFontFace(const std::string& faceName)
...
  lf.lfFaceName = faceName.c_str();
...

或者如果有完全不同的替代方案，请告诉我。

Answer 1

您收到安全警告的原因是，您的 faceName 参数可能指向一个比 LF_FACESIZE 字符长的字符串，然后 < code>strcpy 会盲目地覆盖 LOGFONT 结构中 lfFaceName 之后的任何内容。你确实有一个错误。

您不应该通过将 strcpy 更改为 strcpy_s 来盲目地修复该错误，因为：

1. *_s 函数是不可移植的 Microsoft几乎所有的发明都复制了其他可移植的 C 库函数的功能。永远不应该使用它们，即使是在不打算移植的程序中（正如看起来的那样）。
2. 盲目的改变往往并不能真正修复此类错误。例如，strcpy 的“安全”变体（strncpy、strlcpy、strcpy_s）仅在以下情况下截断字符串：它太长了，在这种情况下会让您尝试加载错误的字体。更糟糕的是，strncpy 在执行此操作时会忽略 NUL 终止符，因此如果您使用了 CreateFontIndirect，您可能会将崩溃移至 CreateFontIndirect 内。 正确的修复方法是预先检查长度，如果太长则整个操作失败。在这一点上，strcpy变得安全（因为你知道它不会太长），尽管我更喜欢memcpy，因为它让未来的代码读者可以清楚地看到我所认为的关于这个。
3. TCHAR 和 char 不是同一件事；将 C 样式 const char * 字符串或 C++ std::string 复制到 TCHAR 数组而不进行适当的编码转换可能会产生完全是胡说八道。 （根据我的经验，使用 TCHAR 总是一个错误，最大的问题是这样的代码在 ASCII 构建中似乎可以正常工作，并且仍然可以编译 在 UNICODE 模式下，但在运行时会发生灾难性的失败。）

您当然可以使用 std::string 来帮助解决这个问题，但它不会帮助您无需检查长度并手动复制字符串。我可能会这样做。请注意，我在 std::string 中使用 LOGFONTW 和 CreateFontIndirectW 以及从 UTF-8 的显式转换。另请注意，其中大部分内容是从 MSDN 中收集而来的，并且没有经过测试。对不起。

void MyControl::SetFontFace(const std::string& faceName)
{
    LOGFONTW lf;
    this->font_.GetLogFontW(&lf);

    int count = MultiByteToWideChar(CP_UTF8, MB_ERR_INVALID_CHARS,
                                    faceName.data(), faceName.length(),
                                    lf.lfFaceName, LF_FACESIZE - 1)
    if (count <= 0)
        throw GetLastError(); // FIXME: use a real exception

    lf.lfFaceName[count] = L'\0'; // MultiByteToWideChar does not NUL-terminate.

    this->font_.DeleteObject();
    if (!this->font_.CreateFontIndirectW(&lf))
        throw GetLastError(); // FIXME: use a real exception

    // ...
}

Answer 2

lf.lfFaceName = faceName.c_str();

不，您不应该这样做，因为您正在将指向器的本地副本复制到 std::string 中保存的数据。如果 c++ 字符串更改或被删除，则指针不再有效，并且如果 lFaceName 决定更改数据，这几乎肯定会破坏 std::string。

由于您需要复制 ac 字符串，因此需要一个 'c' 函数，那么 strcpy_s （或等效的）是安全的替代方案

Answer 3

你尝试过吗？鉴于您帖子中的信息，该赋值应该会生成编译器错误，因为您尝试将指针分配给数组，这在 C(++) 中不起作用。

#include <cstdio>
#include <string>
using namespace std;

struct LOGFONT {
 char lfFaceName[3];
};


int main() {
        struct LOGFONT f;
        string foo="bar";
        f.lfFaceName = foo.c_str();
        return 0;
}

导致

x.c:13: error: incompatible types in assignment of `const char*' to `char[3]'

我建议使用安全的 strcpy 替代方案，如警告所述，前提是您无论如何都知道目标空间的大小。

Answer 4

#include <algorithm>
#include <iostream>
#include <string>

enum { LF_FACESIZE = 256 }; // = 3 // test too-long input
struct LOGFONT
{
    char lfFaceName[LF_FACESIZE];
};

int main()
{
    LOGFONT f;
    std::string foo("Sans-Serif");
    std::copy_n(foo.c_str(), foo.size()+1 > LF_FACESIZE ? LF_FACESIZE : foo.size()+1,
                f.lfFaceName);

    std::cout << f.lfFaceName << std::endl;
    return 0;
}

Answer 5

lf.lfFaceName = faceName.c_str(); 不起作用有两个原因（假设您将faceName 更改为std:string）

1. c_str() 返回的指针的生命周期是临时的。仅当 fileName 对象未更改且处于活动状态时它才有效。
2. 该行将无法编译。 .c_str() 返回一个指向 char 的指针，而 lfFaceName 是一个字符数组，无法赋值。您需要执行一些操作来填充字符串数组，填充 lfFaceName 处的字节，而指针赋值不会执行此操作。

没有任何 C++ 可以帮助这里，因为 lfFaceName 是一个 C“字符串”。您需要使用 C 字符串函数，例如 strcpy 或 strcpy_s。您可以将代码更改为：

strcpy_s(lf.lfFaceName, LF_FACESIZE, faceName);