Swift3 pageViewControllerでボタンでの遷移方法

Question

Swift3にて、pageViewControllerでの遷移でスワイプでの遷移に加えて、ボタンでの遷移ができずに苦戦しております。教えて頂ければ助かります。

スワイプでの遷移はできているのですが、pageViewControllerとは別のviewController（PageContentViewController)にボタンを配置し、ボタン押下でも次のページに遷移できるようにしたいです。
宜しくお願い致します。

＜やってみたこと＞
PageContentViewController内でのボタンイベントでsetViewControllersを呼び出そうとしましたが、[fatal error: Index out of range] のようなエラーがでました。

■ サンプルコード Swift3　PageViewController

class PageViewController: UIPageViewController, UIPageViewControllerDataSource {

    var data: [String] = []

    var selectedIndex = 0

    var contentVCs = [UIViewController]()

    override func viewDidLoad() {
        super.viewDidLoad()

        dataSource = self

        for index in 0..<20 {
            let contentVC = storyboard?.instantiateViewController(withIdentifier: "PageContentViewController") as! PageContentViewController
            contentVC.labelStr = index.description   // index確認用
            contentVC.pageIndex = index
            contentVC.data = data[index]
            contentVCs.append(contentVC)
        }

        setViewControllers([contentVCs[selectedIndex]], direction: .forward, animated: true, completion: nil)

    }  // viewDidLoad()を閉じる


    func pageViewController(_ pageViewController: UIPageViewController, viewControllerBefore viewController: UIViewController) -> UIViewController? {
        guard let index = contentVCs.index(of: viewController as! PageContentViewController), index > 0 else {
            return nil
        }
        let previousIndex = index - 1
        return contentVCs[previousIndex]
    }


    func pageViewController(_ pageViewController: UIPageViewController, viewControllerAfter viewController: UIViewController) -> UIViewController? {
        guard let index = contentVCs.index(of: viewController as! PageContentViewController), index < contentVCs.count - 1 else {
            return nil
        }
        let nextVC = contentVCs[index + 1]
        return nextVC
    }

}   // class()を閉じる

■ サンプルコード Swift3　PageContentViewController

class PageContentViewController: UIViewController, UIScrollViewDelegate {

    var pageIndex:Int = 0   
    var data:String?

    var labelStr: String?
    let indexPageLabel = UILabel()
    var contentVCs = [UIViewController]()

    override func viewDidLoad() {
        super.viewDidLoad()

        （略）

        // indexPageラベル
        indexPageLabel.frame = CGRect(x: 150, y: 10, width: 100, height: 30)
        indexPageLabel.text = "(labelStr!)"
        scrollView.addSubview(indexPageLabel)
     
        // Nextへボタン
         nextButton.frame = CGRect(x:305, y:680, width: 50, height: 30)
         nextButton.setTitle("Next＞", for: .normal)
         nextButton.titleLabel?.font = UIFont.boldSystemFont(ofSize: 14)
         nextButton.setTitleColor(UIColor.white, for: .normal)
         nextButton.backgroundColor = UIColor.green
         nextButton.layer.cornerRadius = 15.0
         nextButton.layer.borderWidth = 2.0
         nextButton.layer.borderColor = UIColor.green.cgColor
         nextButton.addTarget(self, action: #selector(buttonEventNext(sender:)), for: .touchUpInside)
         scrollView.addSubview(nextButton)


        // CSVファイルを読み込む
        let csvArray = loadCSV("test")

        //csvArrayの行をタップされた番号で取得
        dayArray = csvArray[pageIndex].components(separatedBy: ";")

        （略）
    }  // viewDidLoad()を閉じる

    func buttonEventNext(sender: AnyObject) {
        let pageViewController = self.parent as! PageViewController
            let nextIndex = pageIndex + 1
            pageViewController.setViewControllers([contentVCs[nextIndex]], direction: .forward, animated: true, completion: nil)

}   // class()を閉じる

Comments

[nakasho_dev]

エラーが出ましたとありますが、どのようなエラーかを書いた方が良いと考えます。

[torkia]

追記ありがとうございます。エラー内容を本文に追加致しました。

Answer 1

エラー文が何か分かりませんがbuttonEventNextに@objcをつけたら動きませんか。

【追記】
エラー文から配列に含まれている要素数よりも多いIndexを選択していると思われます。
PageContentViewControllerのcontentVCsは空のようです。
以下のようにしてpageViewControllerのcontentVCsを該当箇所で使う必要があります。

pageViewController.setViewControllers([pageViewController.contentVCs[nextIndex]], direction: .forward, animated: true, completion: nil)

また現在のコードでは、pageIndexの数値もゼロのままなので何らかの方法で渡す必要があります。

一般的にはこのような選択はせずPageViewControllerでページ遷移のためのDelegateを実装して、各子要素にDIするほうがコードがきれいになります。

Comments

[torkia]

ご回答ありがとうございます。
> @objcをつけたら動きませんか。
をつけてみましたが、同じエラー[fatal error: Index out of range]がでました。
pageViewController自体を呼び出せていないということでしょうか。

[torkia]

ご回答ありがとうございます。
【追記】のコードで遷移できるようになりました。
配列を呼ぶときもpageViewControllerをつけるべきだったんですね。
丁寧な説明ありがとうございました。

> Delegateを実装して、各子要素にDIするほうがコードがきれいになります。
DelegateやDIなどはまだ使えてませんが、今後勉強していきたいと思います。