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

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

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

Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

Swift

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

Swift 2

Swift 2は、Apple社が独自に開発を行っている言語「Swift」のアップグレード版です。iOSやOS X、さらにLinuxにも対応可能です。また、throws-catchベースのエラーハンドリングが追加されています。

Q&A

1回答

1376閲覧

TableViewの削除機能がうまく働かない

KokokaraYoshiki

総合スコア27

Xcode

Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

Swift

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

Swift 2

Swift 2は、Apple社が独自に開発を行っている言語「Swift」のアップグレード版です。iOSやOS X、さらにLinuxにも対応可能です。また、throws-catchベースのエラーハンドリングが追加されています。

0グッド

0クリップ

投稿2017/07/11 00:08

###前提・実現したいこと
追加したtodoのデータを削除したい
goalのデータは削除できるのですがtodoではエラーになってしまいます
###発生している問題・エラーメッセージ

fatal error: Index out of range

@IBAction func addToDo(_ sender: UIButton) { // テキストフィールド付きアラート表示 let alert = UIAlertController(title: "ToDo", message: "文字を入力してください。", preferredStyle: .alert) // OKボタンの設定 let okAction = UIAlertAction(title: "OK", style: .default, handler: { (action:UIAlertAction!) -> Void in // OKを押した時入力されていたテキストを表示 if let textFields = alert.textFields { // アラートに含まれるすべてのテキストフィールドを調べる for textField in textFields { // AppDelegateのインスタンスを用意しておく let appDelegate:AppDelegate = UIApplication.shared.delegate as! AppDelegate // エンティティを操作するためのオブジェクト let viewContext = appDelegate.persistentContainer.viewContext // ToDoエンティティオブジェクトを作成 let ToDo = NSEntityDescription.entity(forEntityName: "ToDo", in: viewContext) // ToDoエンティティにレコード(行)を挿入するためのオブジェクトを作成 let newRecord = NSManagedObject(entity: ToDo!, insertInto: viewContext) // 追加したいデータ(txtTitleに入力された文字)のセット if textField.text! == "" || textField.text! == nil{ print("nilが入っています。") }else{ newRecord.setValue(textField.text!, forKey: "todotitle") newRecord.setValue(Date(), forKey: "tododeta") // レコード(行)の即時保存 do{ try viewContext.save() }catch{ } print("右の文字が入る\(textField.text!)") self.todoTask.append(textField.text!) // self.todoDeta.append() self.TableView.reloadData() } } } }) alert.addAction(okAction) // キャンセルボタンの設定 let cancelAction = UIAlertAction(title: "Cancel", style: .cancel, handler: nil) alert.addAction(cancelAction) // テキストフィールドを追加 alert.addTextField(configurationHandler: {(textField: UITextField!) -> Void in textField.placeholder = "テキスト" }) alert.view.setNeedsLayout() // シミュレータの種類によっては、これがないと警告が発生 // アラートを画面に表示 self.present(alert, animated: true, completion: nil) } // 削除機能 func tableView(_ tableView: UITableView, commit editingStyle: UITableViewCellEditingStyle, forRowAt indexPath: IndexPath) { if tableView.tag == 1{ if editingStyle == .delete { print("削除する文字は\(todoTask)") print("削除する時間は\(todoDeta)") todoTask.remove(at: indexPath.row) todoDeta.remove(at: indexPath.row) tableView.deleteRows(at: [indexPath], with: .fade) // AooDelegateを使う用意をしておく let appDelegate:AppDelegate = UIApplication.shared.delegate as! AppDelegate // エンティティを操作するためのオブジェクトを作成 let viewContext = appDelegate.persistentContainer.viewContext // どのエンティティからデータを取得してくるか設定 let query:NSFetchRequest<ToDo> = ToDo.fetchRequest() do{ // 削除するデータを取得 let fetchResults = try viewContext.fetch(query) // 削除するデータを取得 for result : AnyObject in fetchResults { let deta: NSDate! = result.value(forKey: "tododeta") as! NSDate let formatter = DateFormatter() formatter.dateFormat = "yyyy-MM-dd HH:mm:ss" //Stringにしたい let detastring:String = formatter.string(from: deta as Date) /// 一行ずつ削除 "ここでエラーになります" if detastring == todoDeta[indexPath.row]{ // 一行ずつ削除 let record = result as! NSManagedObject viewContext.delete(record) } } // 削除した状態を保存(処理の確定) try viewContext.save() }catch{ } } }else{ if editingStyle == .delete { goalTask.remove(at: indexPath.row) goalTaskDetail.remove(at: indexPath.row) tableView.deleteRows(at: [indexPath], with: .fade) // AooDelegateを使う用意をしておく let appDelegate:AppDelegate = UIApplication.shared.delegate as! AppDelegate // エンティティを操作するためのオブジェクトを作成 let viewContext = appDelegate.persistentContainer.viewContext // どのエンティティからデータを取得してくるか設定 let query:NSFetchRequest<Goal> = Goal.fetchRequest() do{ // 削除するデータを取得 let fetchResults = try viewContext.fetch(query) // 削除するデータを取得 for result : AnyObject in fetchResults { let deta: NSDate! = result.value(forKey: "goaldeta") as! NSDate let formatter = DateFormatter() formatter.dateFormat = "yyyy-MM-dd HH:mm:ss" //Stringにしたい let detastring:String = formatter.string(from: deta as Date) // 一行ずつ削除 if detastring == myDeta[indexPath.row]{ // 一行ずつ削除 let record = result as! NSManagedObject viewContext.delete(record) } } // 削除した状態を保存(処理の確定) try viewContext.save() }catch{ } } } } override func viewWillAppear(_ animated: Bool) { // CoreDataからデータをfetchしてくる read1() read() self.subTableView.reloadData() }

###試したこと
課題に対してアプローチしたことを記載してください
/// 一行ずつ削除 "ここでエラーになります"
if detastring == todoDeta[indexPath.row]{
db確認済み
3つtodo リストを作成し再度起動
はじめと2つめ消える。三つ目でエラーになります。
###補足情報(言語/FW/ツール等のバージョンなど)
swift3 xcode10.3

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

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

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

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

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

guest

回答1

0

エラーメッセージは

「配列の要素数を超えた位置にアクセスしようとしているので、それはできませんよ」

という意味になります。

エラーの発生する処理の前に todoDeta (これはスペルミスで、正しくは todoData でしょう)
から値を削除しているのが原因だと思います。

swift

1 todoDeta.remove(at: indexPath.row) 2

投稿2017/07/13 11:48

ykws

総合スコア1236

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問