Cú pháp/định dạng khi lồng khối mục tiêu-01

Question

Tôi đang lồng khối và trông có vẻ UGGGGLY. Có cách nào để viết điều này xấu xí hơn không? Chủ yếu là tìm kiếm các đề xuất cú pháp, thay vì cấu trúc, nhưng tôi sẽ chấp nhận.

My phương pháp nhà máy khối,

-(NSImage *(^)(CGFloat size, BOOL preview))resizeBlock { 

return (NSImage *(^)(CGFloat size, BOOL preview))[[^(CGFloat size, BOOL preview){ 
     // image-resizing code 
     return [[[NSImage alloc] init] autorelease]; 
    } copy] autorelease]; 

} 

nào được gọi từ một số chức năng tương tự như sau,

-(void)queueResize:(CGFloat)targetSize toView:(NSImageView *)targetView { 
    NSImage*(^sizeBlock)(CGFloat,BOOL) = [self resizeBlock]; 
    NSBlockOperation *bo = [NSBlockOperation blockOperationWithBlock:^(void) { 
     NSImage *previewImage = (NSImage*)sizeBlock(targetSize,YES); 
     targetView.image = previewImage; 
    }]; 
    [queue addOperation:bo]; 
} 

hàng đợi là một đối tượng NSOperationQueue. Nó sẽ không biên dịch mà không có tất cả (xấu xí xấu xí) đúc. Amidoinitrite?

Edit: Như mỗi câu trả lời Dave DeLong, và http://www.cimgf.com/2008/02/16/cocoa-tutorial-nsoperation-and-nsoperationqueue/, tôi đã thay đổi dòng

targetView.image = previewImage; 

được,

[targetView performSelectorOnMainThread:@selector(setImage:) withObject:previewImage waitUntilDone:YES]; 

Answer 1

Sử dụng typedef:

typedef NSImage *(^KWResizerBlock)(CGFloat size, BOOL preview); 

Điều này làm cho bạn mã trở thành:

- (KWResizerBlock) resizeBlock { 
    KWResizerBlock block = ^(CGFloat size, BOOL preview){ 
    // image-resizing code 
    return [[[NSImage alloc] init] autorelease]; 
    }; 
    return [[block copy] autorelease]; 
} 

-(void)queueResize:(CGFloat)targetSize toView:(NSImageView *)targetView { 
    KWResizerBlock sizeBlock = [self resizeBlock]; 
    NSBlockOperation *bo = [NSBlockOperation blockOperationWithBlock:^{ 
    NSImage *previewImage = sizeBlock(targetSize, YES); 
    //do something with previewImage 
    }]; 
    [queue addOperation:bo]; 
} 

Một lời cảnh cáo:

bạn NSBlockOperation sẽ được thực hiện trên một sợi đó không phải là chủ đề chính, và vì vậy nó là không an toàn để thao tác bất kỳ yếu tố giao diện người dùng từ bên trong bối cảnh đó. Nếu bạn cần đặt số previewImage vào giao diện người dùng, thì bạn nên dispatch_async() quay lại luồng chính (hoặc chức năng tương đương).

Nó có thể hoạt động ngay bây giờ, nhưng nó là mạnh không khuyến khích và có thể dẫn đến hành vi không xác định.