Удаление аннотации при перемещении ползунка — swift

#swift #mkmapview #uislider #mkannotation #mkpinannotationview

#swift #mkmapview #uislider #mkannotation #mkpinannotationview

Вопрос:

у меня проблема с аннотациями, которую я не могу решить. При нажатии на UIButton запускается функция @IBAction pressPlay, которая заставляет ползунок на моей карте начать двигаться. Ползунок имеет максимальное значение 0 и минимальное -31, а начальное значение равно 0, и он начинает двигаться, только если большой палец находится в нужном положении! = От 0 и перемещается каждые 1 секунду. Это работает правильно, перемещает ползунок.

 @IBAction func pressPlay(_ sender: Any)
    {
        let calendar2 = Calendar.current
        let today = Date()
        var cnt = Int(sliderTime.value)
        let play = UIImage(named: "play")
        let pause = UIImage(named: "pause")
        let format = DateFormatter()
        playButton.setImage(play, for: .normal)
        if control == true amp;amp; Int(sliderTime.value) < 0
        { //mette in play
            control = false
            playButton.setImage(pause, for: .normal)
            //removeSeismometers = true
            if Int(sliderTime.value) < 0
            {
                timer = Timer.scheduledTimer(withTimeInterval: 1,repeats: true)
                { [self]t in //ogni secondo questo timer cambia il valore dell'alpha del pin che sta vibrando
                    
                    if cnt < 0
                    {
                        cnt = Int(self.sliderTime.value)
                        self.sliderTime.value  = 1
                        let newDate2 = calendar2.date(byAdding: .day, value: Int(self.sliderTime.value), to:today)! //sottraggo alla data attuale il vlaore dello slider per tornare indietro nel tempo
                        format.dateStyle = .medium // "MM/GG/AAAA"
                        self.labelTime.text = "(format.string(from: newDate2))"
                        appo  = 1
                        for i in mapEqAnnotation{
                            let str: String = i.eq.eventTime
                            let index = str.index(str.endIndex, offsetBy: -9)
                            let mySubstring = str[..<index]
                            nuovaData = calendario.date(byAdding: .day, value: Int(sliderTime.value), to:dataCorrente)!
                            let format = DateFormatter()
                            format.dateFormat = "yyyy-MM-dd"
                            let dataCntr = "(format.string(from: nuovaData))"
                            if mySubstring == dataCntr{
                                printQuake(quake: i)
                            }else{
                                removeQuake(quake:i)
                            }
                        }
                        //printQuake(sliderValue: appo)
                    }else if cnt == 0{
                        //removeSeismometers = false
                        playButton.setImage(play, for: .normal)
                        timer!.invalidate()
                    }
                }
            }
        }else if control == false amp;amp; Int(sliderTime.value) < 0 {
            playButton.setImage(play, for: .normal)
            control = true
            timer!.invalidate()
        }
    }
  

Моя проблема в том, что каждую секунду ползунок должен перемещаться, когда вы нажимаете на UIButton, и каждую секунду нужно добавлять аннотацию к карте и удалять ее, как только вы снова перемещаете ползунок.
Все работает, за исключением того, что при прокрутке ползунка аннотации предыдущего перемещения не исчезают, а остаются на карте

     func printQuake(quake: MapEarthquakeAnnotation){
        let q = MapEarthquakeAnnotation(eq:quake.eq)
        mapView.addAnnotation(q)
    }

    func mapView(_ mapView: MKMapView, viewFor annotation: MKAnnotation) -> MKAnnotationView?
    {
        if annotation is MapEarthquakeAnnotation{
            annotazioni.append(annotation)
            let EQAnnotation = annotation as! MapEarthquakeAnnotation
            var view: MKPinAnnotationView
            view = MKPinAnnotationView(annotation: annotation, reuseIdentifier: EQAnnotation.identifier)
            view.canShowCallout = true
            view.pinTintColor = UIColor.brown
            return view
            
        }else if (annotation is MapSeismometerAnnotation) {
            if let annotation = annotation as? MapSeismometerAnnotation
            {
                var view: MKPinAnnotationView
                view = MKPinAnnotationView(annotation: annotation, reuseIdentifier: annotation.identifier)
                view.canShowCallout = true
                view.pinTintColor = UIColor.green
                view.image = UIImage(named: "pin-verde")
                return view
            }
            return nil
        }
        return nil
    }
  

Можете ли вы дать мне несколько советов?

Ответ №1:

Было бы полезно просмотреть код, который вы используете, removeQuake но после быстрого просмотра вашего кода очень вероятным кандидатом является код

 func printQuake(quake: MapEarthquakeAnnotation){
    let q = MapEarthquakeAnnotation(eq:quake.eq)
    mapView.addAnnotation(q)
}
  

Здесь вы создаете новую аннотацию каждый раз, когда вызываете этот метод. И эта аннотация нигде не сохраняется. Поэтому, когда вы вызываете removeQuake(quake:i) , вы не можете ожидать, что i это какая-либо из аннотаций, которые вы добавили с помощью printQuake .

Я не уверен, почему этот код был построен так, как он был, но возможно, все, что вам нужно сделать, это изменить этот метод на

 func printQuake(quake: MapEarthquakeAnnotation){
    mapView.addAnnotation(quake)
}
  

