GenerateDiagramCommand::handle() - Code Metrics - Inspection of "Add structured text output for AI model context" - beyondcode/laravel-er-diagram-generator - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#120)

unknown

created 2025-03-11 20:07 UTC

GenerateDiagramCommand::handle() A

↳ Parent: GenerateDiagramCommand

Complexity

Conditions	5
Paths	4

Size

Total Lines	51
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
cc	5
eloc	31
c	2
b	0
f	0
nc	4
nop	0
dl	0
loc	51
rs	9.1128

How to fix Long Method

<?php

namespace BeyondCode\ErdGenerator;

use BeyondCode\ErdGenerator\Model as GraphModel;
use Illuminate\Console\Command;
use Illuminate\Support\Collection;
use phpDocumentor\GraphViz\Graph;
use ReflectionClass;

class GenerateDiagramCommand extends Command
{
    const FORMAT_TEXT = 'text';

    const DEFAULT_FILENAME = 'graph';

    /**
     * The console command name.
     *
     * @var string
     */
    protected $signature = 'generate:erd {filename?} {--format=png} {--text-output : Output as text file instead of image} {--structured : Generate structured text output for AI models}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Generate ER diagram.';

    /** @var ModelFinder */
    protected $modelFinder;

    /** @var RelationFinder */
    protected $relationFinder;

    /** @var Graph */
    protected $graph;

    /** @var GraphBuilder */
    protected $graphBuilder;

    public function __construct(ModelFinder $modelFinder, RelationFinder $relationFinder, GraphBuilder $graphBuilder)
    {
        parent::__construct();

        $this->relationFinder = $relationFinder;
        $this->modelFinder = $modelFinder;
        $this->graphBuilder = $graphBuilder;
    }

    /**
     * @throws \phpDocumentor\GraphViz\Exception
     */
    public function handle()
    {
        $models = $this->getModelsThatShouldBeInspected();

        $this->info("Found {$models->count()} models.");
        $this->info("Inspecting model relations.");

        $bar = $this->output->createProgressBar($models->count());

        $models->transform(function ($model) use ($bar) {
            $bar->advance();
            return new GraphModel(
                $model,
                (new ReflectionClass($model))->getShortName(),
                $this->relationFinder->getModelRelations($model)
            );
        });

        // If structured text output is requested, generate it
        if ($this->option('structured')) {
            $textOutput = $this->graphBuilder->generateStructuredTextRepresentation($models);
            $outputFileName = $this->getTextOutputFileName();
            file_put_contents($outputFileName, $textOutput);
            $this->info(PHP_EOL);
            $this->info('Wrote structured text diagram to ' . $outputFileName);
            return;
        }

        $graph = $this->graphBuilder->buildGraph($models);

        if ($this->option('text-output') || $this->option('format') === self::FORMAT_TEXT) {
            $textOutput = $graph->__toString();
            
            // If text-output option is set, write to file
            if ($this->option('text-output')) {
                $outputFileName = $this->getTextOutputFileName();
                file_put_contents($outputFileName, $textOutput);
                $this->info(PHP_EOL);
                $this->info('Wrote text diagram to ' . $outputFileName);
                return;
            }
            
            // Otherwise just output to console
            $this->info($textOutput);
            return;
        }

        $graph->export($this->option('format'), $this->getOutputFileName());

        $this->info(PHP_EOL);
        $this->info('Wrote diagram to ' . $this->getOutputFileName());
    }

    protected function getOutputFileName(): string
    {
        return $this->argument('filename') ?:
            static::DEFAULT_FILENAME . '.' . $this->option('format');
    }

    protected function getTextOutputFileName(): string
    {
        return $this->argument('filename') ?: static::DEFAULT_FILENAME . '.txt';
    }

    protected function getModelsThatShouldBeInspected(): Collection
    {
        $directories = config('erd-generator.directories');

        $modelsFromDirectories = $this->getAllModelsFromEachDirectory($directories);

        return $modelsFromDirectories;
    }

    protected function getAllModelsFromEachDirectory(array $directories): Collection
    {
        return collect($directories)
        return collect(/** @scrutinizer ignore-type */ $directories)
            ->map(function ($directory) {
                return $this->modelFinder->getModelsInDirectory($directory)->all();
            })
            ->flatten();
    }
}


beyondcode / laravel-er-diagram-generator

Pull Request — master (#120)

GenerateDiagramCommand::handle() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like