`std :: filesystem :: path :: operator /(/ * args * /)`不能按预期工作

时间:2019-05-08 15:58:57

标签: c++ constructor c++17 std-filesystem

我在构造函数中有一个带有初始化程序列表的类,其中我要初始化的字段之一是std::filesystem::path,但似乎没有初始化为期望值。

MyClass::MyClass(
    unsigned int deviceSerial,
    const std::string& processName
) :
    deviceSerial(deviceSerial),
    processName(processName),
    configFilePath(GetBasePath() / std::to_string(deviceSerial) / ("#" + processName + ".json"))
{
    /* Parameter checks */
}

使用调试器,我可以看到GetBasePath()完全返回了我期望的结果(以正确的路径返回了std::filesystem::path),但是/运算符似乎没有作用。进入构造函数的主体后,我可以看到configFilePath被设置为GetBasePath()的结果,而没有附加额外的信息。

我正在使用MSVS-2019,我将C ++语言标准设置为C ++ 17,并且在调试模式下,我禁用了所有优化。

我还在类的主体中测试了以下内容,但我仍然看到path仅是GetBasePath()的结果,并且没有附加多余的项目。

{
    auto path = GetBasePath();             // path = "C:/Users/Me/Desktop/Devices"
    path /= std::to_string(deviceSerial);  // path = "C:/Users/Me/Desktop/Devices"
    path /= ("#" + processName + ".json"); // path = "C:/Users/Me/Desktop/Devices"
}

有一点需要注意的是,我也使用+=而不是/=进行了上述测试,但仍然看到了相同的结果。

编辑

根据要求,下面是一个最小的完整且可验证的示例。

#include <Windows.h>
#include <cstdio>
#include <filesystem>
#include <memory>
#include <string>

std::string ExpandPath(const std::string &str) {
  auto reqBufferLen = ExpandEnvironmentStrings(str.c_str(), nullptr, 0);

  if (reqBufferLen == 0) {
    throw std::system_error((int)GetLastError(), std::system_category(),
                            "ExpandEnvironmentStrings() failed.");
  }

  auto buffer = std::make_unique<char[]>(reqBufferLen);
  auto setBufferLen =
      ExpandEnvironmentStrings(str.c_str(), buffer.get(), reqBufferLen);

  if (setBufferLen != reqBufferLen - 1) {
    throw std::system_error((int)GetLastError(), std::system_category(),
                            "ExpandEnvironmentStrings() failed.");
  }

  return std::string{buffer.get(), setBufferLen};
}

int main() {
  unsigned int serial = 12345;
  std::string procName = "Bake";

  std::filesystem::path p(ExpandPath("%USERPROFILE%\\Desktop\\Devices"));
  std::printf("Path = %s\n", p.string().c_str());
  // p = C:\Users\Me\Desktop\Devices

  p /= std::to_string(serial);
  std::printf("Path = %s\n", p.string().c_str());
  // p = C:\Users\Me\Desktop\Devices

  p /= "#" + procName + ".json";
  std::printf("Path = %s\n", p.string().c_str());
  // p = C:\Users\Me\Desktop\Devices

  std::getchar();
}

I've also used this example and tested with `p.append()` and got the same result.

1 个答案:

答案 0 :(得分:1)

我要感谢@rustyx和@Frank的建议,遵循该建议使我发现了创建初始字符串的一种错误,该初始字符串被传递给路径构造函数(@MM在我输入此答案时发现了确切的错误)

我创建了一个函数(该类正在我的类中使用)std::string ExpandPath(const std::string& path),该函数使用Windows API扩展路径中的所有环境变量并返回字符串。该字符串是由char*和一个计数生成的,该计数包括空字节,因此在使用构造函数变体std::string(char* cstr, size_t len)创建字符串时,该字符串本身包含空字节。

因为我正在使用调试器来询问它读取C样式字符串并在空字节处停止的变量。在我的原始示例中,我也使用printf(),因为我刚好偏爱使用此函数进行输出,但是同样会在空字节处停止打印。如果将输出更改为使用std::cout,我可以看到输出具有预期的路径,但它们具有额外的空间(将空字节打印为空格)。使用std::cout,我看到每个附加项的结果如下:

Path = C:\Users\Me\Desktop\Devices
Path = C:\Users\Me\Desktop\Devices \12345
Path = C:\Users\Me\Desktop\Devices \12345\#Bake.json

摘要:

我的 ExpandPath()(其中

  return std::string{buffer.get(), setBufferLen};

应该是

  return std::string{buffer.get(), setBufferLen - 1};