А можно ли такое делать?

@Zyoma · Регистрация: 05.11.2010

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

Решил я закрепить свои знания по перегрузке операторов и написал свой класс стринга. Естественно я перегрузил оператор извлечения из потока.
Получилось следующие:

C++
1
2
3
4
5
6
7
8
std::istream& operator>>(std::istream& ins, String& st)
{
    if (st.len != -1) delete [] st.str;
    st.str = new char;
    ins >> st.str;
    st.len = strlen(st.str);
    return ins;
}

Меня смущает строка st.str = new char;. Я ж выделяю память под один символ, а потом поток сам расширят его, правильно? Корректно ли это?

@Net_Wanderer · 09.11.2011, 22:47

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

Я ж выделяю память под один символ, а потом поток сам расширят его, правильно? Корректно ли это?

нет, на оба вопроса

@Zyoma · 09.11.2011, 22:50 **[ТС]**

А подробнее?

@Net_Wanderer · 09.11.2011, 22:52

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

А подробнее?

вы выделяете память под один символ, "поток" ничего не расширяет

oxotnik · 09.11.2011, 22:52

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

поток сам расширят его, правильно

а где это такое написано? откуда поток узнает на какой тип расширять?

@Zyoma · 09.11.2011, 22:55 **[ТС]**

Но код работает. Я выделяю под один сивол, ввожу 10,20,30 символов, и он их воспринимает. Почему?

Добавлено через 1 минуту
и в str становится 10,20,30 символов. Как так?

oxotnik · 09.11.2011, 22:56

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

Но код работает. Я выделяю под один сивол, ввожу 10,20,30 символов, и он их воспринимает. Почему?

потому что забиваешь невыделенную память, пока везет, что она не нужна никому, потом может пригодиться и будет ой

@Zyoma · 09.11.2011, 22:58 **[ТС]**

А как тогда сделать правильно?

@Bers · 09.11.2011, 23:05

Честно говоря, я вообще не вижу логики:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
std::istream& operator>>(std::istream& ins, String& st)
{
        if (st.len != -1)   //как длина вообще может быть отрицательным числом?
        {   
             delete [] st.str;    //нарушение инкапсуляции. 
                                       //Доступ к данным в обход интерфейса объекта
         }
         st.str = new char;   //вернётся указатель на один объект типа чар. 
                 //Если str должен указывать на строку, то это явный фейл
          ins >> st.str;
          st.len = strlen(st.str);
           return ins;
}

Ну а главное, я надеюсь вам хватило ума не включать функции взаимодействия между стримами и вашим классом в файлы самого класса?

@Zyoma · 09.11.2011, 23:12 **[ТС]**

len = -1 когда строка пустая, совсем. причем тут нарушение инкапсуляции, если эта функция дружественная? А вам хватит ума написать как надо правильно, а не критиковать вещи совершенно не касающиеся темы?

Добавлено через 4 минуты
str - это указатель на char, причем тут указатель на строку?

@Bers · 09.11.2011, 23:14

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

len = -1 когда строка пустая, совсем. причем тут нарушение инкапсуляции, если эта функция дружественная? А вам хватит ума написать как надо правильно, а не критиковать вещи совершенно не касающиеся темы?

Ну если строка пустая, значит количество символов, которое она содержит рано нуль. Логично?

Мне ума хватит. Я писал когда свой деццкий велосипед строк. Это было первое, что я сделал на с++.

Он у меня до сих пор лежит. Но это деццкий код. То есть сразу видно, что писал его ещё новичок)
Однако, таких вот откровенных ляпов и ошибок логики в нем конечно нет. Он полностью работоспособный.

Добавлено через 1 минуту

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

str - это указатель на char, причем тут указатель на строку?

Указатель на что? На символ чар? или на массив символов чар?

Сам язык не делает различие между указателем на объект, или на массив объектов. За этим следит программист. Его задача знать что это на самом деле - указатель на объект, или на массив. И соотвественно не встрять на этом.

