refactoring

Author: Petr Sramek

Diff for: PolylineAlgorithm.slnx

@@ -5,11 +5,10 @@
   </Folder>
   <Folder Name="/samples/" />
   <Folder Name="/src/">
-    <Project Path="src/PolylineAlgorithm.Abstraction/PolylineAlgorithm.Abstraction.csproj" />
-    <Project Path="src/PolylineAlgorithm.IO.Pipelines/PolylineAlgorithm.IO.Pipelines.csproj" />
     <Project Path="src/PolylineAlgorithm/PolylineAlgorithm.csproj" />
   </Folder>
   <Folder Name="/tests/">
+    <File Path="tests/Directory.Build.props" />
     <Project Path="tests/PolylineAlgorithm.Tests/PolylineAlgorithm.Tests.csproj" />
   </Folder>
 </Solution>

Diff for: benchmarks/PolylineAlgorithm.Benchmarks/AsyncReadWriteBenchmark.cs

@@ -3,7 +3,7 @@
 
     public static class ValueProvider {
         private static readonly ConcurrentDictionary<int, CoordinatePair> _cache = new();
-        private static readonly PolylineEncoder _encoder = new();
+        private static readonly DefaultPolylineEncoder _encoder = new();
 
         public static IEnumerable<Coordinate> GetCoordinates(int count) {
             var entry = GetCaheEntry(count);

Diff for: benchmarks/PolylineAlgorithm.Benchmarks/Internal/CoordinateFactory.cs

@@ -9,7 +9,6 @@ namespace PolylineAlgorithm.Benchmarks;
 using BenchmarkDotNet.Engines;
 using PolylineAlgorithm;
 using PolylineAlgorithm.Benchmarks.Internal;
-using System.Text;
 
 /// <summary>
 /// Benchmarks for the <see cref="PolylineValue"/> struct.
@@ -18,7 +17,7 @@ namespace PolylineAlgorithm.Benchmarks;
 public class PolylineBenchmark {
     private static readonly Consumer consumer = new();
 
-    [Params(1, 10, 100, 1_000, 10_000, 100_000, 1_000_000)]
+    [Params(1, 10, 100, 250, 500, 1_000, 2_500, 5_000, 7_500, 10_000, 15_000, 20_000, 25_000, 50_000, 75_000, 100_000, 250_000, 500_000, 750_000, 1_000_000)]
     public int Length;
 
 #pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.

Diff for: benchmarks/PolylineAlgorithm.Benchmarks/Internal/ValueProvider.cs

@@ -8,9 +8,8 @@ namespace PolylineAlgorithm.Benchmarks;
 using BenchmarkDotNet.Attributes;
 using BenchmarkDotNet.Engines;
 using PolylineAlgorithm;
-using PolylineAlgorithm.Abstraction;
 using PolylineAlgorithm.Benchmarks.Internal;
-using System.Text;
+using PolylineAlgorithm.Extensions;
 
 /// <summary>
 /// Benchmarks for the <see cref="PolylineDecoder"/> class.
@@ -19,7 +18,7 @@ namespace PolylineAlgorithm.Benchmarks;
 public class PolylineDecoderBenchmark {
     private readonly Consumer _consumer = new();
 
-    [Params(1, 10, 100, 1_000, 10_000, 100_000, 1_000_000)]
+    [Params(1, 10, 100, 250, 500, 1_000, 2_500, 5_000, 7_500, 10_000, 15_000, 20_000, 25_000, 50_000, 75_000, 100_000, 250_000, 500_000, 750_000, 1_000_000)]
     public int N;
 
 #pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
@@ -43,7 +42,7 @@ public class PolylineDecoderBenchmark {
     /// <summary>
     /// The polyline decoder instance.
     /// </summary>
-    public PolylineDecoder<Coordinate> Decoder = new(new CoordinateFactory());
+    public DefaultPolylineDecoder Decoder = new();
 
     /// <summary>
     /// Sets up the data for the benchmarks.

Diff for: benchmarks/PolylineAlgorithm.Benchmarks/PolylineBenchmark.cs

@@ -16,7 +16,7 @@ namespace PolylineAlgorithm.Benchmarks;
 /// </summary>
 [RankColumn]
 public class PolylineEncoderBenchmark {
-    [Params(1, 10, 100, 1_000, 10_000, 100_000, 1_000_000)]
+    [Params(1, 10, 100, 250, 500, 1_000, 2_500, 5_000, 7_500, 10_000, 15_000, 20_000, 25_000, 50_000, 75_000, 100_000, 250_000, 500_000, 750_000, 1_000_000)]
     public int N;
 
 #pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
@@ -36,7 +36,7 @@ public class PolylineEncoderBenchmark {
     /// <summary>
     /// The polyline encoder instance.
     /// </summary>
-    public PolylineEncoder Encoder = new();
+    public DefaultPolylineEncoder Encoder = new();
 
     /// <summary>
     /// The async polyline encoder instance.

Diff for: benchmarks/PolylineAlgorithm.Benchmarks/PolylineDecoderBenchmark.cs

@@ -3,7 +3,7 @@
 
     public static class ValueProvider {
         private static readonly ConcurrentDictionary<int, CoordinatePair> _cache = new();
-        private static readonly PolylineEncoder _encoder = new();
+        private static readonly DefaultPolylineEncoder _encoder = new();
 
         public static IEnumerable<Coordinate> GetCoordinates(int count) {
             var entry = GetCaheEntry(count);

Diff for: benchmarks/PolylineAlgorithm.Benchmarks/PolylineEncoderBenchmark.cs

@@ -10,6 +10,7 @@ namespace PolylineAlgorithm.Comparison.Benchmarks;
 using global::PolylineEncoder.Net.Utility;
 using PolylineAlgorithm;
 using PolylineAlgorithm.Comparison.Benchmarks.Internal;
+using PolylineAlgorithm.Extensions;
 using PolylinerNet;
 using PolylineEncoding = Cloudikka.PolylineAlgorithm.Encoding.PolylineEncoding;
 
@@ -20,7 +21,7 @@ namespace PolylineAlgorithm.Comparison.Benchmarks;
 public class PolylineDecoderBenchmark {
     private readonly Consumer _consumer = new();
 
-    [Params(1, 10, 100, 1_000, 10_000, 100_000, 1_000_000)]
+    [Params(1, 10, 100, 250, 500, 1_000, 2_500, 5_000, 7_500, 10_000, 15_000, 20_000, 25_000, 50_000, 75_000, 100_000, 250_000, 500_000, 750_000, 1_000_000)]
     public int N;
 
 #pragma warning disable CS8618 // Non-nullable field must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring as nullable.
@@ -39,7 +40,7 @@ public class PolylineDecoderBenchmark {
     /// <summary>
     /// The polyline decoder instance.
     /// </summary>
-    public PolylineDecoder<Coordinate> PolylineAlgorithm = new(new CoordinateFactory());
+    public DefaultPolylineDecoder PolylineAlgorithm = new();
 
     public PolylineEncoding Cloudikka = new();
 

Diff for: benchmarks/PolylineAlgorithm.Comparison.Benchmarks/Internal/CoordinateFactory.cs

@@ -39,7 +39,7 @@ public class PolylineEncoderBenchmark {
     /// <summary>
     /// The polyline encoder instance.
     /// </summary>
-    public PolylineEncoder PolylineAlgorithm = new();
+    public DefaultPolylineEncoder PolylineAlgorithm = new();
 
     public PolylineEncoding Cloudikka = new();