質問編集履歴

2020/09/10 03:08

投稿

kazuki_user

スコア147

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -178,59 +178,167 @@
-##　追記
-`let json`は、`guard let json`とすべきなのでしょうか？
-```
-            // dataの強制アンラップを修正。
-            guard let data = data else { return }
-            // try!は、例外が発生したときにはクラッシュするので修正。(-> エラーが起こり得ないケースでのみ使用可)
-            // try?で例外を安全に無視できるが、エラーを表示するため do-catch を使用。
-            do {
-                let json = try JSONSerialization.jsonObject(with: data) as? [String: Any]
-                if let items = json?["items"] as? [[String: Any]] {
-                    self.repo = items
-                    // DispatchQueue で一つ以上のタスクを管理し、async で複数のAPIの非同期通信を実行。
-                    DispatchQueue.main.async {
-                        // UIを更新する処理
-                        self.tableView.reloadData()
-                    }
-                }
-            } catch {
-                debugPrint(error.localizedDescription)
-                return
+## SearchRootVC全文
+```SearchRootVC
+import UIKit
+class SearchRootVC: UITableViewController, UISearchBarDelegate {
+    @IBOutlet weak var searchBar: UISearchBar!
+    // Var
+    var word: String!
+    var url: String!
+    var RepoToPass: Int!
+    var task: URLSessionTask?
+    var repo: [[String: Any]]=[]
+    var repositoryApi = RepositoryApi()
+    override func viewDidLoad() {
+        super.viewDidLoad()
+        setupTableView()
+    }
+    // 以下３つ、元から用意されているsearchBar関数名なので、変更NG。
+    func searchBarShouldBeginEditing(_ searchBar: UISearchBar) -> Bool {
+        searchBar.text = ""
+        return true
+    }
+    func searchBar(_ searchBar: UISearchBar, textDidChange searchText: String) {
+        task?.cancel()
+    }
+    func searchBarSearchButtonClicked(_ searchBar: UISearchBar) {
+        repositoryApi.getRandomRepoUrlSession()
+    }
+    func setupTableView() {
+        // UISearchBarのdelegateプロパティに、self(=SearchRootVC)を代入。
+        searchBar.delegate = self
+    }
+    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
+        if segue.identifier == Segues.ToProfileDetail {
+            if let detailVC = segue.destination as? ProfileDetailVC {
+                detailVC.selectedUser = self
             }
+        }
+    }
+}
+// extension
+extension SearchRootVC {
+    override func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
+        return repo.count
+    }
+    override func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
+        // dequeueReusableCellで、セルを再利用。
+        // nilを返さない為、オプショナルバインディングは不要。
+        let cell: RepositoryCell = tableView.dequeueReusableCell(withIdentifier: Identifiers.RepositoryCell, for: indexPath) as! RepositoryCell
+        let UserRepo = repo[indexPath.row]
+        cell.configureCell(UserRepo)
+        cell.tag = indexPath.row
+        return cell
+    }
+    override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
+        RepoToPass = indexPath.row
+        performSegue(withIdentifier: Segues.ToProfileDetail, sender: self)
+    }
+}
-```
+```

あ

2020/09/10 03:08

投稿

kazuki_user

スコア147

test CHANGED Viewed

	@@ -1 +1 @@
1	- self~~が原因の、EXC_BAD_ACCESSが出る~~【URLSessionを他クラスから呼び出したい。】
1	+ do-catch内にてselfエラー【URLSessionを他クラスから呼び出したい。】

test CHANGED Viewed

File without changes

あ

2020/09/09 23:38

投稿

kazuki_user

スコア147

test CHANGED Viewed

	@@ -1 +1 @@
1	- JS~~ONコードにて、guard l~~e~~t j~~son~~とすべきなの~~か。
1	+ selfが原因の、EXC_BAD_ACCESSが出る【URLSessionを他クラスから呼び出したい。】

test CHANGED Viewed

