テキスト文字列をモデルに保存して、VCの関数から参照したい。

Question

##　お願い

本問は解決致しましたので、
こちらに質問を移動しました。

##　やりたいこと

テキスト文字列をConstants.swift(モデル)に保存して、
ProfileDetailVCのsetupUI()から参照したい。

##　エラー

// Constants.swift
Cannot assign to property: 'selectedUser' is immutable

Constants.swiftのプロパティ辺りでエラー。
複数のファイル間に共通する変数(?) の swift基礎文法が欠けているのかな..と思います。

この機会に学びたいので、本問題において 具体的にどの知識が欠けているのか、
差し支えなければ ご指摘お願いします。????‍♀️

##　コード

// Constants.swift

import UIKit
  
let githubBaseUrl = "https://api.github.com/search/repositories?q="

var selectedUser: SearchRootVC!
// 以下でエラー
let repo = selectedUser.repo[selectedUser.RepoToPass]  // Cannot assign to property: 'selectedUser' is immutable


struct Segues {
    static let toProfileDetail = "Detail"
}

struct Identifiers {
    static let repositoryCell = "Repository"
}

struct ApiKey {
    static let language = "language"
    static let stars = "stargazers_count"
    static let watchers = "wachers_count"
    static let forks = "forks_count"
    static let issues = "open_issues_count"
    static let userName = "full_name"
    static let user = "owner"
    static let imgUrl = "avatar_url"
}

struct repoTxt {
    static let title = repo[ApiKey.userName] as? String

    static let language = "Written in (repo["language"] as? String ?? "")"
    static let stars = "(repo["stargazers_count"] as? Int ?? 0) stars"
    static let watchers = "(repo["wachers_count"] as? Int ?? 0) watchers"
    static let forks = "(repo["forks_count"] as? Int ?? 0) forks"
    static let issues = "(repo["open_issues_count"] as? Int ?? 0) open issues"
}

// ProfileDetailVC

import UIKit

class ProfileDetailVC: UIViewController {
    
    @IBOutlet weak var ImgView: UIImageView!
    
    @IBOutlet weak var TitleLbl: UILabel!
    
    @IBOutlet weak var LanguageLbl: UILabel!
    
    @IBOutlet weak var StarsLbl: UILabel!
    @IBOutlet weak var WatchersLbl: UILabel!
    @IBOutlet weak var ForksLbl: UILabel!
    @IBOutlet weak var IssuesLbl: UILabel!
    
    // Var
    var selectedUser: SearchRootVC!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        setupUI()
        getProfileImage()
    }
    

    func setupUI() {
        TitleLbl.text = repoTxt.title
        
        LanguageLbl.text = repoTxt.language
        StarsLbl.text = repoTxt.stars
        WatchersLbl.text = repoTxt.watchers
        ForksLbl.text = repoTxt.forks
        IssuesLbl.text = repoTxt.issues
    }
    
    func getProfileImage() {
        // 以下、省略。

// SearchRootVC

import UIKit

class SearchRootVC: UITableViewController, UISearchBarDelegate {
    
    @IBOutlet weak var searchBar: UISearchBar!
    
    var repoToPass: Int!
    
    var task: URLSessionTask?
    var repo: [[String: Any]] = []
    
    override func viewDidLoad() {
        super.viewDidLoad()
        setupTableView()
        tableViewBgImage()
    }
    
    // 以下３つ、元から用意されているsearchBar関数名なので、変更NG。
    func searchBarShouldBeginEditing(_ searchBar: UISearchBar) -> Bool {
        searchBar.text = ""
        searchBar.autocapitalizationType = .none  // 検索時、先頭を小文字で始める
        return true
    }
    
    func searchBar(_ searchBar: UISearchBar, textDidChange searchText: String) {
        task?.cancel()
    }
    
    func searchBarSearchButtonClicked(_ searchBar: UISearchBar) {
        // キーボード非表示
        searchBar.resignFirstResponder()
        
        // MARK: - 不用意なIUOを削除。
        // let query = searchBar.text!
        guard let query = searchBar.text, query.isEmpty == false else {
            print("検索文字がない")
            return
        }
        
        // word, completion, errorHandler　3つの引数をメソッドに渡す。
        task = SearchAPI.getRandomRepoUrlSession(query, 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: - TableView で背景画像を表示。
    func tableViewBgImage() {
        let imageView = UIImageView(frame: CGRect(x: 0, y: 0, width: self.tableView.frame.width, height: self.tableView.frame.height))
        let image = UIImage(named: "bg_right1")
        
        imageView.image = image
        imageView.alpha = 0.8

        self.tableView.backgroundView = imageView
    }
    
    // MARK: - // Segueが実行される前に呼び出される。
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        // Segueの識別子確認
        if segue.identifier == Segues.toProfileDetail {
            // 遷移先ViewCntrollerの取得
            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: - セルの再利用
        let cell = tableView.dequeueReusableCell(withIdentifier: "RepositoryCell", for: indexPath)
        
        let userRepo = repo[indexPath.row]
        
        // MARK: - nil回避しつつ、nilの場合は「None」を表示。
        if let userName = userRepo[ApiKey.userName] as? String {
            cell.textLabel?.text = userName
        } else {
            cell.textLabel?.text = "--None--"
        }
        // カスタムクラスの参照を修正。(-> detailTextLabelの表示)
        if let language = userRepo[ApiKey.language] as? String {
            cell.detailTextLabel?.text = language
        } else {
            cell.detailTextLabel?.text = "--None--"
        }
        
        return cell
    }
    
    override func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        repoToPass = indexPath.row
        performSegue(withIdentifier: Segues.toProfileDetail, sender: self)
    }
}

// MARK: - Cellの備忘録
// register() ... 用意したviewをcellのテンプレートとして登録するメソッドであり、cellの再利用に必要。
// Right Detail を表示は、コード不可 StoryBoard可なので、register()は使用しない。

// MARK: - クラスメソッドとして定義

class SearchAPI {
    // staticとして宣言すると、クラスのインスタンス化が不要。
    static func getRandomRepoUrlSession(_ query: 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
        //        }
        
        let repositoryUrl = githubBaseUrl + query
        
        guard let url = URL(string: repositoryUrl) else { return nil }
        
        let task = URLSession.shared.dataTask(with: url) { (data, responce, error) in
            
            guard error == nil else {
                // MARK: - 処理をクロージャに任せる
                errorHandler(error)
                return
            }
            
            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)
            }
        }
        task.resume()
        
        return task
    }
}

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

