Лучшие результаты, чтобы найти значение в DataTable? Для ? Linq? Другие?

Question

У меня есть большой файл txt, загруженный в DataTable в программу C#.

Мне нужно найти значения severasl в этой таблице данных.

На данный момент я использую простой цикл For, и он очень длинный! Мне действительно нужно выиграть время.

Есть ли лучший способ выполнить это? Использование Linq? или другой метод?

Вот основной пример моего кода:

foreach (DataRow row in DataTables[0].Rows) 
{ 
    for (int i = 0; i <= DataTables[1].Rows.Count - 1; i++) 
    { 

     if ((DataTables[1].Rows[i]["PRODUCT_CODE"].ToString().Trim() == row["PRODUCT_CODE"].ToString().Trim()) 
     { 
      // Do Some Stuff 
      // When the value is found, don't break the for...continue because there is severals "PRODUCT_CODE", not once. 
     } 
    } 
} 

Answer 1

короткий пример использования более в качестве одного ядра

Parallel.ForEach(dt.AsEnumerable(), row => 
{ 
    if (i["value1"].ToString() == "test") 
    { 
     Console.WriteLine(i["value1"]); 
    } 
}); 

Другое Solution

Сравнить ключи очень быстро

Dictionary<string, Product> file1 = new Dictionary<string, Product>(); 
Dictionary<string, Product> file2 = new Dictionary<string, Product>(); 

//Add ProductCode in key 

var product = new Product(); 
product.Code = "EAN1202"; 
product.Manufacturer = "Company"; 
product.Name = "Test"; 
product.Price = 12.05; 

file1.Add(product.Code, product); 

//One thread 
foreach (var item in file1) 
{ 
    if (file2.ContainsKey(item.Key)) 
    { 
     // Do Some Stuff 
    } 
} 

//Multi thread 
Parallel.ForEach(file1, item => 
{ 
    if (file2.ContainsKey(item.Key)) 
    { 
     // Do Some Stuff 
    } 
}); 

Продукт класса

public class Product 
{ 
    public string Code; 
    public string Manufacturer; 
    public string Name; 
    public double Price; 
} 

Answer 2

HashSet<string> dt0 = new HashSet<string>(); 
foreach (DataRow row in DataTables[0].Rows) 
    dt0.Add(row["PRODUCT_CODE"].ToString().Trim()); 
for (int i = 0; i <= DataTables[1].Rows.Count - 1; i++) 
{ 
    if (dt0.Contains(DataTables[1].Rows[i]["PRODUCT_CODE"].ToString().Trim() == row["PRODUCT_CODE"].ToString().Trim()) 
    { 
     // Do Some Stuff 
     // When the value is found, don't break the for...continue because there is severals "PRODUCT_CODE", not once. 
    } 
} 

Просто пошли из О (п^т) к О (п + т)

Если вам нужен весь ряд затем словарь, а затем HashSet

Dictionary<String, DataRow> dt0 = new Dictionary<String, DataRow>(); 

Вы должны использовать HashSet/Dictionary для большего.

Я бы дал вам больше, но у вас была наглость спросить меня, могу ли я подумать, что это будет быстрее.

Почему вы используете DataTables в первую очередь?

Answer 3

Это, вероятно, может быть немного лучше, если бы мы знали, что вы делали в цикле, но это должно работать:

var dt1=DataTables[0].Rows.AsEnumerable(); 
var dt2=DataTables[1].Rows.AsEnumerable(); 
var results=dt1.Join(
    dt2, 
    d1=>d1.Field<string>("PRODUCT_CODE").Trim(), 
    d2=>d2.Field<string>("PRODUCT_CODE").Trim(), 
    (d1,d2)=>new {d1,d2}); 
foreach(var row in results) 
{ 
    // Do stuff with row.d1/row.d2 
} 

Если, например, ваши данные таблицы создаются из источника SQL, это было бы лучше использовать соединение вместо этого, что позволит серверу SQL выполнять соединение, а не делать это на стороне клиента. Кроме того, использование не используемых данных и использование класса POCO не улучшат вашу производительность, так как вам не нужно будет вставлять/удалять код продукта во время соединения.