Accelerare o eliminare questa funzione lenta in un'app per fotocamera iPhone

Question

Quando si scattano foto con UIImagePickerController su iPhone, io e molti abitanti di internet abbiamo trovato che è necessario eseguire le immagini attraverso questa funzione per orientarle correttamente (e ridimensionarle mentre ci sei)

C'è un filo rosso gigante su di esso qui: http://discussions.apple.com/message.jspa?messageID=7276709

Purtroppo, questa funzione può prendere secondi per eseguire ed è davvero una seccatura. Se si desidera utilizzare questo e consentire agli utenti di scattare foto velocemente, è anche necessario aggiungere un NSLock o si verificheranno alcuni arresti anomali.

Come posso velocizzare o eliminare questo codice?

// orient photos and scale them to kMaxResolution 
+ (UIImage*)scaleAndRotateImage:(UIImage *)image { 

    // we upload photos at a maximum resolution of 2048 x 1536 
    int kMaxResolution = 2048; 

    CGImageRef imgRef = image.CGImage; 

    CGFloat width = CGImageGetWidth(imgRef); 
    CGFloat height = CGImageGetHeight(imgRef); 

    CGAffineTransform transform = CGAffineTransformIdentity; 
    CGRect bounds = CGRectMake(0, 0, width, height); 
    if (width > kMaxResolution || height > kMaxResolution) { 
     CGFloat ratio = width/height; 
     if (ratio > 1) { 
      bounds.size.width = kMaxResolution; 
      bounds.size.height = bounds.size.width/ratio; 
     } else { 
      bounds.size.height = kMaxResolution; 
      bounds.size.width = bounds.size.height * ratio; 
     } 
    } 

    CGFloat scaleRatio = bounds.size.width/width; 
    CGSize imageSize = CGSizeMake(CGImageGetWidth(imgRef), CGImageGetHeight(imgRef)); 
    CGFloat boundHeight; 
    UIImageOrientation orient = image.imageOrientation; 
    switch(orient) { 

     case UIImageOrientationUp: //EXIF = 1 
      transform = CGAffineTransformIdentity; 
      break; 

     case UIImageOrientationUpMirrored: //EXIF = 2 
      transform = CGAffineTransformMakeTranslation(imageSize.width, 0.0); 
      transform = CGAffineTransformScale(transform, -1.0, 1.0); 
      break; 

     case UIImageOrientationDown: //EXIF = 3 
      transform = CGAffineTransformMakeTranslation(imageSize.width, imageSize.height); 
      transform = CGAffineTransformRotate(transform, M_PI); 
      break; 

     case UIImageOrientationDownMirrored: //EXIF = 4 
      transform = CGAffineTransformMakeTranslation(0.0, imageSize.height); 
      transform = CGAffineTransformScale(transform, 1.0, -1.0); 
      break; 

     case UIImageOrientationLeftMirrored: //EXIF = 5 
      boundHeight = bounds.size.height; 
      bounds.size.height = bounds.size.width; 
      bounds.size.width = boundHeight; 
      transform = CGAffineTransformMakeTranslation(imageSize.height, imageSize.width); 
      transform = CGAffineTransformScale(transform, -1.0, 1.0); 
      transform = CGAffineTransformRotate(transform, 3.0 * M_PI/2.0); 
      break; 

     case UIImageOrientationLeft: //EXIF = 6 
      boundHeight = bounds.size.height; 
      bounds.size.height = bounds.size.width; 
      bounds.size.width = boundHeight; 
      transform = CGAffineTransformMakeTranslation(0.0, imageSize.width); 
      transform = CGAffineTransformRotate(transform, 3.0 * M_PI/2.0); 
      break; 

     case UIImageOrientationRightMirrored: //EXIF = 7 
      boundHeight = bounds.size.height; 
      bounds.size.height = bounds.size.width; 
      bounds.size.width = boundHeight; 
      transform = CGAffineTransformMakeScale(-1.0, 1.0); 
      transform = CGAffineTransformRotate(transform, M_PI/2.0); 
      break; 

     case UIImageOrientationRight: //EXIF = 8 
      boundHeight = bounds.size.height; 
      bounds.size.height = bounds.size.width; 
      bounds.size.width = boundHeight; 
      transform = CGAffineTransformMakeTranslation(imageSize.height, 0.0); 
      transform = CGAffineTransformRotate(transform, M_PI/2.0); 
      break; 

     default: 
      [NSException raise:NSInternalInconsistencyException format:@"Invalid image orientation"]; 

    } 

    UIGraphicsBeginImageContext(bounds.size); 

    CGContextRef context = UIGraphicsGetCurrentContext(); 

    if (orient == UIImageOrientationRight || orient == UIImageOrientationLeft) { 
     CGContextScaleCTM(context, -scaleRatio, scaleRatio); 
     CGContextTranslateCTM(context, -height, 0); 
    } 
    else { 
     CGContextScaleCTM(context, scaleRatio, -scaleRatio); 
     CGContextTranslateCTM(context, 0, -height); 
    } 

    CGContextConcatCTM(context, transform); 

    CGContextDrawImage(UIGraphicsGetCurrentContext(), CGRectMake(0, 0, width, height), imgRef); 
    UIImage *imageCopy = UIGraphicsGetImageFromCurrentImageContext(); 
    UIGraphicsEndImageContext(); 

    return imageCopy; 
} 

Answer 1

Avete davvero bisogno di fare questa operazione all'istante? La risoluzione dello schermo dell'iPhone è 320x480 (e raddoppia quella per retina) - ridimensiona l'immagine almeno fino a questo, quindi applica le trasformazioni (almeno per la visualizzazione, quindi più tardi se l'utente vuole fare qualcosa con l'immagine, corri la versione più costosa in background). Date un'occhiata a quella prima linea:

// we upload photos at a maximum resolution of 2048 x 1536 
int kMaxResolution = 2048; 

Per la maggior parte delle applicazioni, si sta ruotando questa immagine gigantesca, procedendo poi a buttare via la maggior parte di questi dati calcolato quando si restringerlo fino in fondo. Buttare via i dati prima ridimensionandoli (afaik, un'operazione molto più economica), quindi ruotare.

Answer 2

Mi chiedo perché pensi che sia necessario ridimensionare e ruotare un'immagine proveniente da UIImagePicker. Ho scritto un paio di app diverse che usano la fotocamera e non ho mai avuto bisogno di farlo. Se spieghi più in dettaglio cosa stai cercando di fare, forse possiamo darti suggerimenti più specifici.