将更改的图像保存到相机胶卷时由于内存问题而终止 [英] Terminated due to memory issue when saving altered image to camera roll

查看:27
本文介绍了将更改的图像保存到相机胶卷时由于内存问题而终止的处理方法,对大家解决问题具有一定的参考价值,需要的朋友们下面随着小编来一起学习吧!

问题描述

我正在构建一个允许用户为照片添加时间戳的应用.选择来自相机卷的照片,我的应用有时崩溃,Xcode显示错误:来自调试器的消息:由于内存问题而终止.我发现如果我尝试将时间戳添加到非常大的图像,或者如果我重复在同一图像上添加时间戳,它会崩溃.

I am building an app that allows users to add a timestamp to a photo. When a photo from the camera roll is selected my app sometimes crashes and Xcode displays the error: "Message from debugger: Terminated due to memory issue". I found it crashes if I try to add a timestamp to a very large image or if I repeatedly at a timestamp to the same image.

以下是我认为与该问题相关的代码.另外,这里是整个项目的链接.

Below is the code I believe is relevant to the issue. Also, here is a link to the entire project.

这是将时间戳添加到图像的方式.我把评论放在它崩溃的地方.它并不总是在同一行代码上崩溃.我不明白为什么我的代码会因内存问题错误而崩溃.

