UIImageViewを、カメラロールから取得した画像のアスペクト比の形にしたい

Question

前提・実現したいこと

すでにUIImageViewを設置しています。
カメラロールから取得した画像をUIImageView(aspect fit)に表示されるようにしているのですが、空白の余白が残るのを避けたいです。
そのため、取得した画像のアスペクト比をUIImageViewにも反映させられたらなと思っています。
ですがなかなかうまくいきません。ご教授よろしくお願いいたします。

理想のイメージはこのようになります。

写真とUIImageViewのアスペクト比が一致しています。

ご教授よろしくお願いいたします。

該当のソースコード

    @IBOutlet weak var getphoto: UIImageView!

   //カメラロールから写真を選択する処理
    @IBAction func choosePicture() {
        // カメラロールが利用可能か？
        if UIImagePickerController.isSourceTypeAvailable(.photoLibrary) {
            // 写真を選ぶビュー
            let pickerView = UIImagePickerController()
            // 写真の選択元をカメラロールにする
            // 「.camera」にすればカメラを起動できる
            pickerView.sourceType = .photoLibrary
            // デリゲート
            pickerView.delegate = self
            // ビューに表示
            self.present(pickerView, animated: true)
        }
    }
    
    
    // 写真を選んだ後に呼ばれる処理
    func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [String : Any]) {
        // 選択した写真を取得する
        let image = info[UIImagePickerControllerOriginalImage] as! UIImage


        //photoの縦横サイズを取得
        pickerphotowidth = image.size.width
        pickerphotoheight = image.size.height
        
        //横幅をスクリーン画面の1/4、アスペクト比維持のRect作成
        let photorect:CGRect = CGRect(x: 0, y: 0, width: screenWidth/4, height: (screenWidth*photoheight)/photowidth*4)
        
        getphoto.frame = photorect;
        getphoto.center = CGPoint(x:screenWidth/2, y:screenHeight/2)
 

        
        // ビューに表示する
        self.getphoto.image = image
        // 写真を選ぶビューを引っ込める
        self.dismiss(animated: true)
    }

###頂いたコードを参考に記述したもの

 extension CGRect {
        func aspectFit(contentSize: CGSize, stretchble: Bool, integer: Bool) -> CGRect {
            let xZoom = width / contentSize.width
            let yZoom = height / contentSize.height
            let zoom = stretchble ? min(xZoom, yZoom) : min(xZoom, yZoom, 1)
            
            if integer {
                let newWidth = max(Int(contentSize.width * zoom), 1)
                let newHeight = max(Int(contentSize.height * zoom), 1)
                let newX = Int(origin.x) + (Int(width) - newWidth) / 2
                let newY = Int(origin.y) + (Int(height) - newHeight) / 2
                return CGRect(x: newX, y: newY, width: newWidth, height: newHeight)
            } else {
                let newWidth = contentSize.width * zoom
                let newHeight = contentSize.height * zoom
                let newX = origin.x + (width - newWidth) / 2
                let newY = origin.y + (height - newHeight) / 2
                return CGRect(x: newX, y: newY, width: newWidth, height: newHeight)
            }
        }
    }
    
    
    
    // 写真を選んだ後に呼ばれる処理
    func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [String : Any]) {
        // 選択した写真を取得する
        let image = info[UIImagePickerControllerOriginalImage] as! UIImage
        
        //透過していた画像を取り戻す
        getphoto.alpha = 1.0
        

        
        
        
        
        getphoto.frame = CGRect(x: 0, y: 0, width: screenWidth / 4, height: screenHeight / 4).aspectFit(contentSize: image.size, stretchble: true, integer: true)
        getphoto.center = CGPoint(x:screenWidth/2, y:screenHeight/2)
 
        
        // ビューに表示する
        self.getphoto.image = image
        // 写真を選ぶビューを引っ込める
        self.dismiss(animated: true)
    }

補足情報（FW/ツールのバージョンなど）

Swiftのversionは4.1.2
Xcodeのversionは9.4.1

Answer 1

横幅をスクリーン画面の1/4と決めてしまうと、極端な話ですが細く縦長い画像を読み込んだ場合に画面から画像がはみ出すことになりますのでおすすめしません（＝高さ制限がない）。

自分の場合はある矩形にaspectFitしたときの座標を算出する関数を作って、
その矩形を使うようにしています。
これならば細長の画像でもはみ出すことはありません。

extension CGRect {
    func aspectFit(contentSize: CGSize, stretchble: Bool, integer: Bool) -> CGRect {
        let xZoom = width / contentSize.width
        let yZoom = height / contentSize.height
        let zoom = stretchble ? min(xZoom, yZoom) : min(xZoom, yZoom, 1)
        
        if integer {
            let newWidth = max(Int(contentSize.width * zoom), 1)
            let newHeight = max(Int(contentSize.height * zoom), 1)
            let newX = Int(origin.x) + (Int(width) - newWidth) / 2
            let newY = Int(origin.y) + (Int(height) - newHeight) / 2
            return CGRect(x: newX, y: newY, width: newWidth, height: newHeight)
        } else {
            let newWidth = contentSize.width * zoom
            let newHeight = contentSize.height * zoom
            let newX = origin.x + (width - newWidth) / 2
            let newY = origin.y + (height - newHeight) / 2
            return CGRect(x: newX, y: newY, width: newWidth, height: newHeight)
        }
    }
}

使い方の例：

getphoto.frame = CGRect(x: 0, y: 0, width: screenWidth / 4, height: screenHeight / 4).aspectFit(contentSize: image.size, stretchble: true, integer: true)

※多少の誤差はでます。

Comments

[mimamo]

ありがとうございます！
無知ですみません、お伺いしたいことがあります。
extension CGRect{ ... } のwidthやorigin.xなどについてですが、これは事前にどこかで宣言しておく必要があるということでしょうか。
また、この場合にwidthはgetphotoの横幅のことでしょうか。

[takabosoft]

上記のソースコードを新規ファイルでもいいですし、どこでもいいのでまるごと貼り付けてください。
extension CGRect {} とあるように、CGRectのメンバ関数を拡張（＝追加）した感じです。
中で使われているwidth（=self.width）はCGRect自身のwidthを指します。
これらはすでにCGRectが持っている変数ですので、「どこかで宣言しておく」必要はありません。

[mimamo]

ありがとうございます。
貼り付けると　extension CGRect {} の範囲で
１、Declaration is only valid at file scope
２、Use of unresolved identifier 'width'
３、Use of unresolved identifier 'height'
４、Did you mean 'right'?
などといろいろ忠告が出てしまいます。

[takabosoft]

extension CGRect {} はクラスの外（グローバル）に置いてますか？すみません、どこでもいいというのは語弊がありました。

[mimamo]

なるほど、無事できました！！
ありがとうございます！