Не написать delete ptr; там, где требуется delete[] ptr;
и тп

@Zyoma · 09.11.2011, 23:16 **[ТС]**

конечно же вы не поделитесь своими деццкими разработками. Молоть языком очень прошу не в этой теме. Я обратился за помощью, а не за критикой тролля.

@ValeryLaptev · 09.11.2011, 23:18

Zyoma, тут проблема с выделением памяти.
Тебе заранее неизвестно, сколько символов будет введено.
Поэтому в конструкторе твоего класса String нужно застолбить некий минимальный размер, например, 32 символа.
В функции ввода надо вводить посимвольно и считать символы.
Так как возможен ввод более 32 символов, то в кллассе надо предусмотреть приватную функцию resize(), которая должна делать следующее:
затребовать новый динамический массив
скопировать туда уже занятые символы
удалить старый массив
Сменить указатель на новый.
Размер нового динамического массива обычно в два раза больше текущего.

@Zyoma · 09.11.2011, 23:21 **[ТС]**

Вот теперь я понял.
Спасибо огромное.

@Bers · 09.11.2011, 23:23

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

конечно же вы не поделитесь своими деццкими разработками. Молоть языком очень прошу не в этой теме. Я обратился за помощью, а не за критикой тролля.

Я могу поделиццо своим деццким лесапедом. Мне не жалко. Если конечно, вы меня об этом вежливо попросите.

А что касается критики, то я вам указал на критические ошибки. Которые приведут к крушению.

Если вы выделяете память вот так: char ptr = new char;
То удалять её вы сможете только вот так: delete ptr; ptr=0;

Если вы выделяете память вот так: char ptr = new char [MAX];
То удалять её вы сможете только вот так: delete [] ptr; ptr=0;

То есть, правило простое: если выделял память под массив, то и удаляй тоже массив
Если выделял под объект, то и удаляй тоже объект.

Если попутаешь - потом прибежишь на этот форум и будишь спрашивать: почему у меня в рантайме лагает.

Потому что компилятор не различает указатель на массив, от указателя на объект.

А теперь смотри как ты создаёшь:

C++
1
st.str = new char;

и как удаляешь

C++
1
if (st.len != -1) delete [] st.str;

Это называется "порча памяти", и "неизвестное поведение".

Человеку говоришь: у тебя ошибка, мина замедленного действия. А он "аццтань, ты тролль!" Ну тупи дальше тогда.

@Zyoma · 09.11.2011, 23:28 **[ТС]**

Все конечно хорошо, спасибо. Но я читал столько высокомерия в твоих указаниях об ошибках, что стало тошно.

@Bers · 10.11.2011, 00:14

Не по теме:

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

Все конечно хорошо, спасибо. Но я читал столько высокомерия в твоих указаниях об ошибках, что стало тошно.

Вы меня извините пожалуйста. Я совершенно не хотел вас ничем задеть.
Я думал вы для того и выложили код, что бы его покритиковали, и указали на ошибки и изъяны
Я не хотел показаться высокомерным.

