do-catch内にてselfエラー【URLSessionを他クラスから呼び出したい。】

Question

##　やりたいこと

SearchRootVCクラスにて、URLSessionのコードをRepositoryApiクラスから呼び出したい。

   var repositoryApi = RepositoryApi()

...

   func searchBarSearchButtonClicked(_ searchBar: UISearchBar) {
        repositoryApi.getRandomRepoUrlSession()
    }

// URLSessionの処理

##　selfが原因の、EXC_BAD_ACCESSが出る

EXC_BAD_ACCESSエラーにて、

self = CodeCheck_Test_Yumemi.SearchRootVC 0x00007ff8c083db00だと エラー原因が特定出来たので、

下記を記述してselfの使用を避けようとしましたが、引き続きEXC_BAD_ACCESSでクラッシュします...

var searchRootVC = SearchRootVC()

##　URLSessionコード

なお、呼び出しをせずにSearchRootVCのみで実行すると
EXC_BAD_ACCESSは無く、正常に動作します。

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()
    }
    
}

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)
    }
}

質問は以上です。
お時間あるときに、ご返信頂けましたら幸いです????

Comments

[TsukubaDepot]

RepositoryApi　は、SearchRootVCにかなり依存しているようなので、SearchRootVC が何者かわからないと回答できないと思います。
ご自身で作られたクラスであれば、依存関係はおわかりだと思うので、もう少し整理できるかと思いますが、そうでないならば、まずは依存関係を整理されたらいかがでしょうか。

[TsukubaDepot]

質問の変更履歴ですが、適当なタイトル「あ、とかaとか」にせず、適切なタイトルをつけることはできないのでしょうか。
改変履歴は誰でも参照できますが、適当なタイトルだと何が変わったのかひと目ではわからないため大変困ります。

[TsukubaDepot]

ちなみに、RepositoryApiのベースクラス（継承元）が SearchRootVC なのは、どういう意図からなのでしょうか。

[kazuki_user]

申し訳ありませんでした。変更タイトルに関して以後、気を付けます。????‍♂️

質問に添った回答か分かりませんが..

現在、コードに対してリファクタリングを行っており、
SearchRootVCにて、レポジトリ検索画面の主な機能(searchBarShouldBeginEditingなど) を実装しております。
URLSessionを他のファイルに分割する為、RepositoryApiを新たに作りURLSessionを記述し、
それをSearchRootVCから呼び出したいと考えています。

[TsukubaDepot]

クラスの継承、という考え方は理解されていますでしょうか。
また、個別のクラス間におけるプロパティ参照の理屈についても理解されていますでしょうか。

[kazuki_user]

: で継承し、親の機能を参照できる..という認識です。

個別のクラス間におけるプロパティ参照に関しては、少し自信がないかもです..
今回のエラーは、self含め、これらが原因でしょうか？

selectedUserがnilで、Unexpectedly found nilエラーに関してもそうですが、
別ファイル間の行き来の問題でエラーが起こっているように思われます????

[TsukubaDepot]

繰り返しになりますが、文法全般の理解が不足していることが原因かと思います。
また、不用意にクラスを継承させたことが原因で、見えなくてもいい変数が見えてしまい、それがかえって混乱を招いている雰囲気があります。
また、型や定数などの名付け方もSwiftの流儀に沿っていないところがあるので読みにくいところがあります。

リファクタリングは重要なことかと思いますが、どこまでを一般化するのか整理しないと、かえって使いづらくなるのではないでしょうか。

たとえば、RepositoryApiから親クラスの tableView などが見えていますが、見えているからといって具体的なインスタンス（値）が入っているわけではありません。プロパティやメソッドは経由しますが、プロパティに対して動的に代入される値までは継承されません。

なので、そのあたりをきちんと考えてリファクタリングする必要があります。

[TsukubaDepot]

ちなみに、このアプリはどこのWebAPIを検索して（つまり、URL_BASEはどんなURLにアクセスして）結果を表示しようとしたのでしょうか。

[kazuki_user]

https://api.github.com/search/repositories?q=\(word!)
GitHubAPIです。

(word = searchBar.text!)

URLSessionがSearchRootVCにあると、煩雑かなと思ったのですが、どうなのでしょうか..?

Answer 1

現状、あえてクラスを作るほどでもない感じもするのですが、たとえばこんな感じでしょうか。

import UIKit

class SearchRootVC: UITableViewController, UISearchBarDelegate {
    @IBOutlet weak var searchBar: UISearchBar!
    
    // MARK: - 以下のプロパティは使わないのでコメントアウト
    // また、不用意に　Implicitly Unrapped Optional を使わない
    // Var
    //var word: String!
    //var url: String!
    
    // MARK: - 変数名は大文字始まりにしない
    //var RepoToPass: Int!
    //var repoToPass: Int!
    
