PHPieces\Framework\Util\Process

Complexity

Total Complexity

11

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	2
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	2
A	runCom()	0	6	1
A	setPid()	0	4	1
A	getPid()	0	4	1
A	status()	0	10	2
A	start()	0	8	2
A	stop()	0	10	2

<?php

namespace PHPieces\Framework\Util;

class Process
{
    private $pid;
    private $command;

    public function __construct($cl = false)
    {
        if ($cl != false) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison !== instead.

            $this->command = $cl;
            $this->runCom();
        }
    }

    private function runCom()
    {
        $command = 'nohup '.$this->command.' > /dev/null 2>&1 & echo $!';
        exec($command, $op);
        $this->pid = (int)$op[0];
    }

    public function setPid($pid)
    {
        $this->pid = $pid;
    }

    public function getPid()
    {
        return $this->pid;
    }

    public function status()
    {
        $command = 'ps -p '.$this->pid;
        exec($command, $op);
        if (!isset($op[1])) {
            return false;
        } else {
            return true;
        }
    }

    public function start()
    {
        if ($this->command != '') {
            $this->runCom();
        } else {
            return true;
        }
    }

    public function stop()
    {
        $command = 'kill '.$this->pid;
        exec($command);
        if ($this->status() == false) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            return true;
        } else {
            return false;
        }
    }
}