CViewBasic::render() - Code Metrics - donami/ctable - Measure and Improve Code Quality continuously with Scrutinizer

CViewBasic::render() C
last analyzed 2016-05-07 20:10 UTC

↳ Parent: CViewBasic

Complexity

Conditions	7
Paths	6

Size

Total Lines	35
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
cc	7
eloc	18
c	1
b	0
f	1
nc	6
nop	0
dl	0
loc	35
rs	6.7272

<?php

namespace Anax\View;

/**
 * A view connected to a template file.
 *
 */
class CViewBasic implements \Anax\DI\IInjectionAware
{
    use \Anax\DI\TInjectable;



    /**
    * Properties
    *
    */
    private $template;          // Template file or array
    private $templateData = []; // Data to send to template file
    private $sortOrder;         // For sorting views
    private $type;              // Type of view



    /**
     * Set values for the view.
     *
     * @param string/array $template the template file, or array 

     * @param array        $data     variables to make available to the view, default is empty
     * @param int          $sort     which order to display the views, if suitable
     * @param string       $type     which type of view
     *
     * @return $this
     */
    public function set($template, $data = [], $sort = 0, $type = 'file')
    {
        $this->template     = $template;
        $this->templateData = $data;
        $this->sortOrder    = $sort;
        $this->type         = $type;
    }



    /**
     * Render the view.
     *
     * @return void
     */
    public function render()
    {
        switch ($this->type) {

            case 'file':
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;

                if (!is_readable($this->template)) {
                    throw new \Exception("Could not find template file: " . $this->template);
                }

                extract($this->templateData);
                include $this->template;

                break;

            case 'callback':
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
    
                if (!isset($this->template['callback']) || !is_callable($this->template['callback'])) {
                    throw new \Exception("View missing callback.");
                }

                echo call_user_func($this->template['callback']);

                break;

            case 'string':
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;

                echo $this->template;

                break;

            default:
                throw new \Exception("Not a valid template type: " . $this->type);
        }
    }



    /**
     * Give the sort order for this view.
     *
     * @return int
     */
    public function sortOrder()
    {
        return $this->sortOrder;
    }
}


1			<?php
2
3			namespace Anax\View;
4
5			/**
6			* A view connected to a template file.
7			*
8			*/
9			class CViewBasic implements \Anax\DI\IInjectionAware
10			{
11			use \Anax\DI\TInjectable;
12
13
14
15			/**
16			* Properties
17			*
18			*/
19			private $template; // Template file or array
20			private $templateData = []; // Data to send to template file
21			private $sortOrder; // For sorting views
22			private $type; // Type of view
23
24
25
26			/**
27			* Set values for the view.
28			*
29			* @param string/array $template the template file, or array
			0 ignored issues – show Documentation introduced 2016-05-07 19:47 UTC by Report Bug Copy Issue Report The doc-type `string/array` could not be parsed: Unknown type name "string/array" at position 0. (view supported doc-types) This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types. Loading history...
30			* @param array $data variables to make available to the view, default is empty
31			* @param int $sort which order to display the views, if suitable
32			* @param string $type which type of view
33			*
34			* @return $this
35			*/
36			public function set($template, $data = [], $sort = 0, $type = 'file')
37			{
38			$this->template = $template;
39			$this->templateData = $data;
40			$this->sortOrder = $sort;
41			$this->type = $type;
42			}
43
44
45
46			/**
47			* Render the view.
48			*
49			* @return void
50			*/
51			public function render()
52			{
53			switch ($this->type) {
54
55			case 'file':
			0 ignored issues – show Coding Style introduced 2016-05-07 19:47 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
56
57			if (!is_readable($this->template)) {
58			throw new \Exception("Could not find template file: " . $this->template);
59			}
60
61			extract($this->templateData);
62			include $this->template;
63
64			break;
65
66			case 'callback':
			0 ignored issues – show Coding Style introduced 2016-05-07 19:47 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
67
68			if (!isset($this->template['callback']) \|\| !is_callable($this->template['callback'])) {
69			throw new \Exception("View missing callback.");
70			}
71
72			echo call_user_func($this->template['callback']);
73
74			break;
75
76			case 'string':
			0 ignored issues – show Coding Style introduced 2016-05-07 19:47 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
77
78			echo $this->template;
79
80			break;
81
82			default:
83			throw new \Exception("Not a valid template type: " . $this->type);
84			}
85			}
86
87
88
89			/**
90			* Give the sort order for this view.
91			*
92			* @return int
93			*/
94			public function sortOrder()
95			{
96			return $this->sortOrder;
97			}
98			}
99

donami / ctable

CViewBasic::render() C last analyzed 2016-05-07 20:10 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

CViewBasic::render() C
last analyzed 2016-05-07 20:10 UTC