Streaming\Utiles

Complexity

Total Complexity

9

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Importance

Changes	1
Bugs	1	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	appendSlash()	0	3	2
A	getOS()	0	11	4
A	concatKeyValue()	0	4	1
A	arrayToFFmpegOpt()	0	8	2

<?php

/**
 * This file is part of the PHP-FFmpeg-video-streaming package.
 *
 * (c) Amin Yazdanpanah <contact@aminyazdanpanah.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */


namespace Streaming;


class Utiles
{
    /**
     * @param string $str
     * @return string
     */
    public static function appendSlash(string $str): string
    {
        return $str ? rtrim($str, '/') . "/" : $str;
    }

    /**
     * @param array $array
     * @param string $glue
     */
    public static function concatKeyValue(array &$array, string $glue = ""): void
    {
        array_walk($array, function (&$value, $key) use ($glue) {
            $value = "$key$glue$value";
        });
    }

    /**
     * @param array $array
     * @param string $start_with
     * @return array
     */
    public static function arrayToFFmpegOpt(array $array, string $start_with = "-"): array
    {
        $new = [];
        foreach ($array as $key => $value) {
            array_push($new, $start_with . $key, $value);
        }

        return $new;
    }

    /**
     * @return string
     */
    public static function getOS(): string
    {
        switch (true) {

It seems like you are loosely comparing stristr(Streaming\PHP_OS, 'WIN') of type string to the boolean true. If you are specifically checking for a non-empty string, consider using the more explicit !== '' instead.

It seems like you are loosely comparing stristr(Streaming\PHP_OS, 'LINUX') of type string to the boolean true. If you are specifically checking for a non-empty string, consider using the more explicit !== '' instead.

It seems like you are loosely comparing stristr(Streaming\PHP_OS, 'DAR') of type string to the boolean true. If you are specifically checking for a non-empty string, consider using the more explicit !== '' instead.

            case stristr(PHP_OS, 'DAR'):
                return "osX";
            case stristr(PHP_OS, 'WIN'):
                return "windows";
            case stristr(PHP_OS, 'LINUX'):
                return "linux";
            default :
                return "unknown";
        }
    }
}