ToDOアプリでToDoListが表示されない

Question

ToDOアプリでToDoListが表示されない

swift4でToDoアプリを作成しているのですが、Add画面でToDoリストに内容を追加したのですが、ToDoListに追加した内容が表示されませんので、アドバイスを頂きたいです。

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

ToDoListに追加した内容が表示されない

該当のソースコード

-ViewController.swift-
import UIKit

class ViewController: UIViewController,UITableViewDelegate,UITableViewDataSource {
    
    @IBOutlet weak var tableView: UITableView!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view, typically from a nib.
        if UserDefaults.standard.object(forKey: "TodoList") != nil {
            todoItem = UserDefaults.standard.object(forKey: "TodoList") as! [String]
        }
    }
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return todoItem.count
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        //変数を作る
        let TodoCell : UITableViewCell = tableView.dequeueReusableCell(withIdentifier: "TodoCell", for: indexPath)
        //変数の中身を作る
        TodoCell.textLabel!.text = todoItem[indexPath.row]
        //戻り値の設定（表示する中身)
        return TodoCell
    }
    



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


}

-AddController.swift-
import UIKit

var todoItem = [String]()

class AddController: UIViewController {
    
    @IBOutlet weak var TodoTextField: UITextField!

    override func viewDidLoad() {
        super.viewDidLoad()
        
        // Do any additional setup after loading the view.
    }

    @IBAction func TodoAddButton(_ sender: Any) {
        todoItem.append(TodoTextField.text!)
        TodoTextField.text = ""
        //変数の中身をUDに追加
        UserDefaults.standard.set(todoItem, forKey: "TodoList" )
    }
    
    

    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/ツールのバージョンなど）

Xcode ver.9.4.1
swift4
Main.storyboardの画面



【コンソール画面に表示される内容】
![]
【TodoAddButton】

【20180926時点でのソースコード】

Comments

[xAxis]

AddControllerのtextFieldでなにかを入力した後にViewControllerに戻ってますがその時Cellは増えていますか？

[kinu221]

cellは増えていません。UserDefaultsでデータを保存しているのですが反映されていない状態です。

Answer 1

まず今書かれているコードにはいくつか問題があります。の前に事前確認を。

確認事項

コード内にtableView.datasource = self等デリゲートの設定が書かれていませんがStoryboard上で設定されているものとします。また、UITableViewCellのStyleはBasicになっているものとします。

では問題点に入ります。

グローバル変数

グローバル変数は少ないコードでは問題にならないことの方が多いですがコード量が増えるとバグの温床になります。なので習慣としてグローバル変数は極力作らない、可能なら全く作らない、とした方がデバッグが楽チンです。
ということでAddController.swift内に書かれているグローバル変数todoItemはViewController.swift内のViewControllerのメンバにしてしまいましょう。
じゃあどうやってAddControllerのtextFieldの内容を反映するか、ですがそれは次の問題点とも関わるので次で触れます。

class ViewController: UIViewController,UITableViewDelegate,UITableViewDataSource {
    var todoItem = [String]()

    //以下略
}

addControllerからViewControllerへ画面遷移する際にunwindsegueが設定されていない

任意のViewController_A（以下ViewControllerをVCと略す）からVC_Bへ画面遷移する際にはsegueを接続します。が、VC_BからVC_Aに画面遷移する際、つまり戻る際にはunwindsegueと言うものを使用します。通常StoryboardでVC_BからVC_Aにsegueで接続する方法とは少々異なるやり方です。

現状のコードを再利用するなら、AddController内の

    @IBAction func TodoAddButton(_ sender: Any) {
        todoItem.append(TodoTextField.text!)
        TodoTextField.text = ""
        //変数の中身をUDに追加
        UserDefaults.standard.set(todoItem, forKey: "TodoList" )
    }

このコードを削除して、ViewController内に以下のように追加します。

