Solver - Code Metrics - Inspection of "current tests bug fixed" - dsd-meetme/backend - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a92f31...97f136 )

by claudio

created 2015-12-30 16:48 UTC

Solver D

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	579
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

87.44%

Importance

Changes	6
Bugs	3	Features	0

Metric	Value
wmc	80
c	6
b	3
f	0
lcom	1
cbo	4
dl	0
loc	579
ccs	188
cts	215
cp	0.8744
rs	4.8718

43 Methods

Rating	Name	Size	Complexity
A	__construct()	7	1
A	getPath()	4	1
A	getSchedule()	4	1
A	setSchedule()	5	1
A	getUsers()	4	1
A	setUsers()	5	1
A	getMeetings()	4	1
A	setMeetings()	5	1
A	getTimeSlots()	4	1
A	setTimeSlots()	8	3
A	getMaxTimeSlots()	4	1
A	setMaxTimeSlots()	8	3
A	getMeetingsAvailability()	4	1
A	checkGlpsol()	5	2
A	__destruct()	4	1
A	setMeetingsAvailability()	15	4
A	arrayPad()	7	2
A	getMeetingsDuration()	4	1
B	setMeetingsDuration()	16	5
A	getUsersAvailability()	4	1
A	setUsersAvailability()	16	4
A	getUsersMeetings()	4	1
A	setUsersMeetings()	14	4
A	solve()	11	2
A	writeData()	10	1
A	checkData()	5	1
A	checkArrayProprieties()	6	3
A	checkIntProprieties()	6	4
A	writeUsers()	4	1
A	writeCSVArrayNoKey()	10	2
A	writeCSV()	14	2
A	writeMeetings()	4	1
A	writeMeetingsDuration()	4	1
A	writeCSVArray()	10	2
A	writeMeetingsAvailability()	4	1
A	writeCSVMatrix()	11	3
A	writeUsersAvailability()	4	1
A	writeUsersMeetings()	4	1
A	writeModelFile()	10	2
A	getXResults()	4	1
B	readCSVFile()	24	6
A	getYResults()	4	1
A	getOutput()	6	2

How to fix Complexity

<?php
/**
 * Created by PhpStorm.
 * User: Claudio Cardinale <[email protected]>
 * Date: 07/12/15
 * Time: 21.18
 * This program is free software; you can redistribute it and/or
 * modify it under the terms of the GNU General Public License
 * as published by the Free Software Foundation; either version 2
 * 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 General Public License for more details.
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
 */

namespace plunner\Console\Commands\Optimise;

use Illuminate\Console\Scheduling\Schedule;
use Illuminate\Foundation\Application;

/**
 * Class Solver
 * @package plunner\Console\Commands\Optimise
 * @author Claudio Cardinale <[email protected]>
 * @copyright 2015 Claudio Cardinale
 * @version 1.0.0
 */
class Solver
{
    /**
     * @var Path
     */
    private $path;
    /**
     * @var string[]
     */
    private $users;
    /**
     * @var string[]
     */
    private $meetings;
    /**
     * @var int
     */
    private $timeSlots = 0;
    /**
     * @var int
     */
    private $maxTimeSlots = 0;
    /**
     * @var string[]
     */
    private $meetingsAvailability;
    /**
     * @var string[]
     */
    private $meetingsDuration;
    /**
     * @var string[]
     */
    private $usersAvailability;
    /**
     * @var string[]
     */
    private $usersMeetings;

    /**
     * @var Schedule laravel schedule object needed to perform command in background
     */
    private $schedule;

    /**
     * @var Application
     */
    private $laravel;

    //TODo clone function rmemeber to clone also path or create a new one
    //TODO mehtod to check if all variables are correctly set
    //TODO check no duplicates
    //TODO exception if glpsol return erros
    //TODO intercept all erros of system calls like mkdir
    //TODO to_string

    /**
     * Solver constructor.
     * @param Schedule $schedule
     * @param Application $laravel
     * @throws OptimiseException on general problems
     */
    public function __construct(Schedule $schedule, Application $laravel)
    {
        self::checkGlpsol();
        $this->path = Path::createPath();
        $this->schedule = $schedule;
        $this->laravel = $laravel;
    }

    /**
     * @throws OptimiseException
     */
    static private function checkGlpsol()
    {
        if (!(`which glpsol`))
            throw new OptimiseException('glpsol is not installed');
    }

    /**
     * @throws OptimiseException
     */
    public function __destruct()
    {
        $this->path = null; //call the path destruct
    }

    /**
     * @return Path
     */
    public function getPath()
    {
        return clone $this->path;
    }

