Собственно проблема такая. 
Дана следующая программа:



----------------------------------

Суть Проблемы: 
Происходит суммирование, результат записывается в Z, Однако информация, находившая в X и Y потеряна. Вместо нее записаны какие то символы.

Испробовано: 

• Заменить строку "void form(str x, str y)" на "void form (str &x, str &y)", тоесть работать не с копиями переменных, а с самими перменными. Проблема исчезла.
• Убрали Диструктор. Проблема Исчезла.

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


Исходник приведен ниже
[*]

Заранее спасибо!

M chipset

Тег [code=cpp] рулез форева. СУВ, Chipset.

Присоединённый файл ( Кол-во скачиваний: 3 )
 TEST.CPP 0,75 Kb

для начала, это неверно

правильно:
s= new char [ strlen( x.s ) +  strlen( y.s) ];
пардон, вернее так:
s= new char [ strlen( x.s ) +  strlen( y.s)  + 1];

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

повезло просто скорее всего. 

Это сообщение отредактировал(а) zkv - 6.4.2007, 21:54

я не так давно начал изучать Объектно ориентированное... не объяснишь про конструктор копирования пару сторк? 

sizeof(st) обычно будет 4.

при передаче в функцию объекта по значению создается копия объекта (есть еще пара таких случаев ;-)), для этого вызывается конструктор копирования, если пользователь не определил этот самый конструктор, то его  создает сам компилятор. 
Конструктор копирования по умолчанию (вроде так он называется), просто копирует значения всех членов-данных. 
У тебя в этом случае копируется указатель на строку, а не сама строка, те объект в памяти остается один, а указатель на него копируется, что может привести к неприятным последствиям, если ты обратишься к этому объекту после вызова деструктора хотя бы в одном из объектов. 
Скорее всего не это привело к ошибке в твоей программе, там и без этого интересностей хватает 

Например:

видимо вместо '\t' ты имел ввиду '\0'? Наверное длину строки определял? Это можно сделать функцией strlen(). 

Да, в твоем случае конструктор копирования должен выглядеть примерно так:


Хотя надо еще исправить этот косяк:


на


Еще советую научиться нормально оформлять код (невозможно читать, место экономишь?), и выбирать более осмысленные имена идентификаторам. Ну еще неплохо сразу научить себя разделять объявление класса и реализацию его методов.
   Удачи!

Это сообщение отредактировал(а) zkv - 6.4.2007, 22:47

не забудь поправит деструктор 
 
иначе будет утечка памяти.
  delete s;
освобождает память размером с один символ 

не хочу быть особо въедливым, но это совсем не факт. в большинстве случаев удалится всё-таки вся память, а вот с вызовом деструкторов будут проблемы. а вообще это UB

Daevaorn, это просто UB без каких-либо предположений 

Да, определял я действительно длинну строки. Ток когда я смотрел через debug-watch, в конце строки почему то стояло '\t'

На счет оформления кода - скопировал пост с сайта, где до этого размещал свою задачку.... там сайт автоматом делал вот такое вот выравнивание. Я тут первый раз, и как то не пришло в голову, что тут по другому. Поэтому и прикрепил cpp файл.  

и что такое UB?

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

http://en.wikipedia.org/wiki/Undefined_behavior

И еще один вопросик. Что делает "const"?
Чем отличается "str(char *st)" от "str( const char *st )" ?

оно делает как раз то, что это слово означает  , а именно делает строку(в данном случае) константной

Это больше касается качества кода. Т.е. объявляя функцию с константным указателем или ссылкой в качестве параметра, ты гарантируешь всем, кто ее использует (а так же компилятору, который компилирует твой код), что ты не изменяешь переменную, на которую указывает указатель или ссылка. Другими словами, если ты по ошибке напишешь это:То компилятор тебя пошлет. А если напишешь так:То компилятор не пискнет.

Больше того, у методов классов тоже есть модификатор const, который означает, что данный метод не меняет аттрибуты класса.
По мимо всего прочего, данные модификаторы помогают при написании программы.
Например, ты пишешь класс, у которого есть метод size():
По идее, он не должен менять содержимое класса, а просто должен посчитать занимаемый данными объем памяти. Ты его реализовал, но компилятор выдал кучу ругательств... И ты выясняешь, что где-то в куче кода, банально перепутал источник с приемником (например). Т.е. ты выловил ошибку на этапе компиляции. А отлов отладчиком занимает намного больше времени.

Все супер! все работает!   
остался последний вопрос: про конструктор копирования.



первая строчка "&obStr". все слово является какой то переменной или str - это имя класса? если так, то почему оно с большой буквы? 
Или это указатель на копируемый объект?

obStr - это идентификатор (типа object класса str) 
знак & - указывает, что объект передается по ссылке. По значению объект в конструктор копирования не передать, так как для этого надо вызвать конструктор копирования , поэтому передаем его по ссылке, а модификатор const говорит о том,  что мы обещаем его не менять внутри ;-)

ну так захотелось