From a44102a5afea310c32bb512a192c3360bdeada15 Mon Sep 17 00:00:00 2001
From: Tomas Votruba <tomas.vot@gmail.com>
Date: Sun, 8 Apr 2018 21:31:43 +0200
Subject: [PATCH] improve commplexity

---
 src/Console/Command/DescribeCommand.php | 61 +++++++++++++++----------
 1 file changed, 38 insertions(+), 23 deletions(-)

diff --git a/src/Console/Command/DescribeCommand.php b/src/Console/Command/DescribeCommand.php
index 40d53f4b347..ee448468cd8 100644
--- a/src/Console/Command/DescribeCommand.php
+++ b/src/Console/Command/DescribeCommand.php
@@ -5,8 +5,10 @@ namespace Rector\Console\Command;
 use Rector\Configuration\Option;
 use Rector\Console\ConsoleStyle;
 use Rector\ConsoleDiffer\DifferAndFormatter;
+use Rector\Contract\Rector\RectorInterface;
 use Rector\Naming\CommandNaming;
 use Rector\NodeTraverser\RectorNodeTraverser;
+use Rector\RectorDefinition\CodeSample;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Input\InputOption;
@@ -57,33 +59,46 @@ final class DescribeCommand extends Command
     protected function execute(InputInterface $input, OutputInterface $output): int
     {
         foreach ($this->rectorNodeTraverser->getRectors() as $rector) {
-            $this->consoleStyle->section(get_class($rector));
-
-            $rectorDefinition = $rector->getDefinition();
-            if ($rectorDefinition->getDescription()) {
-                $this->consoleStyle->writeln(' * ' . $rectorDefinition->getDescription());
-            }
-
-            if ($input->getOption(Option::DESCRIBE_WITH_DIFFS)) {
-                foreach ($rectorDefinition->getCodeSamples() as $codeSample) {
-                    $this->consoleStyle->newLine();
-
-                    $formattedDiff = $this->differAndFormatter->bareDiffAndFormat(
-                        $codeSample->getCodeBefore(),
-                        $codeSample->getCodeAfter()
-                    );
-
-                    if ($formattedDiff) {
-                        $this->consoleStyle->write($formattedDiff);
-                    }
-                }
-            }
-
-            $this->consoleStyle->newLine();
+            $this->describeRector($input, $rector);
         }
 
         $this->consoleStyle->success('Rector is done!');
 
         return 0;
     }
+
+    private function describeRector(InputInterface $input, RectorInterface $rector): void
+    {
+        $this->consoleStyle->section(get_class($rector));
+
+        $rectorDefinition = $rector->getDefinition();
+        if ($rectorDefinition->getDescription()) {
+            $this->consoleStyle->writeln(' * ' . $rectorDefinition->getDescription());
+        }
+
+        if ($input->getOption(Option::DESCRIBE_WITH_DIFFS)) {
+            $this->describeRectorCodeSamples($rectorDefinition->getCodeSamples());
+        }
+
+        $this->consoleStyle->newLine();
+    }
+
+    /**
+     * @param CodeSample[] $codeSamples
+     */
+    private function describeRectorCodeSamples(array $codeSamples): void
+    {
+        foreach ($codeSamples as $codeSample) {
+            $this->consoleStyle->newLine();
+
+            $formattedDiff = $this->differAndFormatter->bareDiffAndFormat(
+                $codeSample->getCodeBefore(),
+                $codeSample->getCodeAfter()
+            );
+
+            if ($formattedDiff) {
+                $this->consoleStyle->write($formattedDiff);
+            }
+        }
+    }
 }