aj/OpenNest
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

refactor(jobs): single-source strategy resolution in adapter

Browse Source 
LegacyPlateNesterAdapter.Create now delegates to PlateNesterFactory
instead of carrying its own minimal Default-only switch, so built-in
strategy resolution has one source of truth. Behavior unchanged:
unknown keys still reject; all four built-ins now resolvable.

44 net8.0 tests pass in Debug and Release; 0 warnings.
This commit is contained in:
aj
2026-09-17 16:26:45 -04:00
parent 5b88d85937
commit 67f5fb8eca
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
OpenNest.Engine/Jobs/Adapters/LegacyPlateNesterAdapter.cs
+4 -4
View File
@@ -18,10 +18,10 @@ public sealed class LegacyPlateNesterAdapter : IPlateNester
this.engineFactory = engineFactory; this.engineFactory = engineFactory;
} }
/// <summary>Minimal built-in selection; never reads or changes NestEngineRegistry.</summary> /// <summary>Convenience overload delegating to <see cref="PlateNesterFactory"/> so strategy
public static IPlateNester Create(string strategy) => strategy == "Default" /// resolution has a single source of truth; rejects unknown keys. Never reads or changes the
? new LegacyPlateNesterAdapter(plate => new DefaultNestEngine(plate)) /// process-global NestEngineRegistry.</summary>
: throw new NotSupportedException($"Unknown placement strategy: {strategy}."); public static IPlateNester Create(string strategy) => PlateNesterFactory.Create(strategy);
public PlateCandidate Place(PlatePlacementRequest request, IProgress<NestJobProgress> progress = null, public PlateCandidate Place(PlatePlacementRequest request, IProgress<NestJobProgress> progress = null,
CancellationToken token = default) CancellationToken token = default)