调用成员初始化列表中的函数有问题吗?

问题描述 投票:0回答:2

我正在编写这个复制构造函数:

//CCtor of RegMatrix                    
RegMatrix::RegMatrix(const RegMatrix &other){

    this-> numRow = other.getRow();
    this-> numCol = other.getCol();

    //Create
    _matrix = createMatrix(other.numRow,other.numCol);

    int i,j;

    //Copy Matrix
    for(i=0;i<numRow; ++i){
        for(j=0;j<numCol; ++j){
            _matrix[i][j] = other._matrix[i][j];
        }
    }
}

像这样初始化列表中的numRow、numCol是不是有问题:

numRow(other.numRow), numCol(other.numCol)
而不是:

this-> numRow = other.getRow();
this-> numCol = other.getCol();

还有,不知道有没有这样的问题,是不是在初始化列表中调用其他类的对象的函数有问题,比如:

numRow(other.getRow())

而不是:

this-> numRow = other.getRow();
c++ initialization member-initialization
2个回答
11
投票

初始化列表中的numRow、numCol初始化是否有问题[...]?

一般来说,这样做有两个问题:

  1. 在初始化初始化列表中的对象时,该对象尚未完全构造。因此,当您调用非静态成员函数时,您是在尚未完全构造的对象上调用它们。如果这些函数尝试使用尚未构造的对象的任何子对象,则您将调用未定义的行为
  2. 初始化的顺序是类定义中成员的声明顺序,它是不是它们在初始化列表中列出的顺序。因此,您需要注意需要其他成员提供数据的成员的初始化。 (这可以看作是前一个问题的子问题:使用尚未构造的子对象。)最好避免这种情况,但如果无法避免,请在声明成员的位置添加一个大而可怕的注释在类的定义中,强调其顺序的重要性。

在您的具体示例中这并不重要,因此您可以安全执行此操作。


2
投票

不,实际上没有问题。但要注意初始化的顺序不是您在 init-list 中指定初始值设定项的顺序。该顺序是您的成员在班级中声明的顺序。除了那个潜在的问题之外,我没有看到任何问题。 HTH

© www.soinside.com 2019 - 2024. All rights reserved.