Это хорошая или плохая практика?
if (!theConnection && !receivedData) { // release the connection, and the data object [theConnection release]; // receivedData is declared as a method instance elsewhere [receivedData release]; }
Отправка любого сообщения на ноль объекта не имеет никакого эффекта, поэтому вы можете безопасно удалить эту проверку.
Кроме того, ваш код теряет память, если только 1 объект не ноль.
Это плохая практика - просто посмотрите образец кода Apple, чтобы понять, как он должен.
Используйте [theConnection release];
[theConnection release];
или [theConnection release]; theConnection = nil;
[theConnection release]; theConnection = nil;
ито же самое для receivedData.
receivedData