delete указателя

@norge_goth · Регистрация: 27.01.2009

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

valueOfWordS - указатель, член класса String_value, конструктору же этого класса передается указатель на переменную того же типа:

C++

String_value::String_value(int* value_pointer)
{
    valueOfWordS = 0;
    valueOfWordS = value_pointer;
    delete value_pointer;
}

Вопрос, почему когда освобождается память

C++

1	delete value_pointer;

вне конструктора например в ф-циях можно всеравно обращаться к указателю valueOfWordS, ведь память должна была затереться так как valueOfWordS = value_pointer; используют одну и ту же область памяти?

@~~NightmareZ~~ · 28.08.2010, 13:59

Тебе не кажется, что передавать int через указатель немного странно?

Не по теме:

Я так полагаю, что передаётся именно указатель на одно число, а не на массив, ибо в том случае глупо бы было делать delete вместо delete[].

@fasked · 28.08.2010, 14:05

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

вне конструктора например в ф-циях можно всеравно обращаться к указателю valueOfWordS, ведь память должна была затереться так как valueOfWordS = value_pointer; используют одну и ту же область памяти?

Куда же память то денется. Максимум, что затрется, так это значения по этому адресу в памяти. Запрещено обращаться только к нулевым указателям.

@norge_goth · 28.08.2010, 14:05 **[ТС]**

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

Тебе не кажется, что передавать int через указатель немного странно?

Не по теме:

Я так полагаю, что передаётся именно указатель на одно число, а не на массив, ибо в том случае глупо бы было делать delete вместо delete[].

там странного ничего нет, объект создается внутри другого объекта, а именно при нажатии кнопки "Run"(это гуи приложение), а указатель нужен для того чтобы при некоторых условиях узменить значение переменной из первого класса(который вызывает констуктор String_value - класса)

@~~NightmareZ~~ · 28.08.2010, 14:07

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

там странного ничего нет, объект создается внутри другого объекта, а именно при нажатии кнопки "Run"(это гуи приложение), а указатель нужен для того чтобы при некоторых условиях узменить значение переменной из первого класса(который вызывает констуктор String_value - класса)

Ну тогда это нарушение инкапсуляции.

@norge_goth · 28.08.2010, 14:10 **[ТС]**

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

Куда же память то денется. Максимум, что затрется, так это значения по этому адресу в памяти. Запрещено обращаться только к нулевым указателям.

так дело и в том *valueOfWordS значением можно свободно пользоваться в классе после конструктора

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

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

Ну тогда это нарушение инкапсуляции.

Может и так, но это осознанная жертва, хотя если посидеть это можно исправить
но дело в другом, дело в сути вопроса

@~~NightmareZ~~ · 28.08.2010, 14:14

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

Может и так, но это осознанная жертва, хотя если посидеть это можно исправить
но дело в другом, дело в сути вопроса

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

@fasked · 28.08.2010, 14:17

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

так дело и в том *valueOfWordS значением можно свободно пользоваться в классе после конструктора

Смотрите, как я понял у Вас примерно такой случай:

C++

int main()
{
    int *p1 = new int;
    int *p2 = p1;
 
    *p1 = 5;
 
    std::cout << "P1: " << *p1 << std::endl
              << "P2: " << *p2 << std::endl;
 
    delete p1;
 
    std::cout << "P1: " << *p1 << std::endl
              << "P2: " << *p2 << std::endl;
}

Результат такой:

Код

fasked@linux-5kgr:~/Documents> g++ test.cpp
fasked@linux-5kgr:~/Documents> ./a.out
P1: 5
P2: 5
P1: 0
P2: 0
fasked@linux-5kgr:~/Documents>

Но! Значение в памяти не обязательно затрется! Там может остаться мусор, это нормально. В любом случае пользоваться дальше указателем p2 неправильно! Это может привести к трудноуловимым ошибкам.

@norge_goth · 28.08.2010, 14:39 **[ТС]**

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

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

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

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

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

Но! Значение в памяти не обязательно затрется! Там может остаться мусор, это нормально. В любом случае пользоваться дальше указателем p2 неправильно! Это может привести к трудноуловимым ошибкам.

даже так работает

C++

String_value::String_value(int* value_pointer)
{
    delete value_pointer;
    valueOfWordS = 0;
    valueOfWordS = value_pointer;
    std::cout << *valueOfWordS << endl;
 
    std::cout << "!!!!" << *value_pointer << endl;
}

в конструктор с внешнего класса передается адрес переменной со значением 3, в конструкторе делаю вывод *valueOfWordS и *value_pointer после delete и тоже получаю 3!

@fasked · 28.08.2010, 14:56

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

даже так работает

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

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

@norge_goth · 28.08.2010, 15:02 **[ТС]**

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

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

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

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

Спасибо

@fasked · 28.08.2010, 15:06

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

наверно даже ф-цию напишу для автоматизма

Это уже перебор по-моему.

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

или delete перегружу

Для int'a?

@norge_goth · 28.08.2010, 15:13 **[ТС]**

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

Это уже перебор по-моему.

Для int'a?

не тут конечно уже не буду так извращаться

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

