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

refactor: replace NestPhase switch statements with attribute-based extensions

Browse Source 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
AJ Isaacs
2026-03-21 19:49:44 -04:00
parent ccd402c50f
commit 8ffdacd6c0
2 changed files with 2 additions and 25 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
OpenNest/Controls/PhaseStepperControl.cs
View File

@@ -59,16 +59,6 @@ namespace OpenNest.Controls
}
}
private static string GetDisplayName(NestPhase phase)
{
switch (phase)
{
case NestPhase.RectBestFit: return "BestFit";
case NestPhase.Nfp: return "NFP";
default: return phase.ToString();
}
}
protected override void OnPaint(PaintEventArgs e)
{
base.OnPaint(e);
@@ -134,7 +124,7 @@ namespace OpenNest.Controls
}
// Label
var label = GetDisplayName(phase);
var label = phase.ShortName();
var font = isVisited || isActive ? BoldLabelFont : LabelFont;
var brush = isVisited || isActive ? activeTextBrush : pendingTextBrush;
var labelSize = g.MeasureString(label, font);

15
OpenNest/Forms/NestProgressForm.cs
View File

@@ -73,7 +73,7 @@ namespace OpenNest.Forms
descriptionValue.Text = !string.IsNullOrEmpty(progress.Description)
? progress.Description
: FormatPhase(progress.Phase);
: progress.Phase.DisplayName();
}
public void ShowCompleted()
@@ -196,18 +196,5 @@ namespace OpenNest.Forms
return DensityMidColor;
return DensityHighColor;
}
private static string FormatPhase(NestPhase phase)
{
switch (phase)
{
case NestPhase.Linear: return "Trying rotations...";
case NestPhase.RectBestFit: return "Trying best fit...";
case NestPhase.Pairs: return "Trying pairs...";
case NestPhase.Extents: return "Trying extents...";
case NestPhase.Nfp: return "Trying NFP...";
default: return phase.ToString();
}
}
}
}