質問

ウィンドウにデータベースのレコードを表示し、数秒ごとにデータベースの新しいレコードをチェックするアプリケーションを作成しました。問題は、新しいレコードをチェックするたびにウィンドウが点滅し、修正したいことです。古いデータテーブルと新しいデータテーブルを比較し、それらが異なる場合にのみ更新することを試みました。 そのような場合のベストプラクティスは何ですか?私はそれを次のようにしようとしましたが、うまくいきません:

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を返します

ライセンス: CC-BY-SA帰属
所属していません StackOverflow
scroll top