Что не так с этим деллоком в ARC?
Я работаю над приложением, которое выполняет обработку изображений и отображает полученное изображение. Я использую UIScrollView чтобы позволить пользователю прокручивать все изображения, потому что изображение не является стандартным jpg или png, для его загрузки требуется время. Я использую GCD для асинхронной загрузки, когда закончена отправка в основную очередь для отображения. фрагмент выглядит следующим образом:
- (void)loadImage:(NSString *)name
{
dispatch_async(dispatch_get_global_queue(DISPATCH_QUEUE_PRIORITY_DEFAULT, 0), ^{
UIImage *image = [Reader loadImage:name];
dispatch_sync(dispatch_get_main_queue(), ^{
[self displayImage:image];
});
});
}
loadImage Метод Reader выглядит так:
+ (UIImage *)loadImage:(NSString *)name
{
UInt8 *data = NULL;
NSString *mfjPath = [TMP stringByAppendingPathComponent:name];
NSData *mfjData = [NSData dataWithContentsOfFile:mfjPath];
if(mfjData){
data = malloc(sizeof(UInt8)*mfjData.length);
[mfjData getBytes:data];
}
if(data){
ResultHolder *result = [sDecoder decodeData:data withOffset:0];// static id<IDecoder> sDecoder; in Reader.m before @implementation Reader.
return [result bitmap];
}
retun nil;
}
IDCoder это протокол, который
@protocol IDecoder <NSObject>
- (ResultHolder *)decodeData:(UInt8 *) withOffset:(int)offset;
@end
ResultHolder класс для загрузки простого изображения и объединения сложного изображения. что следующим образом:
ResultHolder.h
typedef struct color24{
UInt8 R;
UInt8 G;
UInt8 B;
} Color24;
@interface ResultHolder : NSObject
{
unsigned long mWidth;
unsigned long mHeight;
UInt8 *mData;
CGImageRef mBitmap;
BOOL isMonoColor;
Color24 mMonoColor;
}
+ (ResultHolder *)resultHolderWithCGImage:(CGImageRef)image;
+ (ResultHolder *)resultHolderWithData:(UInt8 *)data Width:(unsigned long)width andHeight:(unsigned long)height;
+ (ResultHolder *)resultHolderWithMonoColor:(Color24)monoColor withWidth:(unsigned long)width andHeight:(unsigned long)height;
- (ResultHolder *)initWithData:(UInt8 *)data Width:(unsigned long)width andHeight:(unsigned long) height;
- (ResultHolder *)initWithCGImage:(CGImageRef)image;
- (ResultHolder *)initWithMonoColor:(Color24)monoColor withWidth:(unsigned long)width andHeight:(unsigned long)height;
- (BOOL)isSuccess;
- (UIImage *)bitmap;
- (void)combineFixResultHolder:(ResultHolder *)child Rect:(CGRect)bounds Width:(unsigned long)width andHeight:(unsigned long)height;
- (void)combineResultHolder:(ResultHolder *)child Bounds:(CGRect)bounds Width:(unsigned long)width andHeight:(unsigned long)height;
@end
ResultHolder.m
@implementation ResultHolder
@synthesize width = mWidth;
@synthesize height = mHeight;
@synthesize isMonoColor;
@synthesize monoColor = mMonoColor;
- (ResultHolder *)initWithData:(UInt8 *)data Width:(unsigned long)width andHeight:(unsigned long)height
{
if (self = [super init]) {
mWidth = width;
mHeight = height;
mData = malloc(mWidth*mHeight*sizeof(Color24));
memcpy(mData, data, mWidth*mHeight*sizeof(Color24));
mBitmap = NULL;
}
return self;
}
- (ResultHolder *)initWithCGImage:(CGImageRef)image
{
if (self = [super init]) {
mBitmap = CGImageRetain(image);
mWidth = CGImageGetWidth(image);
mHeight = CGImageGetHeight(image);
}
return self;
}
- (ResultHolder *)initWithMonoColor:(Color24)monoColor withWidth:(unsigned long)width andHeight:(unsigned long)height
{
if (self = [super init]) {
mMonoColor = monoColor;
isMonoColor = YES;
mWidth = width;
mHeight = height;
mBitmap = NULL;
mData = NULL;
}
return self;
}
+ (ResultHolder *)resultHolderWithCGImage:(CGImageRef)image
{
ResultHolder *resultHolder = [[ResultHolder alloc] initWithCGImage:image];
return resultHolder;
}
+ (ResultHolder *)resultHolderWithData:(UInt8 *)data Width:(unsigned long)width andHeight:(unsigned long)height
{
ResultHolder *resultHolder = [[ResultHolder alloc] initWithData:data Width:width andHeight:height];
return resultHolder;
}
+ (ResultHolder *)resultHolderWithMonoColor:(Color24)monoColor withWidth:(unsigned long)width andHeight:(unsigned long)height
{
ResultHolder *resultHolder = [[ResultHolder alloc] initWithMonoColor:monoColor withWidth:width andHeight:height];
return resultHolder;
}
- (BOOL)isSuccess
{
if ([ReaderConfigures CodecDebug])
NSLog(@"ResultHolder isSuccess");
return (mData != NULL || isMonoColor || mBitmap != nil);
}
- (void)fillMonoColor
{
if (isMonoColor) {
if (mData) {
free(mData);
}
mData = (UInt8 *)malloc(mWidth*mHeight*sizeof(Color24));
for (int i = 0; i < mHeight; i++) {
for (int j = 0; j < mWidth; j++) {
memcpy(mData+(i*mWidth+j)*3, &mMonoColor, sizeof(Color24));
}
}
isMonoColor = NO;
}
}
- (void)extractBitmap
{
if (mBitmap) {
CGDataProviderRef dataProvider = CGImageGetDataProvider(mBitmap);
CFDataRef bitmapData = CGDataProviderCopyData(dataProvider);
UInt8 * dataSource = (UInt8 *)CFDataGetBytePtr(bitmapData);
size_t width = CGImageGetWidth(mBitmap);
size_t height = CGImageGetHeight(mBitmap);
if(mData)
free(mData);
mData = malloc(width*height*3);
for (int i = 0; i < height; i++) {
for (int j = 0; j < width; j++) {
memcpy(mData+(i*width+j)*3, dataSource+(i*width+j)*4, sizeof(Color24));
}
}
CFRelease(bitmapData);
CGImageRelease(mBitmap);
mBitmap = NULL;
}
}
- (UInt8 *)getRawData
{
if (mBitmap) {
[self extractBitmap];
}
if (isMonoColor) {
[self fillMonoColor];
}
return mData;
}
- (UIImage *)bitmap
{
if (mBitmap) {
UIImage *image = [[UIImage alloc] initWithCGImage:mBitmap];
CGImageRelease(mBitmap);
mBitmap = NULL;
return image;
}
if (isMonoColor) {
[self fillMonoColor];
}
if (mData) {
CGDataProviderRef dataProvider = CGDataProviderCreateWithData(NULL, mData, mWidth*mHeight*3, NULL);
CGColorSpaceRef colorSpace = CGColorSpaceCreateDeviceRGB();
CGImageRef bitmap = CGImageCreate(mWidth, mHeight, 8, 24, mWidth*3, colorSpace, kCGBitmapByteOrderDefault, dataProvider, NULL, YES, kCGRenderingIntentDefault);
CGColorSpaceRelease(colorSpace);
CGDataProviderRelease(dataProvider);
UIImage *image = [[UIImage alloc] initWithCGImage:bitmap];
CGImageRelease(bitmap);
return image;
}
return nil;
}
- (void)combineResultHolder:(ResultHolder *) child Bounds:(CGRect) bounds Width:(unsigned long)width andHeight:(unsigned long)height
{
CGRect rect = CGRectMake(MAX(0, bounds.origin.x), MAX(0, bounds.origin.y),MIN(width - 1, bounds.origin.x + bounds.size.width), MIN(height - 1, bounds.origin.y + bounds.size.height));
int w = MIN(rect.size.width + 1, child.width);
int h = MIN(rect.size.height + 1, child.height);
int dstPos = (height - 1 - (rect.origin.y + h - 1))*width;
UInt8 *dataParent = [self getRawData];
if (child.isMonoColor) {
Color24 childMonoColor = child.monoColor;
for (int i = 0; i < h; i++) {
memcpy(dataParent+(dstPos+(int)rect.origin.x)*3, &childMonoColor, w*3);
dstPos += width;
}
} else {
UInt8 *dataChild = [child getRawData];
if (dataChild != nil) {
int srcPos = 0;
for (int i = 0; i < h; i++) {
memcpy(dataParent+dstPos*3+((int)rect.origin.x)*3, dataChild+srcPos*3, w*3);
srcPos += child.width;
dstPos += width;
}
}
}
}
- (void)combineFixResultHolder:(ResultHolder *)child Rect:(CGRect)bounds Width:(unsigned long)width andHeight:(unsigned long)height
{
CGRect rect = CGRectMake(bounds.origin.x, height-1-bounds.origin.y-bounds.size.height, bounds.origin.x+bounds.size.width, height-1-bounds.origin.y);
[self combineResultHolder:child Bounds:rect Width:width andHeight:height];
}
- (void)dealloc
{
if (mData) {
free(mData);
mData = NULL;
}
if (mBitmap) {
CGImageRelease(mBitmap);
mBitmap = NULL;
}
}
@end
для простого изображения, например, только для изображения JPEG, + (ResultHolder *)resultHolderWithCGImage:(CGImageRef)image; а также - (UIImage *)bitmap; методы называются. для некоторых сложных, ResultHolder будет извлекать mBitmap в mData, а затем объединить с суб resultHolder"s mData чтобы получить изображение. эти методы хорошо работают, если я загружаю изображение в основной поток, но если я использую GCD или NSThread загрузить изображение в фоновом режиме легко рухнет, особенно при загрузке сложных в фоновом режиме. когда приложение падает, основной поток сообщает об ошибке метода CGSConvertBGR888toRGBA8888, один из других потоков выполняет [ResultHolder dealloc] метод, на самом деле free(mData), Кажется, существует конфликт памяти между потоком загрузки и основным потоком.
когда приложение вылетает, ошибка выглядит так:
Я боролся за эту ошибку в течение нескольких дней, но до сих пор не могу найти, как ее исправить. Я надеюсь, что кто-то может мне помочь. Любые предложения приветствуются.
ОБНОВЛЕНИЕ: я делаю демонстрационный проект ReaderDemo для имитации ситуации. Если вы заинтересованы, вы можете скачать, чтобы увидеть ошибку. В этом проекте 15 изображений, 5,7,14 изображения вызовут сбой при прокрутке, они немного сложнее, чем другие. но если вы прокрутите миниатюру прокрутки, а затем нажмите, все они могут быть отображены.
2 ответа
У вас есть ряд проблем, но давайте начнем с первого, что я нашел:
Неправильный тест
if (index > [mPageNames count]) {Это должно быть>= или вы потерпите крах.
вы вызываете dispatch_sync в mainQueue, это не кажется хорошим решением (но, возможно, у вас действительно хорошее решение) - я изменил его на async, похоже, работает нормально
Если вы включите исключения в этом проекте, это действительно поможет вам. Нажмите кнопку точек останова на панели инструментов Xcode. Затем выберите опцию Break Points слева, вторую справа. Коснитесь нижнего левого значка "+" и добавьте точку останова "Все исключения". Теперь при запуске отладчик останавливается там, где возникает проблема.
Я получил последний сбой, который я позволю вам исправить:
2012-09-26 08:55:12.378 ReaderDemo[787:11303] MFJAtIndex index out of bounds,index:15,bounds:15 2012-09-26 08:55:12.379 ReaderDemo[787:11303] *** Assertion failure in -[ImageScrollView showLoadingForMFJ:], /Volumes/Data/Users/dhoerl/Downloads/ReaderDemo/ReaderDemo/ImageScrollView.m:247
Это должно помочь вам.
РЕДАКТИРОВАТЬ: Ваша проблема связана с управлением mData объем памяти. Вы пытаетесь управлять его жизненным циклом в своем коде, но это управление не синхронизировано с CGImageDataProvider это пытается использовать это. Авария почти наверняка (то есть я на 99,99% убежден) является побочным продуктом CGImageProvided создано CGDataProviderCreateWithData пытаясь получить доступ к данным после того, как ваш класс освободил эту память в dealloc. У меня был подобный опыт с поставщиками данных.
Правильное решение состоит в том, чтобы удалить все free(data) звонки, или, по крайней мере, большинство из них. Учитывая текущую структуру вашего кода, вам нужно будет тщательно обдумать это - вы можете заменить все тесты и malloc / frees на флаг. В конце, что вы хотите сделать, это как только указатель памяти передается ovdr CGDataProviderCreateWithData, вам нужно NULL из mData и пусть провайдер данных обработает удаление.
Способ сделать это - предоставить указатель на функцию CGDataProviderCreateWithData в прошлом параметре:
CGDataProviderReleaseDataCallback
A callback function that releases data you supply to the function CGDataProviderCreateWithData.
typedef void (*CGDataProviderReleaseDataCallback) (
void *info,
const void *data,
size_t size
);
Все, что нужно сделать функции, это просто вызвать free(data);, Поэтому, когда поставщик данных выполняет выделенную память, он освобождает ее (и вам не нужно об этом беспокоиться).
Если вы хотите освободить () или освободить свои ресурсы в любом классе в среде с поддержкой ARC, вы должны установить соответствующие флаги для своего класса в "Фазы сборки". Для этого выберите файл проекта в XCode, выберите цель, перейдите в раздел "Build Phases", найдите свой класс и вставьте -fno-objc-arc флаг для этого класса. Или, возможно, по другой причине, вы вызываете какую-то функцию CoreGraphics, которая должна вызываться из основного потока только в другом потоке.