@@ -1,31 +1,217 @@
-##　リファクタリングしたJSONコードの添削。
-強制アンラップ、不必要なIUOを考慮してリファクタリングしたコードが２です。
-安全性の低い点があれば、指摘して頂きたいです。
-２にて、`guard let`の必要性の有無も伺いたいです。
-##　コード
-####　１
-```
-if let obj = try! JSONSerialization.jsonObject(with: data) as? [String: Any] {
-                if let items = obj["items"] as? [[String: Any]] {
+##　やりたいこと
+`SearchRootVC`クラスにて、**URLSession**のコードを`RepositoryApi`クラスから呼び出したい。
+```SearchRootVC
+   var repositoryApi = RepositoryApi()
+...
+   func searchBarSearchButtonClicked(_ searchBar: UISearchBar) {
+        repositoryApi.getRandomRepoUrlSession()
+    }
+```
+```RepositoryApi
+// URLSessionの処理
+```
+##　selfが原因の、EXC_BAD_ACCESSが出る
+`EXC_BAD_ACCESS`エラーにて、
+`self = CodeCheck_Test_Yumemi.SearchRootVC   0x00007ff8c083db00`だと エラー原因が特定出来たので、
+下記を記述して`self`の使用を避けようとしましたが、引き続き`EXC_BAD_ACCESS`でクラッシュします...
+```RepositoryApi
+var searchRootVC = SearchRootVC()
+```
+##　URLSessionコード
+なお、呼び出しをせずに`SearchRootVC`のみで実行すると
+`EXC_BAD_ACCESS`は無く、正常に動作します。
+```RepositoryApi
+import UIKit
+class RepositoryApi: SearchRootVC {
+    var searchRootVC = SearchRootVC()
+    func getRandomRepoUrlSession() {
+        let word = searchBar.text!
+        let REPOSITORY_URL = URL_BASE + "(word)"
+        // nilは許さない。urlの強制アンラップを修正。
+        guard let url = URL(string: REPOSITORY_URL) else { return }
+        let task = URLSession.shared.dataTask(with: url) { (data, responce, error) in
+            guard error == nil else {
+                debugPrint(error.debugDescription)
+                return
+            }
+            // dataの強制アンラップを修正。
+            guard let data = data else { return }
+            // try!は、例外が発生したときにはクラッシュするので修正。(-> エラーが起こり得ないケースでのみ使用可)
+            // try?で例外を安全に無視できるが、エラーを表示するため do-catch を使用。
+            do {
+                let json = try JSONSerialization.jsonObject(with: data) as? [String: Any]
+                if let items = json?["items"] as? [[String: Any]] {
+                    self.searchRootVC.repo = items
+                    // DispatchQueue で一つ以上のタスクを管理し、async で複数のAPIの非同期通信を実行。
+                    DispatchQueue.main.async {
+                        // UIを更新する処理
+                        self.searchRootVC.tableView.reloadData()
+                    }
+                }
+            } catch {
+                debugPrint(error.localizedDescription)
+                return
+            }
+        }
+        // 新しく初期化されたタスクは一時停止状態で開始されるため、このメソッドを呼び出してタスクを開始する必要がある。
+        task.resume()
+    }
+}
+```
+##　追記
+`let json`は、`guard let json`とすべきなのでしょうか？
+```
+            // dataの強制アンラップを修正。
+            guard let data = data else { return }
+            // try!は、例外が発生したときにはクラッシュするので修正。(-> エラーが起こり得ないケースでのみ使用可)
+            // try?で例外を安全に無視できるが、エラーを表示するため do-catch を使用。
+            do {
+                let json = try JSONSerialization.jsonObject(with: data) as? [String: Any]
+                if let items = json?["items"] as? [[String: Any]] {
                     self.repo = items
+                    // DispatchQueue で一つ以上のタスクを管理し、async で複数のAPIの非同期通信を実行。
                     DispatchQueue.main.async {
                         // UIを更新する処理
@@ -36,54 +222,6 @@
                 }
-}
-```
-####2
-`let json`は、`guard let json`とすべきなのでしょうか？
-```
-            // dataの強制アンラップを修正。
-            guard let data = data else { return }
-            // try!は、例外が発生したときにはクラッシュするので修正。(-> エラーが起こり得ないケースでのみ使用可)
-            // try?で例外を安全に無視できるが、エラーを表示するため do-catch を使用。
-            do {
-                let json = try JSONSerialization.jsonObject(with: data) as? [String: Any]
-                if let items = json?["items"] as? [[String: Any]] {
-                    self.repo = items
-                    // DispatchQueue で一つ以上のタスクを管理し、async で複数のAPIの非同期通信を実行。
-                    DispatchQueue.main.async {
-                        // UIを更新する処理
-                        self.tableView.reloadData()
-                    }
-                }
             } catch {
                 debugPrint(error.localizedDescription)
@@ -96,8 +234,6 @@
+質問は以上です。
 お時間あるときに、ご返信頂けましたら幸いです????

あ

2020/09/09 14:25

投稿

kazuki_user

スコア147

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -1,6 +1,4 @@
-##　JSONコードにて、guard let jsonとすべきなのか。
+##　リファクタリングしたJSONコードの添削。
 強制アンラップ、不必要なIUOを考慮してリファクタリングしたコードが２です。

2020/09/09 13:44

投稿

kazuki_user

スコア147

test CHANGED Viewed

	@@ -1 +1 @@
1	- ~~リファクタリングした~~JSONコードを、~~添削して頂~~き~~たいです~~。
1	+ JSONコードにて、guard let jsonとすべきなのか。

test CHANGED Viewed

@@ -1,4 +1,4 @@
-##　リファクタリングしたJSONコードの添削。
+##　JSONコードにて、guard let jsonとすべきなのか。