0

在我的 Asp.Net Web 服务中，我使用下面的 2 种方法来更改现有客户端的状态，形成一个名为ClientStatus的全局列表对象。此全局列表是从多个客户端修改的，但以安全的方式（锁定）。

private static List<ActiveClient> ClientStatus = new List<ActiveClient>();
public static void SetClinetStatus(string ClientID, int clinetstatus)
{
    ActiveClient activeClient=null;
    try
    {
            activeClient = GetClient(ClientID);
            if (activeClient != null)
            {
                activeClient.statuschanged = true;
                activeClient.status = clinetstatus;
            }
    }
    catch (Exception ex)
    {
        WebserviceLog.Debug(System.Reflection.MethodBase.GetCurrentMethod().DeclaringType + "::" + System.Reflection.MethodBase.GetCurrentMethod().ToString() + ":" + ex.Message);
    }

}

public static ActiveClient GetClient(string clientID)
{
    ActiveClient activeClient = null;
    try
    {
        lock (ClientStatus)
        {
            activeClient = ClientStatus.Find(c => c.clinetID == clientID);
        }
    }
    catch (Exception ex)
    {
        throw ex;
    }
    return activeClient;
}

我使用下面的代码将值传递给SetClinetStatus(string ClientID, int clinetstatus)方法

string errorData = Encoding.Default.GetString(data);
string[] tokens = errorData.Split(new string[] { ":" }, StringSplitOptions.RemoveEmptyEntries);
if (tokens.Length == 2)
    {                         
     SessionVariables.SetClinetStatus(tokens[0],Convert.ToInt32(tokens[1]));
    }

但有时（不是每次）我得到

你调用的对象是空的

形式 

activeClient = GetClient(ClientID);

我不明白为什么会这样，也没有看到任何问题。

有没有人看到任何导致这种异常的问题。

编辑

在全局列表中，我只通过下面的方法添加客户端，这里的 clientID 将来自直接的 webservice 方法。在另一端（来自客户端 ID 的地方），我添加了一个检查，以确保不为空或清空客户端 ID。

 public static void AddClient(string clientID)
        {
            try
            {
                lock (ClientStatus)
                {
                    ClientStatus.Add(new ActiveClient { clinetID = clientID });
                }

            }
            catch (Exception ex)
            {
                WebserviceLog.Debug(System.Reflection.MethodBase.GetCurrentMethod().DeclaringType + "::" + System.Reflection.MethodBase.GetCurrentMethod().ToString() + ":" + ex.Message);
            }

        }

并且ActiveClient类结构是

public class ActiveClient
    {
        public ActiveClient()
        {
            clinetID = string.Empty;
            status = 0;
            statuschanged = false;
        }
        public string clinetID { get; set; }
        public int status { get; set; }
        public bool statuschanged { get; set; } 
    }
4

1 回答 1

0

尝试

public static void SetClinetStatus(string ClientID, int clinetstatus)
{
    ClientID = ClientID.Trim();

    // Cannot run unless there is a ClientID submitted
    if(string.IsNullOrEmpty(ClientID))
    {
        // Log handling of event
        return;
    }

    ActiveClient activeClient=null;
    try
    {
        activeClient = GetClient(ClientID);
        if (activeClient != null)
        {
            activeClient.statuschanged = true;
            activeClient.status = clinetstatus;
        }
    }
    catch (Exception ex)
    {
        WebserviceLog.Debug(System.Reflection.MethodBase.GetCurrentMethod().DeclaringType + "::" + System.Reflection.MethodBase.GetCurrentMethod().ToString() + ":" + ex.Message);
    }

}

您还应该确保 clientID 在 ClientStatus 中

public static ActiveClient GetClient(string clientID)
{
    // Cannot continue without a ClientStatus
    if(ClientStatus == null) 
    {
        return null;
    }

    ActiveClient activeClient = null;
    try
    {
        lock (ClientStatus)
        {
            // Test if there are any matching elements
            if(ClientStatus.Any(c => c.clinetID == clientID))
            {
                activeClient = ClientStatus.Find(c => c.clinetID != null && c.clinetID == clientID);
            }
        }
    }
    catch (Exception ex)
    {
        throw ex;
    }

    // This will return null if there are no matching elements
    return activeClient;
}
于 2013-07-09T10:53:55.147 回答