UIPageControlの最初のページを設定できない

Question

前提・実現したいこと

SwiftでiPhoneの写真アプリを作成しており、現在写真一覧から写真を選択した時にその写真を単体で拡大表示し、横スワイプで前後の写真に切り替えられる機能を実装しようとしております。（一般的な写真アルバムでみられるような写真を選択して開いたのちに、左右にスワイプすることで前後の写真に切り替えられる機能です）

発生している問題・エラーメッセージ

写真が格納されている配列から選択した写真をUIPageControlの開いた時に表示されるページに格納したい（10個格納されている配列の5番目の写真であれば、10ページ中５ページ目が開かれてほしい）のですが、必ず配列の一番最初の写真が表示されてしまいます。（5番目の写真を選んでも、1枚目が表示される10ページ中1ページ目が開かれてしまいます）
UIPageControlのcurrentPageに設定されたページが開かれると認識していますが、間違っていますでしょうか？
もし間違っていましたら、他にそのような挙動にする方法がありましたら、ご教示いただけますと大変助かります。
どうぞよろしくお願いいたします。

該当のソースコード

storyboard上は、ScrollView - View - PageControlの構成になっております。

　　 @IBOutlet weak var scrollView: UIScrollView!
    @IBOutlet weak var pageControl: UIPageControl!
    
    var photoUrlDatas = [FilmData]()//前のページから写真全てのファイルパスが格納されております
    var numberOfImages = Int()//photoUrlDatasの配列の要素の数が入っています
    var currentPage = Int()　//前のページから開いた写真が配列の何番目かを渡しています

    override func viewDidLoad() {
    super.viewDidLoad()
    setupPageControl()
    }
    

    override func viewDidLayoutSubviews() {
    super.viewDidLayoutSubviews()
    setupScrollView()
    (0..<numberOfImages).forEach { page in
        let subScrollView = generateSubScrollView(at: page)
        scrollView.addSubview(subScrollView)
        let imageView = generateImageView(at: page)
        subScrollView.addSubview(imageView)
    }

    private func setupPageControl() {
        pageControl.numberOfPages = numberOfImages
        pageControl.currentPage = currentPage
        pageControl.addTarget(
            self,
            action: #selector(didValueChangePageControl),
            for: .valueChanged
        )
    }
    
    private func setupScrollView() {
        scrollView.delegate = self
        scrollView.isPagingEnabled = true
        scrollView.showsVerticalScrollIndicator = false
        scrollView.showsHorizontalScrollIndicator = false
        scrollView.contentSize = CGSize(
            width: calculateX(at: numberOfImages),
            height: scrollView.bounds.height
        )
    }
    
    private func generateImageView(at page: Int) -> UIImageView {
        let frame = scrollView.bounds
        let imageView = UIImageView(frame: frame)
        
        imageView.contentMode = .scaleAspectFit
        imageView.clipsToBounds = true
        
        guard photoUrlDatas.indices.contains(page)else{return imageView}
        let storageRef = Storage.storage().reference(forURL: "gs://snappin-57194.appspot.com/")
        let photoImageRef = storageRef.child(photoUrlDatas[page].filePath ?? "no filepath")
        imageView.sd_setImage(with: photoImageRef)
        
        return imageView
       }
    }

    ・・・

Comments

[tomato879241]

numberOfImagesは数字を設定しなければ、0です。どの行でそれを0以外に設定しているのでしょうか？

[keroto]

こちら分かりづらくて大変申し訳ございません。
numberOfImagesは前のページからphotoUrlDatasを渡す際に同時にphotoUrlDatasの要素数を渡しております。
ページ数自体は想定通りになっているため、numberOfImages自体は問題なく機能してると思われます。

[tomato879241]

「前のページからphotoUrlDatas」ということはsetupPageControl()を読む場所が違うんではないですか？前のページから変数を現在のview controllerに渡すのであれば、それはviewDidLoadの時点では渡されません。viewDidLoadとviewWillAppearの違いを知っていますか？

[keroto]

不勉強で恐縮ですが、ライフサイクルについては、値渡しのタイミングまではきちんと把握していませんでした。
printによってphotoUrlDatas、numberOfImages、currentPageを確認した際に、viewDidLoadのsetupPageControl()の時点で値が渡されていることが確認できていたので問題ないと思っておりました。
ご教示いただいた通りひとまずviewWillAppear以降のライフサイクルのタイミングでsetupPageControl()を読み込んでみましたが、相変わらず開かれる際のページは1ページ目のまま変わりませんでした...

[hoshi-takanori]

scrollView と pageControl は自動的には連動しないと思いますので、連動させるコードを書く必要があるはず。特に、didValueChangePageControl はユーザーが pageControl を操作した場合しか呼ばれないのでは。

[keroto]

おっしゃる通りscrollViewをpageControlと連動させるようにコードを記載した結果想定の動きになりました。ありがとうございました！

Answer 1

scrollView.contentOffset = CGPoint(x: Int(UIScreen.main.bounds.width) * currentPage, y: 0)

こちらのコードをviewDidLoadに記載し、pageControlとscrollViewを連動させることで、想定の動きを実現できました。