Seboettg\CiteProc\Util\NumberHelper

Complexity

Total Complexity

29

Size/Duplication

Total Lines	140
Duplicated Lines	0 %

Test Coverage

Coverage

81.48%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	extractNumber()	0	6	2
A	roman2Dec()	0	23	6
A	getCompareNumber()	0	12	6
B	evaluateStringPluralism()	0	23	9
A	dec2roman()	0	13	3
A	isRomanNumber()	0	9	3

<?php
/*

You must use "/**" style comments for a file comment

 * citeproc-php
 *
 * @link        http://github.com/seboettg/citeproc-php for the source repository
 * @copyright   Copyright (c) 2016 Sebastian Böttger.
 * @license     https://opensource.org/licenses/MIT
 */

namespace Seboettg\CiteProc\Util;

/**
 * Class Number
 * @package Seboettg\CiteProc\Util

There must be exactly one blank line before the tags in a doc comment

 *
 * @author Sebastian Böttger <seboettg@gmail.com>
 */

Missing @category tag in class comment

Missing @license tag in class comment

Missing @link tag in class comment

class NumberHelper
{

    const PATTERN_ORDINAL = "/\d+(st|nd|rd|th)?\.?$/";

    const PATTERN_ROMAN = "/^[ivxlcdm]+\.?$/i";

    const PATTERN_AFFIXES = "/^[a-z]?\d+[a-z]?$/i";

    const PATTERN_COMMA_AMPERSAND_RANGE = "/\d*([\s?\-&+,;\s])+\d+/";

    const ROMAN_NUMERALS = [
        ["", "i", "ii", "iii", "iv", "v", "vi", "vii", "viii", "ix"],
        ["", "x", "xx", "xxx", "xl", "l", "lx", "lxx", "lxxx", "xc"],
        ["", "c", "cc", "ccc", "cd", "d", "dc", "dcc", "dccc", "cm"],
        ["", "m", "mm", "mmm", "mmmm", "mmmmm"]
    ];

    const ROMAN_DIGITS = [
        "M" => 1000,
        "D" => 500,
        "C" => 100,
        "L" => 50,
        "X" => 10,
        "V" => 5,
        "I" => 1
    ];

    /**

Missing short description in doc comment

     * @return \Closure
     */
    public static function getCompareNumber()
    {
        return function($numA, $numB, $order) {

Expected 1 space after FUNCTION keyword; 0 found

            if (is_numeric($numA) && is_numeric($numB)) {
                $ret = $numA - $numB;
            } else {
                $ret = strcasecmp($numA, $numB);
            }
            if ("descending" === $order) {
                return $ret > 0 ? -1 : 1;
            }
            return $ret > 0 ? 1 : -1;
        };
    }

    /**

Missing short description in doc comment

Parameter $num should have a doc-comment as per coding-style.

     * @param $num

Missing parameter name

Tag value indented incorrectly; expected 2 spaces but found 1

     * @return string

Tag cannot be grouped with parameter tags in a doc comment

     */
    public static function dec2roman($num)
    {
        $ret = "";
        if ($num < 6000) {

            $numStr = strrev($num);
            $len = strlen($numStr);
            for ($pos = 0; $pos < $len; $pos++) {
                $n = $numStr[$pos];
                $ret = self::ROMAN_NUMERALS[$pos][$n] . $ret;
            }
        }
        return $ret;
    }

    /**

Missing short description in doc comment

Parameter $romanNumber should have a doc-comment as per coding-style.

     * @param $romanNumber

Missing parameter name

Tag value indented incorrectly; expected 2 spaces but found 1

     * @return int|mixed

Tag cannot be grouped with parameter tags in a doc comment

     */
    public static function roman2Dec($romanNumber)
    {
        if (is_numeric($romanNumber)) {
            return 0;
        }

        $values = [];
        // Convert the string to an array of roman values:
        for ($i = 0; $i < strlen($romanNumber); ++$i) {
            $char = strtoupper($romanNumber{$i});
            if (self::ROMAN_DIGITS[$char] !== null) {
                $values[] = self::ROMAN_DIGITS[$char];
            }
        }

        $sum = 0;
        while ($current = current($values)) {
            $next = next($values);
            $next > $current ? $sum += $next - $current + 0 * next($values) : $sum += $current;
        }

        // Return the value:
        return $sum;
    }

    public static function isRomanNumber($str)

Missing doc comment for function isRomanNumber()

    {
        for ($i = 0; $i < strlen($str); ++$i) {
            $char = strtoupper($str{$i});
            if (!in_array($char, array_keys(self::ROMAN_DIGITS))) {
                return false;
            }
        }
        return true;
    }

    /**

Missing short description in doc comment

Parameter $str should have a doc-comment as per coding-style.

     * @param $str

Missing parameter name

Tag value indented incorrectly; expected 2 spaces but found 1

     * @return string

Tag cannot be grouped with parameter tags in a doc comment

     */
    public static function evaluateStringPluralism($str)
    {
        $plural = 'single';
        if (!empty($str)) {
            $ranges = preg_split("/[-–&,]/", $str);
            if (count($ranges) > 1) {

It seems like $ranges can also be of type false; however, parameter $var of count() does only seem to accept Countable|array, maybe add an additional type check?

                $isRange = 1;
                foreach ($ranges as $range) {
                    if (NumberHelper::isRomanNumber(trim($range)) || is_numeric(trim($range))) {
                        $isRange &= 1;
                    }
                }
                if ($isRange == 1) {
                    return 'multiple';
                }
            } else {
                if (is_numeric($str) || NumberHelper::isRomanNumber($str)) {
                    return 'single';
                }
            }
        }
        return $plural;
    }

    public static function extractNumber($string)

Missing doc comment for function extractNumber()

    {
        if (preg_match("/(\d+)[^\d]*$/", $string, $match)) {
            return $match[1];
        }
        return $string;
    }
}