refactor: use new code in more places

Author: stefan-tb

CADability/BSpline.cs

@@ -1,4 +1,4 @@
-using CADability.Attribute;
+using CADability.Attribute;
 using CADability.Curve2D;
 using CADability.UserInterface;
 using System;
@@ -61,7 +61,7 @@ public class BSpline : IGeoObjectImpl, IColorDef, ILineWidth, ILinePattern, ISer
         private Plane? plane; // liegt in dieser Ebene, wenn spezifiziert
 #if NET_45
         private WeakReference<ExplicitPCurve3D> explicitPCurve3D;
-#else 
+#else
         private WeakReference explicitPCurve3D;
 #endif
         // NurbsHelper Daten
@@ -197,20 +197,20 @@ private void MakeNurbsHelper()
                 {
                     double dknot = knots[knots.Length - 1] - knots[0];
                     // letztlich ist es komisch, dass zwei knoten vornedran müssen
-                    //for (int i = 0; i < 1; ++i) // 
+                    //for (int i = 0; i < 1; ++i) //
                     //{
                     //    knotslist.Insert(0, knotslist[knotslist.Count - degree - i] - dknot);
                     //}
                     //for (int i = 0; i < 2 * degree - 2; ++i)
                     //{
                     //    knotslist.Add(knotslist[2 * (degree - 1) + i] + dknot);
                     //}
-                    // neue Idee: 
+                    // neue Idee:
                     // 1. es werden immer "degree" poles hinten angehängt
                     // 2. der 1. Knoten muss immer degree+1 mal vorkommen ( siehe STEP/piece0: dort sind alle Knoten 4-fach
                     //    bei degree=5 und FindSpan muss immer eine Stelle finden, an der es gerade wechselt
                     // 3. es werden soviele Knoten hinten angehängt, dass "knotslist.Length-degree-1 ==  poles.length" gilt
-                    //for (int i = 0; i < 1; ++i) // 
+                    //for (int i = 0; i < 1; ++i) //
                     //{
                     //    knotslist.Insert(0, knotslist[knotslist.Count - degree - i] - dknot);
                     //}
@@ -458,7 +458,7 @@ private void InsertInterpol(double sp, double ep, List<GeoPoint> pointList, List
                 //bool ok = false;
                 //if (sa >= 0 && sm >= 0) ok = sm <= sa;
                 //if (sa < 0 && sm < 0) ok = sm >= sa;
-                // Die Frage ist, wie groß ist der Fehler. Man kann nicht einfach in der Mitte testen, 
+                // Die Frage ist, wie groß ist der Fehler. Man kann nicht einfach in der Mitte testen,
                 // da der Verlauf der Kurve nicht bekannt ist, insbesondere ein Wendepunkt kann zu Fehlern
                 // führen. So testen wir hier jeweils noch die 1/4 und 3/4 Punkte um einigerm´ßen sicher
                 // zu gehen
@@ -708,7 +708,7 @@ public bool ThroughPoints(GeoPoint[] points, int maxDegree, bool closed, double[
         //				SetOcasBuddy(edg.GetGeneralEdge());
         //				FireDidChange(ce);
         //				return true;
-        //			} 
+        //			}
         //			catch (OpenCascade.Exception)
         //			{
         //				return false;
@@ -921,7 +921,7 @@ public bool SetData(int degree, GeoPoint[] poles, double[] weights, double[] kno
             //-	the Poles and Weights arrays have the same dimension and this dimension is greater than or equal to 2,
             //-	the Knots and Multiplicities arrays have the same dimension and this dimension is greater than or equal to 2,
             //-	the knots sequence is in ascending order, i.e. Knots(i) is less than Knots(i+1),
-            //-	the multiplicity coefficients are in the range 1 to Degree.  However, on a non-periodic curve, the first and last multiplicities 
+            //-	the multiplicity coefficients are in the range 1 to Degree.  However, on a non-periodic curve, the first and last multiplicities
             //  may be Degree + 1 (this is recommended if you want the curve to start and finish on the first and last poles),
             //-	on a periodic curve the first and last multiplicities must be the same,
             //-	on a non-periodic curve, the number of poles is equal to the sum of the multiplicity coefficients, minus Degree, minus 1,
@@ -1089,7 +1089,7 @@ public void SetThroughPoint(int Index, GeoPoint NewValue)
             //					throughPoints3d[Index] = NewValue;
             //					RecalcFromEdge(edg);
             //				}
-            //			} 
+            //			}
             //			catch (OpenCascade.Exception)
             //			{
             //			}
@@ -1235,17 +1235,17 @@ public bool IsClosed
                         if (!done)
                         {
                             if (multiplicities.Length > 1)
-                            {	// ob das so ganz allgemein gilt, muss noch überprüft werden, 
+                            {	// ob das so ganz allgemein gilt, muss noch überprüft werden,
                                 // die Bedinungen sind so:
-                                // 1.	the multiplicity coefficients are in the range 1 to Degree.  
-                                // However, on a non-periodic curve, the first and last multiplicities may be 
-                                // Degree + 1 (this is recommended if you want the curve to start 
+                                // 1.	the multiplicity coefficients are in the range 1 to Degree.
+                                // However, on a non-periodic curve, the first and last multiplicities may be
+                                // Degree + 1 (this is recommended if you want the curve to start
                                 // and finish on the first and last poles),
                                 // 2.	on a periodic curve the first and last multiplicities must be the same,
-                                // 3.	on a non-periodic curve, the number of poles is equal to the sum of the 
+                                // 3.	on a non-periodic curve, the number of poles is equal to the sum of the
                                 // multiplicity coefficients, minus Degree, minus 1,
-                                // 4.	on a periodic curve, the number of poles is equal to the sum of knot 
-                                // multiplicities, excluding the last knot.							
+                                // 4.	on a periodic curve, the number of poles is equal to the sum of knot
+                                // multiplicities, excluding the last knot.
                                 if (value)
                                 {	// es wird geschlossen
                                     // 1. Bedingung
@@ -1294,7 +1294,7 @@ public bool IsClosed
                                     // die 3. Bedingung muss erfüllt werden
                                     int sum = 0;
                                     for (int i = 0; i < multiplicities.Length; ++i) sum += multiplicities[i];
-                                    // 3.	on a non-periodic curve, the number of poles is equal to the sum of the 
+                                    // 3.	on a non-periodic curve, the number of poles is equal to the sum of the
                                     // multiplicity coefficients, minus Degree, minus 1,
                                     int dif = poles.Length - (sum - degree - 1);
                                     int j = 0; // zuerst die beiden äußeren erhöhen
@@ -3549,13 +3549,10 @@ bool ICurve.TryPointDeriv2At(double position, out GeoPoint point, out GeoVector
         private ColorDef colorDef;
         public ColorDef ColorDef
         {
-            get
-            {
-                return colorDef;
-            }
+            get => colorDef;
             set
             {
-                using (new ChangingAttribute(this, "ColorDef", colorDef))
+                using (ChangingAttribute.Create(this, colorDef))
                 {
                     colorDef = value;
                 }
@@ -3574,13 +3571,10 @@ void IColorDef.SetTopLevel(ColorDef newValue, bool overwriteChildNullColor)
         private LineWidth lineWidth;
         public LineWidth LineWidth
         {
-            get
-            {
-                return lineWidth;
-            }
+            get => lineWidth;
             set
             {
-                using (new ChangingAttribute(this, "LineWidth", lineWidth))
+                using (ChangingAttribute.Create(this, lineWidth))
                 {
                     lineWidth = value;
                 }
@@ -3591,13 +3585,10 @@ public LineWidth LineWidth
         private LinePattern linePattern;
         public LinePattern LinePattern
         {
-            get
-            {
-                return linePattern;
-            }
+            get => linePattern;
             set
             {
-                using (new ChangingAttribute(this, "LinePattern", linePattern))
+                using (ChangingAttribute.Create(this, linePattern))
                 {
                     linePattern = value;
                 }

CADability/BlockRef.cs

@@ -1,4 +1,4 @@
-using CADability.Actions;
+using CADability.Actions;
 using CADability.Attribute;
 using CADability.UserInterface;
 using System;
@@ -176,7 +176,7 @@ string IGeoObjectShowProperty.GetContextMenuId()
         #endregion
     }
     /// <summary>
-    /// 
+    ///
     /// </summary>
     [Serializable()]
     public class BlockRef : IGeoObjectImpl, IColorDef, IGeoObjectOwner
@@ -609,13 +609,10 @@ public override bool AttributeChanged(INamedAttribute attribute)
         private ColorDef colorDef;
         public ColorDef ColorDef
         {
-            get
-            {
-                return colorDef;
-            }
+            get => colorDef;
             set
             {
-                using (new ChangingAttribute(this, "ColorDef", colorDef))
+                using (ChangingAttribute.Create(this, colorDef))
                 {
                     colorDef = value;
                 }

CADability/Ellipse.cs

@@ -1,4 +1,4 @@
-using CADability.Attribute;
+using CADability.Attribute;
 using CADability.Curve2D;
 using CADability.UserInterface;
 using System;
@@ -109,7 +109,7 @@ public ICurve2D Curve2D
 	}
 
 	/// <summary>
-	/// 
+	///
 	/// </summary>
 	[Serializable]
 	public class Ellipse : IGeoObjectImpl, IColorDef, ILineWidth, ILinePattern, ICurve, ISerializable,
@@ -188,7 +188,7 @@ internal virtual EllipseData2D GetProjectionData(Projection pr)
 			// implementieren
 			//EllipseData2D res = (EllipseData2D)projectionData[pr];
 			//if (res==null)
-			//{ 
+			//{
 			//    res = CalculateProjectionData(pr.ProjectionPlane);
 			//    projectionData[pr] = res;
 			//}
@@ -306,7 +306,7 @@ internal virtual EllipseData2D CalculateProjectionData(Plane pr)
 					res.sweepAng = (double)new SweepAngle(StartDir.Angle, EndDir.Angle, ccw);
 					if (Math.Abs(res.sweepAng) < 1e-6 && Math.Abs(this.sweepParameter) > Math.PI)
 					{
-						// Sonderfall: ein fast Vollkreis wird nicht als solcher erkannt und liefert 
+						// Sonderfall: ein fast Vollkreis wird nicht als solcher erkannt und liefert
 						// allerdings exakt 0.0 für sweepAng
 						if (ccw) res.sweepAng = Math.Abs(this.sweepParameter);
 						else res.sweepAng = -Math.Abs(this.sweepParameter);
@@ -358,7 +358,7 @@ internal virtual EllipseData2D CalculateProjectionData(Plane pr)
 					res.sweepParameter = sw.Radian;
 					if (Math.Abs(res.sweepParameter) < 1e-6 && Math.Abs(res.sweepAng) > Math.PI)
 					{
-						// Sonderfall: ein fast Vollkreis wird nicht als solcher erkannt und liefert 
+						// Sonderfall: ein fast Vollkreis wird nicht als solcher erkannt und liefert
 						// allerdings exakt 0.0 für sweepAng
 						if (ccw) res.sweepParameter = Math.Abs(res.sweepAng);
 						else res.sweepParameter = -Math.Abs(res.sweepAng);
@@ -932,7 +932,7 @@ public void SetArcPlane2PointsRadiusLocation(Plane p, GeoPoint arcPoint1, GeoPoi
 							if (Geometry.Dist(p.ToGlobal(pl[0].p), locationP) <
 							    Geometry.Dist(p.ToGlobal(pl[1].p), locationP))
 							{
-								if (distLoc > dist12) // ausserhalb 
+								if (distLoc > dist12) // ausserhalb
 									centerIndex = 0;
 								else centerIndex = 1;
 							}
@@ -943,7 +943,7 @@ public void SetArcPlane2PointsRadiusLocation(Plane p, GeoPoint arcPoint1, GeoPoi
 								else centerIndex = 0;
 							}
 
-							// falls eine abweichende Lösong gewünscht ist (es gibt vier), 
+							// falls eine abweichende Lösong gewünscht ist (es gibt vier),
 							// hier für die selSol-Fälle 1 und drei umkehrung des Mittelpunktes
 							// if ((selSol % 2) != 0)
 							if ((selSol & 0x1) != 0)
@@ -961,7 +961,7 @@ public void SetArcPlane2PointsRadiusLocation(Plane p, GeoPoint arcPoint1, GeoPoi
 
 					this.startParameter = new Angle(this.plane.Project(arcPoint1), this.plane.Project(this.Center));
 					bool dir = (distLoc1 < 0.0);
-					// falls eine abweichende Lösung gewünscht ist (es gibt vier), 
+					// falls eine abweichende Lösung gewünscht ist (es gibt vier),
 					// hier für die selSol-Fälle zwei und drei umkehrung der Richtung
 					// if (((selSol % 4) == 2)||((selSol % 4) == 3))  dir = !dir;
 					if ((selSol & 0x2) != 0) dir = !dir;
@@ -1151,7 +1151,7 @@ public double TangentParameter(GeoVector Direction)
 			// dir ist jetzt im Einheitskreis, aber nicht unbedingt Länge 1, ist aber
 			// für Atan2 nicht wichtig
 			// der Atan2 liefert zu einer Senkrechten zu dir den Winkel, also
-			// den Parameter im Sinne der Ellipse. Der 2. Tangentenpunkt hat den 
+			// den Parameter im Sinne der Ellipse. Der 2. Tangentenpunkt hat den
 			// Parameter u+pi (bei Atan2 sind die Parameter vertauscht Atan2(y,x))
 			return Math.Atan2(dir.x, -dir.y);
 		}
@@ -1963,18 +1963,18 @@ internal Ellipse[] SplitAtZero()
 		private ColorDef colorDef;
 
 		public ColorDef ColorDef
-		{
-			get { return colorDef; }
-			set
-			{
-				using (new ChangingAttribute(this, "ColorDef", colorDef))
-				{
-					colorDef = value;
-				}
-			}
-		}
+        {
+            get => colorDef;
+            set
+            {
+                using (ChangingAttribute.Create(this, colorDef))
+                {
+                    colorDef = value;
+                }
+            }
+        }
 
-		void IColorDef.SetTopLevel(ColorDef newValue)
+        void IColorDef.SetTopLevel(ColorDef newValue)
 		{
 			colorDef = newValue;
 		}
@@ -1992,10 +1992,10 @@ void IColorDef.SetTopLevel(ColorDef newValue, bool overwriteChildNullColor)
 
 		public LineWidth LineWidth
 		{
-			get { return lineWidth; }
+			get => lineWidth;
 			set
 			{
-				using (new ChangingAttribute(this, "LineWidth", lineWidth))
+				using (ChangingAttribute.Create(this, lineWidth))
 				{
 					lineWidth = value;
 				}
@@ -2010,10 +2010,10 @@ public LineWidth LineWidth
 
 		public LinePattern LinePattern
 		{
-			get { return linePattern; }
+			get => linePattern;
 			set
 			{
-				using (new ChangingAttribute(this, "LinePattern", linePattern))
+				using (ChangingAttribute.Create(this, linePattern))
 				{
 					linePattern = value;
 				}
@@ -2044,7 +2044,7 @@ public double StartParameter
 		/// <summary>
 		/// Gets or sets the sweep amount of this arc. A full circle or ellipse must have a sweepparameter of
 		/// either 2.0*Math.PI or -2.0*Math.PI, The sweep parameter of circular or elliptical arcs are
-		/// in the range of -2.0*Math.PI &lt; SweepParameter &lt; 2.0*Math.PI. SweepParameter is often used 
+		/// in the range of -2.0*Math.PI &lt; SweepParameter &lt; 2.0*Math.PI. SweepParameter is often used
 		/// in connection with startParameter
 		/// </summary>
 		public double SweepParameter
@@ -2317,7 +2317,7 @@ GeoPoint ICurve.StartPoint
 					Plane pln = plane;
 					if (!Precision.IsPointOnPlane(value, this.plane))
 					{
-						// die Ebene bleibt nicht erhalten. neue Ebene gegeben durch 
+						// die Ebene bleibt nicht erhalten. neue Ebene gegeben durch
 						// Startpunkt, Endpunkt und Richtung senkrecht zum jetzigen Normalenvektor
 						GeoVector diry = (EndPoint - value) ^ plane.Normal;
 						if (Precision.IsNullVector(diry)) diry = plane.DirectionY;
@@ -2376,7 +2376,7 @@ GeoPoint ICurve.EndPoint
 						Plane pln = plane;
 						if (!Precision.IsPointOnPlane(value, this.plane))
 						{
-							// die Ebene bleibt nicht erhalten. neue Ebene gegeben durch 
+							// die Ebene bleibt nicht erhalten. neue Ebene gegeben durch
 							// Startpunkt, Endpunkt und Richtung senkrecht zum jetzigen Normalenvektor
 							GeoVector diry = (StartPoint - value) ^ plane.Normal;
 							if (Precision.IsNullVector(diry)) diry = plane.DirectionY;
@@ -2423,7 +2423,7 @@ GeoPoint ICurve.EndPoint
 		}
 
 		/// <summary>
-		/// Returns the Parameter of the given point projected into the plane of this ellipse (pp). 
+		/// Returns the Parameter of the given point projected into the plane of this ellipse (pp).
 		/// For a circle or arc this is the
 		/// radian of the angle of the point. For an ellipse this is the value (a), where
 		/// e.x = center.x+majorradius*cos(a), e.y = center.y+minorradius*cos(a) yields a point (e)