0

任何人都可以帮我解决这个问题。有三台服务器。我想将作业分配给未启动作业最少的服务器。我们要考虑已开始的作业而不是已开始的作业。我已经做到了。但是我的代码很长。我不知道如何在不改变我的概念的情况下最小化我的代码。这是我的代码：

    public List<int> GetPaServer()
    {
        List<int> PaServers = new List<int>();
        using (PaEntities pa = new PaEntities())
        {
            var PaServer = from server in pa.AppPM_Pa_Server
                              where server.IsActive == true
                              select server.ServerId;
            foreach (var paServer in PaServer)
            {
                PaServers.Add(paServer);
            }
        }
        return PaServers;
    }


    //Method to get the serverid for each request.
    public int GetPaQueue()
    {
        using (PaEntities server = new PaEntities())
        {
            List<int> Paserver = new List<int>();
            Paserver = GetPaServer();
            string server1 = string.Empty;
            string server2 = string.Empty;
            string server3 = string.Empty;
            foreach (int paserver in Paserver)
            {
                if (paserver == 1)
                {
                    server1 = "active";
                }
                else if (paserver == 2)
                {
                    server2 = "active";
                }
                else if (paserver == 3)
                {
                    server3 = "active";
                }
            }
            int retVal = 0;

            // Get the Server NotStarted details here
            var NotStarted1 = (from serverID in server.AppPM_Pat
                               where serverID.Status == "NotStarted" && serverID.ServerId == 1
                               select serverID.ServerId).Count();
            var NotStarted2 = (from serverID in server.AppPM_Pat
                               where serverID.Status == "NotStarted" && serverID.ServerId == 2
                               select serverID.ServerId).Count();
            var NotStarted3 = (from serverID in server.AppPM_Pat
                               where serverID.Status == "NotStarted" && serverID.ServerId == 3
                               select serverID.ServerId).Count();

            // Get the  Server Started details here
            var server_Started1 = (from serverID in server.AppPM_Pat
                                   where serverID.Status == "Started" && serverID.ServerId == 1
                                   select serverID.ServerId).Count();
            var server_Started2 = (from serverID in server.AppPM_Pat
                                   where serverID.Status == "Started" && serverID.ServerId == 2
                                   select serverID.ServerId).Count();
            var server_Started3 = (from serverID in server.AppPM_Pat
                                   where serverID.Status == "Started" && serverID.ServerId == 3
                                   select serverID.ServerId).Count();

            //Get the server number for each  request
            //control comes here only when the server is active 
            if (server1 == "active" && server2 == "active" && server3 == "active")
            {
                if (NotStarted1 == 0 && NotStarted2 == 0 && NotStarted3 == 0)
                {
                    if ((server_Started1 > server_Started2) && (server_Started1 > server_Started3))
                    {
                        retVal = 2;
                    }
                    else if (server_Started1 == 0 && server_Started2 == 0 && server_Started3 == 0)
                    {
                        retVal = 1;
                    }
                    else if (server_Started1 == 1 && server_Started2 == 1 && server_Started3 == 1)
                    {
                        retVal = 1;
                    }
                    else if ((server_Started2 > server_Started1) && (server_Started2 > server_Started3))
                    {
                        retVal = 1;
                    }
                    else if ((server_Started3 > server_Started1) && (server_Started3 > server_Started2))
                    {
                        retVal = 1;
                    }
                    else if (server_Started1 == server_Started2 && server_Started3 == 0)
                    {
                        retVal = 3;
                    }
                    else if (server_Started1 == server_Started3 && server_Started2 == 0)
                    {
                        retVal = 2;
                    }
                    else if (server_Started2 == server_Started3 && server_Started1 == 0)
                    {
                        retVal = 1;
                    }
                }
                //control comes here only when the third server is active after some time
                else if (NotStarted1 == NotStarted2 && NotStarted3 == 0)
                {
                    if (server_Started1 == 1 & server_Started2 == 1 & server_Started3 == 0)
                    {
                        retVal = 3;
                    }
                    else if (server_Started1 == 1 & server_Started2 == 1 & server_Started3 == 1)
                    {
                        retVal = 3;
                    }
                }
                else if ((NotStarted1 > NotStarted2) && NotStarted3 == 0)
                {
                    if (server_Started1 == 1 & server_Started2 == 1 & server_Started3 == 0)
                    {
                        retVal = 3;
                    }
                    else if (server_Started1 == 1 & server_Started2 == 1 & server_Started3 == 1)
                    {
                        retVal = 3;
                    }
                }
                else if ((NotStarted2 > NotStarted1) && NotStarted3 == 0)
                {
                    if (server_Started1 == 1 & server_Started2 == 1 & server_Started3 == 0)
                    {
                        retVal = 3;
                    }
                    else if (server_Started1 == 1 & server_Started2 == 1 & server_Started3 == 1)
                    {
                        retVal = 3;
                    }
                }
                //control comes here only when the first server is active after some time
                else if (NotStarted2 == NotStarted3 && NotStarted1 == 0)
                {
                    if (server_Started3 == 1 & server_Started2 == 1 & server_Started1 == 0)
                    {
                        retVal = 1;
                    }
                    else if (server_Started1 == 1 & server_Started2 == 1 & server_Started3 == 1)
                    {
                        retVal = 1;
                    }
                }
                else if ((NotStarted2 > NotStarted3) && NotStarted1 == 0)
                {
                    if (server_Started3 == 1 & server_Started2 == 1 & server_Started1 == 0)
                    {
                        retVal = 1;
                    }
                    else if (server_Started1 == 1 & server_Started2 == 1 & server_Started3 == 1)
                    {
                        retVal = 1;
                    }
                }
                else if ((NotStarted3 > NotStarted2) && NotStarted1 == 0)
                {
                    if (server_Started3 == 1 & server_Started2 == 1 & server_Started1 == 0)
                    {
                        retVal = 1;
                    }
                    else if (server_Started1 == 1 & server_Started2 == 1 & server_Started3 == 1)
                    {
                        retVal = 1;
                    }
                }
                //control comes here only when the second server is active after some time
                else if (NotStarted3 == NotStarted1 && NotStarted2 == 0)
                {
                    if (server_Started3 == 1 & server_Started1 == 1 & server_Started2 == 0)
                    {
                        retVal = 2;
                    }
                    else if (server_Started1 == 1 & server_Started2 == 1 & server_Started3 == 1)
                    {
                        retVal = 2;
                    }
                }
                else if ((NotStarted3 > NotStarted1) && NotStarted2 == 0)
                {
                    if (server_Started3 == 1 & server_Started1 == 1 & server_Started2 == 0)
                    {
                        retVal = 2;
                    }
                    else if (server_Started1 == 1 & server_Started2 == 1 & server_Started3 == 1)
                    {
                        retVal = 2;
                    }
                }
                else if ((NotStarted1 > NotStarted3) && NotStarted2 == 0)
                {
                    if (server_Started3 == 1 & server_Started1 == 1 & server_Started2 == 0)
                    {
                        retVal = 2;
                    }
                    else if (server_Started1 == 1 & server_Started2 == 1 & server_Started3 == 1)
                    {
                        retVal = 2;
                    }
                }
                else if (NotStarted1 == 1 && NotStarted2 == 0 && NotStarted3 == 0)
                {
                    if ((server_Started1 > server_Started2) && (server_Started1 > server_Started3))
                    {
                        retVal = 2;
                    }
                    else if (server_Started1 == 0 && server_Started2 == 0 && server_Started3 == 0)
                    {
                        retVal = 1;
                    }
                    else if (server_Started1 == 1 && server_Started2 == 1 && server_Started3 == 1)
                    {
                        retVal = 1;
                    }
                    else if ((server_Started2 > server_Started1) && (server_Started2 > server_Started3))
                    {
                        retVal = 1;
                    }
                    else if ((server_Started3 > server_Started1) && (server_Started3 > server_Started2))
                    {
                        retVal = 1;
                    }
                    else if (server_Started1 == server_Started2 && server_Started3 == 0)
                    {
                        retVal = 3;
                    }
                    else if (server_Started1 == server_Started3 && server_Started2 == 0)
                    {
                        retVal = 2;
                    }
                    else if (server_Started2 == server_Started3 && server_Started1 == 0)
                    {
                        retVal = 1;
                    }
                }
                else if (NotStarted1 == 1 && NotStarted2 == 1 && NotStarted3 == 0)
                {
                    if ((server_Started1 > server_Started2) && (server_Started1 > server_Started3))
                    {
                        retVal = 2;
                    }
                    else if (server_Started1 == 0 && server_Started2 == 0 && server_Started3 == 0)
                    {
                        retVal = 1;
                    }
                    else if (server_Started1 == 1 && server_Started2 == 1 && server_Started3 == 1)
                    {
                        retVal = 1;
                    }
                    else if ((server_Started2 > server_Started1) && (server_Started2 > server_Started3))
                    {
                        retVal = 1;
                    }
                    else if ((server_Started3 > server_Started1) && (server_Started3 > server_Started2))
                    {
                        retVal = 1;
                    }
                    else if (server_Started1 == server_Started2 && server_Started3 == 0)
                    {
                        retVal = 3;
                    }
                    else if (server_Started1 == server_Started3 && server_Started2 == 0)
                    {
                        retVal = 2;
                    }
                    else if (server_Started2 == server_Started3 && server_Started1 == 0)
                    {
                        retVal = 1;
                    }
                }
                else if (NotStarted1 > NotStarted2 && NotStarted1 > NotStarted3)
                {
                    if (server_Started1 == 1 && server_Started2 == 1 && server_Started3 == 1)
                    {
                        if (NotStarted2 > NotStarted3)
                        {
                            retVal = 3;
                        }
                        else
                        {
                            retVal = 2;
                        }
                    }
                }
                else if (NotStarted2 > NotStarted3 && NotStarted2 > NotStarted1)
                {
                    if (server_Started1 == 1 && server_Started2 == 1 && server_Started3 == 1)
                    {
                        if (NotStarted1 > NotStarted3)
                        {
                            retVal = 3;
                        }
                        else
                        {
                            retVal = 1;
                        }
                    }
                }
                else if (NotStarted3 > NotStarted1 && NotStarted3 > NotStarted2)
                {
                    if (server_Started1 == 1 && server_Started2 == 1 && server_Started3 == 1)
                    {
                        if (NotStarted1 > NotStarted2)
                        {
                            retVal = 2;
                        }
                        else
                        {
                            retVal = 1;
                        }
                    }
                }
                else if (NotStarted1 == NotStarted2 && NotStarted2 == NotStarted3 && NotStarted1 == NotStarted3)
                {
                    if (server_Started1 == 1 && server_Started2 == 1 && server_Started3 == 1)
                    {
                        retVal = 1;
                    }
                }
                else if (NotStarted1 == NotStarted2 && NotStarted1 > NotStarted3 && NotStarted2 > NotStarted3)
                {
                    if (server_Started1 == 1 && server_Started2 == 1 && server_Started3 == 1)
                    {
                        retVal = 3;
                    }
                }
                else if (NotStarted2 == NotStarted3 && NotStarted2 > NotStarted1 && NotStarted3 > NotStarted1)
                {
                    if (server_Started1 == 1 && server_Started2 == 1 && server_Started3 == 1)
                    {
                        retVal = 1;
                    }
                }
                else if (NotStarted1 == NotStarted3 && NotStarted1 > NotStarted2 && NotStarted3 > NotStarted2)
                {
                    if (server_Started1 == 1 && server_Started2 == 1 && server_Started3 == 1)
                    {
                        retVal = 2;
                    }
                }                    
            }
            //control comes here only when server1 and server2 is active
            else if (server1 == "active" && server2 == "active")
            {
                if (NotStarted1 > NotStarted2)
                {
                    if (server_Started1 == 1 && server_Started2 == 1)
                    {
                        retVal = 2;
                    }
                }
                else if (NotStarted1 == NotStarted2)
                {
                    if (server_Started1 == 0 && server_Started2 == 0)
                    {
                        retVal = 1;
                    }
                    else if (server_Started1 == 1 && server_Started2 == 0)
                    {
                        retVal = 2;
                    }
                    else if (server_Started1 == 0 && server_Started2 == 1)
                    {
                        retVal = 1;
                    }
                    else if (server_Started1 == 1 && server_Started2 == 1)
                    {
                        retVal = 1;
                    }
                }
                else
                {
                    if (server_Started1 == 1 && server_Started2 == 1)
                    {
                        retVal = 1;
                    }
                }
            }
            //control comes here only when server3 and server2 is active
            else if (server2 == "active" && server3 == "active")
            {
                if (NotStarted2 > NotStarted3)
                {
                    if (server_Started2 == 1 && server_Started3 == 1)
                    {
                        retVal = 3;
                    }
                }
                else if (NotStarted2 == NotStarted3)
                {
                    if (server_Started2 == 0 && server_Started3 == 0)
                    {
                        retVal = 2;
                    }
                    else if (server_Started2 == 1 && server_Started3 == 0)
                    {
                        retVal = 3;
                    }
                    else if (server_Started2 == 0 && server_Started3 == 1)
                    {
                        retVal = 2;
                    }
                    else if (server_Started2 == 1 && server_Started3 == 1)
                    {
                        retVal = 2;
                    }
                }
                else
                {
                    if (server_Started2 == 1 && server_Started3 == 1)
                    {
                        retVal = 2;
                    }
                }
            }
            //control comes here only when server1 and server3 is active
            else if (server1 == "active" && server3 == "active")
            {
                if (NotStarted1 > NotStarted3)
                {
                    if (server_Started1 == 1 && server_Started3 == 1)
                    {
                        retVal = 3;
                    }
                }
                else if (NotStarted1 == NotStarted3)
                {
                    if (server_Started1 == 0 && server_Started3 == 0)
                    {
                        retVal = 1;
                    }
                    else if (server_Started1 == 1 && server_Started3 == 0)
                    {
                        retVal = 3;
                    }
                    else if (server_Started1 == 0 && server_Started3 == 1)
                    {
                        retVal = 1;
                    }
                    else if (server_Started1 == 1 && server_Started3 == 1)
                    {
                        retVal = 1;
                    }
                }
                else
                {
                    if (server_Started1 == 1 && server_Started3 == 1)
                    {
                        retVal = 1;
                    }
                }
            }
            return retVal;
        }
    }
4

1 回答 1

2

使用数组来保存服务器，然后循环遍历数组，而不是为每个服务器重复相同的代码。在循环中，您有一个服务器及其索引，因此您可以在复制代码时仅更改一个数字和服务器。冲洗并重复 NotStarted 和您为每个服务器计算的其他值。

此外，您的代码中似乎有一个展开的排序算法。如果您将服务器及其 NotStarted 和其他计算值组合到一个结构中，您可以只使用一个结构值数组，然后使用自定义排序函数对其进行排序。或者使用 LINQ 表达式。我离我的开发电脑还有七个小时的路程，也许今晚我可以一起破解一些东西......

于 2012-09-27T10:53:33.280 回答