how to run cron with multiple job and specified date

[mnurfilza]

i have code like this

for j, v := range day {
		d := strings.Split(j, ":")
		fmt.Println(v, d)
		sch.Every(1).Weekday(time.Thursday).At("17:59:00").Do(func() {
			// JobFunc()
			fmt.Println(sch.NextRun())
		})
		sch.Every(1).Weekday(time.Saturday).At("17:59:00").Do(func() {
			// JobFunc()
			fmt.Println(sch.NextRun())
		})
		sch.Every(1).Weekday(time.Monday).At("17:59:00").Do(func() {
			// JobFunc()
			fmt.Println(sch.NextRun())
		})

	}
	sch.StartAsync()

when the day come second and third jobs don't work, is there something wrong with my code?

[JohnRoesler]

Hey @mnurfilza a couple things:

1. You definitely shouldn't reference the scheduler inside of the Do() func as that's not safe with the Do func being executed in a separate goroutine. You're asking for concurrency problems with that.

2. I'd also recommend always checking the error when you create a job:

   job, err := sch.Every(1).Weekday(time.Thursday).At("17:59:00").Do(func() { fmt.Println("hello") })
   if err != nil {
       log.Fatal(err) // or something more useful
   }

3. I can't quite tell what you're doing with the loop, but I tried running something similar and it worked just fine:

   package main
   
   import (
       "fmt"
       "log"
       "time"
   
       "github.com/go-co-op/gocron"
   )
   
   func main() {
   
       sch := gocron.NewScheduler(time.Local)
   
       _, err := sch.Every(1).Weekday(time.Friday).At("10:46:00").Do(func() {
   	    // JobFunc()
   	    fmt.Println(sch.NextRun())
       })
       if err != nil {
   	    log.Fatal(err)
       }
       _, err = sch.Every(1).Weekday(time.Friday).At("10:46:00").Do(func() {
   	    // JobFunc()
   	    fmt.Println(sch.NextRun())
       })
       if err != nil {
   	    log.Fatal(err)
       }
       _, err = sch.Every(1).Weekday(time.Friday).At("10:46:00").Do(func() {
   	    // JobFunc()
   	    fmt.Println(sch.NextRun())
       })
       if err != nil {
   	    log.Fatal(err)
       }
       sch.StartAsync()
       select {}
   }