Как использовать конструктор копирования для копирования по постоянным переменным?

Question

Мой класс имеет следующие частные переменные, в том числе константный статический, как вы можете видеть:

private: 
    // class constant for # of bits in an unsigned short int: 
    const static int _USI_BITS = sizeof(usi)*CHAR_BIT; 
    usi* _booArr; 
    int _booArrLen; 
    int _numBoos; 

Я новичок в использовании конструкторы копирования, и я не могу понять, как написать один. Вот моя попытка:

BitPack::BitPack(const BitPack& other) { 
    _USI_BITS = other._USI_BITS; 
    _booArr = new usi[other._booArrLen]; 
    for (int i = 0; i < _booArrLen; ++i) 
     _booArr[i] = other._booArr[i]; 
    _booArrLen = other._booArrLen; 
    _numBoos = other.numBoos; 
} 

Компилятор говорит:

ошибка: назначение только для чтения переменной 'BitPack :: _ USI_BITS'

Пожалуйста разуверить меня из моих глупых способов.

Answer 1

Конструкторы, включая конструкторы копирования, должны установить членов-членов, то есть те, которые не являются static. Статические члены разделяются всеми экземплярами и поэтому должны быть инициализированы вне любых конструкторов.

В вашем случае, вам нужно удалить строку

_USI_BITS = other._USI_BITS; 

: обе стороны относятся к одному static члена, так что присваивание не имеет никакого эффекта.

Остальная часть вашего конструктора копий в порядке. Обратите внимание, что, так как ваш конструктор копирования распределяет ресурсы, то rule of three предполагает, что вы должны добавить оператор пользовательского назначения, а также пользовательские деструктор:

BitPack& operator=(const BitPack& other) { 
    ... 
} 
~BitPack() { 
    ... 
}