How do I pass params on timer selector

Question

  func locationManager(_ manager: CLLocationManager, didUpdateLocations locations: [CLLocation]) {
        guard let mostRecentLocation = locations.last else {
            return
        }

        print(mostRecentLocation.coordinate.latitude)
        print(mostRecentLocation.coordinate.longitude)        
        Timer.scheduledTimer(timeInterval: 60.0, target: self, selector: #selector(StartTestVC.sendDataToServer), userInfo: nil, repeats: true)
    }

    func sendDataToServer (latitude: Double, longitude: Double) {
        SFUserManager.shared.uploadPULocation(latitude, longitude:longitude)
    }

I want send data to server every 1 minute. I am using Timer.scheduledTimer and setting selector. But how could I send lat/lng params to my function?

@pmb you just have to get rid of the input arguments and use the class properties inside your function directly. See my answer below. — Dávid Pásztor, Jul 24 '17 at 16:14

Maulik Pandya · Accepted Answer · 2020-08-20T09:33:31.003

8

For sending the data with Timer you can use the userInfo parameter for pass the data.

Here is the sample by which you can get call of selector method and by that you can pass your location coordinate to it.

Timer.scheduledTimer(timeInterval: 0.5, target: self, selector:#selector(iGotCall(sender:)), userInfo: ["Name": "i am iOS guy"], repeats:true)

For handling that userInfo you need to go according to below.

func iGotCall(sender: Timer) {
    print((sender.userInfo)!)
}

for your case make sure your didUpdateLocations is called frequently.

edited Aug 20 '20 at 09:33

answered Jul 24 '17 at 16:24

Maulik Pandya

2,200
17
26

iGotCall called every one second even I set 1 sec. why? – pmb Jul 24 '17 at 16:57
for one minute the timeInterval should be 60 so it's timeInterval: 60 – Maulik Pandya Jul 24 '17 at 17:00
yeah I know I set it 60 but it delayong 60 seconds and then calling IGotCall every 1 second – pmb Jul 24 '17 at 17:01
are you calling iGotCall from anywhere else? – Maulik Pandya Jul 24 '17 at 17:05
no its not called – pmb Jul 24 '17 at 17:11
from where you are scheduling your timer in your question's code snippet? – Maulik Pandya Jul 24 '17 at 17:12
Let us [continue this discussion in chat](http://chat.stackoverflow.com/rooms/150012/discussion-between-pmb-and-maddy). – pmb Jul 24 '17 at 17:13

score 1 · Answer 2 · answered Jul 24 '17 at 16:13

One way to make sure your sendDataToServer is always uploading the latest coordinates without inputting the coordinates to the function as input arguments would be to store the values in a scope, which can be accessed by the function and use those values inside the function.

Assuming you make mostRecentLocation a class property, you can use below code

func locationManager(_ manager: CLLocationManager, didUpdateLocations locations: [CLLocation]) {
    guard let mostRecentLocation = locations.last else {
        return
    }   

    self.mostRecentLocation = mostRecentLocation
    Timer.scheduledTimer(timeInterval: 60.0, target: self, selector: #selector(StartTestVC.sendDataToServer), userInfo: nil, repeats: true)
}

func sendDataToServer() {
    SFUserManager.shared.uploadPULocation(self.mostRecentLocation.coordinate.latitude, longitude:self.mostRecentLocation.coordinate.longitude)
}

Did it work before the change? You didn't mention it not being called in your question. If it didn't work before either, is didUpdateLocations being called? — Dávid Pásztor, Jul 24 '17 at 16:30

score 0 · Answer 3 · answered Jul 24 '17 at 16:30

0

This is exactly what the userInfo parameter is meant to be used for:

struct SendDataToServerData { //TODO: give me a better name
    let lastLocation: CLLocation
    // Add other stuff if necessary
}

func locationManager(_ manager: CLLocationManager, didUpdateLocations locations: [CLLocation]) {
    guard let mostRecentLocation = locations.last else { return }

    print(mostRecentLocation.coordinate.latitude)
    print(mostRecentLocation.coordinate.longitude)

    Timer.scheduledTimer(
        timeInterval: 60.0,
        target: self,
        selector: #selector(StartTestVC.sendDataToServer(timer:)),
        userInfo: SendDataToServerData(mostRecentLocation: mostRecentLocation),
        repeats: true
    )
}

// Only to be called by the timer
func sendDataToServerTimerFunc(timer: Timer) {
    let mostRecentLocation = timer.userInfo as! SendDataToServerData
    self.sendDataToServer(
        latitude: mostRecentLocation.latitude
        longitude: mostRecentLocation.longitude
    )
}

// Call this function for all other uses
func sendDataToServer(latitude: Double, longitude: Double) {
    SFUserManager.shared.uploadPULocation(latitude, longitude:longitude)
}

answered Jul 24 '17 at 16:30

Alexander

59,041
12
98
151

sendDataToServer called every one second even I set 1 sec. why? – pmb Jul 24 '17 at 17:02
@pmb huh? did you make a typo? – Alexander Jul 24 '17 at 17:06
I copied your code did a little changes but sendDataToServer calling after 1 min but after that it started call every 1 second – pmb Jul 24 '17 at 17:12
@pmb Idk how you expect me to help you based on the little information you provided – Alexander Jul 24 '17 at 17:12
is your code working for you? I am using exactly your code, but it not working. I just copied your code, and fixed the errors – pmb Jul 24 '17 at 17:15
@pmb Of course not, I didn't recreate an entire project just to run this – Alexander Jul 24 '17 at 17:18
it was my mistake I called selector inside location update that's why it's not worked. Sorry man – pmb Jul 24 '17 at 17:29
Let us [continue this discussion in chat](http://chat.stackoverflow.com/rooms/150015/discussion-between-alexander-and-pmb). – Alexander Jul 24 '17 at 17:35

How do I pass params on timer selector

3 Answers3

Linked

Related