好坏：在构造函数中调用析构函数

Question

---

休息：我不认为这实际上是同一个问题，另一个问题是关于手动调用析构函数的一般问题。这是在类本身内部的创建过程中。仍然想知道执行此操作时会发生什么，如下面的问题所述。

---

起初，我认为这很糟糕，真的很糟糕。只需分析这两个人编写的构造函数的这段代码（请参见下文），并需要将其转换为Delphi对象Pascal。它的行为必须与C版本相同。我不喜欢这种风格，非常丑陋，但是没关系。

另一件事，在代码的两个阶段中，当失败时会调用析构函数（我想关闭连接，但是删除后会自动调用析构函数，为什么还是要这样做呢？）。我认为这不是做到这一点或错过某些东西的方法吗？

另外，在调用析构函数之后，他们想抛出一个异常（是吗？），但是我认为这将永远不会执行，并且当您手动访问或删除它时会导致另一个异常。

---

Serial::Serial(
  std::string &commPortName,
  int bitRate,
  bool testOnStartup,
  bool cycleDtrOnStartup
) {
  std::wstring com_name_ws = s2ws(commPortName);

  commHandle =
    CreateFileW(
      com_name_ws.c_str(),
      GENERIC_READ | GENERIC_WRITE,
      0,
      NULL,
      OPEN_EXISTING,
      0,
      NULL
    );

  if(commHandle == INVALID_HANDLE_VALUE)
    throw("ERROR: Could not open com port");
  else {
    // set timeouts
    COMMTIMEOUTS timeouts;

    /* Blocking:
        timeouts.ReadIntervalTimeout = MAXDWORD;
        timeouts.ReadTotalTimeoutConstant = 0;
        timeouts.ReadTotalTimeoutMultiplier = 0;
       Non-blocking:
        timeouts = { MAXDWORD, 0, 0, 0, 0}; */

    // Non-blocking with short timeouts
    timeouts.ReadIntervalTimeout = 1;
    timeouts.ReadTotalTimeoutMultiplier = 1;
    timeouts.ReadTotalTimeoutConstant = 1;
    timeouts.WriteTotalTimeoutMultiplier = 1;
    timeouts.WriteTotalTimeoutConstant = 1;

    DCB dcb;
    if(!SetCommTimeouts(commHandle, &timeouts)) {
      Serial::~Serial();                                      <- Calls destructor!
      throw("ERROR: Could not set com port time-outs");
    }

    // set DCB; disabling harware flow control; setting 1N8 mode
    memset(&dcb, 0, sizeof(dcb));
    dcb.DCBlength = sizeof(dcb);
    dcb.BaudRate = bitRate;
    dcb.fBinary = 1;
    dcb.fDtrControl = DTR_CONTROL_DISABLE;
    dcb.fRtsControl = RTS_CONTROL_DISABLE;
    dcb.Parity = NOPARITY;
    dcb.StopBits = ONESTOPBIT;
    dcb.ByteSize = 8;

    if(!SetCommState(commHandle, &dcb)) {
      Serial::~Serial();                                    <- Calls destructor!
      throw("ERROR: Could not set com port parameters");
    }
  }

  if(cycleDtrOnStartup) {
    if(!EscapeCommFunction(commHandle, CLRDTR))
      throw("ERROR: clearing DTR");
    Sleep(200);
    if(!EscapeCommFunction(commHandle, SETDTR))
      throw("ERROR: setting DTR");
  }

  if(testOnStartup) {
    DWORD numWritten;
    char init[] = "PJON-python init";
    if(!WriteFile(commHandle, init, sizeof(init), &numWritten, NULL))
      throw("writing initial data to port failed");
    if(numWritten != sizeof(init))
      throw("ERROR: not all test data written to port");
  }
};

Serial::~Serial() {
  CloseHandle(commHandle);
};

// and there is more etc .......
// .............

---

下一个问题，执行此代码并调用析构函数时，内存中将实际发生什么？我无法执行和调试它。

Answer 1

这段代码很丑但合法。当构造函数抛出异常时，不会调用相应的析构函数。因此需要在抛出之前手动调用它以防止资源泄漏。这里真正的错误不是在其他情况下在抛出异常之前手动调用析构函数。

当然，更好的方法是使用一个单独的 RAII 对象来封装commHandle. 一个unique_ptr与定制删除可以起到这个作用。

任何超出低级库的析构函数在现代 C++ 中都是一种代码味道。