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

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

ただいまの
回答率

87.37%

swift 値渡しがされない場合の解決方法

解決済

回答 2

投稿

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

score 14

Xcodeでコレクションビューを使ったTODOアプリを作っています。
collectionViewのセルをロングタップすると入力画面に遷移し、次画面にて入力された内容をセル上のラベルにそれぞれ表示したいのですが値が渡らず困っています・・・。
NavigationControllerは使っておらず、segueにはpresentModallyを使っています。エラー等も出ていません。
何か解る方がいましたらご教授お願いしたいです。よろしくお願いします。

import UIKit

struct MyTodoItem {
    var title: String
    var dateString: String? 
}


class ViewController: UIViewController,UICollectionViewDelegate,UICollectionViewDataSource,UIGestureRecognizerDelegate,nextViewControllerDelegate {



    @IBOutlet weak var todoCollection: UICollectionView!

    let toDos = ["1","2","3","4","5","6"]

    var todos: [MyTodoItem] = [
        MyTodoItem(title: "1", dateString: nil),
        MyTodoItem(title: "2", dateString: nil),
        MyTodoItem(title: "3", dateString: nil),
        MyTodoItem(title: "4", dateString: nil),
        MyTodoItem(title: "5", dateString: nil),
        MyTodoItem(title: "6", dateString: nil),
    ]




    override func viewDidLoad() {
        super.viewDidLoad()

        let layout = UICollectionViewFlowLayout()
        layout.sectionInset = UIEdgeInsets(top: 20, left: 20, bottom: 20, right: 20)
        layout.itemSize = CGSize(width: 100,height: 100)
        todoCollection.collectionViewLayout = layout

        let longPressGestureRecognizer = UILongPressGestureRecognizer(target: self, action: #selector(cellLongPressed))
        longPressGestureRecognizer.delegate = self
        longPressGestureRecognizer.allowableMovement = 15
        longPressGestureRecognizer.minimumPressDuration = 0.6
        todoCollection.addGestureRecognizer(longPressGestureRecognizer)

    }


    func nextViewController(_ nextVC: NextViewController, didFinishText text: String?) {

       if let editingItem = self.editingItem {

                 todos[editingItem].dateString = text

                   todoCollection.reloadItems(at: [IndexPath(item: editingItem, section: 0)])
               }


    }



    //表示するセルの数
    func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        return 6
    }

    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {

        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "cell", for: indexPath)
        cell.backgroundColor = .systemPink

        let item = todos[indexPath.row]


        let todoLabel = cell.contentView.viewWithTag(1) as! UILabel
        todoLabel.text = item.title 
        let dateLabel = cell.contentView.viewWithTag(2) as! UILabel
        dateLabel.text = item.dateString       

        return cell


    }
    func collectionview(_ collectionView: UICollectionView, layout collectionViewLayout: UICollectionViewLayout, sizeForItemAt indexPath: IndexPath) -> CGSize {
        let horizontalSpace : CGFloat = 20
        let cellSize : CGFloat = self.view.bounds.width / 3 - horizontalSpace
        return CGSize(width: cellSize, height: cellSize)
    }


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

    @objc func cellLongPressed(sender: UILongPressGestureRecognizer){

        if sender.state == UIGestureRecognizer.State.began{

           performSegue(withIdentifier: "next", sender: nil)

        }



    }



    var  editingItem: Int?




   override func prepare(for segue: UIStoryboardSegue, sender: Any?) {


              let nextVC = segue.destination as! NextViewController
               nextVC.delegate = self

           }



}
import UIKit



protocol nextViewControllerDelegate {

    func nextViewController(_ nextVC:NextViewController,didFinishText text: String?)
}


class NextViewController: UIViewController,UITextFieldDelegate {

    //デートピッカー
    var datePicker :UIDatePicker = UIDatePicker()


    @IBOutlet weak var dayTextField: UITextField!


   var delegate: nextViewControllerDelegate?




    override func viewDidLoad() {
        super.viewDidLoad()

    //デートピッカー
        datePicker.datePickerMode = UIDatePicker.Mode.date
        datePicker.timeZone = NSTimeZone.local
        datePicker.locale = Locale(identifier: "ja")
        dayTextField.inputView = datePicker
    //デートピッカーのツールバー関係
        let toolBar = UIToolbar(frame: CGRect(x: 0, y: 0, width: view.frame.size.width, height: 35))
        let spaceItem = UIBarButtonItem(barButtonSystemItem: .flexibleSpace, target: self, action: #selector(done))
        let doneItem = UIBarButtonItem(barButtonSystemItem: .done, target: self, action: #selector(done)); toolBar.setItems([spaceItem,doneItem],animated: true)

        dayTextField.inputView = datePicker
        dayTextField.inputAccessoryView = toolBar

    }
   //デートピッカーを閉じるメソッド
    func textFieldShouldReturn(_ textField: UITextField) -> Bool {

        textField.resignFirstResponder()

        return true
    }
  //デートピッカーにdoneボタン

    @ objc func done(){
        dayTextField.endEditing(true)

        let formatter = DateFormatter()

        formatter.dateFormat = "yyyy年MM月dd日"

        dayTextField.text = "\(formatter.string(from: datePicker.date))"

    }

    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()

    }


    @IBAction func addButton(_ sender: Any) {


        delegate?.nextViewController(self, didFinishText: dayTextField.text)

               self.dismiss(animated: true, completion: nil)





    }


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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+1

下記の self.editingItem が nil だからじゃないですか?
self.editingItem の値が設定されていないように思います。

func nextViewController(_ nextVC: NextViewController, didFinishText text: String?) {

       if let editingItem = self.editingItem {

                 todos[editingItem].dateString = text

                   todoCollection.reloadItems(at: [IndexPath(item: editingItem, section: 0)])
               }


    }

質問とは関係ないですが循環参照するのでdelegateはweakにした方がいいですよ。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/10/07 02:14

    おっしゃるとおりでした!どうもありがとうございました!

    キャンセル

0

delegateの使い方が間違っていると思います。
viewcontroller自体を引数に取る必要はないですよ。

下記を参考にしてみてください。
https://qiita.com/osamu1203/items/6dedc01e3b975a0ceec4

コードは古かったので、自分で新たに更新し、名前を少し付け直し作成し
githubにあげてみました。見てみてくださいね。

https://github.com/hameji/TestDelegate

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/10/07 02:15

    コード、参考にさせていただきました!ご親切にどうもありがとうございました!

    キャンセル

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

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

関連した質問

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