c – 好的或坏的：在构造函数中调用析构函数

Break：我认为实际上并不是同一个问题,另一个问题是关于手动调用析构函数的一般性问题.这是在创建过程中,在类本身内部.仍然想知道当你这样做时会发生什么,如下面的问题中所述.

起初,我认为这很糟糕,真的很糟糕.只需分析构造函数的这段代码(见下文),由两个人做,需要将它转换为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 .......
// .............

接下来的问题,在执行此代码时它会在内存中实际发生什么并调用析构函数？我无法执行它并进行调试.

解决方法:

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

当然,更好的方法是使用一个单独的RAII对象来封装commHandle.带有自定义删除器的unique_ptr可以担任此角色.

除了低级库之外的任何析构函数都是现代C中的代码味道.

标签：c,destructor,constructor,windows
来源： https://codeday.me/bug/20190910/1801908.html