Semaphore::fetch() - Code Metrics - EVE-KILL/zKillboard - Measure and Improve Code Quality continuously with Scrutinizer

Semaphore::fetch() B
last analyzed 2016-02-10 18:51 UTC

↳ Parent: Semaphore

Complexity

Conditions	5
Paths	10

Size

Total Lines	11
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	11
rs	8.8571
cc	5
eloc	8
nc	10
nop	1

﻿<?php

/* zLibrary
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License as published by
 * the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */

class Semaphore
{
    /**
     * @param int $semaphoreID
     * @return null|resource
     */
    public static function fetch($semaphoreID)
    {
        global $useSemaphores, $semaphoreModulus;
        if (!isset($useSemaphores)) $useSemaphores = false;
        if ($useSemaphores == false) return null;

        if (!isset($semaphoreModulus)) $semaphoreModulus = 10;

        $sem = sem_get($semaphoreID % $semaphoreModulus);
        if (!sem_acquire($sem)) return null;
        return $sem;
    }

    /**
     * @param null|resource $semaphoreResource
     */
    public static function release($semaphoreResource)
    {
        if ($semaphoreResource != null) sem_release($semaphoreResource);
    }
}


1			<?php
			0 ignored issues – show Coding Style introduced 2015-12-13 23:29 UTC by Report Bug Copy Issue Report The opening PHP tag must be the first content in the file Loading history...
2			/* zLibrary
3			*
4			* This program is free software: you can redistribute it and/or modify
5			* it under the terms of the GNU Affero General Public License as published by
6			* the Free Software Foundation, either version 3 of the License, or
7			* (at your option) any later version.
8			*
9			* This program is distributed in the hope that it will be useful,
10			* but WITHOUT ANY WARRANTY; without even the implied warranty of
11			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12			* GNU Affero General Public License for more details.
13			*
14			* You should have received a copy of the GNU Affero General Public License
15			* along with this program. If not, see <http://www.gnu.org/licenses/>.
16			*/
17
18			class Semaphore
19			{
20			/**
21			* @param int $semaphoreID
22			* @return null\|resource
23			*/
24			public static function fetch($semaphoreID)
25			{
26			global $useSemaphores, $semaphoreModulus;
27			if (!isset($useSemaphores)) $useSemaphores = false;
28			if ($useSemaphores == false) return null;
			0 ignored issues – show Coding Style Best Practice introduced 2015-12-13 23:29 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing two booleans. Considering using the strict comparison `===` instead. When comparing two booleans, it is generally considered safer to use the strict comparison operator. Loading history...
29			if (!isset($semaphoreModulus)) $semaphoreModulus = 10;
30
31			$sem = sem_get($semaphoreID % $semaphoreModulus);
32			if (!sem_acquire($sem)) return null;
33			return $sem;
34			}
35
36			/**
37			* @param null\|resource $semaphoreResource
38			*/
39			public static function release($semaphoreResource)
40			{
41			if ($semaphoreResource != null) sem_release($semaphoreResource);
42			}
43			}
44

EVE-KILL / zKillboard

Semaphore::fetch() B last analyzed 2016-02-10 18:51 UTC

Complexity

Size

Duplication

Duplication Side-by-Side

Filter issues like

Semaphore::fetch() B
last analyzed 2016-02-10 18:51 UTC