swift3 他のClassからViewControllerマップにピンを立てたい

Question

subTableVC_1.swiftのクラスからViewController.swiftのマップにピンを立てたいのですが、fatal error: unexpectedly found nil while unwrapping an Optional valueが発生してピンを立てることができません。宜しくお願い申し上げます。

//  ViewController.swift

import UIKit
import MapKit

class ViewController: UIViewController, MKMapViewDelegate, CLLocationManagerDelegate {

    @IBAction func fromSubTableVC_1Segue(_ segue:UIStoryboardSegue){}
    
    var senderFileName : String!
    
    var myMapView : MKMapView!
    var myLocationManager : CLLocationManager!
    

    var menuBtn   : UIButton!
    
    // 縮尺.
    var myLatDist : CLLocationDistance = 10
    var myLonDist : CLLocationDistance = 10
    let DEFAULT_DISTANCE : CLLocationDistance = 10
    
    var center: CLLocationCoordinate2D!
      
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view, typically from a nib.
    }
    
    override func viewWillAppear(_ animated: Bool) {
        super.viewWillAppear(animated)
    
        let maxWidth : CGFloat = self.view.bounds.width
        let maxHeight: CGFloat = self.view.bounds.height
       
        myMapView = MKMapView()
        myMapView.frame = self.view.frame(forAlignmentRect: CGRect(x: 0.0, y: 20.0 + 50.0, width: maxWidth, height: maxHeight - 20.0 - 50.0 - 50.0))
        myMapView.delegate = self
        myMapView.setCenter(self.myMapView.userLocation.coordinate, animated: true)
        myMapView.setUserTrackingMode(.follow, animated: true)
        myMapView.showsUserLocation = true
        view.addSubview(self.myMapView)
        
        menuBtn = UIButton()
        menuBtn.frame = CGRect(x: maxWidth / 5.0 * 2.0, y: maxHeight - 50.0, width: maxWidth / 5.0, height: 50.0)
        menuBtn.setTitleColor(UIColor.blue, for: .normal)
        menuBtn.backgroundColor = UIColor.lightGray
        menuBtn.layer.cornerRadius = 10.0
        menuBtn.setTitle("リスト", for: .normal)
        menuBtn.tag = 4
        menuBtn.addTarget(self, action: #selector(ViewController.onClickMyButton(sender:)), for: .touchDown)
        self.view.addSubview(menuBtn)

        myLocationManager = CLLocationManager()
        myLocationManager.delegate = self
        myLocationManager.distanceFilter = 100.0
        myLocationManager.desiredAccuracy = kCLLocationAccuracyHundredMeters
        
        myLocationManager.startUpdatingLocation()
 
        let status = CLLocationManager.authorizationStatus()
        
        if(status != CLAuthorizationStatus.authorizedWhenInUse){
            print("not determined")
            myLocationManager.requestWhenInUseAuthorization()            
        }
      
        pin(name: "String", address :"String", lati: 33.002184, long: 130.426536)
    }
    
    // GPSから値を取得した際に呼び出されるメソッド.
    func locationManager(_ manager: CLLocationManager, didUpdateLocations locations: [CLLocation]) {
        
        print("didUpdateLocations")
        
        // 配列から現在座標を取得.
        let myLocations: NSArray = locations as NSArray
        let myLastLocation: CLLocation = myLocations.lastObject as! CLLocation
        let myLocation:CLLocationCoordinate2D = myLastLocation.coordinate
        
        print("\(myLocation.latitude), \(myLocation.longitude)")
        
        
        // Regionを作成.
        let myRegion: MKCoordinateRegion = MKCoordinateRegionMakeWithDistance(myLocation, myLatDist, myLonDist);
        
        // MapViewに反映.
        myMapView.setRegion(myRegion, animated: true)
    }
    
    // Regionが変更された時に呼び出されるメソッド.
    func mapView(_ mapView: MKMapView, regionDidChangeAnimated animated: Bool) {
        print("regionDidChangeAnimated")
    }

    
    func onClickMyButton(sender: UIButton) {
        
        if (sender.tag == 4){
            self.performSegue(withIdentifier: "toSubTableVC_1Segue", sender: nil)
        }
    }
    
    
    
    func pin(name: String, address :String, lati: Double, long: Double) {
       
        // 経度、緯度.
        let myLatitude : CLLocationDegrees = (lati as CLLocationDegrees?)!
        let myLongitude : CLLocationDegrees = (long as CLLocationDegrees?)!
        
        let myPin = MKPointAnnotation()

        // 座標を設定.
        let myCoordinate: CLLocationCoordinate2D = CLLocationCoordinate2DMake(myLatitude, myLongitude)
        myPin.coordinate = myCoordinate

        // タイトルを設定.
        myPin.title = name
        
        // サブタイトルを設定.
        myPin.subtitle = address
        
        // MapViewにピンを追加
        self.myMapView.addAnnotation(myPin) //ここでエラー発生  
    }
    
    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }
}

//  SubTableViewCountroller.swift

import UIKit
import MapKit

class SubTableVC_1: UIViewController, UITableViewDelegate, UITableViewDataSource, MKMapViewDelegate {
    
    // ステータスバーの高さ
    let statusBarHeight = UIApplication.shared.statusBarFrame.height
    
    var senderRegion : String!
    
    var filename : Int!
    var filenameStr : String!
    
    let array1 = ["北海道"]
    
    var arrays = [""]
    
    var exitBtn : UIButton!
    
