Рефакторинг

Bim4Everyone · Oct 28, 2024 · 12a7c9f · 12a7c9f
1 parent f1907cd
commit 12a7c9f
Show file tree

Hide file tree

Showing 8 changed files with 96 additions and 218 deletions.
diff --git a/src/RevitOpeningPlacement/Services/CableTrayOffsetFinder.cs b/src/RevitOpeningPlacement/Services/CableTrayOffsetFinder.cs
@@ -6,10 +6,9 @@
 using RevitClashDetective.Models.Extensions;
 
 using RevitOpeningPlacement.Models.Configs;
-using RevitOpeningPlacement.OpeningModels;
 
 namespace RevitOpeningPlacement.Services {
-    internal class CableTrayOffsetFinder : OutcomingTaskOffsetFinder<CableTray>, IOutcomingTaskOffsetFinder {
+    internal class CableTrayOffsetFinder : OutcomingTaskOffsetFinder<CableTray> {
         private MepCategory _category;
 
         public CableTrayOffsetFinder(
@@ -51,29 +50,5 @@ protected override Solid GetMepSolid(CableTray cableTray) {
                 .ToList();
             return ElementExtensions.UniteSolids(solids);
         }
-
-        public double FindHorizontalOffsetsSum(OpeningMepTaskOutcoming opening, Element mepElement) {
-            return base.FindHorizontalOffsetsSum(opening, mepElement as CableTray);
-        }
-
-        public double FindVerticalOffsetsSum(OpeningMepTaskOutcoming opening, Element mepElement) {
-            return base.FindHorizontalOffsetsSum(opening, mepElement as CableTray);
-        }
-
-        public double GetMinHorizontalOffsetSum(Element mepElement) {
-            return base.GetMinHorizontalOffsetSum(mepElement as CableTray);
-        }
-
-        public double GetMaxHorizontalOffsetSum(Element mepElement) {
-            return base.GetMaxHorizontalOffsetSum(mepElement as CableTray);
-        }
-
-        public double GetMinVerticalOffsetSum(Element mepElement) {
-            return base.GetMinVerticalOffsetSum(mepElement as CableTray);
-        }
-
-        public double GetMaxVerticalOffsetSum(Element mepElement) {
-            return base.GetMaxVerticalOffsetSum(mepElement as CableTray);
-        }
     }
 }
diff --git a/src/RevitOpeningPlacement/Services/ConduitOffsetFinder.cs b/src/RevitOpeningPlacement/Services/ConduitOffsetFinder.cs
@@ -4,10 +4,9 @@
 using RevitClashDetective.Models.Extensions;
 
 using RevitOpeningPlacement.Models.Configs;
-using RevitOpeningPlacement.OpeningModels;
 
 namespace RevitOpeningPlacement.Services {
-    internal class ConduitOffsetFinder : OutcomingTaskOffsetFinder<Conduit>, IOutcomingTaskOffsetFinder {
+    internal class ConduitOffsetFinder : OutcomingTaskOffsetFinder<Conduit> {
         private MepCategory _category;
 
         public ConduitOffsetFinder(
@@ -39,29 +38,5 @@ protected override MepCategory GetCategory(Conduit mepElement) {
         protected override Solid GetMepSolid(Conduit mepElement) {
             return mepElement.GetSolid();
         }
-
-        public double FindHorizontalOffsetsSum(OpeningMepTaskOutcoming opening, Element mepElement) {
-            return base.FindHorizontalOffsetsSum(opening, mepElement as Conduit);
-        }
-
-        public double FindVerticalOffsetsSum(OpeningMepTaskOutcoming opening, Element mepElement) {
-            return base.FindVerticalOffsetsSum(opening, mepElement as Conduit);
-        }
-
-        public double GetMinHorizontalOffsetSum(Element mepElement) {
-            return base.GetMinHorizontalOffsetSum(mepElement as Conduit);
-        }
-
-        public double GetMaxHorizontalOffsetSum(Element mepElement) {
-            return base.GetMaxHorizontalOffsetSum(mepElement as Conduit);
-        }
-
-        public double GetMinVerticalOffsetSum(Element mepElement) {
-            return base.GetMinVerticalOffsetSum(mepElement as Conduit);
-        }
-
-        public double GetMaxVerticalOffsetSum(Element mepElement) {
-            return base.GetMaxVerticalOffsetSum(mepElement as Conduit);
-        }
     }
 }
diff --git a/src/RevitOpeningPlacement/Services/DuctOffsetFinder.cs b/src/RevitOpeningPlacement/Services/DuctOffsetFinder.cs
@@ -4,10 +4,9 @@
 using RevitClashDetective.Models.Extensions;
 
 using RevitOpeningPlacement.Models.Configs;
-using RevitOpeningPlacement.OpeningModels;
 
 namespace RevitOpeningPlacement.Services {
-    internal class DuctOffsetFinder : OutcomingTaskOffsetFinder<Duct>, IOutcomingTaskOffsetFinder {
+    internal class DuctOffsetFinder : OutcomingTaskOffsetFinder<Duct> {
         private MepCategory _roundCategory;
         private MepCategory _rectangleCategory;
 
@@ -26,10 +25,12 @@ public DuctOffsetFinder(
 
         protected override MepCategory GetCategory(Duct mepElement) {
             if(mepElement.DuctType.Shape == ConnectorProfileType.Round) {
-                _roundCategory = _roundCategory ?? OpeningConfig.Categories[Models.MepCategoryEnum.RoundDuct];
+                _roundCategory = _roundCategory
+                    ?? OpeningConfig.Categories[Models.MepCategoryEnum.RoundDuct];
                 return _roundCategory;
             } else {
-                _rectangleCategory = _rectangleCategory ?? OpeningConfig.Categories[Models.MepCategoryEnum.RectangleDuct];
+                _rectangleCategory = _rectangleCategory
+                    ?? OpeningConfig.Categories[Models.MepCategoryEnum.RectangleDuct];
                 return _rectangleCategory;
             }
         }
@@ -53,29 +54,5 @@ protected override double GetWidth(Duct mepElement) {
         protected override Solid GetMepSolid(Duct mepElement) {
             return mepElement.GetSolid();
         }
-
-        public double FindHorizontalOffsetsSum(OpeningMepTaskOutcoming opening, Element mepElement) {
-            return base.FindHorizontalOffsetsSum(opening, mepElement as Duct);
-        }
-
-        public double FindVerticalOffsetsSum(OpeningMepTaskOutcoming opening, Element mepElement) {
-            return base.FindVerticalOffsetsSum(opening, mepElement as Duct);
-        }
-
-        public double GetMinHorizontalOffsetSum(Element mepElement) {
-            return base.GetMinHorizontalOffsetSum(mepElement as Duct);
-        }
-
-        public double GetMaxHorizontalOffsetSum(Element mepElement) {
-            return base.GetMaxHorizontalOffsetSum(mepElement as Duct);
-        }
-
-        public double GetMinVerticalOffsetSum(Element mepElement) {
-            return base.GetMinVerticalOffsetSum(mepElement as Duct);
-        }
-
-        public double GetMaxVerticalOffsetSum(Element mepElement) {
-            return base.GetMaxVerticalOffsetSum(mepElement as Duct);
-        }
     }
 }
diff --git a/src/RevitOpeningPlacement/Services/FamilyInstanceOffsetFinder.cs b/src/RevitOpeningPlacement/Services/FamilyInstanceOffsetFinder.cs
@@ -8,10 +8,9 @@
 using RevitClashDetective.Models.Extensions;
 
 using RevitOpeningPlacement.Models.Configs;
-using RevitOpeningPlacement.OpeningModels;
 
 namespace RevitOpeningPlacement.Services {
-    internal class FamilyInstanceOffsetFinder : OutcomingTaskOffsetFinder<FamilyInstance>, IOutcomingTaskOffsetFinder {
+    internal class FamilyInstanceOffsetFinder : OutcomingTaskOffsetFinder<FamilyInstance> {
         public FamilyInstanceOffsetFinder(
             OpeningConfig openingConfig,
             OutcomingTaskGeometryProvider geometryProvider,
@@ -61,29 +60,5 @@ protected override double GetWidth(FamilyInstance mepElement) {
         protected override Solid GetMepSolid(FamilyInstance mepElement) {
             return mepElement.GetSolid();
         }
-
-        public double FindHorizontalOffsetsSum(OpeningMepTaskOutcoming opening, Element mepElement) {
-            return base.FindHorizontalOffsetsSum(opening, mepElement as FamilyInstance);
-        }
-
-        public double FindVerticalOffsetsSum(OpeningMepTaskOutcoming opening, Element mepElement) {
-            return base.FindVerticalOffsetsSum(opening, mepElement as FamilyInstance);
-        }
-
-        public double GetMinHorizontalOffsetSum(Element mepElement) {
-            return base.GetMinHorizontalOffsetSum(mepElement as FamilyInstance);
-        }
-
-        public double GetMaxHorizontalOffsetSum(Element mepElement) {
-            return base.GetMaxHorizontalOffsetSum(mepElement as FamilyInstance);
-        }
-
-        public double GetMinVerticalOffsetSum(Element mepElement) {
-            return base.GetMinVerticalOffsetSum(mepElement as FamilyInstance);
-        }
-
-        public double GetMaxVerticalOffsetSum(Element mepElement) {
-            return base.GetMaxVerticalOffsetSum(mepElement as FamilyInstance);
-        }
     }
 }
diff --git a/src/RevitOpeningPlacement/Services/IOutcomingTaskOffsetFinder.T.cs b/src/RevitOpeningPlacement/Services/IOutcomingTaskOffsetFinder.T.cs
diff --git a/src/RevitOpeningPlacement/Services/IOutcomingTaskOffsetFinder.cs b/src/RevitOpeningPlacement/Services/IOutcomingTaskOffsetFinder.cs
@@ -1,11 +1,62 @@
 using Autodesk.Revit.DB;
 
+using RevitOpeningPlacement.OpeningModels;
+
 namespace RevitOpeningPlacement.Services {
     /// <summary>
     /// Интерфейс, предоставляющий методы для определения отступа от элемента ВИС из активного файла<br/>
     /// до граней исходящего задания на отверстие из активного файла.<br/>
     /// Ожидается, что элемент ВИС пересекается с заданием на отверстие, но не выходит за габариты этого задания.
     /// </summary>
-    internal interface IOutcomingTaskOffsetFinder : IOutcomingTaskOffsetFinder<Element> {
+    internal interface IOutcomingTaskOffsetFinder {
+        /// <summary>
+        /// Находит сумму горизонтальных отступов от элемента ВИС до граней задания на отверстие
+        /// </summary>
+        /// <param name="opening">Задание на отверстие из активного файла</param>
+        /// <param name="mepElement">Элемент ВИС из активного файла, проходящий через задание</param>
+        /// <returns>Значение суммы горизонтальных отступов в единицах Revit (футах) от элемента ВИС 
+        /// до граней задания на отверстие.</returns>
+        double FindHorizontalOffsetsSum(OpeningMepTaskOutcoming opening, Element mepElement);
+
+        /// <summary>
+        /// Находит сумму вертикальных отступов от элемента ВИС до граней задания на отверстие
+        /// </summary>
+        /// <param name="opening">Задание на отверстие из активного файла</param>
+        /// <param name="mepElement">Элемент ВИС из активного файла, проходящий через задание</param>
+        /// <returns>Значение суммы вертикальных отступов в единицах Revit (футах) от элемента ВИС 
+        /// до граней задания на отверстие.</returns>
+        double FindVerticalOffsetsSum(OpeningMepTaskOutcoming opening, Element mepElement);
+
+        /// <summary>
+        /// Определяет минимальное допустимое значение суммы горизонтальных отступов от элемента ВИС 
+        /// до граней задания на отверстие
+        /// </summary>
+        /// <param name="mepElement">Элемент ВИС из активного файла</param>
+        /// <returns>Минимальное допустимое значение суммы горизонтальных отступов в единицах Revit (футах)</returns>
+        double GetMinHorizontalOffsetSum(Element mepElement);
+
+        /// <summary>
+        /// Определяет максимальное допустимое значение суммы горизонтальных отступов от элемента ВИС 
+        /// до граней задания на отверстие
+        /// </summary>
+        /// <param name="mepElement">Элемент ВИС из активного файла</param>
+        /// <returns>Максимальное допустимое значение суммы горизонтальных отступов в единицах Revit (футах)</returns>
+        double GetMaxHorizontalOffsetSum(Element mepElement);
+
+        /// <summary>
+        /// Определяет минимальное допустимое значение суммы вертикальных отступов от элемента ВИС 
+        /// до граней задания на отверстие
+        /// </summary>
+        /// <param name="mepElement">Элемент ВИС из активного файла</param>
+        /// <returns>Минимальное допустимое значение суммы вертикальных отступов в единицах Revit (футах)</returns>
+        double GetMinVerticalOffsetSum(Element mepElement);
+
+        /// <summary>
+        /// Определяет максимальное допустимое значение суммы вертикальных отступов от элемента ВИС 
+        /// до граней задания на отверстие
+        /// </summary>
+        /// <param name="mepElement">Элемент ВИС из активного файла</param>
+        /// <returns>Максимальное допустимое значение суммы вертикальных отступов в единицах Revit (футах)</returns>
+        double GetMaxVerticalOffsetSum(Element mepElement);
     }
 }