質問
ウィンドウにデータベースのレコードを表示し、数秒ごとにデータベースの新しいレコードをチェックするアプリケーションを作成しました。問題は、新しいレコードをチェックするたびにウィンドウが点滅し、修正したいことです。古いデータテーブルと新しいデータテーブルを比較し、それらが異なる場合にのみ更新することを試みました。 そのような場合のベストプラクティスは何ですか?私はそれを次のようにしようとしましたが、うまくいきません:
private bool GetBelongingMessages()
{
bool result = false;
DataTable dtTemp = OleDbWorks.GetBelongingMessages(currentCallID);
if(dtTemp != dtMessages)
{
dtMessages = dtTemp;
result = true;
}
else
{
result = false;
}
return result;
}
解決
まず、コードで比較しているのはデータテーブルのコンテンツではなく、データテーブルの参照であることを認識することが重要です。両方のデータテーブルのコンテンツが同じかどうかを判断するには、すべての行と列をループして、それらが等しいかどうかを確認する必要があります。
//This assumes the datatables have the same schema...
public bool DatatablesAreSame(DataTable t1, DataTable t2) {
if (t1.Rows.Count != t2.Rows.Count)
return false;
foreach (DataColumn dc in t1.Columns) {
for (int i = 0; i < t1.Rows.Count; i++) {
if (t1.Rows[i][dc.ColumnName] != t2.Rows[i][dc.ColumnName]) {
return false;
}
}
}
return true;
}
他のヒント
しばらくの間、DataTableの比較を行う方法を探していましたが、最終的には独自の関数を作成することになりました。
bool tablesAreIdentical = true;
// loop through first table
foreach (DataRow row in firstTable.Rows)
{
foundIdenticalRow = false;
// loop through tempTable to find an identical row
foreach (DataRow tempRow in tempTable.Rows)
{
allFieldsAreIdentical = true;
// compare fields, if any fields are different move on to next row in tempTable
for (int i = 0; i < row.ItemArray.Length && allFieldsAreIdentical; i++)
{
if (!row[i].Equals(tempRow[i]))
{
allFieldsAreIdentical = false;
}
}
// if an identical row is found, remove this row from tempTable
// (in case of duplicated row exist in firstTable, so tempTable needs
// to have the same number of duplicated rows to be considered equivalent)
// and move on to next row in firstTable
if (allFieldsAreIdentical)
{
tempTable.Rows.Remove(tempRow);
foundIdenticalRow = true;
break;
}
}
// if no identical row is found for current row in firstTable,
// the two tables are different
if (!foundIdenticalRow)
{
tablesAreIdentical = false;
break;
}
}
return tablesAreIdentical;
Dave Markleのソリューションと比較して、私の場合、同じレコードを持つが順序が異なる2つのテーブルを同一として扱います。これが、このスレッドに再びつまずく人の助けになることを願っています。
オブジェクトt1.Rows [i] [dc.ColumnName]およびt1.Rows [i] [dc.ColumnName]をキャストする必要があります。そうでなければ、ステートメントt1.Rows [i] [dc.ColumnName]!= t2.Rows [i] [dc.ColumnName]は常にtrueです。コードを次のように変更しました。
for(int i = 0; i < t1.Rows.Count; i++)
{
if((string)t1.Rows[i][1] != (string)t2.Rows[i][1])
return false;
}
それは動作しますが、エレガントなソリューションではありません。
public Boolean CompareDataTables(DataTable table1, DataTable table2)
{
bool flag = true;
DataRow[] row3 = table2.Select();
int i = 0;// row3.Length;
if (table1.Rows.Count == table2.Rows.Count)
{
foreach (DataRow row1 in table1.Rows)
{
if (!row1.ItemArray.SequenceEqual(row3[i].ItemArray))
{
flag = false;
break;
}
i++;
}
}
else
{
flag = false;
}
return flag;
}
//ここでは、この関数は結果としてブール値を返します。両方が同じ場合はtrueを返し、そうでない場合はfalseを返します