    var task: URLSessionTask?
    var repo: [[String: Any]] = []
    
    
    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) {
        searchBar.resignFirstResponder()
        
        // MARK: - UITextField がからの場合には検索しない
        //let word = searchBar.text!
        guard let word = searchBar.text, word.isEmpty == false else {
            print("検索文字がない")
            return
        }
        
        task = SearchAPI.getRandomRepoUrlSession(word, completionHandler: { items in
            self.repo = items
            // DispatchQueue で一つ以上のタスクを管理し、async で複数のAPIの非同期通信を実行。
            DispatchQueue.main.async {
                // UIを更新する処理
                self.tableView.reloadData()
            }
        }, errorHandler:  { error in
            debugPrint(error?.localizedDescription)
        })
    }
    
    func setupTableView() {
        // UISearchBarのdelegateプロパティに、self(=SearchRootVC)を代入。
        searchBar.delegate = self
        
        // MARK: - TableViewCell は StoryBoard で追加する
        // Identifier は Cell
        // Style は Subtitle
    }
    
    // MARK: - 画面遷移は今回は行わない
    //    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を返さない為、オプショナルバインディングは不要。
        
        // MARK: - 今回は標準の TableViewCell を使う
        //let cell: RepositoryCell = tableView.dequeueReusableCell(withIdentifier: Identifiers.RepositoryCell, for: indexPath) as! RepositoryCell
        let cell = tableView.dequeueReusableCell(withIdentifier: "Cell", for: indexPath)
        
        // MARK: - 大文字始まりはクラス名でのみ使う。変数名は小文字始まり
        //let UserRepo = repo[indexPath.row]
        let userRepo = repo[indexPath.row]
        
        //MARK: - GitHub から得られる情報のうち、"name" と "description" だけ表示させる
        if let userName = userRepo["name"] as? String {
            cell.textLabel?.text = userName
        } else {
            cell.textLabel?.text = "（名無し）"
        }
        
        if let description = userRepo["description"] as? String {
            cell.detailTextLabel?.text = description
        } else {
            cell.detailTextLabel?.text = "（詳細無し）"
        }
        
        // MARK: - とりあえず以下はコメントアウト
        //        cell.configureCell(UserRepo)
        //        cell.tag = indexPath.row
        return cell
    }
    
    override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        // MARK: - 画面遷移は今回は行わない
        //repoToPass = indexPath.row
        //performSegue(withIdentifier: Segues.ToProfileDetail, sender: self)
    }
}

// MARK: - クラスメソッドとして定義
class SearchAPI {
    static private let githubBaseUrl = "https://api.github.com/search/repositories?q="
    static func getRandomRepoUrlSession(_ word: String, completionHandler completion: @escaping ([[String: Any]]) -> (), errorHandler: @escaping (Error?) -> ()) -> URLSessionTask? {
        // MARK: -  下記の部分はメソッドに渡す前に処理しておく
        //        let word = searchBar.text!
        //        guard let word = searchBar.text, word.isEmpty == false else {
        //            print("検索文字がない")
        //            return
        //        }
        
        // MARK: - 変数名を全て大文字にしない（流儀）
        //let REPOSITORY_URL = URL_BASE + "(word)"
        let repositoryUrl = githubBaseUrl + word
        
        // nilは許さない。urlの強制アンラップを修正。
        guard let url = URL(string: repositoryUrl) else { return nil }
        
        
        //let task = URLSession.shared.dataTask(with: url) { (data, responce, error) in
        let task = URLSession.shared.dataTask(with: url) { (data, responce, error) in
            
            guard error == nil else {
                // MARK: - 処理をクロージャに任せる
                errorHandler(error)
                // MARK: - 以下の処理はクロージャに任せる
                //                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]] {
                    // MARK: - 処理をクロージャに任せる
                    completion(items)
                    // MARK: - 以下の処理はクロージャに任せる
                    //                    self.repo = items
                    //                    // DispatchQueue で一つ以上のタスクを管理し、async で複数のAPIの非同期通信を実行。
                    //                    DispatchQueue.main.async {
                    //                        // UIを更新する処理
                    //                        self.tableView.reloadData()
                    //                    }
                }
            } catch {
                // MARK: - 処理をクロージャに任せる
                errorHandler(error)
                // MARK: - 以下の処理は クロージャに任せる
                //                debugPrint(error.localizedDescription)
                //                return
            }
        }
        // 新しく初期化されたタスクは一時停止状態で開始されるため、このメソッドを呼び出してタスクを開始する必要がある。
        task.resume()
        
        return task
    }
}

Comments

[TsukubaDepot]

変更点の詳細はコメントをご参照ください。
また、TableViewCell については、StoryBoard で作り必要がありますが、詳細については、同じくコメントをご参照ください。

[kazuki_user]

勉強になります。
Implicitly Unrapped Optional始め、Swiftの文法基礎についてしっかり学び直します。
また、おかしな点がありましたら、ご指摘頂けると嬉しいです。

[TsukubaDepot]

udemyで色々と学ばれていると思いますが、まずはごく簡単なアプリから（たとえば、TableViewならTableView、画面遷移なら遷移だけを行うアプリなど）を作り、一つひとつのパーツなどの動作についてクリアしてから組み合わせたほうが、結局は習得が早いかと思います。

[kazuki_user]

なるほどです????‍♂️、了解しました。