В целом ваш код немного сложен для чтения. Вам следует изучить более современные подходы с использованием Swift, и вы должны попытаться разделить части кода, чтобы их было легче читать и поддерживать. Неанглоязычные языки также не очень полезны.

Я попытался снести и восстановить ваш код. Не все есть, и я не уверен, что он делает то, что вы хотите. Но все же, пожалуйста, проверьте его. Возможно, это поможет вам устранить возникшую у вас проблему.

 import UIKit
import MapKit

class ViewController: UIViewController {
    
    @IBOutlet private var mapView: MKMapView!
    @IBOutlet private var sliderTime: UISlider!
    @IBOutlet private var playButton: UIButton!
    @IBOutlet private var labelTime: UILabel!

    private var timer: Timer?
    private var earthquakeAnnotations: [MapEarthquakeAnnotation] = []
    
    private var dayOffset: Int = -30 {
        didSet {
            refresh()
        }
    }
    
    private var isPlaying: Bool = false {
        didSet {
            playButton.setImage(isPlaying ? UIImage(named: "play") : UIImage(named: "pause"), for: .normal)
        }
    }
    
    private func refresh() {
        let beginningOfToday: Date = Calendar.autoupdatingCurrent.date(from: Calendar.autoupdatingCurrent.dateComponents([.year, .month, .day], from: Date()))!
        let presentedDay = Calendar.autoupdatingCurrent.date(byAdding: .day, value: dayOffset, to: beginningOfToday)!
        refreshSlider()
        refreshTimeLabel(date: presentedDay)
        refreshAnnotations(date: presentedDay)
    }
    
    private func refreshSlider() {
        sliderTime.value = Float(dayOffset)
    }
    
    private func refreshTimeLabel(date: Date) {
        labelTime.text = {
            let formatter = DateFormatter()
            formatter.dateStyle = .medium
            return formatter.string(from: date)
        }()
    }
    
    private func refreshAnnotations(date: Date) {
        earthquakeAnnotations.forEach { annotation in
            if annotation.eq.date == date {
                if !mapView.annotations.contains(where: { $0 === annotation }) {
                    mapView.addAnnotation(annotation)
                }
            } else {
                if mapView.annotations.contains(where: { $0 === annotation }) {
                    mapView.removeAnnotation(annotation)
                }
            }
        }
    }
    
    private func stopPlaying() {
        timer?.invalidate()
        timer = nil
        isPlaying = false
    }
    
    private func startPlaying() {
        if dayOffset < 0 {
            isPlaying = true
            timer?.invalidate() // Just in case
            timer = Timer.scheduledTimer(withTimeInterval: 1.0, repeats: true, block: { [weak self] timer in
                guard let self = self else {
                    timer.invalidate()
                    return
                }
                
                if self.dayOffset < 0 {
                    self.dayOffset  = 1
                } else {
                    self.stopPlaying()
                }
            })
        } else {
            // Already at the end. Should restart?
        }
    }
    
    @IBAction func pressPausePlay(_ sender: Any) {
        if isPlaying {
            stopPlaying()
        } else {
            startPlaying()
        }
    }
    
}

// MARK: - MKMapViewDelegate

extension ViewController: MKMapViewDelegate {
    
    func mapView(_ mapView: MKMapView, viewFor annotation: MKAnnotation) -> MKAnnotationView? {
        if let annotation = annotation as? MapEarthquakeAnnotation {
            let view: MKPinAnnotationView = MKPinAnnotationView(annotation: annotation, reuseIdentifier: annotation.identifier)
            view.canShowCallout = true
            view.pinTintColor = UIColor.brown
            return view
        } else if let annotation = annotation as? MapSeismometerAnnotation {
            let view: MKPinAnnotationView = MKPinAnnotationView(annotation: annotation, reuseIdentifier: annotation.identifier)
            view.canShowCallout = true
            view.pinTintColor = UIColor.green
            view.image = UIImage(named: "pin-verde")
            return view
        } else {
            return nil
        }
    }
    
}

// MARK: - MapEarthquakeAnnotation

private extension ViewController {
    
    class MapEarthquakeAnnotation: NSObject, MKAnnotation {
        var identifier: String { "MapEarthquakeAnnotationID" }
        
        let coordinate: CLLocationCoordinate2D
        let eq: Earthquake
        
        init(earthquake: Earthquake, coordinate: CLLocationCoordinate2D) { self.eq = earthquake; self.coordinate = coordinate }
    }
    
}

// MARK: - MapSeismometerAnnotation

private extension ViewController {
    
    class MapSeismometerAnnotation: NSObject, MKAnnotation {
        var identifier: String { "MapSeismometerAnnotationID" }
        
        let coordinate: CLLocationCoordinate2D
        
        init(coordinate: CLLocationCoordinate2D) { self.coordinate = coordinate }
    }
    
}

// MARK: - Earthquake

private extension ViewController {
    
    struct Earthquake {
        let eventTime: String
        
        var date: Date {
            let index = eventTime.index(eventTime.endIndex, offsetBy: -9)
            let format = DateFormatter()
            format.dateFormat = "yyyy-MM-dd"
            return format.date(from: String(eventTime[..<index])) ?? Date()
        }
    }
    
}