Утечка памяти с использованием (аннулируются) Alloc

Question

Я видел подобную строку кода носятся в яблоках код:

(void)[[URLRequest alloc] initializeRequestWithValues:postBody url:verifySession httpHeader:nil delegate:self]; 

URLRequest мой собственный пользовательский класс. Я не писал этого, и я думаю, что парень, который просто схватил его с примера Apple. Для меня это должно протекать, и когда я его тестирую, я уверен, что он протекает через 16 байт. Не так ли? Я знаю, как это исправить, если это так, но не было уверен, как это было взято из кода Apple.

EDIT: Проблема была с SDK, а не с указанным выше кодом. См. Ответ ниже для получения дополнительной информации.

Answer 1

Думал, что я мог бы это обновить, поскольку после дальнейших испытаний и выпуска iOS4 он изменился.

Приведенный выше код не течет, а объем памяти приложения возвращается в нормальное состояние даже после 200 итераций кода. Утечка произошла в iOS3, но была очень маленькой, в iOS4 она полностью исчезла как в симуляторе, так и в устройстве.

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

Answer 2

Совершенно не уверен, что этот код должен выполнить. Кажется, что он нарушает каждое соглашение о методах инициализации. Какой смысл возвращать указатель void из метода инициализации? Вся цель метода инициализации - вернуть объект. Где вы видели это в примерах кода Apple?

Сказав это, я не понимаю, почему это просочится. Поскольку он не возвращает объект, нет ничего, чтобы утечка внешнего метода. Возможно, что-то внутренне течет.

Edit:

It basically does an NSURLConnection. Because we are submitting a lot of forms with a lot of different values we put it in an external class. All the delegate methods like didFailWithError: are in NSURLRequest and connectionDidFinishLoading just passes the data to its delegate. So it doesn't really need to return anything as it is done through a delegate method.

Да, вам нужно переделать это. В настоящее время этот метод является просто катастрофой, ожидающей своего появления. Если ничего другого, все остальные, смотрящие на этот код, будут совершенно смущены тем, что вы делаете.

Если у вас нет необходимости сохранять созданный объект, затем перемещайте его выделение и полностью очищайте внутри метода. Измените префикс имени метода с «initialize» на что-то вроде «setup», «configure», «purchase» и т. Д., Чтобы имя не подразумевало, что оно создает и возвращает и объект.

Если вам нужен экземпляр с одним выстрелом для определенного класса, используйте метод класса, например, предложенный Майклом Аароном Сафяном (опять же без инициализации имени). Метод класса должен инициализировать экземпляр внутри, выполнить необходимые операции, вернуть данные в любом месте, а затем исключить экземпляр.

Таким образом, вам не придется беспокоиться о утечках, и все, кто может читать ваш код (включая вас, месяцы в пути), сразу поймут, что делает код.

Answer 3

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

 
[[[URLRequest alloc] initializeRequestWithValues:postBody url:verifySession httpHeader:nil delegate:self] autorelease]; 

Может быть, лучше исправить было бы создать функцию класса, что делает это:

 
@interface URLRequest 
{ 
    // ... 
} 
// ... 
+ (void) requestWithValues:/* ... */ 
// ... 
@end 

Тогда вы могли бы просто использовать [ URLRequest requestWithValues:/* ... * /] без вызова alloc.