Новые блоги и статьи Все статьи Все блоги /
Программный отбор элементов справочника Номенклатура по группе 1С Maks 22.03.2026 Установка программного отбора элементов справочника "Номенклатура" из модуля формы документа. В качестве фильтра для отбора справочника служит группа номенклатуры. Отбор под наименованию группы (на. . .	Как я обхитрил таблицу Word Alexander-7 21.03.2026 Когда мигает курсор у внешнего края таблицы, и нам надо перейти на новую строку, а при нажатии Enter создается новый ряд таблицы с ячейками, то мы вместо нервных нажатий Энтеров мы пишем любые буквы. . .	Krabik - рыболовный бот для WoW 3.3.5a AmbA 21.03.2026 без регистрации и смс. Это не торговля, приложение не содержит рекламы. Выполняет свою непосредственную задачу - автоматизацию рыбалки в WoW - и ничего более. Однако если админы будут против -. . .	Программный отбор элементов справочника Сотрудники по перечислениям 1С Maks 21.03.2026 Установка программного отбора элементов справочника "Сотрудники" из модуля формы документа. В качестве фильтра для отбора служит предопределенное значение перечислений. Процедура. . .
Переходник USB-CAN-GPIO Eddy_Em 20.03.2026 Достаточно давно на работе возникла необходимость в переходнике CAN-USB с гальваноразвязкой, оный и был разработан. Однако, все меня терзала совесть, что аж 48-ногий МК используется так тупо: просто. . .	Оттенки серого Argus19 18.03.2026 Оттенки серого Нашёл в интернете 3 прекрасных модуля: Модуль класса открытия диалога открытия/ сохранения файла на Win32 API; Модуль класса быстрого перекодирования цветного изображения в оттенки. . .	SDL3 для Desktop (MinGW): Рисуем цветные прямоугольники с помощью рисовальщика SDL3 на Си и C++ 8Observer8 17.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-rectangles-sdl3-c. zip finish-rectangles-sdl3-cpp. zip	Символические и жёсткие ссылки в Linux. algri14 15.03.2026 Существует два типа ссылок — символические и жёсткие. Ссылка в Linux — это запись в каталоге, которая может указывать либо на inode «файла-ИСТОЧНИКА», тогда это будет «жёсткая ссылка» (hard link),. . .

@Zyoma 10 / 10 / 3 Регистрация: 05.11.2010 Сообщений: 30
	09.11.2011, 22:50 [ТС]
	А подробнее? 0

@Zyoma 10 / 10 / 3 Регистрация: 05.11.2010 Сообщений: 30
	09.11.2011, 22:55 [ТС]
	Но код работает. Я выделяю под один сивол, ввожу 10,20,30 символов, и он их воспринимает. Почему? Добавлено через 1 минуту и в str становится 10,20,30 символов. Как так? 0

@Zyoma 10 / 10 / 3 Регистрация: 05.11.2010 Сообщений: 30
	09.11.2011, 22:58 [ТС]
	А как тогда сделать правильно? 0

@Zyoma 10 / 10 / 3 Регистрация: 05.11.2010 Сообщений: 30
	09.11.2011, 23:12 [ТС]
	len = -1 когда строка пустая, совсем. причем тут нарушение инкапсуляции, если эта функция дружественная? А вам хватит ума написать как надо правильно, а не критиковать вещи совершенно не касающиеся темы? Добавлено через 4 минуты str - это указатель на char, причем тут указатель на строку? 0

@Zyoma 10 / 10 / 3 Регистрация: 05.11.2010 Сообщений: 30
	09.11.2011, 23:16 [ТС]
	конечно же вы не поделитесь своими деццкими разработками. Молоть языком очень прошу не в этой теме. Я обратился за помощью, а не за критикой тролля. 0

@ValeryLaptev 1069 / 848 / 60 Регистрация: 30.04.2011 Сообщений: 1,659
	09.11.2011, 23:18
	Zyoma, тут проблема с выделением памяти. Тебе заранее неизвестно, сколько символов будет введено. Поэтому в конструкторе твоего класса String нужно застолбить некий минимальный размер, например, 32 символа. В функции ввода надо вводить посимвольно и считать символы. Так как возможен ввод более 32 символов, то в кллассе надо предусмотреть приватную функцию resize(), которая должна делать следующее: затребовать новый динамический массив скопировать туда уже занятые символы удалить старый массив Сменить указатель на новый. Размер нового динамического массива обычно в два раза больше текущего. 1

@Zyoma 10 / 10 / 3 Регистрация: 05.11.2010 Сообщений: 30
	09.11.2011, 23:21 [ТС]
	Вот теперь я понял. Спасибо огромное. 0

@Zyoma 10 / 10 / 3 Регистрация: 05.11.2010 Сообщений: 30
	09.11.2011, 23:28 [ТС]
	Все конечно хорошо, спасибо. Но я читал столько высокомерия в твоих указаниях об ошибках, что стало тошно. 0