Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.67/9: Рейтинг темы: голосов - 9, средняя оценка - 4.67
61 / 28 / 24
Регистрация: 28.09.2017
Сообщений: 399

Пользовательский класс String: реализовать копирование строки и вывод её на консоль (найти ошибки в коде)

02.05.2018, 09:41. Показов 1887. Ответов 12
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Привет. Мне нужен ответ: я дурак, или это c++ подвел (естественно первое). Я решил попрактиковаться и написать собственный класс string, все методы отлично работают, кроме метода, производящего копирование строки.
C++
1
2
3
4
5
6
7
8
9
10
    const char* _copy(char* begin, char* end) const {  //begin - указатель на начало интервала, end - на конец
        char *copy_str = new char[5];                       //5 элеметов для примера
 
        unsigned int str_i = 0;
        for (char *str_ = begin; str_ != end + 1; str_++){ //интервал - [begin; end];
            copy_str[str_i] = *str_;
            str_i++;
        }
        return copy_str;
    }
Но дело даже не в самом копировании, а в том, сколько места резервируется в памяти для copy_str. Как видим их 5, но при выводе выходит мусор. Например:
C++
1
2
3
4
string str_1 = "Hello World!!!";  //создаю и инициализирую 1 объект класса
string str_2; //создаю 2 объект класса
 
str_2 = str_1._copy(&str_1.data[0], &str_1.data[4]); //копируем "Hello"
Вывод: Hello¤¤¤¤▌▌▌▌▌▌▌╫╬╗N▌
Сразу скажу, что оператор присваивания работает верно. Вот код:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
    const void operator=(const string& right_) {
        if (this->data != nullptr) {
            delete[] this->data;
            delete this->begin;
            delete this->end;
            this->length = 0;
        }
 
        this->data = new char[255];
 
        for (const char *str_ = right_.begin; str_ != right_.end; str_++) {
            this->data[this->length] = *str_;
            this->length++;
        }
    }
Слово копируется, но далее - мусор. Что не так?
0
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
02.05.2018, 09:41
Ответы с готовыми решениями:

Реализовать пользовательский класс Matrix (найти причины возникновения ошибок в коде и исправить эти ошибки)
Доброго времени суток,задание написать класс Matrix,все вроде бы понятно,но почему-то моя матрица инициализируется мусором,не знаю почему,и...

