[Swift] delegateがnilになってしまう

Question

実現したいこと

swift初学者です。delegateについて自力で解決できないところがあり、ご回答いただけますとありがたいです。

位置情報を元に天気に関する情報をAPIから取得し、その内容を表示するアプリを作成しています。
APIから取得したデータをdelegateを用いて、WeatherManagerクラスからWeatherViewControllerクラスにWeatherModelオブジェクトを渡して処理を実行しようとしています。

現在の問題

WeatherManagerのdelegateがnilになったままで、処理が実行されません。

環境

Xcode(ver 13.3.1)
macOS Monterey(ver12.3.1)

該当のコード

import Foundation

// delegate design parttern
protocol WeatherManagerDelegate:AnyObject {
    func didUpdateWeather(inputWeatherModel: WeatherModel)
}

class WeatherManager {
    
    // delegate
    weak var delegate: WeatherManagerDelegate?
    
    // fetchCoordinate
    func fetchWeather(urlString: String) {
        performRequest(inputURLString: urlString)
    }
    
    // performRequest
    func performRequest(inputURLString: String) {
        // 1. Create URL
        if let url = URL(string: inputURLString) {
            // 2. Create URLSession
            let session = URLSession(configuration: .default)
            // 3. Give the URLSession a task
            let task = session.dataTask(with: url) { (data, response, error) in
                if error != nil {
                    print(error!)
                    return
                }
                if let safeData = data {
                    if let weather = self.parseJSON(JSONobject: safeData) {
                        // ---------- 問題の箇所 ----------
                        self.delegate?.didUpdateWeather(inputWeatherModel: weather)
                    }
                }
            }
            // 4. Start the task
            task.resume()
        }
    }
    
    // parse JSON object
    func parseJSON(JSONobject: Data) -> WeatherModel? {
        let decoder = JSONDecoder()
        do {
            let decodedData = try decoder.decode(WeatherData.self, from: JSONobject)
            
            let id = decodedData.weather[0].id
            let temp = decodedData.main.temp
            let name = decodedData.name
            
            let weather = WeatherModel(conditionId: id, cityName: name, temperature: temp)
            return weather
            
        } catch {
            print(error)
            return nil
        }
    }

}

import UIKit

class WeatherViewController: UIViewController, UITextFieldDelegate, WeatherManagerDelegate {
    
    @IBOutlet weak var conditionImageView: UIImageView!
    @IBOutlet weak var temperatureLabel: UILabel!
    @IBOutlet weak var searchTextField: UITextField!
    @IBOutlet weak var cityLabel: UILabel!
    
    var weatherManager = WeatherManager()
    var coordinateManager = CoordinateManager()
    
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view.
        searchTextField.delegate = self
        searchTextField.keyboardType = .asciiCapable
        weatherManager.delegate = self
    }

    @IBAction func searchPressed(_ sender: UIButton) {
        searchTextField.endEditing(true)
    }
    
    func textFieldShouldReturn(_ textField: UITextField) -> Bool {
        searchTextField.endEditing(true)
        return true
    }
    
    func textFieldShouldEndEditing(_ textField: UITextField) -> Bool {
        if textField.text != "" {
            return true
        } else {
            textField.placeholder = "Type Something"
            return false
        }
    }
    
    func textFieldDidEndEditing(_ textField: UITextField) {
        
        if let city = searchTextField.text {
            coordinateManager.fetchCoordinate(cityName: city)
        }
        searchTextField.text = ""
    }
    // ---------- プロトコルのメソッド ----------
    func didUpdateWeather(inputWeatherModel: WeatherModel) {
        print(inputWeatherModel.temperature)
    }
}

import Foundation

struct WeatherModel {
    let conditionId: Int
    let cityName: String
    let temperature: Double
}

import Foundation

struct CoordinateManager {

    // WeatherManager
    let weathetManager = WeatherManager()
    // geographical coordinates(lat, lon)　地理座標
    let coordinateURL = "https://api.openweathermap.org/geo/1.0/direct?limit=1&appid=c8e60c6317c653a1789294c00f54ae19"
    
    // fetchCoordinate
    func fetchCoordinate(cityName: String) {
        let urlString = "\(coordinateURL)&q=\(cityName)"
        performRequest(inputURLString: urlString)
    }
    
    // transformURLString
    func transformURLString(_ string: String) -> URLComponents? {
        guard let urlPath = string.components(separatedBy: "?").first else {
            return nil
        }
        var components = URLComponents(string: urlPath)
        if let queryString = string.components(separatedBy: "?").last {
            components?.queryItems = []
            let queryItems = queryString.components(separatedBy: "&")
            for queryItem in queryItems {
                guard let itemName = queryItem.components(separatedBy: "=").first,
                      let itemValue = queryItem.components(separatedBy: "=").last else {
                        continue
                }
                components?.queryItems?.append(URLQueryItem(name: itemName, value: itemValue))
            }
        }
        return components!
    }
    
    // performRequest
    func performRequest(inputURLString: String) {
        // 1. Create URL
        let components = transformURLString(inputURLString)
        if let url = components?.url {
            // 2. Create URLSession
            let session = URLSession(configuration: .default)
            // 3. Give the URLSession a task
            let task = session.dataTask(with: url) {(data, response, error) in
                if error != nil {
                    print(error!)
                    return
                }
                if let safeData = data {
                    self.parseJSON(JSONobject: safeData)
                }
            }
            // 4. Start the task
            task.resume()
        }
    }

    // parseJSON
    func parseJSON(JSONobject: Data) {
        let decoder = JSONDecoder()
        do {
            let decodedData = try decoder.decode([CoordinateData].self, from: JSONobject)
            let name = decodedData[0].name
            let lat = decodedData[0].lat
            let lon = decodedData[0].lon
            
            let coordinateModel = CoordinateModel(name: name, lat: lat, lon: lon)
            let weatherURL = coordinateModel.urlString

           // ------- WeatherManager fetchWeather ---------
           weathetManager.fetchWeather(urlString: weatherURL)
        } catch {
            print(error)
        }
    }
}

import Foundation

struct CoordinateModel {
    let name: String
    let lat: Double
    let lon: Double
    
    // wheather　地理座標を元に検索した天気の情報
    let weatherstr = "https://api.openweathermap.org/data/2.5/weather?"
    let str = "units=metric&appid=c8e60c6317c653a1789294c00f54ae19#"

    // computedproperty
    var urlString: String {
        return "\(weatherstr)&lat=\(lat)&lon=\(lon)&\(str)"
    }
}

よろしくお願いいたします。

Comments

[hoshi-takanori]

fetchWeather はどこで呼んでますか？

[Adam_Yoneda]

失礼しました。
追加したコードのstruct CoordinateManager内の下の方で呼んでいます。

CoordinateManager内では先に都市名から地理座標を得るAPIを利用して、天気の情報を得るURLを作成しています。

Answer 1

stackoverflowでも質問したところ、
WeatherManagerの2つのインスタンスを書いており、そのためWeatherVC内のWeatherManager()が使われずにいたためnilのままでした。

解決策としてはCoordinateManagerの内容をWeatherManager内に持ってきて、WeatherManagerのインスタンスをWeatherVC内のみにすると機能しました。

検討いただいた方々ありがとうございました！