TicketGenerator::generateOne() - Code Metrics - Inspection of "add proxy" - leo108/laravel_cas_server - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — proxy (a8ed88)

by leo

created 2016-10-26 10:11 UTC

TicketGenerator::generateOne() A

↳ Parent: TicketGenerator

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
cc	1
eloc	2
nc	1
nop	2
dl	0
loc	4
ccs	0
cts	2
cp	0
crap	2
rs	10
c	0
b	0
f	0

<?php
/**
 * Created by PhpStorm.
 * User: leo108
 * Date: 2016/10/26
 * Time: 10:22
 */

namespace Leo108\CAS\Services;

use Illuminate\Support\Str;

class TicketGenerator
{

    /**
     * @param integer  $totalLength
     * @param string   $prefix
     * @param callable $checkFunc
     * @param integer  $maxRetry
     * @return string|false
     */
    public function generate($totalLength, $prefix, callable $checkFunc, $maxRetry)
    {
        $ticket = false;
        $flag   = false;
        for ($i = 0; $i < $maxRetry; $i++) {
            $ticket = $this->generateOne($totalLength, $prefix);
            if (!call_user_func_array($checkFunc, [$ticket])) {
                $flag = true;
                break;
            }
        }

        if (!$flag) {
            return false;
        }

        return $ticket;
    }

    public function generateOne($totalLength, $prefix)
    {
        return $prefix.Str::random($totalLength - strlen($prefix));
    }
}


1			<?php
2			/**
3			* Created by PhpStorm.
4			* User: leo108
5			* Date: 2016/10/26
6			* Time: 10:22
7			*/
8
9			namespace Leo108\CAS\Services;
10
11			use Illuminate\Support\Str;
12
13			class TicketGenerator
14			{
15
16			/**
17			* @param integer $totalLength
18			* @param string $prefix
19			* @param callable $checkFunc
20			* @param integer $maxRetry
21			* @return string\|false
22			*/
23			public function generate($totalLength, $prefix, callable $checkFunc, $maxRetry)
24			{
25			$ticket = false;
26			$flag = false;
27			for ($i = 0; $i < $maxRetry; $i++) {
28			$ticket = $this->generateOne($totalLength, $prefix);
29			if (!call_user_func_array($checkFunc, [$ticket])) {
30			$flag = true;
31			break;
32			}
33			}
34
35			if (!$flag) {
36			return false;
37			}
38
39			return $ticket;
40			}
41
42			public function generateOne($totalLength, $prefix)
43			{
44			return $prefix.Str::random($totalLength - strlen($prefix));
45			}
46			}
47

leo108 / laravel_cas_server

Branch — proxy (a8ed88)

TicketGenerator::generateOne() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like