Comments

[unhappychoice]

省略せずに Constants.swift VC のコードを全文掲載をお願いしますmm

[kazuki_user]

追記致しました。????‍♂️

[TsukubaDepot]

あるいは、Cannot assign to property: 'selectedUser' is immutable というエラーが出るような、最低限（できれば Playground で実行できるレベル）のコードを提示してもらうことはできますでしょうか。
仮に定義は間違っていなくとも、その使い方が間違っていればエラーがおきますので、それを確認できるレベルでコードを示していただいた方が有意義な議論ができるかと思います。

あと、
> (structを使用しているので、「値渡し」？というものでしょうか????)
とありますが、struct と値渡しは全く関係ありません。

新たにインスタンス化した構造体やクラスのプロパティに対し、値を代入している操作を便宜的に「値渡し」と呼んでいるだけであり、構造体だけに特化した話ではありませんし、特別な操作でもありません。

Udemy の英語講座を受けていらっしゃったかと思いますが、おそらく pass variables to other properties of the class (or struct) みたいなタイトルで言及されていると思いますし、View Controller 間の値渡しであれば「pass variable to next view controller」というキーワードでも参考記事は見つかるかと思います。

[TsukubaDepot]

あと、SearchRootVCのコードも追記いただけないでしょうか。
細かい点はともかく、上記のコードを真似して作ってみても、コンパイルは通ってしまいます。そうなると、SearchRootVC の方と関連する可能性も否めません。

あと、「selectedUserがnilで、Unexpectedly found nilエラー」というご質問で、Implicitly Unwrapped Optinal Value の扱いについて指摘いたしましたが、その対策は取られているのでしょうか。
今回のコードにも改善された形跡がないため、仮に今回のエラーが解決しても、次は実行時エラーが出てしまうことは明白です。
https://teratail.com/questions/290812

[kazuki_user]

> 今回のコードにも改善された形跡がないため、

..。見直してみます。
var selectedUser: SearchRootVC! のことでしょうか？

SearchRootVCに関しては、コードを追記しました。

[TsukubaDepot]

はい、
var selectedUser: SearchRootVC!
この部分です。
これだと、selectedUser は nil のままなので（かつ、nil のまま操作することを許されていないプロパティなので）、結局はselectedUser.repo[selectedUser.RepoToPass]を実行しようとした時点でエラーとなってしまいます。

[kazuki_user]

オプショナル型の変数を宣言する際に付ける「!」と、 強制的アンラップの 「! 」とを混同していました。

オプショナル型の基礎については学びましたが、
クラスのデータ型を持つ変数の参照する際に、別のエラーが発生してしまい、

テキストデータのモデル格納が未解決なので、それに関しては別の質問として投稿させて頂きます。

Answer 1

オプショナル型の変数を宣言する際に付ける「!」と、 強制的アンラップの 「!」とを混同していました。

使用するとき必ず強制的アンラップを行い、(= 自動でアンラップをしてくれる)
nilでアンラップするとアプリが落ちるため、絶対に値がある(= nilでない) 場合のみ使用可能なImplicitly Unwrapped Optionalの基礎についてはしっかりと学んだつもりですが、

未だ別のエラーが発生してしまい、テキストデータのモデル格納が未解決なので、
それに関しては別の質問として投稿させて頂きます。

Answer 2

let repo = selectedUser.repo[selectedUser.RepoToPass]  // Cannot assign to property: 'selectedUser' is immutable

上記のエラーが出る理由は単純な理由です。

class SearchRootVC: UITableViewController, UISearchBarDelegate {

    var repoToPass: Int!

SearchRootVC で定義されている repoToPass は小文字始まりです。
なので、selectedUser.repoToPassと変更すればコンパイルは通るのではないでしょうか。

しかし、次は実行時エラー(Unexpectedly found nil while implicitly unwrapping an Optional value）になるかと思います。

これを解決しようとするには、（おそらく以前も指摘しましたが）SearchRootVC のインスタンスを作ることになりなりますが、そもそも SearchRootVC というクラスはこういう流れでインスタンス化するように意図されていませんので、様々なところで不都合が起きてくると思います。