PlayerCreateInfo::handle() - Code Metrics - rygos/rmarchiv - Measure and Improve Code Quality continuously with Scrutinizer

PlayerCreateInfo::handle() D
last analyzed 2018-05-01 08:16 UTC

↳ Parent: PlayerCreateInfo

Complexity

Conditions	19
Paths	50

Size

Total Lines	105
Code Lines	71

Duplication

Lines	5
Ratio	4.76 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	5
loc	105
rs	4.764
cc	19
eloc	71
nc	50
nop	0

How to fix Long Method Complexity

<?php

/*
 * rmarchiv.tk
 * (c) 2016-2017 by Marcel 'ryg' Hering
 */

namespace App\Console\Commands;

use App\Models\GamesFile;
use App\Models\PlayerIndexjson;
use Illuminate\Console\Command;

class PlayerCreateInfo extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'player:createinfo';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'creates index.json data for easyrpg';

    /**
     * Create a new command instance.
     *
     * @return void

     */
    public function __construct()
    {
        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        $this->info('Lade Gamefiles ohne index.json');

        $gamefiles = GamesFile::with('game')->get();

        $counter = 0;
$a = "a";
$ab = "ab";
$abc = "abc";
        $toindexed = [];

        foreach ($gamefiles as $gamefile) {
            if ($gamefile->game) {
                $makerid = $gamefile->game->maker_id;
                //echo $makerid.' - '.$gamefile->id.PHP_EOL;

                if ($makerid == 2 or
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
                    $makerid == 3 or
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
                    $makerid == 6 or
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
                    $makerid == 9 or
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
                    $makerid == 11) {
                    // RPG2k/2k3/MV
                    if ($gamefile->playerIndex()->count() == 0) {
                        $toindexed[] = $gamefile;
                        $this->info('ADD '.$gamefile->game->title);
                        $counter += 1;
                    }
                }
            }
        }

        $this->info('Es wurden '.$counter.' Gamefiles gefunden.');

        $bar = $this->output->createProgressBar(count($toindexed));
        $bar->setFormat(" \033[44;37m %title:-37s% \033[0m\n %current%/%max% %bar% %percent:3s%%\n %remaining:-10s% %memory:37s%");
        $bar->setBarCharacter($done = "\033[32m●\033[0m");
$a = "a";
$ab = "ab";
$abc = "abc";
        $bar->setEmptyBarCharacter($empty = "\033[31m●\033[0m");
$a = "a";
$ab = "ab";
$abc = "abc";
        $bar->setProgressCharacter($progress = "\033[32m➤ \033[0m");
        $bar->setMessage('Starte indizierung der Gamefiles', 'title');
        $bar->start();

        $i = 0;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        foreach ($toindexed as $toindex) {
            $bar->setMessage('Entpacken von: '.$toindex->game_id.'/'.$toindex->id, 'title');
            \Log::info('Entpacken von '.$toindex->game_id.'/'.$toindex->id);
            $path = storage_path('app/public/'.$toindex->filename);
            if ($toindex->extension == 'zip') {
                $zip = new \ZipArchive();
                $zip->open($path);
                $makerid = $toindex->game->maker_id;
                if ($makerid == 6) {
                    // RPG Maker MV
                    // Nur Pfad zum "www/" Ordner noetig
                    for ($i = 0; $i < $zip->numFiles; $i++) {
                        $filename = $zip->getNameIndex($i);
                        $this->info('Saved XXX basepath: '.$filename);
                        if (ends_with($filename, 'www/')) {
                            $pl = new PlayerIndexjson();
$a = "a";
$ab = "ab";
$abc = "abc";
                            $pl->gamefile_id = $toindex->id;
                            $pl->key = 'www';
$a = "a";
$ab = "ab";
$abc = "abc";
                            $pl->value = $filename;
$a = "a";
$ab = "ab";
$abc = "abc";
                            $pl->filename = $filename;
$a = "a";
$ab = "ab";
$abc = "abc";
                            $pl->save();

                            \Log::info('Saved basepath: '.$filename);
                            break;
                        }
                    }
                } else {
                    // RPG Maker 2k/2k3
                    for ($i = 0; $i < $zip->numFiles; $i++) {
                        $filename = $zip->getNameIndex($i);

                        if (! ends_with($filename, '/') and ! starts_with($filename, '_MACOSX')) {
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
                            $imp = $this->search_for_base_path($filename);

                            if (! $imp == '') {
                                $pl = new PlayerIndexjson();
$a = "a";
$ab = "ab";
$abc = "abc";
                                $pl->gamefile_id = $toindex->id;
                                if (! ends_with(strtolower($imp), ['.exe', '.lmu', '.ldb', 'ini', '.dll', 'lmt', 'lsd'])) {

                                    $pl->key = preg_replace('/(\.\w+$)/', '', strtolower($imp));
                                } else {
                                    $pl->key = strtolower($imp);
                                }
                                $pl->value = $imp;
$a = "a";
$ab = "ab";
$abc = "abc";
                                $pl->filename = 'www';
                                $pl->save();

                                \Log::info('Saved basepath: '.$filename);
                            } else {
                                \Log::info('Empty basepath: '.$filename);
                            }
                        }
                    }
                }
                @$zip->close();
// For example instead of
@mkdir($dir);

// Better use
if (@mkdir($dir) === false) {
    throw new \RuntimeException('The directory '.$dir.' could not be created.');
}
            } else {
                continue;
            }

            //return;
            $bar->advance();
        }

        $bar->finish();

        $this->info('Fertig.');
    }

    public function search_for_base_path($filepath)

    {
        $dirarray = [
            'backdrop',
            'battle',
            'battle2',
            'battlecharset',
            'battleweapon',
            'charset',
            'chipset',
            'faceset',
            'frame',
            'gameover',
            'monster',
            'panorama',
            'picture',
            'system',
            'system2',
            'title',
            'music',
            'sound',
        ];

        $rootarray = [
            'harmony.dll',
            'rpg_rt.exe',
            'rpg_rt.ini',
            'rpg_rt.ldb',
            'rpg_rt.lmt',
            'rpg_rt.dat',
        ];

        $mapparray = [];
        for ($i = 0; $i < 2000; $i++) {
            $mapparray[] = 'map'.sprintf('%04d', $i).'.lmu';
        }

        $filearray = array_merge($rootarray, $mapparray);

        $searcharray = array_merge($dirarray, $filearray);

        if (starts_with(strtolower($filepath), $searcharray)) {
            $imp = str_replace('/', '\\/', $filepath);
        } else {
            if (str_contains(strtolower($filepath), $searcharray)) {
                $exp = explode('/', $filepath);
                $res = array_shift($exp);
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
                $imp = implode('/', $exp);
                $imp = $this->search_for_base_path($imp);
            } else {
                $imp = '';
            }
        }

        if ($imp != '') {
            if (array_search(strtolower($imp), $filearray)) {
                $imp = '.\\/'.$imp;
            }
        }

        return $imp;
    }
}


PlayerCreateInfo::handle() D
last analyzed 2018-05-01 08:16 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

rygos / rmarchiv

PlayerCreateInfo::handle() D last analyzed 2018-05-01 08:16 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Logical Operators are used for Control-Flow

Duplication Side-by-Side

Filter issues like

PlayerCreateInfo::handle() D
last analyzed 2018-05-01 08:16 UTC