    /**
     * @return Schedule
     */
    public function getSchedule()
    {
        return $this->schedule;
    }

    /**
     * @param Schedule $schedule
     * @return Solver
     */
    public function setSchedule($schedule)
    {
        $this->schedule = $schedule;
        return $this;
    }

    /**
     * @return \string[]
     */
    public function getUsers()
    {
        return $this->users;
    }

    /**
     * @param string[] $users
     * @return Solver
     */
    public function setUsers($users)
    {
        $this->users = $users;
        return $this;
    }

    /**
     * @return string[]
     */
    public function getMeetings()
    {
        return $this->meetings;
    }

    /**
     * @param string[] $meetings
     * @return Solver
     */
    public function setMeetings($meetings)
    {
        $this->meetings = $meetings;
        return $this;
    }

    /**
     * @return int
     */
    public function getTimeSlots()
    {
        return $this->timeSlots;
    }

    /**
     * @param int $timeSlots
     * @return Solver
     * @throws OptimiseException
     */
    public function setTimeSlots($timeSlots)
    {
        if (!is_int($timeSlots) || $timeSlots <= 0)
            throw new OptimiseException('$timeSlots is not integer or it is not >0');

        $this->timeSlots = $timeSlots;
        return $this;
    }

    /**
     * @return int
     */
    public function getMaxTimeSlots()
    {
        return $this->maxTimeSlots;
    }

    /**
     * @param int $maxTimeSlots
     * @return Solver
     * @throws OptimiseException
     */
    public function setMaxTimeSlots($maxTimeSlots)
    {
        if (!is_int($maxTimeSlots) || $maxTimeSlots <= 0)
            throw new OptimiseException('$maxTimeSlots is not integer or it is not >0');

        $this->maxTimeSlots = $maxTimeSlots;
        return $this;
    }

    /**
     * @return string[]
     */
    public function getMeetingsAvailability()
    {
        return $this->meetingsAvailability;
    }

    /**
     * @param string[] $meetingsAvailability
     * @return Solver
     * @throws OptimiseException
     */
    public function setMeetingsAvailability($meetingsAvailability)
    {
        $meetings = array_keys($meetingsAvailability);
        if (array_diff($meetings, $this->meetings))
            throw new OptimiseException('meetings different from meetings set');
        foreach ($meetingsAvailability as $key => $meetingsAvailabilityS) {
            $timeSlots = array_keys($meetingsAvailabilityS);
            if (count($timeSlots) != $this->timeSlots)
                throw new OptimiseException('timeSlots different from timeSlots set');
            $meetingsAvailability[$key] = self::arrayPad($meetingsAvailabilityS, $this->timeSlots + $this->maxTimeSlots, 0);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
        }

        $this->meetingsAvailability = $meetingsAvailability;

        return $this;
    }

    /**
     * implementation of arraypad that doesn't change original keys<br/>
     * <strong>CAUTION: Only positive $len</strong>
     * @param array $array
     * @return array
     */
    static private function arrayPad(array $array, $len, $pad)
    {
        $len = $len - count($array);
        for ($i = 0; $i < $len; $i++)
            $array[] = $pad;
        return $array;
    }

    /**
     * @return string[]
     */
    public function getMeetingsDuration()
    {
        return $this->meetingsDuration;
    }

    /**
     * @param string[] $meetingsDuration
     * @return Solver
     * @throws OptimiseException
     */
    public function setMeetingsDuration($meetingsDuration)
    {
        $meetings = array_keys($meetingsDuration);
        if (array_diff($meetings, $this->meetings)) {
            print "";
            throw new OptimiseException('meetings different from meetings set');
        }
        foreach ($meetingsDuration as $duration) {
            $duration = (int)$duration; //TODO fix this (fix for optimise)
            if (!is_int($duration) || $duration <= 0)
                throw new OptimiseException('duration is not integer or it is not >0');
        }

        $this->meetingsDuration = $meetingsDuration;
        return $this;
    }

    /**
     * @return string[]
     */
    public function getUsersAvailability()
    {
        return $this->usersAvailability;
    }

    /**
     * @param string[] $usersAvailability
     * @return Solver
     * @throws OptimiseException
     */
    public function setUsersAvailability($usersAvailability)
    {
        $users = array_keys($usersAvailability);
        if (array_diff($users, $this->users))
            throw new OptimiseException('users different from users set');
        foreach ($usersAvailability as $key => $usersAvailabilityS) {
            $timeSlots = array_keys($usersAvailabilityS);
            if (count($timeSlots) != $this->timeSlots)
                throw new OptimiseException('timeSlots different from timeSlots set');

            $usersAvailability[$key] = self::arrayPad($usersAvailabilityS, $this->timeSlots + $this->maxTimeSlots, 0);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
        }

        $this->usersAvailability = $usersAvailability;

        return $this;
    }