    override func viewDidLoad() {
        super.viewDidLoad()

        let tableView = UITableView()
        tableView.frame = CGRect(x: 0, y: statusBarHeight, width: self.view.frame.width, height: self.view.frame.height - statusBarHeight - 50)
        
        tableView.delegate = self
        tableView.dataSource = self
        
        self.view.addSubview(tableView)

        arrays = array1
        filename = 000
        
     }
    
    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        
    }
    
    
    // MARK: - UITableViewDataSource
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        // セルを作る
        let cell = UITableViewCell(style: .default, reuseIdentifier: "cell")
        cell.textLabel?.text = arrays[indexPath.row]
        return cell
    }
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        // セルの数を設定
        return arrays.count
    }
    
    // MARK: - UITableViewDelegate
    
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        // セルがタップされた時の処理
        csvToArray(index: indexPath.row)
    }
    
    func tableView(_ tableView: UITableView, heightForRowAt indexPath: IndexPath) -> CGFloat {
        // セルの高さを設定
        return 50
    }
    
    func tableView(_ tableView: UITableView, accessoryButtonTappedForRowWith indexPath: IndexPath) {
        // アクセサリボタン（セルの右にあるボタン）がタップされた時の処理
        print("タップされたアクセサリがあるセルのindex番号: \(indexPath.row)")
    }

    var csvArr : [String] = []
    
    func csvToArray (index: Int) {
        filenameStr = "assets/sh/" + (String(format: "%03d", filename + index))
        print(filenameStr)
        if let csvPath = Bundle.main.path(forResource: filenameStr, ofType: "csv") {
            do {
                let csvStr = try String(contentsOfFile:csvPath, encoding:String.Encoding.utf8)
                let csvChg = csvStr.replacingOccurrences(of: "\r", with: "\n")
                csvArr = csvChg.components(separatedBy: .newlines)
                
                var csvShrine : [String] = []
                var shrineName : String!
                var shrineAddress : String!
                var shrineLatitude : String!
                var shrineLongitude : String!
                
                for i in 0...(csvArr.count - 1) {
                    csvShrine = csvArr[i].components(separatedBy: ",")
                    
                    if (csvShrine[0] != ""){
                        
                        shrineName = csvShrine[2]
                        shrineAddress = csvShrine[1]
                        shrineLatitude = csvShrine[3]
                        shrineLongitude = csvShrine[4]
                        
                        print(("\(i / 2 + 1)") + ":" + (shrineName) + ":" + (shrineAddress) + ":" + (shrineLatitude) + ":" + (shrineLongitude))
                    
                        ViewController().pin(name: shrineName, address: shrineAddress, lati: atof(shrineLatitude), long: atof(shrineLongitude))
                    }
                }
            } catch let error as NSError {
                print(error.localizedDescription)
            }
        }
        self.performSegue(withIdentifier: "fromSubTableVC_1Segue", sender: nil)
    }
}

assets/sh/000.csv このデータに神社名と緯度・軽度が示されている

1,北海道野付郡別海町別海宮舞町23,別海神社社務所,43.395813,145.124554
2,北海道小樽市塩谷2丁目20－20,塩谷神社社務所,43.207892,140.924224
3,北海道岩見沢市2条東4丁目1－3,御嶽神社,43.206618,141.768348
4,北海道札幌市中央区南15条西5丁目1－1,札幌護国神社,43.040716,141.352921
5,北海道札幌市厚別区大谷地西二丁目2番1号,大谷地神社,43.026356,141.447487
6,北海道虻田郡洞爺湖町青葉町54番,虻田神社社務所,42.55366,140.762298
7,北海道檜山郡江差町字姥神町99-1,姥神大神宮,41.866344,140.12503
8,北海道檜山郡江差町字柏町216番,柏森神社,41.84801,140.126248
9,北海道函館市新八幡町30番,椴法華八幡神社,41.830463,141.14611
10,北海道稚内市大字宗谷村字宗谷99番の2,嚴島神社,45.485779,141.8836977

Answer 1

csvToArray(index: indexPath.row)
self.dismiss(animated: true, completion: nil)

でマップにピンが刺さった状態で、ViewControllerに戻れました。

Answer 2

ViewController().pin(...)

これでは、遷移元のViewControllerのpin()を呼んでいるのではなく、新たに生成したViewControllerのpin()を呼んでいることになります。

また、今回のエラーとは関係ありませんが、viewWillAppear(_:)でmapViewを生成しているので、SubTableVC_1でいくらピンを立てても、戻った時にはmapViewは初期化されてしまいますよ。

Comments

[harima]

おっしゃていることは、理解しました。ですが、pin(...)をどのように呼んだらいいのか、さっぱり分かりません。すみませんが、どのようにして、呼んだらいいのでしょうか？宜しくお願い致します。

[fuzzball]

delegateという言葉を調べれば何か分かるかも知れませんし、delegateでなくても、とにかくViewControllerのインスタンスを渡してやればいいです。

ちなみにナビゲーションコントローラは使っていないのでしょうか？
遷移の種類はShow?Modally?

[harima]

ナビゲーションコントローラーは使っておりません。
遷移の種類は、showです。

[fuzzball]

今回のケースでは、self.presentingViewController で遷移元のViewControllerを取得できると思います。

[harima]

self.presentingViewController.pin(....)でよろしいのですか？エラーになりました。

[fuzzball]

そのままではUIViewControllerと認識してしまうので、ViewControllerにキャストして下さい。

[harima]

(self.presentingViewController as! ViewController).pin(...)としました。エラーは出なかったのですが、ピンが立っていませんでした。viewWillAppear(_:)も削除しています。

[fuzzball]

SubTableVC_1に遷移せずに、まずはViewControllerの中でピンを立ててみて、ピンを立てるコードが間違っていないかどうか確認して下さい。修正したのであれば、質問内のコードも更新しておいて下さい。

あと、自分で何も調べずに質問を繰り返すことはやめましょうね。

[harima]

ViewControllerの中ではピンは立っています。毎度毎度すみません。自分で調べることもこれからはします。ありがとうございました。