TorowTocol::byRow() - Code Metrics - Inspection of "TOCOL and TOROW" - PHPOffice/PhpSpreadsheet - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#4493)

by Owen

created 2025-05-29 06:17 UTC

TorowTocol::byRow() B

↳ Parent: TorowTocol

Complexity

Conditions	10
Paths	13

Size

Total Lines	23
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	14
dl	0
loc	23
rs	7.6666
c	1
b	0
f	0
cc	10
nc	13
nop	2

How to fix Complexity

<?php

namespace PhpOffice\PhpSpreadsheet\Calculation\LookupRef;

use PhpOffice\PhpSpreadsheet\Calculation\Functions;
use PhpOffice\PhpSpreadsheet\Calculation\Information\ErrorValue;
use PhpOffice\PhpSpreadsheet\Calculation\Information\ExcelError;

class TorowTocol
{
    /**
     * Excel function TOCOL.
     *
     * @return mixed[]|string
     */
    public static function tocol(mixed $array, mixed $ignore = 0, mixed $byColumn = false): array|string
    {
        $result = self::torow($array, $ignore, $byColumn);
        if (is_array($result)) {
            return array_map((fn ($x) => [$x]), $result);
        }

        return $result;
    }

    /**
     * Excel function TOROW.
     *
     * @return mixed[]|string
     */
    public static function torow(mixed $array, mixed $ignore = 0, mixed $byColumn = false): array|string
    {
        if (!is_numeric($ignore)) {
            return ExcelError::VALUE();
        }
        $ignore = (int) $ignore;
        if ($ignore < 0 || $ignore > 3) {
            return ExcelError::VALUE();
        }
        if (is_int($byColumn) || is_float($byColumn)) {
            $byColumn = (bool) $byColumn;
        }
        if (!is_bool($byColumn)) {
            return ExcelError::VALUE();
        }
        if (!is_array($array)) {
            $array = [$array];
        }
        if ($byColumn) {
            $temp = [];
            foreach ($array as $row) {
                if (!is_array($row)) {
                    $row = [$row];
                }
                $temp[] = Functions::flattenArray($row);
            }
            $array = ChooseRowsEtc::transpose($temp);
        } else {
            $array = Functions::flattenArray($array);
        }

        return self::byRow($array, $ignore);
    }

    /**
     * @param mixed[] $array
     *
     * @return mixed[]
     */
    private static function byRow(array $array, int $ignore): array
    {
        $returnMatrix = [];
        foreach ($array as $row) {
            if (!is_array($row)) {
                $row = [$row];
            }
            foreach ($row as $cell) {
                if ($cell === null) {
                    if ($ignore === 1 || $ignore === 3) {
                        continue;
                    }
                    $cell = 0;
                } elseif (ErrorValue::isError($cell)) {
                    if ($ignore === 2 || $ignore === 3) {
                        continue;
                    }
                }
                $returnMatrix[] = $cell;
            }
        }

        return $returnMatrix;
    }
}


1			<?php
2
3			namespace PhpOffice\PhpSpreadsheet\Calculation\LookupRef;
4
5			use PhpOffice\PhpSpreadsheet\Calculation\Functions;
6			use PhpOffice\PhpSpreadsheet\Calculation\Information\ErrorValue;
7			use PhpOffice\PhpSpreadsheet\Calculation\Information\ExcelError;
8
9			class TorowTocol
10			{
11			/**
12			* Excel function TOCOL.
13			*
14			* @return mixed[]\|string
15			*/
16			public static function tocol(mixed $array, mixed $ignore = 0, mixed $byColumn = false): array\|string
17			{
18			$result = self::torow($array, $ignore, $byColumn);
19			if (is_array($result)) {
20			return array_map((fn ($x) => [$x]), $result);
21			}
22
23			return $result;
24			}
25
26			/**
27			* Excel function TOROW.
28			*
29			* @return mixed[]\|string
30			*/
31			public static function torow(mixed $array, mixed $ignore = 0, mixed $byColumn = false): array\|string
32			{
33			if (!is_numeric($ignore)) {
34			return ExcelError::VALUE();
35			}
36			$ignore = (int) $ignore;
37			if ($ignore < 0 \|\| $ignore > 3) {
38			return ExcelError::VALUE();
39			}
40			if (is_int($byColumn) \|\| is_float($byColumn)) {
41			$byColumn = (bool) $byColumn;
42			}
43			if (!is_bool($byColumn)) {
44			return ExcelError::VALUE();
45			}
46			if (!is_array($array)) {
47			$array = [$array];
48			}
49			if ($byColumn) {
50			$temp = [];
51			foreach ($array as $row) {
52			if (!is_array($row)) {
53			$row = [$row];
54			}
55			$temp[] = Functions::flattenArray($row);
56			}
57			$array = ChooseRowsEtc::transpose($temp);
58			} else {
59			$array = Functions::flattenArray($array);
60			}
61
62			return self::byRow($array, $ignore);
63			}
64
65			/**
66			* @param mixed[] $array
67			*
68			* @return mixed[]
69			*/
70			private static function byRow(array $array, int $ignore): array
71			{
72			$returnMatrix = [];
73			foreach ($array as $row) {
74			if (!is_array($row)) {
75			$row = [$row];
76			}
77			foreach ($row as $cell) {
78			if ($cell === null) {
79			if ($ignore === 1 \|\| $ignore === 3) {
80			continue;
81			}
82			$cell = 0;
83			} elseif (ErrorValue::isError($cell)) {
84			if ($ignore === 2 \|\| $ignore === 3) {
85			continue;
86			}
87			}
88			$returnMatrix[] = $cell;
89			}
90			}
91
92			return $returnMatrix;
93			}
94			}
95

PHPOffice / PhpSpreadsheet

Pull Request — master (#4493)

TorowTocol::byRow() B

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like