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

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

新規登録して質問してみよう
ただいま回答率
85.50%
Swift

Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

Q&A

解決済

1回答

2382閲覧

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

RyousukeTanaka

総合スコア12

Swift

Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

0グッド

0クリップ

投稿2018/04/08 13:08

前提・実現したいこと

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

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

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

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

該当のソースコード

遷移元

swift

1import UIKit 2 3class ViewController: UIViewController,UICollectionViewDataSource { 4 5 override func viewDidLoad() { 6 super.viewDidLoad() 7 } 8 9 10 11 @IBAction func backToTop(segue: UIStoryboardSegue) {} 12 13 func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int { 14 return 30 15 } 16 17 func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell { 18 let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "testcell", for: indexPath) as! MyCollectionViewCell 19 // セルのインデックスを取得 20 let columnIndex = indexPath.row 21 // 画像名を作成 22 let imageName = "image" + String(columnIndex + 1) 23 // MyCollectionViewCellのimageに画像を設定 24 cell.cellimage.image = UIImage(named: imageName) 25 26return cell 27 28 } 29 var selectedImage: UIImage? 30 31 func collectionView(_ collectionView: UICollectionView, 32 didSelectItemAt indexPath: IndexPath) { 33 34 35 let columnIndex = indexPath.row 36 let imageName = "image" + String(columnIndex + 1) 37 selectedImage = UIImage(named:imageName) 38 if selectedImage != nil { 39 // SubViewController へ遷移するために Segue を呼び出す 40 performSegue(withIdentifier: "toDetailViewController",sender: nil) 41 } 42 } 43 // Segue 準備 44 override func prepare(for segue: UIStoryboardSegue, sender: Any!) { 45 if (segue.identifier == "toDetailViewController") { 46 let subVC: DetailViewController = (segue.destination as? DetailViewController)! 47 // SubViewController のselectedImgに選択された画像を設定する 48 subVC.testImage = selectedImage 49 } 50 } 51 52 53 54 override func didReceiveMemoryWarning() { 55 super.didReceiveMemoryWarning() 56 } 57 58 59 60}

遷移先

swift

1import UIKit 2 3class DetailViewController: UIViewController { 4 5 6 @IBOutlet weak var recievesideview: UIImageView! 7var testImage:UIImage! 8 override func viewDidLoad() { 9 super.viewDidLoad() 10 recievesideview.image=testImage 11 } 12 13 override func didReceiveMemoryWarning() { 14 super.didReceiveMemoryWarning() 15 // Dispose of any resources that can be recreated. 16 } 17 18 19 /* 20 // MARK: - Navigation 21 22 // In a storyboard-based application, you will often want to do a little preparation before navigation 23 override func prepare(for segue: UIStoryboardSegue, sender: Any?) { 24 // Get the new view controller using segue.destinationViewController. 25 // Pass the selected object to the new view controller. 26 } 27 */ 28 29}

試したこと

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

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

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

516k

2018/04/08 13: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 14:08 編集

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

2018/04/08 14:14

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

2018/04/08 14:20

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

回答1

0

ベストアンサー

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

swift

1class ViewController:UIViewController,UICollectionViewDataSource,UICollectionViewDelegate 2

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

CollectionViewのDelegateを使わない実装であれば
下記のコードは不要で
```swift
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の中で選択された画像の取り出しを完結させれば動くような気がします。 ```swift // 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/08 14:21

編集2018/04/10 03:58
516k

総合スコア189

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

RyousukeTanaka

2018/04/08 15:11 編集

・この位置にprint("ok")でコンソールに正しく表示がされます。 ・上記のコードで行うと、特に変わりはありません。プリント文も問題なしです。 DetailViewController→UINavigationControllerで入れ替えたところ、「Cannot convert value of type 'UINavigationController' to specified type 'DetailViewController'」との表示で警告が出ている状況です。 ・segueの種類は「present modally」とかそういうことじゃないですよね..?
516k

2018/04/08 15:24

上記 let subVC: DetailViewController = segue.destination as! DetailViewController でもクラッシュはしませんでしたか? present modallyで遷移する先のDetailViewControllerはUINavigationControllerの中ではなく直接表示されるって事でOKですか?
RyousukeTanaka

2018/04/08 15:44 編集

はい、クラッシュはしていないです。 okだと思います。UINavigationControllerは使っていないはずなので..? 自分の状況がよく分かっていないのと、うまく補足できておらずにすみません。
516k

2018/04/08 15:51

うむむなかなか手強いですね。。 override func viewDidLoad() { super.viewDidLoad() print(testImage) recievesideview.image=testImage } は何が表示されますかね?
516k

2018/04/08 16:07

殆ど問題なさそうに見えるのですが、なぜかイメージが次の画面に引き継げていない状況です。明日もう少し詳しく調査して見ますね。
516k

2018/04/09 02:49

// 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 print(selectedImage) } } これの結果はどうでしょうか?noneが表示されますか?
RyousukeTanaka

2018/04/09 03:32

noneとの違いは分からないのですが、「nil」と表示されました。
516k

2018/04/10 01:22

コードの流れ的にはそこはnilにならない流れになっていますね。。。。 ちょっと観点が変わるのですが、このcollectionViewのDelegateはどこに設定されていますか? UICollectionViewDataSourceだけがこのViewControllerに設定されていて、かつ、Storyboard上のセルから直接セグエで次の画面をpresentしている可能性は無いでしょうか?
RyousukeTanaka

2018/04/10 03:31

・{ let indexPath = collectionView?.indexPathsForSelectedItems?.first,} の「collectionView」について、「Ambiguous reference to member 'collectionView(_:numberOfItemsInSection:)'」と警告が出ています。(本当に何度も申し訳ないです。。) ・ViewControllerにcollectionVIewのDelegateは紐付けています。
516k

2018/04/10 03:36

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

2018/04/10 03:54

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

2018/04/10 03:55

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問