ボタンを押すとすぐにインジゲーターが出るようにし、その間に別の処理を行いたい。

Question

前提・実現したいこと

Navigationbarの右側に配置したボタンを押すことでPDFを作成できるようにしているのですが、その処理が重いためかボタンを押すとしばらく動かせなくなります。

そこでインジゲーターを表示させるようにしたいのですが、そのようにコーディングしてもPDF作成の処理が先に施され、インジゲーターの表示が後になってしまいます。

ボタンはstoryboardで以下のように配置しています。

ボタンを押すとインジゲーター表示、PDF作成、画面遷移を行いたいと思っています。

ボタンをひとつにするとインジゲーターは表示されるようになったのですが、PDF作成や画面遷移をしなくなりました。

Navigationbarのところに配置したボタンがゆえに処理が上手くいってないのかなと思うのですが、、、ご教授よろしくお願いいたします。

該当のソースコード

インジゲーター表示させっぱなしのコード

class ViewController2: UIViewController, UIGestureRecognizerDelegate,UIImagePickerControllerDelegate,UINavigationControllerDelegate {
      var ActivityIndicator: UIActivityIndicatorView!
    override func viewDidLoad() {
        super.viewDidLoad()
        
        
        // ActivityIndicatorを作成＆中央に配置
        ActivityIndicator = UIActivityIndicatorView()
        ActivityIndicator.frame = CGRect(x: 0, y: 0, width: 50, height: 50)
        ActivityIndicator.center = self.view.center
        
        // クルクルをストップした時に非表示する
        ActivityIndicator.hidesWhenStopped = true
        
        // 色を設定
        ActivityIndicator.activityIndicatorViewStyle = UIActivityIndicatorViewStyle.gray
        
        //Viewに追加
        self.view.addSubview(ActivityIndicator)
        
    }

 @IBAction func push(_ sender: Any) {
 print("push")
 
    // クルクルスタート
    ActivityIndicator.startAnimating()
    
    
 
        let x = self.getimage.frame.origin.x
        let y = self.getimage.frame.origin.y
 UserDefaults.standard.set(y, forKey: "uekaranokyori")
        let imageHeight = self.screenWidth * 210.0 / 143.5
 
 
 ///////////////////////////////////////////////////////////VIEWのため変更
 
 
 
 //VIEW.frame = pdfImageView.bounds
 
 let pdfData = NSMutableData()
 
 //binsenの画像
 let binhoge = UserDefaults.standard.integer(forKey: "selectedTag")
 let bin: UIImage = UIImage(named:"binsen(binhoge)-1.jpg")!
 //元画像の大きさのUIImageViewを作る
 let pdfImageView = UIImageView(image: bin)
 
 
 //どんな大きさ、位置のものを表示するのか
 //UIGraphicsBeginImageContextWithOptions(getimage.frame.size, false, 0.0)
        let  gamen:CGRect = CGRect(x: 0, y: 0, width: self.screenWidth, height: imageHeight)
 UIGraphicsBeginPDFContextToData(pdfData, pdfImageView.bounds, nil)
 print("gamen = (gamen)")
 ////////CGRect(x: 0, y: 0, width: screenWidth, height: imageHeight)

    //print("pdfContex1 = (pdfContext1)")
 UIGraphicsBeginPDFPage()
 
//サイズをbinの元々のサイズにする
    let pdfContext1 = UIGraphicsGetCurrentContext() ////
        pdfContext1?.scaleBy(x: pdfImageView.bounds.width/self.self.screenWidth, y: pdfImageView.bounds.height/imageHeight)////
        UserDefaults.standard.set(pdfImageView.bounds.width/self.screenWidth, forKey: "X")
    UserDefaults.standard.set(pdfImageView.bounds.height/imageHeight, forKey: "Y")
    print("X=(pdfImageView.bounds.width)")
    print("Y=(pdfImageView.bounds.height)")
 
 guard let pdfContext = UIGraphicsGetCurrentContext() else { return }
 //VIEW.layer.render(in: pdfContext)

 bin.draw(in: gamen)
        self.draw(imageView: self.getphoto)
    
    //image?.draw(in: CGRect(x:,y:,width:,height:))
 //CGRect(x: 0, y: 0, width: pdfImageView.bounds.width, height: pdfImageView.bounds.height)
 
 
 
 UIGraphicsEndPDFContext()
 
 
 
 if let documentDirectories = NSSearchPathForDirectoriesInDomains(.documentDirectory, .userDomainMask, true).first {
 let documentsFileName = documentDirectories + "/test.pdf"
 //パスを保存して次の画面で表示させる
 UserDefaults.standard.set(documentsFileName, forKey:"filePath")
 debugPrint(documentsFileName)
 pdfData.write(toFile: documentsFileName, atomically: true)
 print("確認")
 }

    
        
        
    

        
        
   
 //let getimagerectback:CGRect = CGRect(x:x, y:y, width:screenWidth, height:imageHeight)
 //getimage.frame = getimagerectback;
 
 ///////////////////////////////////////////////////////////VIEWのため変更
 //VIEW.frame = imagerect;
 
 }
    
