将日期转换为 const char*

Question

我创建了一个Date 类，我正在尝试创建一个转换方法，以便您可以将Date 对象转换为const char*“字符串”。必须是const char*，不是 string。

但是，我发现了一些我无法解决的问题。

这是我的方法：

Date::operator char*() const {
    tm date = { 0, 0, 0, day_, month_ - 1, year_ - 1900, 0, 0, -1 };
    mktime(&date);

    char* weekday[] = { "sunday", ..., "saturday" };
    char* month[] = { "january", ..., "diciembre" };

    char string[50];
    sprintf(string, "%s %d / %s / %d", weekday[fch.tm_wday], day_, month[fch.tm_mon], year_);

    return string;
}

编译器说：

warning: address of local variable ‘cadena’ returned [-Wreturn-local-addr]
  char string[50];

如果我为字符串变量添加static 标签，它将被共享，我不希望这样。

我该怎么做？

Answer 1

在类级别定义您的缓冲区，并在写入后从您的方法中返回它。

但请记住，这是一种愚蠢的工作方式，因为 (1) 对该方法的后续调用会改变缓冲区，如果之前的调用者仍然保留该指针，则会影响他们，以及 (2) 因为它违反了单一责任原则- 最后格式化操作的细节在逻辑上根本不属于日期类的状态。此外，缓冲区必须标记为 mutable 才能从您的 const 方法进行修改，这清楚地表明您违反了类的预期职责。

另外，“经典 C”方法将更改方法以从调用者接收缓冲区（连同其大小），然后直接写入其中。这将内存管理的负担转移到调用者（它在逻辑上属于哪里），但使使用该方法的语法变得相当繁重。

这两种方法都很笨拙和/或功能欠佳；我什至不会提到返回指向动态分配内存的指针的可能性，因为它具有与std::string 相同的开销，但没有自动内存管理（因此它绝对是最愚蠢的选择）。

正如在其他地方已经说过的，正确的 C++ 方法是返回一个std::string，它自己处理分配内存的生命周期。除非您处于极其特殊的条件下，否则没有理由不使用它。

Answer 2

您在函数内部创建了一个名为 string 的变量，一旦函数返回，该变量将超出范围（即变为无效引用）。您还返回了一个指向它的指针，这意味着您正在引用一个不再有效的内存区域。

考虑返回一个std::string，它会为您自动管理内存。

编辑：如果不这样做，您始终可以拥有一个 static 数组并返回一个指向它的指针。

Date::operator const char*() const
{
    static char str[50];
    // fill in array
    return str;
}

当然，您现在不能执行以下操作：

Date d1{...};
Date d2{...};

const char* d1_str = s1;
const char* d2_str = s2;

因为用于将日期转换为字符串的数组由该类的所有实例共享。