创建任务计划程序

Question

好吧,在我正在创建的应用程序中,我希望有一个调度程序,其中包含有关传入作业的数据以及执行它们的剩余时间,因此我创建了一个非常天真的实现,我认为它远非应有的.

履行

接口

public interface IQueue
{
    string Name { get; set; }
    int Priority { get; set; }
}

类

public class TaskScheduler : Queue, IQueue
{
    public string Name { get; set; }
    public int Priority { get; set; }

    public TaskScheduler(string name, int priority)
    {
        Name = name;
        Priority = priority;
    }

    public void Add(string work, TimeSpan execution)
    {
        Enqueue(new Job {Work = work, Execution = execution});
    }

    public Job Get()
    {
        return (Job) Dequeue();
    }
}

public class Job
{
    public string Work { get; set; }
    public TimeSpan Execution { get; set; }

    public override string ToString()
    {
        return string.Format("{0} will be excuted in {1}", Work, Execution);
    }
}

用法

    var schedulerss = new List<TaskScheduler>
                         {
                             new TaskScheduler("Meetings", 2), 
                             new TaskScheduler("Jobs", 1)
                         };

    schedulerss = schedulerss.OrderBy(element => element.Priority).ToList(); //ORDER THE schedulers according to the Priority

    schedulerss.Find(schedulers => schedulers.Name == "Meetings").Add("Meet Barack Obama", new TimeSpan(1, 0, 0, 15));
    schedulerss.Find(schedulers => schedulers.Name == "Jobs").Add("Make a cheese sandwich :D", new TimeSpan(0, 2, 0, 15));

    var meetingschedulers = schedulerss.Find(schedulers => schedulers.Name == "Meetings");

    if (null != meetingschedulers)
    {
        foreach (var job in meetingschedulers)
        {
            Console.WriteLine(job);
        }
    }

    Console.Read();

问题(S)

这段代码是否运作良好,或者我只是错过了所有的东西,并且有更好的方法来做这样的事情？

请求

我想要一个非常详细的答案,关于我提供的代码的缺点是什么,以及如何创建更好的重用代码(仅适用于可能会发现此主题有用的其他人).

Answer 1

正如您从用法中看到的那样，在调度程序中包含调度程序的名称会使代码有点奇怪......调度程序真的需要知道自己的名字吗？我会用 aDictionary<string, TaskScheduler>()代替，并将他们的名字放在那里..

此外，您的队列始终返回 a Job，因此您应该使用通用队列，Queue<Job>

我做了一些修改

执行

接口

public interface IQueue
{
    int Priority { get; set; }
}

课程

public class TaskScheduler : Queue<Job>, IQueue
{
    public int Priority { get; set; }

    public TaskScheduler(int priority)
    {
        Priority = priority;
    }

    public void Add(string work, TimeSpan execution)
    {
        Enqueue(new Job { Work = work, Execution = execution });
    }

    public Job Get()
    {
        return Dequeue();
    }
}

public class Job
{
    public string Work { get; set; }
    public TimeSpan Execution { get; set; }

    public override string ToString()
    {
        return string.Format("{0} will be excuted in {1}", Work, Execution);
    }
}

用法

        var schedulers = new Dictionary<string, TaskScheduler>();

        schedulers.Add("Meetings", new TaskScheduler(2));
        schedulers.Add("Jobs", new TaskScheduler(1));

        schedulers["Meetings"].Add("Meet Barack Obama", new TimeSpan(1, 0, 0, 15));
        schedulers["Jobs"].Add("Make a cheese sandwich :D", new TimeSpan(0, 2, 0, 15));

        if (schedulers.ContainsKey("Meetings"))
        {
            foreach (var job in schedulers["Meetings"])
            {
                Console.WriteLine(job);
            }
        }

其他建议

由于您确实使用多个调度程序，因此我建议制作某种调度程序控制器，它将为您处理优先级以及您可能想要添加的其他内容。

根据请求编辑示例

我不知道你到底想用你的调度程序做什么，但我的意思是这样的，一个实用程序类：

public class TaskSchedulerController
{
    private Dictionary<string, TaskScheduler> _scedulers;

    public TaskSchedulerController()
    {
        _scedulers = new Dictionary<string, TaskScheduler>();
    }

    public void Add(string name, int priority)
    {
        _scedulers.Add(name, new TaskScheduler(priority));
    }

    public IEnumerable<string> GetJobsOfScheduler(string name)
    {
        if (_scedulers.ContainsKey(name))
        {
            foreach (var job in _scedulers[name])
            {
                yield return job.ToString();
            }
        }
    }
}