    /**
     * @return string[]
     */
    public function getUsersMeetings()
    {
        return $this->usersMeetings;
    }

    /**
     * @param string[] $usersMeetings
     * @return Solver
     * @throws OptimiseException
     */
    public function setUsersMeetings($usersMeetings)
    {
        $users = array_keys($usersMeetings);
        if (array_diff($users, $this->users))
            throw new OptimiseException('users different from users set');
        foreach ($usersMeetings as $usersMeetingsS) {
            $meetings = array_keys($usersMeetingsS);
            if (array_diff($meetings, $this->meetings))
                throw new OptimiseException('meetings different from meetings set');
        }

        $this->usersMeetings = $usersMeetings;
        return $this;
    }

    /**
     * @return Solver
     * @throws OptimiseException
     */
    public function solve()
    {
        $this->writeData();
        $this->writeModelFile();
        $event = $this->schedule->exec('glpsol --math ' . $this->path->getModelPath())->sendOutputTo($this->path->getOutputPath())->after(function () {
        }); //this just to execute in foreground
        if ($event->isDue($this->laravel))
            $event->run($this->laravel);
        //TODO catch glpsol errors
        return $this;
    }

    /**
     * @throws OptimiseException
     */
    private function writeData()
    {
        $this->checkData();
        $this->writeUsers();
        $this->writeMeetings();
        $this->writeMeetingsDuration();
        $this->writeMeetingsAvailability();
        $this->writeUsersAvailability();
        $this->writeUsersMeetings();
    }

    /**
     * @throws OptimiseException
     */
    private function checkData()
    {
        $this->checkArrayProprieties(['users', 'meetings', 'meetingsAvailability', 'meetingsDuration', 'usersAvailability', 'usersMeetings']);
        $this->checkIntProprieties(['timeSlots', 'maxTimeSlots']);
    }

    /**
     * @param $proprieties
     * @throws OptimiseException
     */
    private function checkArrayProprieties($proprieties)
    {
        foreach ($proprieties as $propriety)
            if (count($this->$propriety) == 0)
                throw new OptimiseException($propriety . ' is not set correctly');
    }

    /**
     * @param $proprieties
     * @throws OptimiseException
     */
    private function checkIntProprieties($proprieties)
    {
        foreach ($proprieties as $propriety)
            if (!is_int($this->$propriety) || $this->$propriety <= 0)
                throw new OptimiseException($propriety . ' is not set correctly');
    }

    /**
     * @throws OptimiseException
     */
    private function writeUsers()
    {
        self::writeCSVArrayNoKey($this->path->getUsersPath(), $this->users);
    }

    /**
     * @param string $file
     * @param array $data
     * @throws OptimiseException
     */
    static private function writeCSVArrayNoKey($file, $data)
    {
        $f = function ($fp, $data) {
            foreach ($data as $field) {
                fputcsv($fp, [$field]);
            }
        };

        self::writeCSV($file, $data, ['i'], $f);
    }

    /**
     * @param string $file
     * @param array $data
     * @param array $heading
     * @param \Closure $writer
     * @throws OptimiseException
     */
    static private function writeCSV($file, $data, $heading, \Closure $writer)
    {
        $fp = @fopen($file, 'w');
        if (!$fp)
            throw new OptimiseException('problem during creation of a file');

        fputcsv($fp, $heading);

        $writer($fp, $data);

        //fputcsv($fp, []); //empty line

        fclose($fp);
    }

    /**
     * @throws OptimiseException
     */
    private function writeMeetings()
    {
        self::writeCSVArrayNoKey($this->path->getMeetingsPath(), $this->meetings);
    }

    /**
     * @throws OptimiseException
     */
    private function writeMeetingsDuration()
    {
        self::writeCSVArray($this->path->getMeetingsDurationPath(), $this->meetingsDuration, 'MeetingsDuration');
    }

    /**
     * @param string $file
     * @param array $data
     * @param string $name
     * @throws OptimiseException
     */
    static private function writeCSVArray($file, $data, $name)
    {
        $f = function ($fp, $data) {
            foreach ($data as $key => $field) {
                fputcsv($fp, [$key, $field]);
            }
        };

        self::writeCSV($file, $data, ['i', $name], $f);
    }

    /**
     * @throws OptimiseException
     */
    private function writeMeetingsAvailability()
    {
        self::writeCSVMatrix($this->path->getMeetingsAvailabilityPath(), $this->meetingsAvailability, 'MeetingsAvailability');
    }

