5

好吧，在我正在创建的应用程序中，我希望有一个调度程序来保存有关传入作业的数据以及执行它们的剩余时间，因此我创建了一个非常幼稚的实现，我认为它远非应有的样子。

执行

接口

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();

问题）

这段代码会很好地工作还是我错过了所有的事情并且有更好的方法来做这样的事情？

要求

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

4

2 回答 2

2

好吧，从你的用法可以看出，在调度器中包含调度器的名称会使代码有点古怪……调度器真的需要知道他自己的名字吗？我会用 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();
            }
        }
    }
}
于 2013-07-20T17:29:59.027 回答
0

我看到您的抽象存在一些问题......您似乎抽象了队列，而不是 Job 类。这应该是相反的（或者至少你的生活会更轻松）。

public interface IJob
{
    MyJobType Type{ get; set; }
    string Name { get; set; }
    int Priority { get; set; }
    void RunJob();
}

  1. 您可以删除 IQueue，因为无论如何您都没有使用它。

  2. 仅使用一个队列，除非有#3 无法解决的充分理由

  3. 抽象你的工作类别——毕竟创建奶酪三明治的工作与处理会议的工作不同。使用上面的接口，您可以按类型（您创建的枚举）和优先级对作业进行排序，然后使用 IJob.RunJob 运行它们，而不必知道它们在每个实现中实际做了什么。

于 2013-07-20T16:53:31.003 回答