Проблемы с полем класса в виде массива char

@Homa_Sapiens · Регистрация: 25.05.2023

Студворк — интернет-сервис помощи студентам

Имеется класс Array (привожу заголовочный файл)

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
class Array {
protected:
    char* array;
    unsigned int length;
    static unsigned int limit;
public:
    class IndexError : public std::exception {
        std::string m_msg;
    public:
        IndexError(unsigned int i);
        const char* what() const noexcept override;
    };
    Array(unsigned int len, char val);
    Array(unsigned int len) : Array(len, '0') {};
    Array& operator=(const Array& other);
    Array(const Array& other);
    char operator[](unsigned int i) const;
    char& operator[](unsigned int i);
    virtual Array plus(const Array& other) const;
};

И унаследованный от него класс String

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
class String : public Array {
public:
    String() : Array(100, '0') {};
    String(unsigned int len) : Array(len, '0') {};
    String(char* source_string);
    const char* c_str() const;
    String& operator=(const String&);
    String plus(const String&) const;
    int search_substring(const String&) const;
    String& delete_substring(const String&);
    String& insert_substring(const String&, unsigned int);
    unsigned int get_length() const;
    ~String();
    friend std::ostream& operator<<(std::ostream& out, const String& str);
    friend std::istream& operator>>(std::istream& in, String& str);
};

Стандартный вывод в консоль перегружен следующим образом (пробовал по-разному, результат идентичен):

C++
1
2
3
4
5
6
std::ostream& operator<<(std::ostream& out, const String& str) {
    for (int i = 0; i < str.length; i++) {
        std::cout << (char)str.array[i];
    }
    return out;
}

Стандартный ввод из консоли перегружен так:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
std::istream& operator>>(std::istream& in, String& str) {
    std::string tmp;
    in >> tmp;
 
 
    auto len = std::min((unsigned int)(tmp.length()), Array::limit);
    std::cout << len << std::endl;
    delete[] str.array;
    str.array = new char[len+1];
 
    for (int i = 0; i < len; i++) {
        str.array[i] = tmp[i];
    }
    str.array[len] = '\0';
    
    str.length = len;
    std::cout << str.array << " " << str.length << std::endl;
    return in;
}

При попытке вывода массива array с помощью std::cout выводятся иные символы, отличные от введенных, хотя в функции ввода массив array выводится корректно (делал исключительно для проверки).
Что не так? Заранее благодарю

@zss · 05.10.2023, 06:49

Приведите код копиконструктора и operator=

@Homa_Sapiens · 05.10.2023, 07:02 **[ТС]**

C++
1
2
3
4
5
6
7
8
String::String(const String& str)
{
    length = str.length;
    array = new char[length + 1];
    for (unsigned int i = 0; i < length; i++) {
        array[i] = str.array[i];
    }
}

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
String& String::operator=(const String& obj) {
    if (this == &obj) return *this;
 
    unsigned int length = get_length();
    unsigned int obj_length = obj.get_length();
 
    if (length != obj_length)
    {
        delete[] array;
        array = new char[obj_length];
    }
 
    for (unsigned int i = 0; i < obj_length; i++)
    {
        array[i] = obj.array[i];
    }
 
    return *this;
}

Алексей1153 · 05.10.2023, 07:56

Сообщение от Homa_Sapiens

с полем класса в виде массива char

какой же там массив, это просто указатель

Сообщение от Homa_Sapiens

String& String::operator=(const String& obj)

тут не обновляется поле Array::length

Сообщение от Homa_Sapiens

String::String(const String& str)
{
length = str.length;
array = new char[length + 1];
for (unsigned int i = 0; i < length; i++) {
array[i] = str.array[i];
}
}

а тут завершающий ноль не пишется

И вообще, выложи ПОЛНЫЙ код, а не обрывки

@Homa_Sapiens · 05.10.2023, 08:02 **[ТС]**

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
#include <string>
#include "algorithm"
#include "exception"
 
class Array {
protected:
    char* array;
    unsigned int length;
    static unsigned int limit;
public:
    class IndexError : public std::exception {
        std::string m_msg;
    public:
        IndexError(unsigned int i);
        const char* what() const noexcept override;
    };
    Array() : Array(0) {};
    Array(unsigned int len, char val);
    Array(unsigned int len) : Array(len, '0') {};
    Array& operator=(const Array& other);
    Array(const Array& other);
    char operator[](unsigned int i) const;
    char& operator[](unsigned int i);
    virtual Array plus(const Array& other) const;
};

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
#include "Array.h"
 
unsigned int Array::limit = 100;
 
Array::Array(unsigned int len, char val) {
    if (len > limit) throw;
    array = new char[len];
    length = len;
    for (int i = 0; i < len; i++) {
        array[i] = val;
    }
}
 
 
char Array::operator[](unsigned int i) const {
    if (i >= length) throw IndexError(i);
    return array[i];
}
 
char& Array::operator[](unsigned int i) {
    if (i >= length) throw IndexError(i);
    return array[i];
}
 
Array Array::plus(const Array& other) const {
    return Array(0, 0);
}
 
Array& Array::operator=(const Array& other) {
    length = other.length;
    array = new char[length];
    for (unsigned int i = 0; i < length; i++) {
        array[i] = other[i];
    }
    return *this;
}
 
Array::Array(const Array& other) {
    length = other.length;
 
    array = new char[length];
 
    for (unsigned int i = 0; i < length; i++)
    {
        array[i] = other.array[i];
    }
}
 
Array::IndexError::IndexError(unsigned int i) {
    m_msg.assign("ERROR: Wrong index " + std::to_string(i));
}
 
