Миграция данных из одной таблицы в другую. Гибкое решение.

Буквально пару дней назад один мой знакомый, начинающий .NET программист показал мне код, который занимается переносом данных из одной базы в другую, причем паралельно делает преобразование типов:

copy to clipboardподсветка кода

1. string nsvid="";
2. string nsit="";
3. string type="";
4. string nzav="";
5. string ninv="";
6. string lim="";
7. string tochnost="";
8. string where="";
9. System.DateTime dtp = System.DateTime.Today;
10. Int64 pov=0;
11. System.DateTime dtn = System.DateTime.Today;
12. string texsos="";
14. for (Int64 i = 0; i <>
15. {
16. if (!bdt[int.Parse(i.ToString())].Is___п_пNull())
17. pp = long.Parse(bdt[int.Parse(i.ToString())].___п_п.ToString());
18. if (!bdt[int.Parse(i.ToString())].Is___свид_Null())
19. nsvid = bdt[int.Parse(i.ToString())].___свид_;
20. if (!bdt[int.Parse(i.ToString())].IsНаименование_СИТNull())
21. nsit = bdt[int.Parse(i.ToString())].Наименование_СИТ;
22. if (!bdt[int.Parse(i.ToString())].IsТипNull())
23. type = bdt[int.Parse(i.ToString())].Тип;
24. if (!bdt[int.Parse(i.ToString())].Is_Заводской__Null())
25. nzav = bdt[int.Parse(i.ToString())]._Заводской__;
26. if (!bdt[int.Parse(i.ToString())].Is_Инвентарный___Null())
27. ninv = bdt[int.Parse(i.ToString())]._Инвентарный___;
28. if (!bdt[int.Parse(i.ToString())].IsПредел_измеренийNull())
29. lim = bdt[int.Parse(i.ToString())].Предел_измерений;
30. if (!bdt[int.Parse(i.ToString())].Is_Класс__разряд__ц_д___погрешностьNull())
31. tochnost = bdt[int.Parse(i.ToString())]._Класс__разряд__ц_д___погрешность;
32. if (!bdt[int.Parse(i.ToString())].IsВладелецNull())
33. where = bdt[int.Parse(i.ToString())].Владелец;
34. if (!bdt[int.Parse(i.ToString())].IsДата_поверкиNull())
35. dtp = bdt[int.Parse(i.ToString())].Дата_поверки;
36. if (!bdt[int.Parse(i.ToString())].IsПериодичность_поверкиNull())
37. pov = long.Parse(bdt[int.Parse(i.ToString())].Периодичность_поверки.ToString());
38. if (!bdt[int.Parse(i.ToString())].IsДата_следующей_поверкиNull())
39. dtn = bdt[int.Parse(i.ToString())].Дата_следующей_поверки;
40. if (!bdt[int.Parse(i.ToString())].Is_Тех_состояниеNull())
41. texsos = bdt[int.Parse(i.ToString())]._Тех_состояние;
43. pdt.AddPriborRow(i+1, nsvid, nsit, type, nzav, ninv, lim, tochnost, where, dtp, pov, dtn, tochnost);
45. }
46. priborTableAdapter.Update(pdt);
47. pdt.AcceptChanges();

Достаточно беглого взгляда на этот код чтобы понять - реализация метода неоптимальна.

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

Поэтому, для иллюстрации я на коленке набросал более лаконичный подход, который можно повторно использовать.

1. Делаем универсальный конвертер данных:

copy to clipboardподсветка кода

1. public static class ConvertUtils
2. {
3. public static object ToType(object originalValue, Type targetType)
4. {
5. if (targetType == typeof(string))
6. {
7. return originalValue.ToString();
8. }
9. if (targetType == typeof(int))
10. {
11. return int.Parse(originalValue.ToString());
12. }
14. throw new Exception(
15. string.Format(
16. "Type {0} is not expected.",
17. originalValue.GetType().FullName));
18. }
19. }

Если есть необходимость можно расширить метод до ToType(object originalValue, Type originalType, Type targetType)

2. Заводим "словарик" для преобразования данных:

copy to clipboardподсветка кода

1. var migrateDictionary = new SortedDictionary<string, type="">()
2. {
3. { "___п_п", typeof(long) },
4. { "___свид_", typeof(string) },
5. { "Наименование_СИТ", typeof(string) },
6. };
7. 
   

Можно расширить словарик до такого вида:

Ключ = Имя исходного поля + его тип
Значение = Имя целевого поля + его тип

3. И непосредственно реализация метода по переносу данных. Ни зависит ни от чего, разве что от ADO.NET, а именно типов DataRow, DataTable, DataSet :)

copy to clipboardподсветка кода

1. // Объекты DataRow естественно реально существуют :)
2. DataRow sourceDataRow = null;
3. DataRow targetDataRow = null;
5. foreach (var migrateInfo in migrateDictionary)
6. {
7. var columnName = migrateInfo.Key;
8. var targetType = migrateInfo.Value;
10. var sourceValue = sourceDataRow[columnName];
12. targetDataRow[columnName] = ConvertUtils.ToType(sourceValue, targetType);
14. }

Итого задача сводится только к заполнению словарей, которые описывают алгоритм миграции. Т.е. по сути метод на вход ожидает предельно простые структуры данных, а реализация самого метода пишется один раз для миграции разных типов данных из разных таблиц.

Надеюсь что идея понятна. Не сочтите меня за велосипедостроителя. Я знаю что подобных примеров масса в интернете, и просто хочу донести эту мысль до тех, кто читает этот блог :)