Когда класс содержит указатель на динамически выделенную память, то вам необходимо обеспечить в явном виде по крайней мере конструктор копирования, копирующий оператор присваивания и деструктор. Иначе может оказаться. что два объекта класса в результате копирования или присваивания будут содержать указатели на одну и ту же область памяти, для которой будет осуществлена попытка ее удалить дважды каждым объектом.
Кроме того член класса size1
во многих случаях у вас не отражает длину хранящейся строки, так как ему не присваивается никакого значения. И вы забываете выделять память для завершающего нуля копируемых строк.
Также конструктор по умолчанию и конструктор с целочисленным параметром не инициализируют выделенную память. При выводе таких строк с помощью метода show
возникнет неопределенное поведение программы.
Поэтому нет никакого смысла иметь статический член класса size0
равным 80
. По умолчанию у вас должна быть пустая строка.
Также деструктор у вас не удаляет выделенную строку.
В методе input
вы увеличиваете размер текущей строки вместо того, чтобы сделать его равным длине строки параметра
for (int a = 0; uStr[a] != '\0'; a++) {
size1++;
Здесь член класса size1
не был изначально установлен в 0.
Обратите внимание, что лучше, когда длина строки имеет тип size_t
так как именно этот тип имеет возвращаемое значение оператора sizeof
и функции работы со строками std::strlen
.
Класс может выглядеть следующим образом:
class MyString {
private:
size_t n;
char *s;
public:
MyString() : n( 0 ), s( new char[1] ) {
MyString::s[0] = '\0';
}
MyString( const char *s ) : n( 0 ) {
while ( s[n] != '\0' ) ++n;
MyString::s = new char[MyString::n + 1];
std::strcpy( MyString::s, s );
}
MyString( const MyString &rhs ) : n( rhs.n ), s( new char[n + 1] ) {
std::strcpy( MyString::s, rhs.s );
}
~MyString() { delete [] MyString::s; }
MyString & operator =( const MyString &rhs ) {
if ( this != &rhs ) {
if ( MyString::n != rhs.n ) {
delete [] MyString::s;
MyString::n = rhs.n;
MyString::s = new char[n + 1];
}
std::strcpy( MyString::s, rhs.s );
}
return *this;
}
size_t size() const { return MyString::n; }
void input( const char *s ) {
size_t i = 0;
while ( s[i] != '\0' ) ++i;
if ( MyString::n != i ) {
delete [] MyString::s;
MyString::n = i;
MyString::s = new char[MyString::n + 1];
}
std::strcpy( MyString::s, s );
}
void show() const {
std::cout << "\n\n You String: " << MyString::s << "\n\n";
}
};
int main() { MyString str = "qwe"; str.show(); }
size1
в двух конструкторах.delete
. Но такой пример тоже работает: int main() { MyString str = "qwe"; str.input("qwerty"); str.show(); }