Files
gf/os/gcron/gcron_schedule_check.go
houseme 7ffdff37e4 chore: upgrade golangci-lint configuration and optimize codebase (#4236)
This PR includes the following changes:

- **Upgrade `.golangci.yml`**: Updated the configuration file to align
with the latest golangci-lint version, ensuring compatibility and
leveraging new features.
- **Refactor GitHub Action workflow**: Modified `golangci-lint.yml` in
the GitHub Actions workflow to reflect the updated configuration and
improve CI performance.
- **Codebase optimization**: Refactored code to address issues and
warnings raised by the updated golangci-lint rules, including:
  - Improved function length and complexity.
  - Enhanced error handling and variable naming conventions.
- Fixed minor issues such as unused imports and formatting
inconsistencies.

These changes aim to maintain code quality, ensure compatibility with
the latest tools, and improve overall maintainability.
2025-08-22 13:29:09 +08:00

140 lines
3.6 KiB
Go

// Copyright GoFrame Author(https://goframe.org). All Rights Reserved.
//
// This Source Code Form is subject to the terms of the MIT License.
// If a copy of the MIT was not distributed with this file,
// You can obtain one at https://github.com/gogf/gf.
package gcron
import (
"context"
"time"
"github.com/gogf/gf/v2/os/gtime"
)
// checkMeetAndUpdateLastSeconds checks if the given time `t` meets the runnable point for the job.
// This function is called every second.
func (s *cronSchedule) checkMeetAndUpdateLastSeconds(ctx context.Context, currentTime time.Time) (ok bool) {
var (
lastCheckTimestamp = s.getAndUpdateLastCheckTimestamp(ctx, currentTime)
lastCheckTime = gtime.NewFromTimeStamp(lastCheckTimestamp)
lastMeetTime = gtime.NewFromTimeStamp(s.lastMeetTimestamp.Val())
)
defer func() {
if ok {
s.lastMeetTimestamp.Set(currentTime.Unix())
}
}()
return s.checkMinIntervalAndItemMapMeet(lastMeetTime.Time, lastCheckTime.Time, currentTime)
}
func (s *cronSchedule) checkMinIntervalAndItemMapMeet(
lastMeetTime, lastCheckTime, currentTime time.Time,
) (ok bool) {
if s.everySeconds != 0 {
// It checks using interval.
secondsAfterCreated := lastCheckTime.UnixNano()/1e9 - s.createTimestamp
if secondsAfterCreated > 0 {
return secondsAfterCreated%s.everySeconds == 0
}
return false
}
if !s.checkMeetSecond(lastMeetTime, currentTime) {
return false
}
if !s.checkMeetMinute(currentTime) {
return false
}
if !s.checkMeetHour(currentTime) {
return false
}
if !s.checkMeetDay(currentTime) {
return false
}
if !s.checkMeetMonth(currentTime) {
return false
}
if !s.checkMeetWeek(currentTime) {
return false
}
return true
}
func (s *cronSchedule) checkMeetSecond(lastMeetTime, currentTime time.Time) (ok bool) {
if s.ignoreSeconds {
if currentTime.Unix()-lastMeetTime.Unix() < 60 {
return false
}
} else {
// If this pattern is set in precise second time,
// it is not allowed executed in the same time.
if len(s.secondMap) == 1 && lastMeetTime.Format(time.RFC3339) == currentTime.Format(time.RFC3339) {
return false
}
if !s.keyMatch(s.secondMap, currentTime.Second()) {
return false
}
}
return true
}
func (s *cronSchedule) checkMeetMinute(currentTime time.Time) (ok bool) {
return s.keyMatch(s.minuteMap, currentTime.Minute())
}
func (s *cronSchedule) checkMeetHour(currentTime time.Time) (ok bool) {
return s.keyMatch(s.hourMap, currentTime.Hour())
}
func (s *cronSchedule) checkMeetDay(currentTime time.Time) (ok bool) {
return s.keyMatch(s.dayMap, currentTime.Day())
}
func (s *cronSchedule) checkMeetMonth(currentTime time.Time) (ok bool) {
return s.keyMatch(s.monthMap, int(currentTime.Month()))
}
func (s *cronSchedule) checkMeetWeek(currentTime time.Time) (ok bool) {
return s.keyMatch(s.weekMap, int(currentTime.Weekday()))
}
func (s *cronSchedule) keyMatch(m map[int]struct{}, key int) bool {
_, ok := m[key]
return ok
}
func (s *cronSchedule) checkItemMapMeet(lastMeetTime, currentTime time.Time) (ok bool) {
// second.
if s.ignoreSeconds {
if currentTime.Unix()-lastMeetTime.Unix() < 60 {
return false
}
} else {
if !s.keyMatch(s.secondMap, currentTime.Second()) {
return false
}
}
// minute.
if !s.keyMatch(s.minuteMap, currentTime.Minute()) {
return false
}
// hour.
if !s.keyMatch(s.hourMap, currentTime.Hour()) {
return false
}
// day.
if !s.keyMatch(s.dayMap, currentTime.Day()) {
return false
}
// month.
if !s.keyMatch(s.monthMap, int(currentTime.Month())) {
return false
}
// week.
if !s.keyMatch(s.weekMap, int(currentTime.Weekday())) {
return false
}
return true
}