This is how the timestamp gets added to the image. I put comments where it crashes. It does not always crash on the same line of code. I can't figure out why my code is making it crash with the memory issue error.

  func textToImage(drawText text: NSString, inImage image: UIImage, atPoint point: CGPoint) -> UIImage {

    let color = UserDefaults.standard.colorForKey(key: "color")!
    let font = UserDefaults.standard.value(forKey: "font") as! String
    let size = UserDefaults.standard.value(forKey: "size") as! Int
    let fontAndSize = UIFont(name: font, size: CGFloat(size))!
    let location = Locations(rawValue: UserDefaults.standard.value(forKey: "location") as! String)!
    let scale = UIScreen.main.scale


    UIGraphicsBeginImageContextWithOptions(image.size, false, scale)
    // sometimes terminates here

    let textFontAttributes = [
      NSFontAttributeName: fontAndSize,
      NSForegroundColorAttributeName: color
      ] as [String : Any]
    image.draw(in: CGRect(origin: CGPoint.zero, size: image.size))

    let rectSize = text.boundingRect(with: CGSize(width:CGFloat(MAXFLOAT), height: CGFloat(MAXFLOAT)), options: NSStringDrawingOptions.usesLineFragmentOrigin, attributes: textFontAttributes, context: nil).size
    // sometimes terminates here
    if location == .topRight || location == .bottomRight || location == .center || location == .topCenter || location == .bottomCenter {
      // Calculate the text size

      if location == .center || location == .topCenter || location == .bottomCenter {
        // Subtract half the text width from the x origin
        let rect = CGRect(x:point.x-(rectSize.width / 2), y:point.y-(rectSize.height / 2), width:rectSize.width, height: rectSize.height)
        text.draw(in: rect, withAttributes: textFontAttributes)
      } else {
        // Subtract the text width from the x origin
        let rect = CGRect(x:point.x-rectSize.width, y:point.y-(rectSize.height / 2), width:rectSize.width, height: rectSize.height)
        text.draw(in: rect, withAttributes: textFontAttributes)
      }
    } else {
      let rect = CGRect(x:point.x, y:point.y-(rectSize.height / 2), width:rectSize.width, height: rectSize.height)
      text.draw(in: rect, withAttributes: textFontAttributes)
    }

    let newImage = UIGraphicsGetImageFromCurrentImageContext()
    UIGraphicsEndImageContext()

    return newImage!
  }

  func selectPhotoFromCameraRoll(mediaType: String) {
    imagePicker.sourceType = .photoLibrary
    if mediaType == "Video" {
      imagePicker.mediaTypes = [kUTTypeMovie as String]
    }
    newMedia = false
    present(imagePicker, animated: true, completion: nil)
  }

  func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [String : Any]) {
    if let mediaType = info[UIImagePickerControllerMediaType] as? String {
      if mediaType.isEqual((kUTTypeImage as String)) {
        if let pickedImage = info[UIImagePickerControllerOriginalImage] as? UIImage {
          let width = pickedImage.size.width
          let height = pickedImage.size.height
          let location = Locations(rawValue: UserDefaults.standard.value(forKey: "location") as! String)!
          var point = CGPoint(x: 0, y: 0)
          switch location {
          case .topLeft:
            point = CGPoint(x: 30, y: 50)
          case .topRight:
            point = CGPoint(x: width - 30, y: 50)
          case .bottomLeft:
            point = CGPoint(x: 30, y: height - 50)
          case .bottomRight:
            point = CGPoint(x: width - 30, y: height - 50)
          case .center:
            point = CGPoint(x: width / 2, y: height / 2)
          case .topCenter:
            point = CGPoint(x: width / 2, y: 50)
          case .bottomCenter:
            point = CGPoint(x: width / 2, y: height - 50)
          }
          let savedFormat = UserDefaults.standard.value(forKey: "format") as! String
          var date = Date()
          if !currentDateBool {
            date = UserDefaults.standard.value(forKey: "selectedDate") as! Date
          }
          let timestampText = getFormattedDateFromFormatType(formatType: savedFormat, date: date) as NSString
          let timestampImage = textToImage(drawText: timestampText, inImage: pickedImage, atPoint: point)
          if newMedia {
            UIImageWriteToSavedPhotosAlbum(timestampImage, self, #selector(image(_:didFinishSavingWithError:contextInfo:)), nil)
          } else {
            UIImageWriteToSavedPhotosAlbum(timestampImage, self, #selector(image(_:didFinishSavingWithError:contextInfo:)), nil)
          }
        }
      } else if mediaType.isEqual((kUTTypeMovie as String)) {
        if let videoUrl = info[UIImagePickerControllerMediaURL] as? NSURL {
          if let videoPath = videoUrl.relativePath {
            if newMedia {
              UISaveVideoAtPathToSavedPhotosAlbum(videoPath, nil, nil, nil)
            }
          }
        }
      }
    }
    dismiss(animated: true, completion: nil)
  }


  func image(_ image: UIImage, didFinishSavingWithError error: NSError?, contextInfo: UnsafeRawPointer) {
    if let error = error {
      // we got back an error!
      let ac = UIAlertController(title: "Save error", message: error.localizedDescription, preferredStyle: .alert)
      ac.addAction(UIAlertAction(title: "OK", style: .default))
      present(ac, animated: true)
    } else {
      let ac = UIAlertController(title: "Saved!", message: "Your altered image has been saved to your photos.", preferredStyle: .alert)
      ac.addAction(UIAlertAction(title: "OK", style: .default))
      present(ac, animated: true)
    }
  }

推荐答案

您将相机胶卷中的照片视为 UIImage.不是.

You are treating a photo in the camera roll as if it were a UIImage. It isn't.

让我们假设一个最坏的情况.假设我们有一个 12MP 的相机.那么您派生的 UIImage 将是 4032x3024 像素.现在您要求创建一个图形上下文缩放屏幕分辨率.假设此设备的屏幕具有 3 倍分辨率.然后您要求图形上下文 12096x9072,即 109734912.这就是 .要存储颜色数据,您必须将其乘以每个点的颜色信息占用的空间.我不知道那是多少,但我们肯定会再上升一个数量级.无论如何,无论您如何切片,这都是您要求的大量内存.

Let's take a worst-case scenario. Suppose we have a 12MP camera. Then your derived UIImage would be 4032x3024 pixels. Now you are asking to make a graphics context with scaling for the screen resolution. Suppose the screen of this device has 3x resolution. Then you are asking for a graphics context 12096x9072, which is 109734912. That's points. To store the color data, you have to multiply that by how much space the color info takes for each point. I don't know how much that is, but we're certainly going to go up another order of magnitude. In any case, no matter how you slice it, that is a boatload of memory you're asking for.

这篇关于将更改的图像保存到相机胶卷时由于内存问题而终止的文章就介绍到这了,希望我们推荐的答案对大家有所帮助,也希望大家多多支持IT屋!

查看全文
登录 关闭
扫码关注1秒登录
发送“验证码”获取 | 15天全站免登陆