improved rendering of SummaryView

Author: DeGraciaMathieu

app/Presenter/Analyze/Summary/SummaryView.php

@@ -5,22 +5,22 @@
 use function Laravel\Prompts\outro;
 use function Laravel\Prompts\table;
 use function Laravel\Prompts\warning;
-use function Laravel\Prompts\note;
 use App\Presenter\Analyze\Summary\SummaryViewModel;
 
 class SummaryView
 {
     public function show(SummaryViewModel $viewModel): void
+    {
+        $this->displayMetrics($viewModel);
+        $this->displayInfo($viewModel);
+        $this->displayCount($viewModel);
+    }
+
+    private function displayMetrics(SummaryViewModel $viewModel): void
     {
         count($viewModel->metrics()) === 0
             ? warning('No classes found')
             : $this->showTable($viewModel);
-
-        $viewModel->needInfo()
-            ? $this->showInfo($viewModel)
-            : outro('Add --info to get more information on metrics.');
-
-        outro(sprintf('Found %d classes in the given path', $viewModel->count()));
     }
 
     private function showTable(SummaryViewModel $viewModel): void
@@ -31,6 +31,13 @@ private function showTable(SummaryViewModel $viewModel): void
         );
     }
 
+    private function displayInfo(SummaryViewModel $viewModel): void
+    {
+        $viewModel->needInfo()
+            ? $this->showInfo($viewModel)
+            : outro('Add --info to get more information on metrics.');
+    }
+
     private function showInfo(SummaryViewModel $viewModel): void
     {
         $viewModel->isHumanReadable()
@@ -40,7 +47,8 @@ private function showInfo(SummaryViewModel $viewModel): void
 
     private function showHumanReadableInfo(): void
     {
-        outro('A stable and concrete class may be challenging to modify, as it likely does not follow the open/closed principle.');
+        outro('A stable and concrete class is heavily used by the application and has few abstractions.');
+        outro('It is probably not open to extension and it will be necessary to modify it to add behaviors.');
         outro('An unstable and concrete class will likely suffer from side effects caused by its dependencies, making it harder to test.');
         outro('For more information, see the documentation: https://php-quality-tools.com/class-dependencies-analyzer');
     }
@@ -63,4 +71,9 @@ private function showMetricsInfo(): void
         outro('Class with a high abstractness (A close to 1) is totally abstract and should not have any concrete code.');
         outro('See the documentation for more information : https://php-quality-tools.com/class-dependencies-analyzer');
     }
+
+    private function displayCount(SummaryViewModel $viewModel): void
+    {
+        outro(sprintf('Found %d classes in the given path', $viewModel->count()));
+    }
 }