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

2019-09-10 23:08:28 阅读：224 来源： 互联网

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

本站声明： 1. iCode9 技术分享网（下文简称本站）提供的所有内容，仅供技术学习、探讨和分享；
2. 关于本站的所有留言、评论、转载及引用，纯属内容发起人的个人观点，与本站观点和立场无关；
3. 关于本站的所有言论和文字，纯属内容发起人的个人观点，与本站观点和立场无关；
4. 本站文章均是网友提供，不完全保证技术分享内容的完整性、准确性、时效性、风险性和版权归属；如您发现该文章侵犯了您的权益，可联系我们第一时间进行删除；
5. 本站为非盈利性的个人网站，所有内容不会用来进行牟利，也不会利用任何形式的广告来间接获益，纯粹是为了广大技术爱好者提供技术内容和技术思想的分享性交流网站。

ICode9

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