Реализовать структуру Student (не работает вывод на консоль - найти и исправить ошибки в коде)
Описать структуру с именем STUDENT, содержащую следующие поля:-фамилия и инициалы;-номер группы;-успеваемость (массив из пяти...

Найти причины и способы исправления ошибок в коде (пользовательский класс "String", перегрузка функций)
Не пойму почему этот код не работает...Подскажите что не так,пожалуйста. #include "stdafx.h" #include <iostream> ...

12
7804 / 6568 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
02.05.2018, 10:00
А вывод-то ты реализовал свой, который будет выводить именно length символов?
0
1617 / 1182 / 553
Регистрация: 08.01.2012
Сообщений: 4,561
02.05.2018, 10:01
нормально же копирует, но про 0 прикрывающий зад доводилось слышать?
0
61 / 28 / 24
Регистрация: 28.09.2017
Сообщений: 399
02.05.2018, 10:07  [ТС]
nmcf, вывод:
C++
1
2
3
4
    const void operator<<(std::ostream& out_stream) {
        for (int i = 0; i < this->length; i++)
            out_stream << this->data[i];
    }
MansMI, естественно, не раз использовал в этой программе. Но в данной ситуации он роли особой не играет.

Добавлено через 1 минуту
nmcf, вывод конечно не ахти, но я не парился, потом допилю.
0
7804 / 6568 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
02.05.2018, 10:11
Взялся делать свой string, значит и вывод делай тоже свой. Или обеспечивай везде символ конца строки.
0
61 / 28 / 24
Регистрация: 28.09.2017
Сообщений: 399
02.05.2018, 10:16  [ТС]
nmcf,
Цитата Сообщение от nmcf Посмотреть сообщение
вывод делай тоже свой
я же только что показал код вывода

Добавлено через 20 секунд
Но проблема то точно не в выводе.

Добавлено через 15 секунд
Я же тестил.

Добавлено через 2 минуты
Вот, например, функция вставки:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
    const void _insert(unsigned ind, const char* str) {
        int str_length = 0;
        for (const char* str_ = str + '\0'; *str_ != '\0'; str_++)
            str_length++;
 
        for (char *str_ = this->end; str_ != &this->data[ind - 1]; str_--)
            if (str_ + str_length < &this->data[0] + 255) {
                char *p = str_ + str_length;
                *p = *str_;
            }
            else
                throw std::logic_error("Incorrect index");
 
        int str_i = 0;
        for (int i = ind; i < ind + str_length; i++) {
            this->data[i] = str[str_i];
            str_i++;
        }
 
        this->length += str_length;
    }
После нее вывод происходит правильно.
0
7804 / 6568 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
02.05.2018, 10:43
Для чего вот это?
C++
1
2
            delete this->begin;
            delete this->end;
Помимо data там ещё массивы, что ли?

И где учитывается реальная длина строки? В copy() у тебя 5 выделяется, хотя end от begin может отстоять на любую длину.
А length вообще не видно.
Копирование вообще должно быть через конструктор копирования.
0
61 / 28 / 24
Регистрация: 28.09.2017
Сообщений: 399
02.05.2018, 11:21  [ТС]
Цитата Сообщение от nmcf Посмотреть сообщение
Для чего вот это?
мой косяк.
Цитата Сообщение от nmcf Посмотреть сообщение
Копирование вообще должно быть через конструктор копирования.
вот я кстати ну ооочень долго пытался понять суть такого конструктора, помнится весь инет облазил в поисках, но так до конца и не понял, зачем он нужен. Можете объяснить?
0
7804 / 6568 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
02.05.2018, 13:38
Чтобы получить копию объекта.
0
61 / 28 / 24
Регистрация: 28.09.2017
Сообщений: 399
02.05.2018, 15:13  [ТС]
nmcf, а как?
0
7804 / 6568 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
02.05.2018, 19:36
Что как? Ну был один объект, а через конструктор копирования получается ещё один с такими же значениями полей.
0
61 / 28 / 24
Регистрация: 28.09.2017
Сообщений: 399
02.05.2018, 19:59  [ТС]
nmcf, а, ну я понял, конструктор копирования тоже самое, если написать:
C++
1
2
string str_1 = "Hello World";
string str_2 = str_1; //str_2 = "Hello World"
Но это не то, что мне нужно, в своем методе я хочу сделать не полную копию, а лишь копию выбранного интервала.
0
7804 / 6568 / 2988
Регистрация: 14.04.2014
Сообщений: 28,705
02.05.2018, 23:07
Тогда типа substr() делай. Но конструктор всё равно должен быть.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
02.05.2018, 23:07
Помогаю со студенческими работами здесь

Реализовать класс, выводящий сообщение на консоль (найти и исправить ошибки)
Здравствуйте! Разбираю тему &quot;классы&quot; в С++. Программа должна выдавать сообщение: website: cppstudio.com theme: Classes and Objects...

Реализовать класс Bank (найти и исправить ошибки и недочёты в коде)
Что тут не так? Подскажите пж! 2 дня сижу и не понимаю как сделать этот чёртов банк! #include &lt;iostream&gt; #include...

Реализовать базовый класс «Буква» в соответствии с условием (найти и исправить ошибки в коде)
Создать базовый класс «буква», такой, что: a) он содержит перечислительная тип, что обозначает 4 цвета. b) его экземпляр содержит код...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
13
Ответ Создать тему
Новые блоги и статьи
Символьное дифференцирование
igorrr37 13.02.2026
/ * Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2). Унарный минус обозначается как ! */ #include <iostream> #include <stack> #include <cctype>. . .
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров
8Observer8 12.02.2026
Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 11.02.2026
Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image
8Observer8 10.02.2026
Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .
Установка Qt-версии Lazarus IDE в Debian Trixie Xfce
volvo 10.02.2026
В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru