IndexOutOfRangeException已投入串.split C＃

Question

我使用MySQL的.Net連接，但由於某種原因，我遇到解析結果返回他們問題的應用程序，

public NameValueCollection[] query(string query) 
{ 
    connect(); 
    NameValueCollection[] resultSet; 

    MySqlCommand command = connection.CreateCommand(); 
    MySqlDataReader Reader; 

    command.CommandText = query; 
    connection.Open(); 
    Reader = command.ExecuteReader(); 
    string theserows = ""; 
    while (Reader.Read()) 
    { 
     for (int i = 0; i < Reader.FieldCount; i++){ 
      theserows += Reader.GetName(i)+"="+Reader.GetValue(i).ToString() + ","; 
      Count = i; 
     } 
     theserows += "\n"; 
    } 
    connection.Close(); 
    string[] results = theserows.Split(new char[] {'\n'}, StringSplitOptions.RemoveEmptyEntries); 
    int countResultRows = 0; 
    resultSet = new NameValueCollection[Count]; 
    foreach (string s in results) 
    { 
     resultSet[countResultRows] = new NameValueCollection(); 
     string[] result = s.Split(new char[] {','}, StringSplitOptions.RemoveEmptyEntries); 
     foreach (string col in results) 
     { 
      if (string.IsNullOrEmpty(col)) 
       continue; 

      string[] kv = col.Split('='); 
      resultSet[countResultRows].Add(kv[0], kv[1]); 
     } 
     countResultRows++; 
    } 
    return resultSet; 
} 

在這種theserows = "site_id=1,\n"，但我除了string[] result = s.Split(',');有異常IndexOutOfRangeException

任何人都可以給出任何洞察，爲什麼會發生這種錯誤嗎？

在另一方面的原因我讀這一切，然後建設的NameValueCollection是我想補充一個記錄系統，它記錄了完整的查詢和它的業績

編輯::

「計數」（小寫）改爲countResultRows

調用堆棧

HTML5Streamer.exe!HTML5Streamer.Classes.MySQL.query(string query) Line 53 C# 
HTML5Streamer.exe!HTML5Streamer.Classes.Query.getSiteId(string domain) Line 17 + 0x10 bytes C# 
HTML5Streamer.exe!HTML5Streamer.Classes.Query.checkUserPass(string username, string password, string domain) Line 31 + 0xb bytes C# 
HTML5Streamer.exe!HTML5Streamer.Classes.Service.ProccessAdminRequest(System.Net.HttpListenerContext context) Line 239 + 0xd9 bytes C# 
HTML5Streamer.exe!HTML5Streamer.Classes.Service.ProcessRequest(System.Net.HttpListenerContext context) Line 176 + 0xb bytes C# 
HTML5Streamer.exe!HTML5Streamer.Classes.Service.ListenerCallback(System.IAsyncResult result) Line 150 + 0xb bytes C# 

http://i.stack.imgur.com/EST4w.png 52行是在65

Answer 1

2個不同的值（計數，計數），則不應使用相同的變量名，即使是語法正確的，這可能會造成混淆。

其次，你應該，檢查是否KV長度爲> 1訪問陣列

resultSet[count].Add(kv[0], kv[1]); 

如果我跟着你程序的邏輯之前給定的字符串「SITE_ID = 1，\ n」個，分割後， result = {「site_id = 1」，「\ n」}，對於第二個字符串kv = {「\ n」}，因此kv [1]將返回一個OutOfRangeException。

這是我會怎麼寫你的代碼

string[] results = theserows.Split(new char[]{'\n'},StringSplitOptions.RemoveEmptyEntries); 
resultSet = new NameValueCollection[Count]; 
for(int i = 0; i < results.Length && i < Count; i++) 
{ 
    string s = results[i]; 
    resultSet[i] = new NameValueCollection(); 
    string[] result = s.Split(new char[]{','},StringSplitOptions.RemoveEmptyEntries); 
    foreach (string col in result) // As pointed by other users, this should be result insead of results, again bad var name choice 
    { 
     string[] kv = col.Split('='); 
     if(kv.Length >= 2) 
     resultSet[i].Add(kv[0], kv[1]); 
    } 
} 

避免任何可能的OutOfRangeExceptions

Answer 2

只要你嘗試分割字符串第二環路之前在線路resultSet[countResultRows] = new NameValueCollection(); 功能開始26個端「SITE_ID = 1」，用逗號作爲一個分離器 - 的結果將是一個兩個字符串的數組與第二個是空的。只要改變你內心的foreach循環以下幾點：

foreach (string col in result) 
{ 
    if (string.IsNullOrEmpty(col)) 
     continue; 

    string[] kv = col.Split('='); 
    resultSet[count].Add(kv[0], kv[1]); 
} 

Answer 3

您有尾隨逗號和空格的問題。另外，當您通過result迭代時，您正在遍歷results。嘗試這個。

string[] results = theserows.Trim().Split(new string[] { "\n" }, StringSplitOptions.RemoveEmptyEntries); ; 
int count = 0; 
resultSet = new System.Collections.Specialized.NameValueCollection[results.Length]; 
foreach (string s in results) 
{ 
    resultSet[count] = new System.Collections.Specialized.NameValueCollection(); 
    string[] result = s.Trim().Split(new string[] { ","}, StringSplitOptions.RemoveEmptyEntries); 
    foreach (string col in result) 
    { 
     string[] kv = col.Split('='); 
     resultSet[count].Add(kv[0], kv[1]); 
    } 
    count++; 
}