あなたのコードには複数の誤りがあります。
Viewは複数の場所を掛け持ちできない
user17014 さんの回答にも書かれている事柄です。
詳細不詳のイニシャライザでUIImageView
のインスタンスを作成している
UIImageView
で引数のないイニシャライザの動作は不詳です。そのようなイニシャライザでインスンタンスを作成(let imageView = UIImageView()
のこと)しても、結果は保障されません。
dequeueReusableCell(withReuseIdentifier:for:)
で取り出したセルにaddSubview(_:)
で子ビューを追加している
dequeueReusableCell
で取得したセルは常に再利用品である可能性があります。従って、以前に追加した子ビューを削除することなく子ビューを追加すると、使っているうちにどんどん子ビューが増えていき、動作速度の上でもメモリ使用量の上でもシステムを圧迫することになります。
Storyboard上でPrototype Cellに子ビューを追加した上で、カスタムセルクラスを使用すれば、上記を全部いっぺんに解消できます。
以下のカスタムセルクラスを用意する。
MyCustomCell.swift (他の部分とつじつまを合わせてくれれば、クラス名は任意です。)
import UIKit
class MyCustomCell: UICollectionViewCell {
@IBOutlet weak var imageView: UIImageView!
}
Storyboard上のIdentifierをcell
としたプロトタイプセルで
- Custom Classを
MyCustomCell
に変更する
UIImageView
をレイアウトエディタ上で該当のプロトタイプセルに貼り付ける
(必要に応じてConstraintを設定してください。)
- 該当のプロトタイプセルを選択した状態でAssistantエディタを開き、レイアウトエディタ上の
UIImageView
をMyCustomCell
クラスのimageView
に接続する
ViewController
は以下のように修正
import UIKit
class ViewController: UICollectionViewController, UICollectionViewDelegateFlowLayout {
//画面に表示するviewは掛け持ちはできない
//let imageView = UIImageView()
let img = UIImage(named: "image")
override func viewDidLoad() {
super.viewDidLoad()
// Do any additional setup after loading the view, typically from a nib.
}
override func didReceiveMemoryWarning() {
super.didReceiveMemoryWarning()
// Dispose of any resources that can be recreated.
}
override func numberOfSections(in collectionView: UICollectionView) -> Int {
return 1
}
override func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
return 3
}
override func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
//次の行が実行時エラーになるならstoryboardでのcellの設定がうまくいっていない
let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "cell", for: indexPath) as! MyCustomCell
//`dequeueReusableCell(withReuseIdentifier:for:)`で取得したセルは常に再利用品である可能性がある
//単純に取り出すたびにaddSubview()していくと、子ビューが増える一方でメモリ使用料・処理時間とも増え続ける
cell.imageView.image = img
return cell
}
}
(let cell = ...
の行は後ろの方に修正点があるのでご注意を。)
UICollectionViewCell
のデフォルトは空っぽなので、storyboardを使用したくないと言う理由が特にあるのでない限り、storyboardでプロトタイプセルのレイアウトを行なって、カスタムセルクラスを使用することをお勧めします。