Tree - Code Metrics - Inspection of "Namespaces, autoload" - mambax7/instruction - Measure and Improve Code Quality continuously with Scrutinizer

Completed
Push — master ( a4e09c...825da9 )

by Michael
created 2017-10-09 08:26 UTC
Tree B

↳ Parent: Project
Complexity

Total Complexity
Size/Duplication

Total Lines	422
Duplicated Lines	4.74 %
Coupling/Cohesion

Components	0
Dependencies	0
Importance

Changes
Metric	Value
dl	20
loc	422
rs	8.5454
c	0
b	0
f	0
wmc	49
lcom	0
cbo	0
7 Methods

Rating	Name	Duplication	Size	Complexity
C	_makePagesAdminOptions()	5	68	9
B	makePagesAdmin()	0	29	1
C	_makeCatsAdminOptions()	5	59	10
A	makeCatsAdmin()	0	20	1
F	_makePagesUserTree()	5	101	16
C	_makePagesUserCalc()	5	48	11
A	makePagesUser()	0	22	1
How to fix Duplicated Code Complexity

Duplicated Code

Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.
Common duplication problems, and corresponding solutions are:
If you have the same expression in different places: Extract expression to a method
If you have the same method in different sub-classes: Extract method, and pull up field to the parent class
If you have the same code in unrelated classes: Consider extracting the code to a new class, and injecting that class
Complex Class

Tip: Before tackling complexity, make sure that you eliminate any duplication first. This often can reduce the size of classes significantly.
Complex classes like Tree often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use Tree, and based on these observations, apply Extract Interface, too.
<?php namespace Xoopsmodules\instruction;

// Автор: andrey3761
// Вывод древообразного списка страниц в панели администрирования

//defined('XOOPS_ROOT_PATH') || exit('Restricted access');

define('INST_DIRNAME', basename(dirname(__DIR__)));

include_once $GLOBALS['xoops']->path('include/common.php');
// Подключаем трей
include_once $GLOBALS['xoops']->path('class/tree.php');

// Наследник класса XoopsObjectTree

/**
 * Class Tree
 * @package Xoopsmodules\instruction
 */
class Tree extends \XoopsObjectTree
{
    //    public function __construct()

    //    {
    //    }