    /**
     * @param string $file
     * @param array $data
     * @param string $name
     * @throws OptimiseException
     */
    static private function writeCSVMatrix($file, $data, $name)
    {
        $f = function ($fp, $data) {
            foreach ($data as $key => $field) {
                foreach ($field as $key2 => $field2)
                    fputcsv($fp, [$key, $key2, $field2]);
            }
        };

        self::writeCSV($file, $data, ['i', 'j', $name], $f);
    }

    /**
     * @throws OptimiseException
     */
    private function writeUsersAvailability()
    {
        self::writeCSVMatrix($this->path->getUsersAvailabilityPath(), $this->usersAvailability, 'UsersAvailability');
    }

    /**
     * @throws OptimiseException
     */
    private function writeUsersMeetings()
    {
        self::writeCSVMatrix($this->path->getUsersMeetingsPath(), $this->usersMeetings, 'UsersMeetings');
    }

    /**
     * @throws OptimiseException
     */
    private function writeModelFile()
    {
        $strReplaceS = array('{USERS_PATH}', '{MEETINGS_PATH}', '{USER_AVAILABILITY_PATH}', '{MEETINGS_AVAILABILITY_PATH}', '{USER_MEETINGS_PATH}', '{MEETINGS_DURATION_PATH}', '{TIME_SLOTS}', '{MAX_TIME_SLOTS}', '{X_OUT_PATH}', '{Y_OUT_PATH}');
        $strReplaceR = array($this->path->getUsersPath(), $this->path->getMeetingsPath(), $this->path->getUsersAvailabilityPath(), $this->path->getMeetingsAvailabilityPath(), $this->path->getUsersMeetingsPath(), $this->path->getMeetingsDurationPath(), $this->timeSlots, $this->maxTimeSlots, $this->path->getXPath(), $this->path->getYPath());
        $f = @fopen($this->path->getModelPath(), "w");
        if (!$f)
            throw new OptimiseException('problem during creation of a file');
        fwrite($f, str_replace($strReplaceS, $strReplaceR, file_get_contents(__DIR__ . "/model.stub")));
        fclose($f);
    }

    /**
     * @return array
     * @throws OptimiseException
     */
    public function getXResults()
    {
        return self::readCSVFile($this->path->getXPath());
    }

    /**
     * @param string $file
     * @return array
     * @throws OptimiseException
     */
    static private function readCSVFile($file)
    {
        if (!file_exists($file) || !filesize($file))
            throw new OptimiseException('no results file');

        $handle = @fopen($file, "r");
        if (!$handle)
            throw new OptimiseException('problems during reading the file');

        $ret = [];
        fgetcsv($handle); //skip head
        while (($data = fgetcsv($handle)) !== FALSE) {
            if (count($data) != 3) {
                fclose($handle);
                throw new OptimiseException('problems during parsing the file');
            }

            $ret[$data[0]][$data[1]] = $data[2];
        }

        fclose($handle);

        return $ret;
    }

    /**
     * @return array
     * @throws OptimiseException
     */
    public function getYResults()
    {
        return self::readCSVFile($this->path->getYPath());
    }

    /**
     * @return string
     * @throws OptimiseException
     */
    public function getOutput()
    {
        if (!($data = file_get_contents($this->path->getOutputPath())))
            throw new OptimiseException('problems during reading the file');
        return $data;
    }
}