    @IBAction func TodoAddButton(segue: UIStoryboardSegue) {
　　　　　//segue.sourceとsegue.destinationはセットでよく使います
        //sourceは遷移元、destinationは遷移先です
        guard let VC = segue.source as? AddController else {
            fatalError("ViewController does not exist")
        }
        guard let text = VC.TodoTextField.text else {
            fatalError()
        }
        todoItem.append(text)
        UserDefaults.standard.set(todoItem, forKey: "TodoList" )
    }

こうすることで画面遷移元と画面遷移先の情報のやり取りが可能になります。

その後、Main.storyboardを開き、AddControllerからViewControllerへ伸びているsegueと「戻る」と書かれたBarButtonItem(かな？)を削除します（ついでにNavigationBarも削除しちゃってもいいと思います）。次に画面左部のViewController SceneをクリックするとViewControllerの画面の上に三つボタンが現れます。今回使うのは三つのボタンの一番右側のサーモンピンクっぽい色のボタン(exit)です。「add」と書かれているUIButtonを普段segueを接続するようにexitと接続します。するとToDoAddButtonWithSegueという項目が現れるのでその項目をクリックします。
これでSimulatorで「add」と書かれているUIButtonをタップするとViewControllerへ戻れるようになります。

本題

質問では内容が表示されない、とのことですが実は今回色々書いた中でこれが一番簡単です。todoAddButton関数内にtableView.reloadData()と書き込むだけです。

@IBAction func TodoAddButton(segue: UIStoryboardSegue) {

　　　　　//segue.sourceとsegue.destinationはよく使います
        //sourceは遷移元、destinationは遷移先です
        guard let VC = segue.source as? AddController else {
            fatalError("ViewController does not exist")
        }

        guard let text = VC.TodoTextField.text else {
            fatalError()
        }
        todoItem.append(text)

        UserDefaults.standard.set(todoItem, forKey: "TodoList" )

        tableView.reloadData()//<-ここ
    }

注意すべき点をあえてあげるならばtodoItemにtextを入れた後にreloadする、と言うことくらいでしょうか。でないと新しいデータが反映されませんからね。

Comments

[kinu221]

ご回答ありがとうございます。
>guard let text = VC.textField.text else {
→上記の箇所でValue of type 'ViewController' has no member 'textField'
　と出ており、VC.textField.textを宣言していない為に出ていると思うのです
が、ViewController.swift内に宣言すれば良いとの認識で合っていますでしょうか？
お手数ですが、ご回答をお願い致します

[xAxis]

エラーの読み方は合ってます。
これならキャストが抜けてるか、私のコードのtextFieldの部分をTodoTextFieldに書き換えてないかのどちらかですね。

[xAxis]

というか回答を書き換えればいいですね。修正します。

[kinu221]

ご回答ありがとうございます。
TodoTextField(UITextFieldも確認済み)に変更しても同様のエラーが表示されています。

[xAxis]

自分のコードのキャスト間違ってました。失礼しました。修正します。

[kinu221]

ご回答ありがとうございます。
エラーの方は解消できたのですが、動作確認するとAddボタンを押すと、フリーズし、Thread 1: signal SIGABRTエラーが表示されます。
segueの接続の方法にミスがあるのかだと思います。

[kinu221]

上記エラーは解決しました。しかし、動作確認すると本題であるデータの反映がされません。tableView.reloadData()処理は記述してあります。

[xAxis]

となるといくつか確認です。CustomCellを作ったりしてませんか？またCellのStyleはBasicになっていますか？あとTodoAddButton関数内にprint("add")とか書いてみてコンソールに表示されるか確認してもらえますか？

[xAxis]

ベストアンサーもらっちゃいましたけども無事解決しましたかね？だったら良いのですけども。

[kinu221]

いえ、まだ未解決です。TodoAddButton関数内にprint("add")を追加したらaddが表示されました。確認事項に記載されているtableView.datasource = selfが記載していなくて、追記したりしましたが、表示はされずThread 1: signal SIGABRTエラーがでるだけなのでお手上げ状態です。

[xAxis]

ではもう少しトラブルシューティングしてみましょうか。
print("add")を含んだTodoAddButton関数とコンソール画面に表示されるreason以下を貼ってもらえますか？

[kinu221]

申し訳ありませんがお願い致します。
コンソールの内容の画像を添付致しましたが、reasonが見当たらなく・・表示されているところ全てを貼りました。

[xAxis]

では少々時間が経っていますし今どこまで出来ているのか分からないので追記でやったところまで書いたコードを貼ってもらえますか？

[xAxis]

>>tyobigorouさん
アシストありがとうございます。

本家stackoverflowで_BSMachErrorを調べたのですが本家ですら５件だったかな？しかヒットしなかったですねwこれはちょっと手間取るかもしれないなぁ。
コードを見る限りコードそのものには問題なさそうなんですよね。となるとStoryboardとかinfo.plistとか。
>>tyobigorouさんのコメント欄にある分を確認してみてください。
その確認が済んだら（もしあるなら）breakpointを全部削除して動かしてみてください。その後Storyboardを見直していきましょうか。
Storyboard上にある各ViewControllerのConnections Inspectrorで接続が間違っているものがないか見直してみてください。

[kinu221]

viewDidLoadにtableview.dataSource = self、tableview.delegate = selfを追加するとアプリが起動できない時があります。
また、breakpointを全削除したり、Connections Inspectrorの確認をしました。
exitとaddを接続しているToDoAddButtonWithSegueにSegues initiated directly from view contorollers must have an identifierの警告が表示されています。
storybordのAddとexitの接続あたりが怪しいと思っていますが、Connections Inspectrorを確認すると接続は正しいので、どこに問題があるのか見当がつかない状態です。

[xAxis]

> アプリが起動できない時があります
こういう状態を見聞きしたことが今までないのでこれはちょっとお手上げな感じがありますね・・・。
最後の希望ですが>>tyobigorouさんのコメントから判断するにCellのStyleがBasicになってるか確認してみてください。
それと接続されているSegueを一旦全部切って再接続、というのは試されましたか？もし試されてないなら試してみてください。

これでダメだったら・・・コード量にもよりますが自分ならプロジェクトを新しく立ち上げて移植を考えます（個人で開発されているとか勉強中でアプリを作成してる、とかだったらの話限定ですけどね）。

あとはこちらでは解決まで至らなかった旨をしっかり書いてstackoverflowの日本語版に質問をしてみるとかですかね（ここでもあちらでもマルチポストは禁止事項なので）。

ただ、とにかく早く動く状態にしたい、とかなら自分なら迷わず移植します。

[kinu221]

１から作り直すことで解決致しました。本当にここまでありがとうございます。様々なことを最後までアドバイスいただき、感謝しております。

Answer 2

ViewControllerのviewDidLoadを以下に差し替えて、実行してみてください。
それで、TableViewに何も表示されなければ、xAxisさんが回答の中で指摘されている「確認事項」の部分が問題かと。

 override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view, typically from a nib.
        if UserDefaults.standard.object(forKey: "TodoList") != nil {
            todoItem = UserDefaults.standard.object(forKey: "TodoList") as! [String]
        }
    todoItem = ["AAA", "BBB", "CCC"]
    }

Comments

[退会済みユーザー]

20180926時点でのソースコードを見させていただいて、質問とお願いがあります。
質問は、このソースコードでアプリを立ち上げた時に、表示されるTableViewに3つのセル"AAA","BBB","CCC"は表示されていますか？
お願いは、上記を確認されたあとにViewDidLad内の、todoItem = ["AAA", "BBB", "CCC"]は削除しておいてください。

[kinu221]

アプリを立ち上げた時、"AAA","BBB","CCC"は表示されませんでした。ViewDidLoad内の、todoItem = ["AAA", "BBB", "CCC"]は削除しておきました。

[退会済みユーザー]

別件で偶然同じエラーに遭遇したので、まだ試していない場合、試してほしいことがあります。
アプリをシミュレータもしくは実機から一度削除したのち再度インストールしてみたらどうなるでしょうか？

また、StoryboardEntryPointはviewControllerに入ってるのに、NavigationControllerの残骸のようなものがあるのも気になります。

基本的にはxAxisさんのおっしゃっているようにコードはコピペでいいのでプロジェクトを作り直しするのが早いと思います。

-- 念の為記入 --
別件とは、任意のプロジェクトとは関係のないsimulatorのデータフォルダをfinderから削除したのち、そのプロジェクト上でデータの保存をかけると同様のエラーが発生したというものです。

[kinu221]

アプリをシミュレータもしくは実機から一度削除したのち再度インストールし、1から作り直すと上手くいきました。
本当にありがとうございます。また、最後までお付き合いいただき、様々なアドバイスをありがとうございます。

Answer 3

コメント欄に書くべきところを回答欄に書いてしまいましたごめんなさい。
20180926時点でのソースコードを見させていただいて、質問とお願いがあります。
質問は、このソースコードでアプリを立ち上げた時に、表示されるTableViewに3つのセル"AAA","BBB","CCC"は表示されていますか？
お願いは、上記を確認されたあとにViewDidLad内の、todoItem = ["AAA", "BBB", "CCC"]は削除しておいてください。