Рефакторинг двух шагов, чтобы избежать избыточности ⇐ C#
-
Anonymous
Рефакторинг двух шагов, чтобы избежать избыточности
У нас есть спецификация, в которой
[*]собрать 84 конфигурации [*]определить 167 связей между 84 конфигурациями [*]объединить все 84 конфигурации в одну общую конфигурацию
Моя проблема двоякая:
[*]Последняя часть кажется мне повторяющейся: мы используем 84 LOC при добавлении всех конфигураций в общую конфигурацию. [*]Когда мы добавляем новую конфигурацию, мы часто забываем о необходимости добавления новой конфигурации к общей.
Моя собственная мысль заключалась в том, чтобы объединить первый и третий этапы, но я не нашел способа сделать это, сохраняя при этом возможность ссылаться на конфигурации для второго шага.
Есть ли у вас предложения, как это улучшить?
Соответствующие части кода выглядят следующим образом:
var x = X.Build() вар у = Y.Build() вар z = Z.Build() // 84 раза x.AddRelation(y) x.AddRelation(z) // 167 раз вар конфигурация = новая конфигурация (...) { Сущности = { Икс, да, з, // 84 раза } }
У нас есть спецификация, в которой
[*]собрать 84 конфигурации [*]определить 167 связей между 84 конфигурациями [*]объединить все 84 конфигурации в одну общую конфигурацию
Моя проблема двоякая:
[*]Последняя часть кажется мне повторяющейся: мы используем 84 LOC при добавлении всех конфигураций в общую конфигурацию. [*]Когда мы добавляем новую конфигурацию, мы часто забываем о необходимости добавления новой конфигурации к общей.
Моя собственная мысль заключалась в том, чтобы объединить первый и третий этапы, но я не нашел способа сделать это, сохраняя при этом возможность ссылаться на конфигурации для второго шага.
Есть ли у вас предложения, как это улучшить?
Соответствующие части кода выглядят следующим образом:
var x = X.Build() вар у = Y.Build() вар z = Z.Build() // 84 раза x.AddRelation(y) x.AddRelation(z) // 167 раз вар конфигурация = новая конфигурация (...) { Сущности = { Икс, да, з, // 84 раза } }
Мобильная версия