const char* Array::IndexError::what() const noexcept {
    return m_msg.c_str();
}

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
#include "Array.h"
 
class String : public Array {
public:
    String() : Array(100, '0') {};
    String(unsigned int len) : Array(len, '0') {};
    String(char* source_string);
    String(const String& str);
    const char* c_str() const;
    String& operator=(const String&);
    String plus(const String&) const;
    int search_substring(const String&) const;
    String& delete_substring(const String&);
    String& insert_substring(const String&, unsigned int);
    unsigned int get_length() const;
    ~String();
    friend std::ostream& operator<<(std::ostream& out, const String& str);
    friend std::istream& operator>>(std::istream& in, String& str);
};

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
#include "String.h"
#include <iostream>
#include "cstring"
#include <cstdio>
#include <string>
#include <algorithm>
 
String::String(char* source_string) :
    Array(strlen(source_string))
{
    auto len = (unsigned int)(strlen(source_string));
    for (unsigned int i = 0; i < len; i++) {
        array[i] = source_string[i];
    }
}
 
String::String(const String& str)
{
    length = str.length;
    array = new char[length + 1];
    for (unsigned int i = 0; i < length; i++) {
        array[i] = str.array[i];
    }
    array[length] = '\0';
}
 
String String::plus(const String& other) const {
    String tmp(length + other.length);
    for (unsigned int i = 0; i < length; i++) {
        tmp[i] = array[i];
    }
    for (unsigned int i = length; i < length + other.length; i++) {
        tmp[i] = other[i];
    }
    return tmp;
}
 
String& String::operator=(const String& obj) {
    if (this == &obj) return *this;
 
    unsigned int length = get_length();
    unsigned int obj_length = obj.get_length();
 
    if (length != obj_length)
    {
        delete[] array;
        length = obj_length;
        array = new char[obj_length];
    }
 
    for (unsigned int i = 0; i < obj_length; i++)
    {
        array[i] = obj.array[i];
    }
 
    return *this;
}
 
int String::search_substring(const String& sub) const {
    int pos = -1;
    if (length < sub.length) return pos;
    for (unsigned int i = 0; i <= length - sub.length; ++i) {
        bool found = true;
        for (unsigned int j = i, k = 0; j < i + sub.length; ++j, ++k) {
            std::cout << array[j] << " " << sub.array[k];
            if (array[j] != sub.array[k]) {
                std::cout << "!=" << std::endl;
                found = false;
                break;
            }
            std::cout << "==" << std::endl;
        }
        if (found) {
            pos = i;
            return pos;
        }
    }
    return pos;
}
 
String& String::delete_substring(const String& sub) {
    int del_pos = search_substring(sub);
    if (del_pos == -1) {
        return *this;
    }
 
    char* tmp = new char[length - sub.length];
 
    if (length - sub.length == 0)
    {
        delete[] array;
        length = 0;
        return *this;
    }
 
    for (unsigned int i = 1, j = 1; i < length; i++, j++)
    {
        if (i == del_pos) i += (sub.length);
        tmp[j] = array[i];
    }
 
    delete[] array;
    array = tmp;
    length = length - sub.length;
    return *this;
}
 
String& String::insert_substring(const String& sub, unsigned int n) {
    try {
        if (n >= length) throw;
        String tmp(length + sub.length);
        for (unsigned int i = 0; i < n; i++) {
            tmp[i] = array[i];
        }
        for (unsigned int i = n; i < n + sub.length; i++) {
            tmp[i] = sub[i];
        }
        for (unsigned int i = n; i < length; i++) {
            tmp[i + sub.length] = array[i];
        }
        *this = tmp;
        return *this;
    }
    catch (const char* e) {
        std::cout << e << '\n';
    }
    return *this;
}
 
unsigned int String::get_length() const {
    return length;
}
 
String::~String() {
    delete[] array;
}
 
const char* String::c_str() const {
    return array;
}
 
std::ostream& operator<<(std::ostream& out, const String& str) {
    for (int i = 0; i < str.length; i++) {
        std::cout << (char)str.array[i];
    }
    return out;
}
 
std::istream& operator>>(std::istream& in, String& str) {
    std::string tmp;
    in >> tmp;
 
 
    auto len = std::min((unsigned int)(tmp.length()), Array::limit);
    std::cout << len << std::endl;
    delete[] str.array;
    str.array = new char[len+1];
 
    for (int i = 0; i < len; i++) {
        str.array[i] = tmp[i];
    }
    str.array[len] = '\0';
    
    str.length = len;
    std::cout << str.array << " " << str.length << std::endl;
    return in;
}

Алексей1153 · 05.10.2023, 08:25

Homa_Sapiens, с ходу подправил

String(const char* source_string);

а ещё - почему память освобождается в деструкторе String, а не Array?
и деструктор Array нужно сделать виртуальным

---------
но к заявленной ошибке вывода это всё не относится (однако, это нужно тоже исправить)

Не ковыряясь более в коде (хотя, там ковыряться и ковыряться), я попробовал так, всё вывелось

C++
1
2
3
4
5
6
7
int main()
{
    String s1="111";
    auto s2=s1;
    std::cout<<"s1="<<s1<<'\n';
    std::cout<<"s2="<<s2<<'\n';
}

s1=111

s2=111

Новые блоги и статьи Все статьи Все блоги /
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Ниже машинный перевод статьи The Thinkpad X220 Tablet is the best budget school laptop period . Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы,. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .

@zss Модератор 13769 / 10962 / 6491 Регистрация: 18.12.2011 Сообщений: 29,238
	05.10.2023, 06:49
	Приведите код копиконструктора и operator= 0