@norge_goth 62 / 62 / 13 Регистрация: 27.01.2009 Сообщений: 279
	28.08.2010, 15:02 [ТС]	11
	Сообщение от fasked Вы читать умеете? Я же сказал, что в памяти может остаться некоторое значение, оно не обязательно будет затерто. При переносе кода на другую платформу все может измениться и тогда Ваш код рухнет при первой подобной операции. Хорошей привычкой считается после операций освобождения памяти обнулять указатели все теперь понятно, после обнуления указателя программа рухнула, будем теперь знать что платформа тоже вносит свои коррективы, теперь буду постоянно их обнулять, наверно даже ф-цию напишу для автоматизма или delete перегружу Спасибо 0

@fasked 5043 / 2622 / 241 Регистрация: 07.10.2009 Сообщений: 4,310 Записей в блоге: 1
	28.08.2010, 15:06	12
	Сообщение от norge_goth наверно даже ф-цию напишу для автоматизма Это уже перебор по-моему. Сообщение от norge_goth или delete перегружу Для int'a? 0

@~~NightmareZ~~ Заблокирован
	28.08.2010, 13:59	2
	Тебе не кажется, что передавать int через указатель немного странно? Не по теме: Я так полагаю, что передаётся именно указатель на одно число, а не на массив, ибо в том случае глупо бы было делать delete вместо delete[]. 0

@fasked 5043 / 2622 / 241 Регистрация: 07.10.2009 Сообщений: 4,310 Записей в блоге: 1
	28.08.2010, 14:05	3
	Сообщение от norge_goth вне конструктора например в ф-циях можно всеравно обращаться к указателю valueOfWordS, ведь память должна была затереться так как valueOfWordS = value_pointer; используют одну и ту же область памяти? Куда же память то денется. Максимум, что затрется, так это значения по этому адресу в памяти. Запрещено обращаться только к нулевым указателям. 0

@norge_goth 62 / 62 / 13 Регистрация: 27.01.2009 Сообщений: 279
	28.08.2010, 14:05 [ТС]	4
	Сообщение от NightmareZ Тебе не кажется, что передавать int через указатель немного странно? Не по теме: Я так полагаю, что передаётся именно указатель на одно число, а не на массив, ибо в том случае глупо бы было делать delete вместо delete[]. там странного ничего нет, объект создается внутри другого объекта, а именно при нажатии кнопки "Run"(это гуи приложение), а указатель нужен для того чтобы при некоторых условиях узменить значение переменной из первого класса(который вызывает констуктор String_value - класса) 0

@~~NightmareZ~~ Заблокирован
	28.08.2010, 14:07	5
	Сообщение от norge_goth там странного ничего нет, объект создается внутри другого объекта, а именно при нажатии кнопки "Run"(это гуи приложение), а указатель нужен для того чтобы при некоторых условиях узменить значение переменной из первого класса(который вызывает констуктор String_value - класса) Ну тогда это нарушение инкапсуляции. 0

@norge_goth 62 / 62 / 13 Регистрация: 27.01.2009 Сообщений: 279
	28.08.2010, 14:10 [ТС]	6
	Сообщение от fasked Куда же память то денется. Максимум, что затрется, так это значения по этому адресу в памяти. Запрещено обращаться только к нулевым указателям. так дело и в том valueOfWordS значением можно свободно пользоваться в классе после конструктора Добавлено через 2 минуты* Сообщение от NightmareZ Ну тогда это нарушение инкапсуляции. Может и так, но это осознанная жертва, хотя если посидеть это можно исправить но дело в другом, дело в сути вопроса 0

@~~NightmareZ~~ Заблокирован
	28.08.2010, 14:14	7
	Сообщение от norge_goth Может и так, но это осознанная жертва, хотя если посидеть это можно исправить но дело в другом, дело в сути вопроса Да нет, как раз таки в этом дело. Ты сначала открываешь доступ к переменным класса неизвестно кому, потом этот неизвестно кто (в данном случае объект класса String_value) удаляет переменную, на которую указывает value_pointer, хотя по логике владеть временем её жизни должен класс, которому она пренадлежит.... А потом удивляешься: чего это оно криво работает? Да потому что архитектура отвратная. Сейчас ты пофиксишь тут, а отвалится в другом месте. 0

@fasked 5043 / 2622 / 241 Регистрация: 07.10.2009 Сообщений: 4,310 Записей в блоге: 1
	28.08.2010, 14:56	10
	Сообщение от norge_goth даже так работает Вы читать умеете? Я же сказал, что в памяти может остаться некоторое значение, оно не обязательно будет затерто. При переносе кода на другую платформу все может измениться и тогда Ваш код рухнет при первой подобной операции. Хорошей привычкой считается после операций освобождения памяти обнулять указатели 0

@norge_goth 62 / 62 / 13 Регистрация: 27.01.2009 Сообщений: 279
	28.08.2010, 15:13 [ТС]	13
	Сообщение от fasked Это уже перебор по-моему. Для int'a? не тут конечно уже не буду так извращаться , тут обойдемся простым обнулением эт когда много надо будет чего удалить, хотя я встречал как некоторые делают свои ф-ции для освобождения и помещают их в заголовочный файл, а потом где надо просто подключают его 0