    /**
     * @param        $key
     * @param        $ret
     * @param        $prefix_orig
     * @param        $objInsinstr
     * @param string $class
     * @param string $prefix_curr
     */
    public function _makePagesAdminOptions($key, &$ret, $prefix_orig, $objInsinstr, $class = 'odd', $prefix_curr = '')
    {
        if ($key > 0) {

            //
            $class = ('even' === $class) ? 'odd' : 'even';
            // ID инструкции ( Можно сделать статической )
            $instrid = $objInsinstr->getVar('instrid');

            // ID страницы
            $pageid = $this->tree[$key]['obj']->getVar('pageid');
            // Название страницы
            $pagetitle = $this->tree[$key]['obj']->getVar('title');
            // Вес
            $pageweight = $this->tree[$key]['obj']->getVar('weight');
            // Статус
            $pagestatus = $this->tree[$key]['obj']->getVar('status');
            // Тип страницы
            $pagetype = $this->tree[$key]['obj']->getVar('type');

            // Дочернии страницы
            $page_childs = $this->getAllChild($pageid);
            // Число дочерних страниц
            $num_childs = count($page_childs);

            // Действие - удаление
            $act_del = ($num_childs > 0) ? '<img src="../assets/icons/no_delete_mini.png" alt="' . _AM_INSTR_NODELPAGE . '" title="' . _AM_INSTR_NODELPAGE . '" />' : '<a href="instr.php?op=delpage&pageid='
                                                                                                                                                                      . $pageid
                                                                                                                                                                      . '"><img src="../assets/icons/delete_mini.png" alt="'
                                                                                                                                                                      . _AM_INSTRUCTION_DEL
                                                                                                                                                                      . '" title="'
                                                                                                                                                                      . _AM_INSTRUCTION_DEL
                                                                                                                                                                      . '"></a>';
            //
            $page_link = '<a name="pageid_' . $pageid . '" ' . ($pagetype ? 'href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '#pagetext"' : '') . '>' . $pagetitle . '</a>';

            $ret .= '<tr class="' . $class . '">
      <td>' . $prefix_curr . ' ' . $page_link . '</td>
      <td align="center" width="50">
        <input type="text" name="weights[]" size="2" value="' . $pageweight . '" />
        <input type="hidden" name="pageids[]" value="' . $pageid . '" />
      </td>
      <td align="center" width="180">';
            // Просмотре без кэша
            $ret .= ' <a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '&amp;nocache=1"><img src="../assets/icons/no_cache.png" alt="' . _AM_INSTR_DISPLAY_NOCACHE . '" title="' . _AM_INSTR_DISPLAY_NOCACHE . '" /></a> ';
            // Добавить подстраницу
            $ret .= ' <a href="instr.php?op=editpage&instrid=' . $instrid . '&pid=' . $pageid . '"><img src="../assets/icons/add_mini.png" alt="' . _AM_INSTRUCTION_ADDSUBPAGE . '" title="' . _AM_INSTRUCTION_ADDSUBPAGE . '" /></a> ';

            if ($pagestatus) {
                $ret .= ' <img src="../assets/icons/lock_mini.png" alt="' . _AM_INSTRUCTION_LOCK . '" title="' . _AM_INSTRUCTION_LOCK . '"> ';
            } else {
                $ret .= ' <img src="../assets/icons/unlock_mini.png" alt="' . _AM_INSTRUCTION_UNLOCK . '" title="' . _AM_INSTRUCTION_UNLOCK . '"> ';
            }

            $ret .= ' <a href="instr.php?op=editpage&pageid=' . $pageid . '"><img src="../assets/icons/edit_mini.png" alt="' . _AM_INSTRUCTION_EDIT . '" title="' . _AM_INSTRUCTION_EDIT . '"></a> ' . $act_del . '
      </td>
    </tr>';

            // Устанавливаем префикс
            $prefix_curr .= $prefix_orig;
        }

        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {

            foreach ($this->tree[$key]['child'] as $childkey) {
                $this->_makePagesAdminOptions($childkey, $ret, $prefix_orig, $objInsinstr, $class, $prefix_curr);
            }
        }
    }

    /**
     * @param        $objInsinstr
     * @param string $prefix
     * @param int    $key
     * @return string
     */
    public function makePagesAdmin(&$objInsinstr, $prefix = '-', $key = 0)
    {
        $ret = '<form name="inspages" action="instr.php" method="post">
  <table width="100%" cellspacing="1" class="outer">
    <tr>
      <th align="center" colspan="3">' . sprintf(_AM_INSTRUCTION_LISTPAGESININSTR, $objInsinstr->getVar('title')) . '</th>
    </tr>
    <tr>
      <td class="head" align="center">' . _AM_INSTRUCTION_TITLE . '</td>
      <td class="head" align="center" width="50">' . _AM_INSTRUCTION_WEIGHT . '</td>
      <td class="head" align="center" width="180">' . _AM_INSTRUCTION_ACTION . '</td>
    </tr>';

        // Выводим все страницы
        $this->_makePagesAdminOptions($key, $ret, $prefix, $objInsinstr);

        $ret .= '<tr class="foot">
      <td><a href="instr.php?op=editpage&instrid=' . $objInsinstr->getVar('instrid') . '"><img src="../assets/icons/add_mini.png" alt="' . _AM_INSTRUCTION_ADDPAGE . '" title="' . _AM_INSTRUCTION_ADDPAGE . '"></a></td>
      <td colspan="2">
        <input type="hidden" name="instrid" value="' . $objInsinstr->getVar('instrid') . '" />
        <input type="hidden" name="op" value="updpage" />
        <input type="submit" value="' . _SUBMIT . '" />
      </td>
    </tr>
  </table>
</form>';

        return $ret;
    }

    // ==================================
    // === Дерево категорий в админке ===
    // ==================================

    /**
     * @param        $key
     * @param        $ret
     * @param        $prefix_orig
     * @param array  $cidinstrids
     * @param string $class
     * @param string $prefix_curr
     */
    public function _makeCatsAdminOptions($key, &$ret, $prefix_orig, $cidinstrids = [], &$class = 'odd', $prefix_curr = '')
    {
        if ($key > 0) {

            //
            $class = ('even' === $class) ? 'odd' : 'even';

            // ID категории
            $catid = $this->tree[$key]['obj']->getVar('cid');
            // Название категории
            $cattitle = $this->tree[$key]['obj']->getVar('title');
            // Вес
            $catweight = $this->tree[$key]['obj']->getVar('weight');
            // Статус
            $pagestatus = $this->tree[$key]['obj']->getVar('status');
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

            // Дочернии категории
            $cat_childs = $this->getAllChild($catid);
            // Число дочерних категорий
            $num_childs = count($cat_childs);
            // Число инструкций
            $num_instrs = isset($cidinstrids[$catid]) ? $cidinstrids[$catid] : 0;

            // Действие - удаление
            $act_del = (($num_instrs > 0) || ($num_childs > 0)) ? '<img src="../assets/icons/no_delete_mini.png" alt="' . _AM_INSTR_NODELCAT . '" title="' . _AM_INSTR_NODELCAT . '" />' : '<a href="cat.php?op=delcat&cid='
                                                                                                                                                                                           . $catid
                                                                                                                                                                                           . '"><img src="../assets/icons/delete_mini.png" alt="'
                                                                                                                                                                                           . _AM_INSTRUCTION_DEL
                                                                                                                                                                                           . '" title="'
                                                                                                                                                                                           . _AM_INSTRUCTION_DEL
                                                                                                                                                                                           . '" /></a>';
            // Действие - просмотр
            $act_view = ($num_instrs > 0) ? '<a href="instr.php?cid=' . $catid . '"><img src="../assets/icons/view_mini.png" alt="' . _AM_INSTR_VIEWINSTR . '" title="' . _AM_INSTR_VIEWINSTR . '" /></a>' : '<img src="../assets/icons/no_view_mini.png" alt="'
                                                                                                                                                                                                             . _AM_INSTR_NOVIEWINSTR
                                                                                                                                                                                                             . '" title="'
                                                                                                                                                                                                             . _AM_INSTR_NOVIEWINSTR
                                                                                                                                                                                                             . '" />';

            $ret .= '<tr class="' . $class . '">
      <td>' . $prefix_curr . ' <a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/index.php?cid=' . $catid . '">' . $cattitle . '</a></td>
      <td align="center" width="50">' . $catweight . '</td>
      <td align="center" width="100">' . $num_instrs . '</td>
      <td align="center" width="150">
        ' . $act_view . '
        <a href="cat.php?op=editcat&cid=' . $catid . '"><img src="../assets/icons/edit_mini.png" alt="' . _AM_INSTRUCTION_EDIT . '" title="' . _AM_INSTRUCTION_EDIT . '"></a>
        ' . $act_del . '
      </td>
    </tr>';

            // Устанавливаем префикс
            $prefix_curr .= $prefix_orig;
        }

        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {

            foreach ($this->tree[$key]['child'] as $childkey) {
                $this->_makeCatsAdminOptions($childkey, $ret, $prefix_orig, $cidinstrids, $class, $prefix_curr);
            }
        }
    }

    /**
     * @param string $prefix
     * @param array  $cidinstrids
     * @param int    $key
     * @return string
     */
    public function makeCatsAdmin($prefix = '-', $cidinstrids = [], $key = 0)
    {
        $ret = '<table width="100%" cellspacing="1" class="outer">
    <tr>
      <th align="center" colspan="4">' . _AM_INSTR_LISTALLCATS . '</th>
    </tr>
    <tr>
      <td class="head">' . _AM_INSTRUCTION_TITLE . '</td>
      <td class="head" align="center" width="50">' . _AM_INSTRUCTION_WEIGHT . '</td>
	  <td class="head" align="center" width="100">' . _AM_INSTR_INSTRS . '</td>
      <td class="head" align="center" width="150">' . _AM_INSTRUCTION_ACTION . '</td>
    </tr>';

        // Выводим все страницы
        $this->_makeCatsAdminOptions($key, $ret, $prefix, $cidinstrids);

        $ret .= '</table>';

        return $ret;
    }

    // ======================================
    // Список страниц на стороне пользователя
    // ======================================

    /**
     * @param       $key
     * @param       $ret
     * @param int   $currpageid
     * @param array $lastpageids
     * @param int   $level
     */
    public function _makePagesUserTree($key, &$ret, $currpageid = 0, &$lastpageids = [], $level = 0)
    {

        // Сохраняем значение предыдущей страницы
        //static $stat_prevpages;

        if ($key > 0) {

            // ID страницы
            $pageid = $this->tree[$key]['obj']->getVar('pageid');
            // Название страницы
            $pagetitle = $this->tree[$key]['obj']->getVar('title');
            // Тип страницы
            $pagetype = $this->tree[$key]['obj']->getVar('type');

            // Дочернии категории
            $page_childs = $this->getAllChild($pageid);
            // Число дочерних страниц
            $num_childs = count($page_childs);

            // Генерируем класс
            // InstrTreeNode InstrTreeIsRoot InstrTreeExpandClosed InstrTreeIsLast
            $class = [];
            // Данный класс должен быть у любого узла
            $class[] = 'InstrTreeNode';
            // Если узел нулевого уровня, добавляем InstrTreeIsRoot
            if (0 === $level) {
                $class[] = 'InstrTreeIsRoot';
            }
            // Тип узла InstrTreeExpandClosed|InstrTreeExpandLeaf
            // Если у узла нет потомков - InstrTreeExpandLeaf
            if (0 == $num_childs) {
                $class[] = 'InstrTreeExpandLeaf';
                // Если у искомого элемента есть потомки - открываем список
            } elseif ($currpageid == $pageid) {
                $class[] = 'InstrTreeExpandOpen';
                // Если искомый элемент есть в потомках текущего, то ставим класс InstrTreeExpandOpen
            } elseif (array_key_exists($currpageid, $page_childs)) {
                $class[] = 'InstrTreeExpandOpen';
                //
            } else {
                $class[] = 'InstrTreeExpandClosed';
            }

            // Данный класс нужно добавлять последнему узлу в каждом уровне

            if (isset($lastpageids[$level]) && ($pageid == $lastpageids[$level])) {
                $class[] = 'InstrTreeIsLast';
            }

            //$class[] = 'InstrTreeIsLast';


            // Test
            //$ret .= '<div id="' . $pageid . '">';

            // Создаём запись
            $ret .= '<li class="' . implode(' ', $class) . '">';
            //
            $ret .= '<div class="InstrTreeExpand"></div>';
            //
            $ret .= '<div class="InstrTreeContent">';

            // Если это лист дерева
            if (0 == $pagetype) {
                $ret .= '<span class="InstrTreeEmptyPage">' . $pagetitle . '</span>';
                //
            } elseif ($currpageid == $pageid) {
                $ret .= $pagetitle;
                //
            } else {
                $ret .= '<a href="' . XOOPS_URL . '/modules/' . INST_DIRNAME . '/page.php?id=' . $pageid . '#pagetext">' . $pagetitle . '</a>';
            }

            $ret .= '</div>';

            // Если есть потомки
            if ($num_childs > 0) {
                $ret .= '<ul class="InstrTreeContainer">';
            }

            // Инкримент уровня
            $level++;
        }

        // Рекурсия
        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {

            foreach ($this->tree[$key]['child'] as $childkey) {
                $this->_makePagesUserTree($childkey, $ret, $currpageid, $lastpageids, $level);
            }
        }

        // Test
        if ($key > 0) {
            // Если есть потомки
            if ($num_childs > 0) {
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
                $ret .= '</ul>';
            }
            // Конец текущей записи
            $ret .= '</li>';
        }
    }

    // Находим предыдущую и следующую страницы.
    // Находим последнии страницы на каждом уровне.
    /**
     * @param       $key
     * @param int   $currpageid
     * @param array $prevpages
     * @param array $nextpages
     * @param array $lastpageids
     * @param int   $level
     */
    public function _makePagesUserCalc($key, $currpageid = 0, &$prevpages = [], &$nextpages = [], &$lastpageids = [], $level = 0)
    {

        // Сохраняем значение предыдущей страницы
        static $stat_prevpages;

        if ($key > 0) {
            // ID страницы
            $pageid = $this->tree[$key]['obj']->getVar('pageid');
            // Название страницы
            $pagetitle = $this->tree[$key]['obj']->getVar('title');
            // Тип страницы
            $pagetype = $this->tree[$key]['obj']->getVar('type');

            // Если мы передали ID текущей страницы, то находить предыдудую и следующую страницы
            // Не находить предыдущие и следующие для "Пустой страницы"
            if ($currpageid && $pagetype) {
                // Если элемент равен текущей странице
                if (isset($stat_prevpages) && ($currpageid == $pageid)) {
                    // Забиваем массив предыдущей страницы
                    $prevpages['pageid'] = $stat_prevpages['pageid'];
                    $prevpages['title']  = $stat_prevpages['title'];

                    // Если предыдущий равен текущей странице
                } elseif (isset($stat_prevpages) && ($currpageid == $stat_prevpages['pageid'])) {
                    // Забиваем массив следующей страницы
                    $nextpages['pageid'] = $pageid;
                    $nextpages['title']  = $pagetitle;
                }
                // Заносим текущие данные в массив предыдущей страницы
                $stat_prevpages['pageid'] = $pageid;
                $stat_prevpages['title']  = $pagetitle;
            }

            // Заносим текущую страницу в массив "последних страний"
            $lastpageids[$level] = $pageid;

            // Инкримент уровня
            $level++;
        }

        // Рекурсия
        if (isset($this->tree[$key]['child']) && !empty($this->tree[$key]['child'])) {

            foreach ($this->tree[$key]['child'] as $childkey) {
                $this->_makePagesUserCalc($childkey, $currpageid, $prevpages, $nextpages, $lastpageids, $level);
            }
        }
    }

    //

    /**
     * @param int   $currpageid
     * @param array $prevpages
     * @param array $nextpages
     * @param int   $key
     * @return string
     */
    public function makePagesUser($currpageid = 0, &$prevpages = [], &$nextpages = [], $key = 0)
    {

        // Массив последней страницы на каждом уровне
        // level => pageid
        $lastpageids = [];

        // Расчёт
        $this->_makePagesUserCalc($key, $currpageid, $prevpages, $nextpages, $lastpageids);

        $ret = '<div onclick="instr_tree_toggle(arguments[0])">
<div>' . _MD_INSTRUCTION_LISTPAGES . '</div>
<div><ul class="InstrTreeContainer">';

        // Генерируем дерево
        $this->_makePagesUserTree($key, $ret, $currpageid, $lastpageids);

        $ret .= '</ul>
</div>';

        return $ret;
    }
}

mambax7 / instruction
