Всем привет!

Код и ниже 2 вопрома по нему



Все работает до вопроса N2

1) Почему не работает просто new pperson , а приходится приводить тип
2) На delete  pp программа виснет. Что не так?

Всем спасибо...............


Модератор: не забываем пользоваться кнопочкой "Код"

Это сообщение отредактировал(а) bsa - 24.1.2012, 17:15

потому-что надо создавать объект а не указатель.



проблема в том же.

данная конструкция выделяет память под указатель (4 байта на 32-х битной ОС и 8 - на 64-х битной), а не под структуру, и возвращает указатель на указатель на структуру, а не указатель на структуру
сделай так, и будет тебе счастье 

и можешь спокойно делать


и пользуйся кнопочкой код

Все понял, спасибо, здОрово.

Добавлено через 9 минут и 23 секунды
Осталось, все-таки непонятным, почему в "правильном" и "неправильном" корректно отработалпа функция changePerson2. ?
С указателем и с указателем на указатель в качестве параметра..

jeff, это тебе кажется, что "корректно". Результат ошибки может проявиться значительно позже.
Никогда не делай "typedef something *psomething". Значительно проще писать каждый раз звездочку для создания указателя. Да и ошибок будет меньше. Например: "const something *x" и "const psomething x" - это разные вещи!!!

А с чего ты взял, что она отрабатывала корректно?

Добавлено через 38 секунд

И совершенно в другом месте  

Я неопытный.

 

и




дают одинаковый(и предсказуемый) результат работы changePerson2 

Можно ли считать что при pp = pperson(new pperson); выполняется как бы инициализация массива person* и в функцию передается первый(и единственный)  элемент массива, и поэтому changePerson2 работает?

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

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

Это сообщение отредактировал(а) azesmcar - 25.1.2012, 08:53

jeff, посмотри внимательно этот код
http://liveworkspace.org/code/6c737cc1d8e5...635a191a48ca81f
и представь, что arr описан как int arr[ 1 ];
в твоём случае происходит примерно то же самое.
ты выделил память под указатель, т.е. 4 байта (на 32-х разрядной ОС), а считаешь, что выделил память под структуру, т.е. вторая запись ( p->height = 100; ) ведётся по неразрешённой (не выделенной для этих целей) памяти.
Если есть желание поэкпериментировать и убедиться в правильности этого

то измени структуру след. образом


Добавлено через 1 минуту и 25 секунд
и оставь свой первоначальный код без изменений. результат должен быть примерно как сказал azesmcar

jeff, смотри, типичный размер твоей структуры 8 байт. Типичный размер указателя - 4 байта. оператор new выделяет под указатель 4 байта памяти и возвращает адрес начала выделенной области памяти. ты с ними работаешь. Потом ты решил еще выделить память. Так как менеджер памяти считает, что занято только 4 байта, то он выделяет сразу за ними. В итоге, меняя данные в новой выделенной области ты портишь в старой... Но это при такой простой работе менеджера памяти. А теперь представим, что менеджер памяти выделяет какую-то область, в начало ее пишет какие-то служебные данные, а возвращает уже адрес за ними. Таким образом, после второго выделения ты можешь потереть служебные данные. И ошибка возникнет при попытке освобождения памяти или еще одного выделения...

Спасибо всем ответившим, все разъяснили, все понял.  С++ коварен, однако

  
Есть простое правило: никогда не используй приведение типов, за исключением:
1) Без этого вообще никак не обойтись
2) Ты на 0x64% знашь, что ты делаешь и как это будет работать