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
    13 дек 2012 в 13:24
  • @hash3r: (там комментарии закончились) вот видите, ваш пример только подтверждает то, что я говорю. Объект удалён, но p не nil. Вопрос был не в том, как удалить объект, а о том, как обнулить указатель.
    – VladD
    13 дек 2012 в 15:34
  • 1
    я так поняла, что self = nil; не работает потому, что в self находится копия адреса и его изменение на p никак не повлияет (p, кстати, тоже копия). (stackoverflow.com/questions/6767916/…)
    – Anastasia
    13 дек 2012 в 16:10
  • @VladD: почему не обнуляется не пойму пока. я думал освобождения достаточно) @Anastasia: в releaseAndSetNil передается не копия. зайдите дебаггером в эту функцию и убедитесь, что адреса p и self одинаковы
    – hash3r
    13 дек 2012 в 17:03
  • 1
    @hash3r: да, p и self хранят одинаковый адрес объекта, но &p и &self - разные
    – Anastasia
    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
    13 дек 2012 в 13:45
  • @hash3r: а чем это противоречит моему утверждению? self-то не параметр. И в исходном вопросе надо было убить другой объект, а не самого себя.
    – VladD
    13 дек 2012 в 13:50
  • @hash3r: хорошо, не убить, лишь убить свою ссылку. Кстати, а что будет с вашим кодом, если refcount не 0? Не знаю, как в Objective C, а в C++ self -- константа, так что я не уверен, скомпилируется ли ваш код.
    – VladD
    13 дек 2012 в 13:54
  • @hash3r: а в чем неправда? хочу разобраться.
    – Anastasia
    13 дек 2012 в 14:00
  • 1
    (lldb) po p (MyClass *) $0 = 0x00000001023003a0 <MyClass: 0x1023003a0> (lldb) p [(int)p retainCount] (int) $1 = 1 Не зарелизился и не обнулился...
    – Anastasia
    13 дек 2012 в 14:26

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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