    /// カレントコンテキストに対してUIImageViewが持つimageをUIImageViewの姿勢のまま描画します。
    private func draw(imageView: UIImageView) {
        guard let image = imageView.image else { return }
        guard let ctx = UIGraphicsGetCurrentContext() else { return }
        let orgTransform = imageView.transform
        imageView.transform = .identity
        let orgFrame = imageView.frame
        imageView.transform = orgTransform
        
        ctx.saveGState()
        ctx.translateBy(x: +orgFrame.midX, y: +orgFrame.midY)
        ctx.concatenate(orgTransform)
        ctx.translateBy(x: -orgFrame.midX, y: -orgFrame.midY)
        
//binsenの画像
        let binhoge = UserDefaults.standard.integer(forKey: "selectedTag")
        let bin: UIImage = UIImage(named:"binsen(binhoge)-1.jpg")!
//元画像の大きさのUIImageViewを作る
        let pdfImageView = UIImageView(image: bin)
        screenWidth = self.view.bounds.width
let imageHeight = screenWidth * 210.0 / 143.5
        
        
        image.draw(in: orgFrame.aspectFit(contentSize: image.size, stretchble: true, integer: false))
        //image.size
        
        ctx.restoreGState()
    }

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

Swiftのversionは4.1.2
Xcodeのversionは9.4.1

Answer 1

PDF作成処理をメインスレッドで行っているので、その間UIは更新されません。
PDF作成処理を別スレッドで行って下さい。

「ボタンをひとつにすると〜」以降は意味（状況）がいまいち分かりません。急に遷移の話が出てきて？？？です。

Comments

[mimamo]

ご回答ありがとうございます。
説明不足ですみません、ボタンを押すとインジゲーター表示、PDF作成、画面遷移を行いたいと思っています。

別のスレッドで処理する方法を調べてコーディングしてみました。

質問本文に載せた、ボタンを押すとされる処理をpdsSakuseiという関数にしました。
そして
@IBAction func push(_ sender: Any) {
// クルクルスタート
ActivityIndicator.startAnimating()
// キューを生成してサブスレッドで実行
DispatchQueue(label: "jp.classmethod.app.queue").async {
self.pdfSakusei()
}
}
としました。

次の画面で作成したPDFを表示するようになっているのですが、ボタンを１回押すと空のPDFが作成され、ボタンを２回押すと１回目で作成されるはずのPDFが作成されました。

インジゲーターも表示されないので、このコードだと別スレッドで処理することになっていないみたいなのですが、他にどのようにコーディングすべきがわからず手が止まってしました、、ご教授よろしくお願いします。

[fuzzball]

一気に全部解決しようとするのではなく、一つ一つ解決していってはどうでしょうか？
・インジケータの表示
・別スレッドで実行
・PDF作成（これは出来てるんですよね）

今のプロジェクトでやるのではなく、一つ一つ別プロジェクトを作って検証しましょう。
現状、何が問題になっているのか、切り分けができていないと思います。

例えば、今までにインジケータを一度でも表示させたことはあるのでしょうか？
無いのであれば、ボタンでON/OFFするようなものを作ってみて下さい。

[fuzzball]

あー、インジケータは表示したことあるって書いてましたね。失礼。
ということは「別スレッドで実行」ですね。

何か調べるときは一つの記事ではなく、複数の記事を参照して下さい。（なるべく新しいの）
あと、self.pdfSakusei() で試すのではなく、例えば、

print("start")
Thread.sleep(forTimeInterval: 5) //5秒間停止
print("finish")

のような疑似処理で作るのがいいと思います。
（これで5秒間クルクルが表示されれば成功）