Mtolhuys\LaravelSchematics\Actions\Relation\DeleteRelationAction

Complexity

Total Complexity

12

Size/Duplication

Total Lines	94
Duplicated Lines	48.94 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	execute()	0	15	2
A	removeLeading()	19	19	3
A	removeTrailing()	19	19	3
A	endOfMethod()	0	4	1
A	startOfMethod()	8	8	3

<?php

namespace Mtolhuys\LaravelSchematics\Actions\Relation;

class DeleteRelationAction
{
    /**
     * @param $request
     * @return false|int
     */
    public function execute($request)
    {
        $file = $request['method']['file'];
        $lines = file($file, FILE_IGNORE_NEW_LINES);
        $index = $request['method']['line'] - 1;

        $removeLines = $this->removeLeading($lines, $index - 1);
        $removeLines = array_merge($removeLines, $this->removeTrailing($lines, $index));

        foreach ($removeLines as $removeLine) {
            unset($lines[$removeLine]);
        }

        return file_put_contents($file, implode("\n", $lines));
    }

    /**
     * @param $lines
     * @param $index
     * @return array
     */
    private function removeLeading($lines, $index): array

This method seems to be duplicated in your project.

    {
        $line = '';
        $remove = [];

        while(! $this->startOfMethod($line)) {
            $line = $lines[$index];

            $remove[] = $index;

            $index--;
        }

        if (trim($lines[$index]) === '') {
            $remove[] = $index;
        }

        return $remove;
    }

    /**
     * @param $lines
     * @param $index
     * @return array
     */
    private function removeTrailing($lines, $index): array

This method seems to be duplicated in your project.

    {
        $line = '';
        $remove = [];

        while(! $this->endOfMethod($line)) {
            $line = $lines[$index];

            $remove[] = $index;

            $index++;
        }

        if (trim($lines[$index]) === '') {
            $remove[] = $index;
        }

        return $remove;
    }

    /**
     * @param $line
     * @return bool
     */
    private function endOfMethod($line): bool
    {
        return str_replace(' ', '', $line) === '}';
    }

    /**
     * @param $line
     * @return bool
     */
    private function startOfMethod($line): bool

This method seems to be duplicated in your project.

    {
        $line = str_replace(' ', '', $line);

        return $line === '/**'
            || $line === '//'
            || $line === '}';
    }
}