質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.60%

遷移先の画面に選択したセル(画像)が表示されない。

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 1,470

RyousukeTanaka

score 12

 前提・実現したいこと

セルの画面を選択して、画面遷移をしようとしています。同時に、選択したセルの画像を受け渡しを行いたいです。

・今できていること
・セルを作り、画像は表示されている。
・セルをタップした際に、画面遷移自体はsegueを付けてできている。

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

画面遷移はするが、選択したセル(画像)が何も表示されない。
(問題は、セルをタップした時のアクションの記述のところだろうと一応見当をつけています。)

 該当のソースコード

遷移元

import UIKit

class ViewController: UIViewController,UICollectionViewDataSource {

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



    @IBAction func backToTop(segue: UIStoryboardSegue) {}

    func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        return 30
    }

    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
     let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "testcell", for: indexPath) as! MyCollectionViewCell
        // セルのインデックスを取得
        let columnIndex = indexPath.row
        // 画像名を作成
        let imageName = "image" + String(columnIndex + 1)
        // MyCollectionViewCellのimageに画像を設定
        cell.cellimage.image = UIImage(named: imageName)

return cell

    }
    var selectedImage: UIImage?

    func collectionView(_ collectionView: UICollectionView,
                        didSelectItemAt indexPath: IndexPath) {


        let columnIndex = indexPath.row
        let imageName = "image" + String(columnIndex + 1)
        selectedImage = UIImage(named:imageName)
        if selectedImage != nil {
            // SubViewController へ遷移するために Segue を呼び出す
            performSegue(withIdentifier: "toDetailViewController",sender: nil)
        }
    }
    // Segue 準備
    override func prepare(for segue: UIStoryboardSegue, sender: Any!) {
        if (segue.identifier == "toDetailViewController") {
            let subVC: DetailViewController = (segue.destination as? DetailViewController)!
            // SubViewController のselectedImgに選択された画像を設定する
            subVC.testImage = selectedImage
        }
    }



    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
    }



}


遷移先

import UIKit

class DetailViewController: UIViewController {


    @IBOutlet weak var recievesideview: UIImageView!
var testImage:UIImage!
    override func viewDidLoad() {
        super.viewDidLoad()
        recievesideview.image=testImage
    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }


    /*
    // MARK: - Navigation

    // In a storyboard-based application, you will often want to do a little preparation before navigation
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        // Get the new view controller using segue.destinationViewController.
        // Pass the selected object to the new view controller.
    }
    */

}

 試したこと

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

始めたばかりで質問自体も的確でないかもしれないので、もし回答するにあたって必要な情報があればすぐに追記します。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • 516k

    2018/04/08 22:47

    全体的に理にかなったコードだと思いました。なのでどちらかというとDetailViewControllerのrecievesideview周りの気がします。 override func viewDidLoad() { super.viewDidLoad() recievesideview.image=testImage } ここにたとえば前から引き継いだtestImageではなく let imageName = "image" + "1" // recievesideview.image = UIImage(named: imageName) 等でハードコードして正しく画像は表示されますかね?

    キャンセル

  • RyousukeTanaka

    2018/04/08 23:06 編集

    コードをおっしゃる通りに編集しました。画像が表示されました! ひとまず、ありがとうございます。
    しかし、もう1つだけ課題が残っています。現在、どのセルをクリックしても同じ画像が出てきている状況です。(おそらく、「"image" + "1"」の部分かと思います。ややこしくて申し訳ありませんが、「image1」という画像名をAssetとして保存しているためです。)

    キャンセル

  • 516k

    2018/04/08 23:14

    すみません、言葉足らずでしたね。私のお願いしたコードでは全て同じ画像が表示される状態で正しいです。 表示の問題なのか、画像の受け渡しの問題なのかの切り分けのためでした。 この切り分けでDetailViewControllerに問題がないのはわかりましたので、DetailViewControllerのコードは元の状態に戻してください。

    キャンセル

  • RyousukeTanaka

    2018/04/08 23:20

    こちらこそ、失礼しました。元の状態に戻しておきます。私の方でも引き続き、「画像の受け渡し」の周辺をもう一度確かめてみます。

    キャンセル

回答 1

checkベストアンサー

+1

ControllerがDelegateに適合していないように見受けられます。

class ViewController:UIViewController,UICollectionViewDataSource,UICollectionViewDelegate


にてデリゲートを設定することで、セルタップ時のデリゲートメソッドが呼び出されるようになります。

CollectionViewのDelegateを使わない実装であれば
下記のコードは不要で

 var selectedImage: UIImage?

    func collectionView(_ collectionView: UICollectionView,
                        didSelectItemAt indexPath: IndexPath) {


        let columnIndex = indexPath.row
        let imageName = "image" + String(columnIndex + 1)
        selectedImage = UIImage(named:imageName)
        if selectedImage != nil {
            // SubViewController へ遷移するために Segue を呼び出す
            performSegue(withIdentifier: "toDetailViewController",sender: nil)
        }
    }


アウトレットにてcollectionViewを参照し、
PrepareForSegueの中で選択された画像の取り出しを完結させれば動くような気がします。

// Segue 準備
    override func prepare(for segue: UIStoryboardSegue, sender: Any!) {
        if segue.identifier == "toDetailViewController",
            let subVC = segue.destination as? DetailViewController,
            let indexPath = collectionView?.indexPathsForSelectedItems?.first,
            let image = UIImage(named: "image" + String(indexPath.item + 1)) {
            subVC.testImage = image
        }
    }

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/04/10 12:36

    class ViewController: UIViewController,UICollectionViewDataSource {
    ご提示いただいているコードではViewControllerはUICollectionViewDataSource には適合していますが、UICollectionViewDelegateに適合していないように見受けられますが、その他の場所でextensionで適合させていると言う事でしょうか?

    キャンセル

  • 2018/04/10 12:54

    そこでした。delegateはstoryboard上のみで完結していました。コードにUICollectionViewDelegateを追記で表示されるようになりました!
    ここまで本当にありがとうございます。

    キャンセル

  • 2018/04/10 12:55

    よかったですね。回答の方を書き換えておきます。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.60%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る