2

У меня возникла необходимость перенести release и обнуление объекта в отдельный метод. Но после вызова этого метода указатель не становится nil. Вот код, описывающий проблему:

- (void)releaseAndSetNil:(MyClass *)inP
{
    [inP release];
    inP = nil;
}

- (void)mainFunction
{
    // this code works
    MyClass *p = [[MyClass alloc] init];
    [p release];
    p = nil; // (MyClass *) $0 = 0x0000000000000000 <nil>

    // this code does not work
    MyClass *p = [[MyClass alloc] init];
    [self releaseAndSetNil:p]; // (MyClass *) $1 = 0x0000000100108340 [no Objective-C description available]
}

Что я делаю не так?

6
  • +1 за минимальный пример, воспроизводящий проблему
    – VladD
    Commented 13 дек. 2012 в 13:24
  • @hash3r: (там комментарии закончились) вот видите, ваш пример только подтверждает то, что я говорю. Объект удалён, но p не nil. Вопрос был не в том, как удалить объект, а о том, как обнулить указатель.
    – VladD
    Commented 13 дек. 2012 в 15:34
  • 1
    я так поняла, что self = nil; не работает потому, что в self находится копия адреса и его изменение на p никак не повлияет (p, кстати, тоже копия). (stackoverflow.com/questions/6767916/…)
    – Anastasia
    Commented 13 дек. 2012 в 16:10
  • @VladD: почему не обнуляется не пойму пока. я думал освобождения достаточно) @Anastasia: в releaseAndSetNil передается не копия. зайдите дебаггером в эту функцию и убедитесь, что адреса p и self одинаковы
    – hash3r
    Commented 13 дек. 2012 в 17:03
  • 1
    @hash3r: да, p и self хранят одинаковый адрес объекта, но &p и &self - разные
    – Anastasia
    Commented 13 дек. 2012 в 17:47

2 ответа 2

3

Хм. В Objective C, как и в чистом C, параметры передаются по значению. То есть, в процедуре releaseAndSetNil вы обнуляете копию вашего p.

Передать сам p вы не можете, поэтому передавайте его адрес. Имея адрес, вы сможете сделать с переменной что угодно.

Чтобы правильно работало, передавайте указатель на указатель:

- (void)releaseAndSetNil:(MyClass **)inPP
{
    [*inPP release];
    *inPP = nil;
}

(если не наврал с синтаксисом).

2

Хм. В Objective C, как и в чистом C, параметры передаются по значению

Вам кот-то сказал не правду.

// метод класса MyClass

 - (void)releaseAndSetNil
    {
        [self release];
        self = nil;
    }

- (void)mainFunction
{

    // this code  works
    MyClass *p = [[MyClass alloc] init];
    [p releaseAndSetNil];
}

UPD (lldb) p [(int)p retainCount] никогда не покажет Вам нуль, т. к. когда у объекта 0 ссылок на него он вызывает dealloc

UPD2 если все еще не верите, вот более убедительные доказательства) Релиз объекта не обязательно происходит мгновенно. Когда он происходит это уже его детали реализации.

Если запустить такой код

MyClass *p = [[MyClass alloc] init];
    [p releaseAndSetNil];
    [p retain];

то получим *** -[MyClass retain]: message sent to deallocated instance что есть доступ к деаллоцированной памяти. это свидетельствует об релизе объекта р в releaseAndSetNil:

10
  • почему не получается комментировать чужие записи? кармы не хватае что ли)?
    – hash3r
    Commented 13 дек. 2012 в 13:45
  • @hash3r: а чем это противоречит моему утверждению? self-то не параметр. И в исходном вопросе надо было убить другой объект, а не самого себя.
    – VladD
    Commented 13 дек. 2012 в 13:50
  • @hash3r: хорошо, не убить, лишь убить свою ссылку. Кстати, а что будет с вашим кодом, если refcount не 0? Не знаю, как в Objective C, а в C++ self -- константа, так что я не уверен, скомпилируется ли ваш код.
    – VladD
    Commented 13 дек. 2012 в 13:54
  • @hash3r: а в чем неправда? хочу разобраться.
    – Anastasia
    Commented 13 дек. 2012 в 14:00
  • 1
    (lldb) po p (MyClass *) $0 = 0x00000001023003a0 <MyClass: 0x1023003a0> (lldb) p [(int)p retainCount] (int) $1 = 1 Не зарелизился и не обнулился...
    – Anastasia
    Commented 13 дек. 2012 в 14:26

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.