1		<?php
2		/**
3		* Created by PhpStorm.
4		* User: Claudio Cardinale <[email protected]>
5		* Date: 07/12/15
6		* Time: 21.18
7		* This program is free software; you can redistribute it and/or
8		* modify it under the terms of the GNU General Public License
9		* as published by the Free Software Foundation; either version 2
10		* of the License, or (at your option) any later version.
11		* This program is distributed in the hope that it will be useful,
12		* but WITHOUT ANY WARRANTY; without even the implied warranty of
13		* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
14		* GNU General Public License for more details.
15		* You should have received a copy of the GNU General Public License
16		* along with this program; if not, write to the Free Software
17		* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
18		*/
19
20		namespace plunner\Console\Commands\Optimise;
21
22		use Illuminate\Console\Scheduling\Schedule;
23		use Illuminate\Foundation\Application;
24
25		/**
26		* Class Solver
27		* @package plunner\Console\Commands\Optimise
28		* @author Claudio Cardinale <[email protected]>
29		* @copyright 2015 Claudio Cardinale
30		* @version 1.0.0
31		*/
32		class Solver
33		{
34		/**
35		* @var Path
36		*/
37		private $path;
38		/**
39		* @var string[]
40		*/
41		private $users;
42		/**
43		* @var string[]
44		*/
45		private $meetings;
46		/**
47		* @var int
48		*/
49		private $timeSlots = 0;
50		/**
51		* @var int
52		*/
53		private $maxTimeSlots = 0;
54		/**
55		* @var string[]
56		*/
57		private $meetingsAvailability;
58		/**
59		* @var string[]
60		*/
61		private $meetingsDuration;
62		/**
63		* @var string[]
64		*/
65		private $usersAvailability;
66		/**
67		* @var string[]
68		*/
69		private $usersMeetings;
70
71		/**
72		* @var Schedule laravel schedule object needed to perform command in background
73		*/
74		private $schedule;
75
76		/**
77		* @var Application
78		*/
79		private $laravel;
80
81		//TODo clone function rmemeber to clone also path or create a new one
82		//TODO mehtod to check if all variables are correctly set
83		//TODO check no duplicates
84		//TODO exception if glpsol return erros
85		//TODO intercept all erros of system calls like mkdir
86		//TODO to_string
87
88		/**
89		* Solver constructor.
90		* @param Schedule $schedule
91		* @param Application $laravel
92		* @throws OptimiseException on general problems
93		*/
94	4	public function __construct(Schedule $schedule, Application $laravel)
95		{
96	4	self::checkGlpsol();
97	4	$this->path = Path::createPath();
98	4	$this->schedule = $schedule;
99	4	$this->laravel = $laravel;
100	4	}
101
102		/**
103		* @throws OptimiseException
104		*/
105	4	static private function checkGlpsol()
106		{
107	4	if (!(`which glpsol`))
108	4	throw new OptimiseException('glpsol is not installed');
109	4	}
110
111		/**
112		* @throws OptimiseException
113		*/
114	2	public function __destruct()
115		{
116	2	$this->path = null; //call the path destruct
117	2	}
118
119		/**
120		* @return Path
121		*/
122		public function getPath()
123		{
124		return clone $this->path;
125		}
126
127		/**
128		* @return Schedule
129		*/
130		public function getSchedule()
131		{
132		return $this->schedule;
133		}
134
135		/**
136		* @param Schedule $schedule
137		* @return Solver
138		*/
139		public function setSchedule($schedule)
140		{
141		$this->schedule = $schedule;
142		return $this;
143		}
144
145		/**
146		* @return \string[]
147		*/
148	2	public function getUsers()
149		{
150	2	return $this->users;
151		}
152
153		/**
154		* @param string[] $users
155		* @return Solver
156		*/
157	4	public function setUsers($users)
158		{
159	4	$this->users = $users;
160	4	return $this;
161		}
162
163		/**
164		* @return string[]
165		*/
166	2	public function getMeetings()
167		{
168	2	return $this->meetings;
169		}
170
171		/**
172		* @param string[] $meetings
173		* @return Solver
174		*/
175	4	public function setMeetings($meetings)
176		{
177	4	$this->meetings = $meetings;
178	4	return $this;
179		}
180
181		/**
182		* @return int
183		*/
184		public function getTimeSlots()
185		{
186		return $this->timeSlots;
187		}
188
189		/**
190		* @param int $timeSlots
191		* @return Solver
192		* @throws OptimiseException
193		*/
194	4	public function setTimeSlots($timeSlots)
195		{
196	4	if (!is_int($timeSlots) \|\| $timeSlots <= 0)
197	4	throw new OptimiseException('$timeSlots is not integer or it is not >0');
198
199	4	$this->timeSlots = $timeSlots;
200	4	return $this;
201		}
202
203		/**
204		* @return int
205		*/
206		public function getMaxTimeSlots()
207		{
208		return $this->maxTimeSlots;
209		}
210
211		/**
212		* @param int $maxTimeSlots
213		* @return Solver
214		* @throws OptimiseException
215		*/
216	4	public function setMaxTimeSlots($maxTimeSlots)
217		{
218	4	if (!is_int($maxTimeSlots) \|\| $maxTimeSlots <= 0)
219	4	throw new OptimiseException('$maxTimeSlots is not integer or it is not >0');
220
221	4	$this->maxTimeSlots = $maxTimeSlots;
222	4	return $this;
223		}
224
225		/**
226		* @return string[]
227		*/
228		public function getMeetingsAvailability()
229		{
230		return $this->meetingsAvailability;
231		}
232
233		/**
234		* @param string[] $meetingsAvailability
235		* @return Solver
236		* @throws OptimiseException
237		*/
238	4	public function setMeetingsAvailability($meetingsAvailability)
239		{
240	4	$meetings = array_keys($meetingsAvailability);
241	4	if (array_diff($meetings, $this->meetings))
242	4	throw new OptimiseException('meetings different from meetings set');
243	4	foreach ($meetingsAvailability as $key => $meetingsAvailabilityS) {
244	4	$timeSlots = array_keys($meetingsAvailabilityS);
245	4	if (count($timeSlots) != $this->timeSlots)
246	4	throw new OptimiseException('timeSlots different from timeSlots set');
247	4	$meetingsAvailability[$key] = self::arrayPad($meetingsAvailabilityS, $this->timeSlots + $this->maxTimeSlots, 0);
		0 ignored issues – show Bug introduced 2015-12-18 14:36 UTC by Report Bug Copy Issue Report It seems like `$meetingsAvailabilityS` defined by `$meetingsAvailabilityS` on line `243` can also be of type `string`; however, `plunner\Console\Commands...mise\Solver::arrayPad()` does only seem to accept `array`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
248	4	}
249
250	4	$this->meetingsAvailability = $meetingsAvailability;
		0 ignored issues – show Documentation Bug introduced 2015-12-18 14:26 UTC by Report Bug Copy Issue Report It seems like `$meetingsAvailability` of type `array<integer,string\|array>` is incompatible with the declared type `array<integer,string>` of property `$meetingsAvailability`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
251	4	return $this;
252		}
253
254		/**
255		* implementation of arraypad that doesn't change original keys<br/>
256		* <strong>CAUTION: Only positive $len</strong>
257		* @param array $array
258		* @return array
259		*/
260	4	static private function arrayPad(array $array, $len, $pad)
261		{
262	4	$len = $len - count($array);
263	4	for ($i = 0; $i < $len; $i++)
264	4	$array[] = $pad;
265	4	return $array;
266		}
267
268		/**
269		* @return string[]
270		*/
271		public function getMeetingsDuration()
272		{
273		return $this->meetingsDuration;
274		}
275
276		/**
277		* @param string[] $meetingsDuration
278		* @return Solver
279		* @throws OptimiseException
280		*/
281	4	public function setMeetingsDuration($meetingsDuration)
282		{
283	4	$meetings = array_keys($meetingsDuration);
284	4	if (array_diff($meetings, $this->meetings)) {
285		print "";
286		throw new OptimiseException('meetings different from meetings set');
287		}
288	4	foreach ($meetingsDuration as $duration) {
289	4	$duration = (int)$duration; //TODO fix this (fix for optimise)
290	4	if (!is_int($duration) \|\| $duration <= 0)
291	4	throw new OptimiseException('duration is not integer or it is not >0');
292	4	}
293
294	4	$this->meetingsDuration = $meetingsDuration;
295	4	return $this;
296		}
297
298		/**
299		* @return string[]
300		*/
301		public function getUsersAvailability()
302		{
303		return $this->usersAvailability;
304		}
305
306		/**
307		* @param string[] $usersAvailability
308		* @return Solver
309		* @throws OptimiseException
310		*/
311	4	public function setUsersAvailability($usersAvailability)
312		{
313	4	$users = array_keys($usersAvailability);
314	4	if (array_diff($users, $this->users))
315	4	throw new OptimiseException('users different from users set');
316	4	foreach ($usersAvailability as $key => $usersAvailabilityS) {
317	4	$timeSlots = array_keys($usersAvailabilityS);
318	4	if (count($timeSlots) != $this->timeSlots)
319	4	throw new OptimiseException('timeSlots different from timeSlots set');
320
321	4	$usersAvailability[$key] = self::arrayPad($usersAvailabilityS, $this->timeSlots + $this->maxTimeSlots, 0);
		0 ignored issues – show Bug introduced 2015-12-28 22:43 UTC by Report Bug Copy Issue Report It seems like `$usersAvailabilityS` defined by `$usersAvailabilityS` on line `316` can also be of type `string`; however, `plunner\Console\Commands...mise\Solver::arrayPad()` does only seem to accept `array`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
322	4	}
323
324	4	$this->usersAvailability = $usersAvailability;
		0 ignored issues – show Documentation Bug introduced 2015-12-18 14:26 UTC by Report Bug Copy Issue Report It seems like `$usersAvailability` of type `array<integer,string\|array>` is incompatible with the declared type `array<integer,string>` of property `$usersAvailability`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
325	4	return $this;
326		}
327
328		/**
329		* @return string[]
330		*/
331		public function getUsersMeetings()
332		{
333		return $this->usersMeetings;
334		}
335
336		/**
337		* @param string[] $usersMeetings
338		* @return Solver
339		* @throws OptimiseException
340		*/
341	4	public function setUsersMeetings($usersMeetings)
342		{
343	4	$users = array_keys($usersMeetings);
344	4	if (array_diff($users, $this->users))
345	4	throw new OptimiseException('users different from users set');
346	4	foreach ($usersMeetings as $usersMeetingsS) {
347	4	$meetings = array_keys($usersMeetingsS);
348	4	if (array_diff($meetings, $this->meetings))
349	4	throw new OptimiseException('meetings different from meetings set');
350	4	}
351
352	4	$this->usersMeetings = $usersMeetings;
353	4	return $this;
354		}
355
356		/**
357		* @return Solver
358		* @throws OptimiseException
359		*/
360	4	public function solve()
361		{
362	4	$this->writeData();
363	4	$this->writeModelFile();
364		$event = $this->schedule->exec('glpsol --math ' . $this->path->getModelPath())->sendOutputTo($this->path->getOutputPath())->after(function () {
365	4	}); //this just to execute in foreground
366	4	if ($event->isDue($this->laravel))
367	4	$event->run($this->laravel);
368		//TODO catch glpsol errors
369	4	return $this;
370		}
371
372		/**
373		* @throws OptimiseException
374		*/
375	4	private function writeData()
376		{
377	4	$this->checkData();
378	4	$this->writeUsers();
379	4	$this->writeMeetings();
380	4	$this->writeMeetingsDuration();
381	4	$this->writeMeetingsAvailability();
382	4	$this->writeUsersAvailability();
383	4	$this->writeUsersMeetings();
384	4	}
385
386		/**
387		* @throws OptimiseException
388		*/
389	4	private function checkData()
390		{
391	4	$this->checkArrayProprieties(['users', 'meetings', 'meetingsAvailability', 'meetingsDuration', 'usersAvailability', 'usersMeetings']);
392	4	$this->checkIntProprieties(['timeSlots', 'maxTimeSlots']);
393	4	}
394
395		/**
396		* @param $proprieties
397		* @throws OptimiseException
398		*/
399	4	private function checkArrayProprieties($proprieties)
400		{
401	4	foreach ($proprieties as $propriety)
402	4	if (count($this->$propriety) == 0)
403	4	throw new OptimiseException($propriety . ' is not set correctly');
404	4	}
405
406		/**
407		* @param $proprieties
408		* @throws OptimiseException
409		*/
410	4	private function checkIntProprieties($proprieties)
411		{
412	4	foreach ($proprieties as $propriety)
413	4	if (!is_int($this->$propriety) \|\| $this->$propriety <= 0)
414	4	throw new OptimiseException($propriety . ' is not set correctly');
415	4	}
416
417		/**
418		* @throws OptimiseException
419		*/
420	4	private function writeUsers()
421		{
422	4	self::writeCSVArrayNoKey($this->path->getUsersPath(), $this->users);
423	4	}
424
425		/**
426		* @param string $file
427		* @param array $data
428		* @throws OptimiseException
429		*/
430	4	static private function writeCSVArrayNoKey($file, $data)
431		{
432		$f = function ($fp, $data) {
433	4	foreach ($data as $field) {
434	4	fputcsv($fp, [$field]);
435	4	}
436	4	};
437
438	4	self::writeCSV($file, $data, ['i'], $f);
439	4	}
440
441		/**
442		* @param string $file
443		* @param array $data
444		* @param array $heading
445		* @param \Closure $writer
446		* @throws OptimiseException
447		*/
448	4	static private function writeCSV($file, $data, $heading, \Closure $writer)
449		{
450	4	$fp = @fopen($file, 'w');
451	4	if (!$fp)
452	4	throw new OptimiseException('problem during creation of a file');
453
454	4	fputcsv($fp, $heading);
455
456	4	$writer($fp, $data);
457
458		//fputcsv($fp, []); //empty line
459
460	4	fclose($fp);
461	4	}
462
463		/**
464		* @throws OptimiseException
465		*/
466	4	private function writeMeetings()
467		{
468	4	self::writeCSVArrayNoKey($this->path->getMeetingsPath(), $this->meetings);
469	4	}
470
471		/**
472		* @throws OptimiseException
473		*/
474	4	private function writeMeetingsDuration()
475		{
476	4	self::writeCSVArray($this->path->getMeetingsDurationPath(), $this->meetingsDuration, 'MeetingsDuration');
477	4	}
478
479		/**
480		* @param string $file
481		* @param array $data
482		* @param string $name
483		* @throws OptimiseException
484		*/
485	4	static private function writeCSVArray($file, $data, $name)
486		{
487		$f = function ($fp, $data) {
488	4	foreach ($data as $key => $field) {
489	4	fputcsv($fp, [$key, $field]);
490	4	}
491	4	};
492
493	4	self::writeCSV($file, $data, ['i', $name], $f);
494	4	}
495
496		/**
497		* @throws OptimiseException
498		*/
499	4	private function writeMeetingsAvailability()
500		{
501	4	self::writeCSVMatrix($this->path->getMeetingsAvailabilityPath(), $this->meetingsAvailability, 'MeetingsAvailability');
502	4	}
503
504		/**
505		* @param string $file
506		* @param array $data
507		* @param string $name
508		* @throws OptimiseException
509		*/
510		static private function writeCSVMatrix($file, $data, $name)
511		{
512	4	$f = function ($fp, $data) {
513	4	foreach ($data as $key => $field) {
514	4	foreach ($field as $key2 => $field2)
515	4	fputcsv($fp, [$key, $key2, $field2]);
516	4	}
517	4	};
518
519	4	self::writeCSV($file, $data, ['i', 'j', $name], $f);
520	4	}
521
522		/**
523		* @throws OptimiseException
524		*/
525	4	private function writeUsersAvailability()
526		{
527	4	self::writeCSVMatrix($this->path->getUsersAvailabilityPath(), $this->usersAvailability, 'UsersAvailability');
528	4	}
529
530		/**
531		* @throws OptimiseException
532		*/
533	4	private function writeUsersMeetings()
534		{
535	4	self::writeCSVMatrix($this->path->getUsersMeetingsPath(), $this->usersMeetings, 'UsersMeetings');
536	4	}
537
538		/**
539		* @throws OptimiseException
540		*/
541	4	private function writeModelFile()
542		{
543	4	$strReplaceS = array('{USERS_PATH}', '{MEETINGS_PATH}', '{USER_AVAILABILITY_PATH}', '{MEETINGS_AVAILABILITY_PATH}', '{USER_MEETINGS_PATH}', '{MEETINGS_DURATION_PATH}', '{TIME_SLOTS}', '{MAX_TIME_SLOTS}', '{X_OUT_PATH}', '{Y_OUT_PATH}');
544	4	$strReplaceR = array($this->path->getUsersPath(), $this->path->getMeetingsPath(), $this->path->getUsersAvailabilityPath(), $this->path->getMeetingsAvailabilityPath(), $this->path->getUsersMeetingsPath(), $this->path->getMeetingsDurationPath(), $this->timeSlots, $this->maxTimeSlots, $this->path->getXPath(), $this->path->getYPath());
545	4	$f = @fopen($this->path->getModelPath(), "w");
546	4	if (!$f)
547	4	throw new OptimiseException('problem during creation of a file');
548	4	fwrite($f, str_replace($strReplaceS, $strReplaceR, file_get_contents(__DIR__ . "/model.stub")));
549	4	fclose($f);
550	4	}
551
552		/**
553		* @return array
554		* @throws OptimiseException
555		*/
556	4	public function getXResults()
557		{
558	4	return self::readCSVFile($this->path->getXPath());
559		}
560
561		/**
562		* @param string $file
563		* @return array
564		* @throws OptimiseException
565		*/
566	4	static private function readCSVFile($file)
567		{
568	4	if (!file_exists($file) \|\| !filesize($file))
569	4	throw new OptimiseException('no results file');
570
571	4	$handle = @fopen($file, "r");
572	4	if (!$handle)
573	4	throw new OptimiseException('problems during reading the file');
574
575	4	$ret = [];
576	4	fgetcsv($handle); //skip head
577	4	while (($data = fgetcsv($handle)) !== FALSE) {
578	4	if (count($data) != 3) {
579		fclose($handle);
580		throw new OptimiseException('problems during parsing the file');
581		}
582
583	4	$ret[$data[0]][$data[1]] = $data[2];
584	4	}
585
586	4	fclose($handle);
587
588	4	return $ret;
589		}
590
591		/**
592		* @return array
593		* @throws OptimiseException
594		*/
595	4	public function getYResults()
596		{
597	4	return self::readCSVFile($this->path->getYPath());
598		}
599
600		/**
601		* @return string
602		* @throws OptimiseException
603		*/
604		public function getOutput()
605		{
606		if (!($data = file_get_contents($this->path->getOutputPath())))
607		throw new OptimiseException('problems during reading the file');
608		return $data;
609		}
610		}

dsd-meetme / backend

Push — master ( a92f31...97f136 )

Solver D

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

43 Methods

How to fix Complexity

Complex Class

Duplication Side-by-Side

Filter issues like