ConnpassEventService::getConnpassApiOneMonthEventJson() - Code Metrics - Inspection of "Merge pull request #73 from igara/68_create_event" - igara/syonet - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 45bcc8...fd9927 )

by Syo

created 2016-03-07 14:37 UTC

getConnpassApiOneMonthEventJson() B

↳ Parent: ConnpassEventService

Complexity

Conditions	4
Paths	4

Size

Total Lines	48
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	48
rs	8.7396
cc	4
eloc	31
nc	4
nop	1

<?php

namespace App\Http\Services\Contents\It\Event;

/**
 * ConnpassEventService.
 *
 * ConnpassEventを扱うService
 */
class ConnpassEventService
{
    /**
     * Create a new instance.
     */
    public function __construct()
    {
    }
    
    /**
     * ConnpassAPIにリクエストするURLの生成
     * @param リクエストパラム
     * @return string ConnpassAPIにリクエストするURL
     */
    private function _createRequestConnpassApiUrl($param) {
        $year = $param['year'];
        $month = $param['month'];
        $count = $param['count'];
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $firstMonth = date("$year-$month-01");
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $lastMonth = date("$year-$month-t");
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

        return "http://connpass.com/api/v1/event/?keyword=python";
    }

    /**
     * connpassのAPIからJSONを取得する.
     * 
     * @param array リクエストパラム
     * @return array connpassのイベントを取得
     */
    public function getConnpassApiOneMonthEventJson($param)
    {   
        $context = stream_context_create([
           'http' => array('ignore_errors' => true)
        ]);
        $response = file_get_contents(
            $this->_createRequestConnpassApiUrl($param),
            false,
            $context
        );

        // ステータスコードを取得する
        preg_match(
            '/HTTP\/1\.[0|1|x] ([0-9]{3})/',
            $http_response_header[0],
            $matches
        );
        $status_code = $matches[1];
        
        // APIのデータ取得用変数
        $eventData = [];
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

        switch ($status_code) {
            case '200':
                // JSONファイルは配列に変換しておく
                $jsonData = mb_convert_encoding($response, 'UTF8', 'auto');
                $eventData = json_decode($jsonData, true);
                break;
            case '403';
switch ($expr) {
    case "A": { //wrong
        doSomething();
        break;
    }
    case "B": //right
        doSomething();
        break;
}
                // 403の場合
                $eventData = [
                    'status' => '403'
                ];
                break;
            case '404':
                // 404の場合
                $eventData = [
                    'status' => '404'
                ];
                break;
            default:
                $eventData = [
                    'status' => 'error'
                ];
                break;
        }
        return $eventData;
    }
}


1			<?php
2
3			namespace App\Http\Services\Contents\It\Event;
4
5			/**
6			* ConnpassEventService.
7			*
8			* ConnpassEventを扱うService
9			*/
10			class ConnpassEventService
11			{
12			/**
13			* Create a new instance.
14			*/
15			public function __construct()
16			{
17			}
18
19			/**
20			* ConnpassAPIにリクエストするURLの生成
21			* @param リクエストパラム
22			* @return string ConnpassAPIにリクエストするURL
23			*/
24			private function _createRequestConnpassApiUrl($param) {
25			$year = $param['year'];
26			$month = $param['month'];
27			$count = $param['count'];
			0 ignored issues – show Unused Code introduced 2016-03-07 14:39 UTC by Report Bug Copy Issue Report `$count` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
28			$firstMonth = date("$year-$month-01");
			0 ignored issues – show Unused Code introduced 2016-03-07 14:39 UTC by Report Bug Copy Issue Report `$firstMonth` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
29			$lastMonth = date("$year-$month-t");
			0 ignored issues – show Unused Code introduced 2016-03-07 14:39 UTC by Report Bug Copy Issue Report `$lastMonth` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
30
31			return "http://connpass.com/api/v1/event/?keyword=python";
32			}
33
34			/**
35			* connpassのAPIからJSONを取得する.
36			*
37			* @param array リクエストパラム
38			* @return array connpassのイベントを取得
39			*/
40			public function getConnpassApiOneMonthEventJson($param)
41			{
42			$context = stream_context_create([
43			'http' => array('ignore_errors' => true)
44			]);
45			$response = file_get_contents(
46			$this->_createRequestConnpassApiUrl($param),
47			false,
48			$context
49			);
50
51			// ステータスコードを取得する
52			preg_match(
53			'/HTTP\/1\.[0\|1\|x] ([0-9]{3})/',
54			$http_response_header[0],
55			$matches
56			);
57			$status_code = $matches[1];
58
59			// APIのデータ取得用変数
60			$eventData = [];
			0 ignored issues – show Unused Code introduced 2016-03-07 14:39 UTC by Report Bug Copy Issue Report `$eventData` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
61
62			switch ($status_code) {
63			case '200':
64			// JSONファイルは配列に変換しておく
65			$jsonData = mb_convert_encoding($response, 'UTF8', 'auto');
66			$eventData = json_decode($jsonData, true);
67			break;
68			case '403';
			0 ignored issues – show Coding Style introduced 2016-03-07 14:39 UTC by Report Bug Copy Issue Report CASE statements must be defined using a colon As per the PSR-2 coding standard, case statements should not be wrapped in curly braces. There is no need for braces, since each case is terminated by the next `break`. switch ($expr) { case "A": { //wrong doSomething(); break; } case "B": //right doSomething(); break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
69			// 403の場合
70			$eventData = [
71			'status' => '403'
72			];
73			break;
74			case '404':
75			// 404の場合
76			$eventData = [
77			'status' => '404'
78			];
79			break;
80			default:
81			$eventData = [
82			'status' => 'error'
83			];
84			break;
85			}
86			return $eventData;
87			}
88			}
89

igara / syonet

GitHub Access Token became invalid

Push — master ( 45bcc8...fd9927 )

getConnpassApiOneMonthEventJson() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like