修正cron解析错误
This commit is contained in:
parent
690feef066
commit
6e68a8ad8c
|
@ -48,7 +48,9 @@ public class CronExpression
|
|||
/// <para>取值可以用,分割,表示取值枚举,比如0 10,20,30 5 表示匹配每天早上5点10分、5点20分和5点30分。</para>
|
||||
/// <para>表达式右侧的*可以省略。比如0 15 4表示每天上午4点15分,和0 15 4 * * * *相同</para>
|
||||
/// <para>星期取值范围0-6,0表示星期天,1表示星期一,以此类推。</para>
|
||||
/// <para>日取值范围1-31,不用考虑大月小月和二月的特殊情况,方法会自动过滤这些特殊日期</para>
|
||||
/// <para>日取值范围1-31,不用考虑大月小月和二月的特殊情况,方法会自动处理这些特殊日期</para>
|
||||
/// <para>关于日期和星期匹配问题。一般的cron标准中日期和星期只要满足一个即可匹配,但本方法不同,我认为既然其他所有部分(比如分和秒)都是必须同时满足,那么日期和星期也是必须同时满足。</para>
|
||||
/// <para>目前方法不支持*-/,分隔符的混合使用,等待后续改进</para>
|
||||
/// </summary>
|
||||
/// <param name="cronExpression">cron表达式</param>
|
||||
/// <exception cref="ArgumentException"></exception>
|
||||
|
@ -57,7 +59,7 @@ public class CronExpression
|
|||
|
||||
Seconds = fields.Length > 0 ? GetRange(fields[0],0,59) : Enumerable.Range(0,60).ToList();
|
||||
Minutes = fields.Length > 1 ? GetRange(fields[1],0,59) : Enumerable.Range(0,60).ToList();
|
||||
Hours = fields.Length > 2 ? GetRange(fields[2],0,23) : Enumerable.Range(0,23).ToList();
|
||||
Hours = fields.Length > 2 ? GetRange(fields[2],0,23) : Enumerable.Range(0,24).ToList();
|
||||
DaysOfMonth = fields.Length > 3 ? GetRange(fields[3],1,31) : Enumerable.Range(1,31).ToList();
|
||||
Months = fields.Length > 4 ? GetRange(fields[4],1,12) : Enumerable.Range(1,12).ToList();
|
||||
DaysOfWeek = fields.Length > 5 ? GetRange(fields[5],0,6) : Enumerable.Range(0,7).ToList();
|
||||
|
@ -187,9 +189,14 @@ public class CronExpression
|
|||
|
||||
private static List<int> GetRange(string exp,int min,int max) {
|
||||
var charList = "0123456789*-/,".ToArray();
|
||||
if(exp.ToCharArray().Any(a => a.NotIn(charList))) {
|
||||
throw new ArgumentOutOfRangeException(nameof(exp));
|
||||
var expChars = exp.ToCharArray().Distinct();
|
||||
var outRangeChars = expChars.Where(a => a.NotIn(charList)).ToArray();
|
||||
if(outRangeChars.Any()) {
|
||||
throw new ArgumentOutOfRangeException(nameof(exp),$"非法字符{new string(outRangeChars)} 在表达式 {exp} 中");
|
||||
}
|
||||
//if(exp.ToCharArray().Any(a => a.NotIn(charList))) {
|
||||
// throw new ArgumentOutOfRangeException(nameof(exp));
|
||||
//}
|
||||
var list = new List<int>();
|
||||
if(exp == "*") {
|
||||
return Enumerable.Range(min,max - min + 1).ToList();
|
||||
|
@ -213,7 +220,7 @@ public class CronExpression
|
|||
var g = exp.Split('/',StringSplitOptions.RemoveEmptyEntries);
|
||||
var f = Math.Max(min,int.Parse(g[0]));
|
||||
var s = int.Parse(g[1]);
|
||||
while(f < max) {
|
||||
while(f <= max) {
|
||||
list.Add(f);
|
||||
f += s;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue
Block a user