Amenadiel\JpGraph\Plot\Plot

Complexity

Total Complexity

44

Size/Duplication

Total Lines	219
Duplicated Lines	31.05 %

Coupling/Cohesion

Components	8
Dependencies	1

Importance

Changes

0

19 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	0	25	6
A	Stroke()	0	4	1
A	HideLegend()	0	4	1
A	DoLegend()	0	6	2
A	StrokeDataValue()	0	4	1
A	SetCSIMTargets()	0	6	1
A	GetCSIMareas()	0	4	1
A	PreStrokeAdjust()	0	7	3
A	PreScaleSetup()	0	4	1
D	Min()	30	30	9
D	Max()	31	31	9
A	SetColor()	0	4	1
A	SetLegend()	7	7	1
A	SetWeight()	0	4	1
A	SetLineWeight()	0	4	1
A	SetCenter()	0	4	1
A	StrokeMargin()	0	4	1
A	Legend()	0	6	2
A	Clear()	0	6	1

<?php

namespace Amenadiel\JpGraph\Plot;

//===================================================
// CLASS Plot
// Description: Abstract base class for all concrete plot classes
//===================================================
class Plot
{
    public $numpoints = 0;
    public $value;
    public $legend           = '';
    public $coords           = array();
    public $color            = 'black';
    public $hidelegend       = false;
    public $line_weight      = 1;
    public $csimtargets      = array();
    public $csimwintargets      = array(); // Array of targets for CSIM
    public $csimareas        = ''; // Resultant CSIM area tags
    public $csimalts         = null; // ALT:s for corresponding target
    public $legendcsimtarget = '';
    public $legendcsimwintarget = '';
    public $legendcsimalt    = '';
    protected $weight        = 1;
    protected $center        = false;

    protected $inputValues;
    protected $isRunningClear = false;

    public function __construct($aDatay, $aDatax = false)
    {
        $this->numpoints = count($aDatay);
        if ($this->numpoints == 0) {
            Util\JpGraphError::RaiseL(25121); //("Empty input data array specified for plot. Must have at least one data point.");
        }

        if (!$this->isRunningClear) {
            $this->inputValues           = array();
            $this->inputValues['aDatay'] = $aDatay;
            $this->inputValues['aDatax'] = $aDatax;
        }

        $this->coords[0] = $aDatay;
        if (is_array($aDatax)) {
            $this->coords[1] = $aDatax;
            $n               = count($aDatax);
            for ($i = 0; $i < $n; ++$i) {
                if (!is_numeric($aDatax[$i])) {
                    Util\JpGraphError::RaiseL(25070);
                }
            }
        }
        $this->value = new DisplayValue();
    }

    // Stroke the plot
    // "virtual" function which must be implemented by
    // the subclasses
    public function Stroke($aImg, $aXScale, $aYScale)
    {
        Util\JpGraphError::RaiseL(25122); //("JpGraph: Stroke() must be implemented by concrete subclass to class Plot");
    }

    public function HideLegend($f = true)
    {
        $this->hidelegend = $f;
    }

    public function DoLegend($graph)
    {
        if (!$this->hidelegend) {
            $this->Legend($graph);
        }
    }

    public function StrokeDataValue($img, $aVal, $x, $y)
    {
        $this->value->Stroke($img, $aVal, $x, $y);
    }

    // Set href targets for CSIM
    public function SetCSIMTargets($aTargets, $aAlts = '', $aWinTargets = '')
    {
        $this->csimtargets    = $aTargets;
        $this->csimwintargets = $aWinTargets;

It seems like $aWinTargets of type string is incompatible with the declared type array of property $csimwintargets.

        $this->csimalts       = $aAlts;
    }

    // Get all created areas
    public function GetCSIMareas()
    {
        return $this->csimareas;
    }

    // "Virtual" function which gets called before any scale
    // or axis are stroked used to do any plot specific adjustment
    public function PreStrokeAdjust($aGraph)
    {
        if (substr($aGraph->axtype, 0, 4) == "text" && (isset($this->coords[1]))) {
            Util\JpGraphError::RaiseL(25123); //("JpGraph: You can't use a text X-scale with specified X-coords. Use a \"int\" or \"lin\" scale instead.");

75% of this comment could be valid code. Did you maybe forget this after debugging?

        }
        return true;
    }

    // Virtual function to the the concrete plot class to make any changes to the graph
    // and scale before the stroke process begins
    public function PreScaleSetup($aGraph)
    {
        // Empty
    }

    // Get minimum values in plot
    public function Min()

This method seems to be duplicated in your project.

    {
        if (isset($this->coords[1])) {
            $x = $this->coords[1];
        } else {
            $x = '';
        }
        if ($x != '' && count($x) > 0) {
            $xm = min($x);
        } else {
            $xm = 0;
        }
        $y   = $this->coords[0];
        $cnt = count($y);
        if ($cnt > 0) {
            $i = 0;
            while ($i < $cnt && !is_numeric($ym = $y[$i])) {
                $i++;
            }
            while ($i < $cnt) {
                if (is_numeric($y[$i])) {
                    $ym = min($ym, $y[$i]);

The variable $ym does not seem to be defined for all execution paths leading up to this point.

                }
                ++$i;
            }
        } else {
            $ym = '';
        }
        return array($xm, $ym);
    }

    // Get maximum value in plot
    public function Max()

This method seems to be duplicated in your project.

    {
        if (isset($this->coords[1])) {
            $x = $this->coords[1];
        } else {
            $x = '';
        }

        if ($x != '' && count($x) > 0) {
            $xm = max($x);
        } else {
            $xm = $this->numpoints - 1;
        }
        $y = $this->coords[0];
        if (count($y) > 0) {
            $cnt = count($y);
            $i   = 0;
            while ($i < $cnt && !is_numeric($ym = $y[$i])) {
                $i++;
            }
            while ($i < $cnt) {
                if (is_numeric($y[$i])) {
                    $ym = max($ym, $y[$i]);

The variable $ym does not seem to be defined for all execution paths leading up to this point.

                }
                ++$i;
            }
        } else {
            $ym = '';
        }
        return array($xm, $ym);
    }

    public function SetColor($aColor)
    {
        $this->color = $aColor;
    }

    public function SetLegend($aLegend, $aCSIM = '', $aCSIMAlt = '', $aCSIMWinTarget = '')

This method seems to be duplicated in your project.

    {
        $this->legend              = $aLegend;
        $this->legendcsimtarget    = $aCSIM;
        $this->legendcsimwintarget = $aCSIMWinTarget;
        $this->legendcsimalt       = $aCSIMAlt;
    }

    public function SetWeight($aWeight)
    {
        $this->weight = $aWeight;
    }

    public function SetLineWeight($aWeight = 1)
    {
        $this->line_weight = $aWeight;
    }

    public function SetCenter($aCenter = true)
    {
        $this->center = $aCenter;
    }

    // This method gets called by Graph class to plot anything that should go
    // into the margin after the margin color has been set.
    public function StrokeMargin($aImg)

The parameter $aImg is not used and could be removed.

    {
        return true;
    }

    // Framework function the chance for each plot class to set a legend
    public function Legend($aGraph)
    {
        if ($this->legend != '') {
            $aGraph->legend->Add($this->legend, $this->color, '', 0, $this->legendcsimtarget, $this->legendcsimalt, $this->legendcsimwintarget);
        }
    }

    public function Clear()
    {
        $this->isRunningClear = true;
        $this->__construct($this->inputValues['aDatay'], $this->inputValues['aDatax']);
        $this->isRunningClear = false;
    }
} // Class