Make the pipeline form look presentable

LittleBigRefresh · Sep 19, 2024 · 0a0ea4e · 0a0ea4e
1 parent 7b9c5d8
commit 0a0ea4e
Show file tree

Hide file tree

Showing 2 changed files with 33 additions and 12 deletions.
diff --git a/Refresher.Core/Pipelines/Pipeline.cs b/Refresher.Core/Pipelines/Pipeline.cs
@@ -44,8 +44,11 @@ public void Initialize()
 
     public async Task ExecuteAsync(CancellationToken cancellationToken = default)
     {
-        if(this.State != PipelineState.NotStarted)
+        if (this.State != PipelineState.NotStarted)
+        {
+            this.State = PipelineState.Error;
             throw new InvalidOperationException("Pipeline must be restarted before it can be executed again.");
+        }
 
         this.State = PipelineState.Running;
 

diff --git a/Refresher/UI/PipelineForm.cs b/Refresher/UI/PipelineForm.cs
@@ -8,44 +8,62 @@ namespace Refresher.UI;
 {
     private TPipeline? _pipeline;
 
-    private ProgressBar _progressBar;
+    private readonly Label _stateLabel;
+    private readonly ProgressBar _progressBar;
 
     public PipelineForm() : base(typeof(TPipeline).Name, new Size(700, 1), false)
     {
-        this.InitializePipeline();
-
         this.Content = new StackLayout([
-            new Button(this.ExecutePipeline),
+            this._stateLabel = new Label(),
+            new Button(this.ExecutePipeline) { Text = "Execute" },
             this._progressBar = new ProgressBar(),
-        ]);
+        ])
+        {
+            Padding = new Padding(0, 10, 0, 0),
+            Spacing = 5,
+            HorizontalContentAlignment = HorizontalAlignment.Stretch,
+            VerticalContentAlignment = VerticalAlignment.Bottom,
+        };
+
+        this.InitializePipeline();
 
         Thread progressThread = new(() =>
         {
             while (!this.IsDisposed && !Application.Instance.IsDisposed)
             {
-                Application.Instance.Invoke(() =>
-                {
-                    this._progressBar.Value = (int)((this._pipeline?.Progress ?? 0) * 100);
-                    this._progressBar.Enabled = this._pipeline?.State == PipelineState.Running;
-                });
-                Thread.Sleep(100);
+                Application.Instance.Invoke(this.UpdateFormState);
+                Thread.Sleep(this._pipeline?.State == PipelineState.Running ? 50 : 250);
             }
         });
 
         progressThread.Start();
     }
 
+    private void UpdateFormState()
+    {
+        this._progressBar.Value = (int)((this._pipeline?.Progress ?? 0) * 100);
+        this._progressBar.Enabled = this._pipeline?.State == PipelineState.Running;
+        this._stateLabel.Text = this._pipeline?.State.ToString() ?? "Uninitialized";
+    }
+
     private void InitializePipeline()
     {
         this._pipeline = new TPipeline();
         this._pipeline.Initialize();
+
+        this.UpdateFormState();
     }
 
     private void ExecutePipeline(object? sender, EventArgs e)
     {
+        if (this._pipeline == null)
+            return;
+
         Task.Run(async () =>
         {
             await this._pipeline.ExecuteAsync();
         });
+
+        this.UpdateFormState();
     }
 }