Tests\Compiling\Statements\UnexpectedUseOfThis

Complexity

Total Complexity

16

Size/Duplication

Total Lines	123
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes	1
Bugs	0	Features	1

12 Methods

Rating	Name	Duplication	Size	Complexity
A	thisAsArgument()	0	4	1
A	thisInCatch()	0	8	2
A	thisAsLoopVariable()	0	6	2
A	thisAsStaticVariable()	0	6	1
A	thisAsGlobalVariable()	0	6	1
A	unsetThis()	0	6	1
A	OtherAsArgument()	0	4	1
A	OtherInCatch()	0	8	2
A	OtherAsLoopVariable()	0	6	2
A	OtherAsStaticVariable()	0	6	1
A	OtherAsGlobalVariable()	0	6	1
A	unsetOther()	0	7	1

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 5 and the first side effect is on line 129.

namespace Tests\Compiling\Statements;

class UnexpectedUseOfThis
{
    /**
     * @return string
     */
    public function thisAsArgument($this)

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

    {
        return 'Cannot use $this as parameter';
    }

    /**
     * @return string
     */
    public function thisInCatch()
    {
        try {
            throw new \LogicException();
        } catch (\Exception $this) {
            return 'Fatal error: Cannot re-assign $this';
        }
    }

    /**
     * @return string
     */
    public function thisAsLoopVariable()
    {
        foreach (['foo'] as $this) {
            return 'Fatal error: Cannot re-assign $this';
        }
    }

    /**
     * @return string
     */
    public function thisAsStaticVariable()
    {
        static $this;

        return 'Fatal error: Cannot use $this as static variable';
    }

    /**
     * @return string
     */
    public function thisAsGlobalVariable()
    {
        global $this;

        return 'Fatal error: Cannot use $this as global variable';
    }

    /**
     * @return string
     */
    public function unsetThis()
    {
        unset($this);

        return 'Fatal error: Cannot unset $this';
    }

    /**
     * @return string
     */
    public function OtherAsArgument($a)

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

Method name "UnexpectedUseOfThis::OtherAsArgument" is not in camel caps format

    {
        return 'Ok';
    }

    /**
     * @return string
     */
    public function OtherInCatch()

Method name "UnexpectedUseOfThis::OtherInCatch" is not in camel caps format

    {
        try {
            throw new \LogicException();
        } catch (\Exception $a) {
            return 'Ok';
        }
    }

    /**
     * @return string
     */
    public function OtherAsLoopVariable()

Method name "UnexpectedUseOfThis::OtherAsLoopVariable" is not in camel caps format

    {
        foreach (['foo'] as $a) {
            return 'Ok';
        }
    }

    /**
     * @return string
     */
    public function OtherAsStaticVariable()

Method name "UnexpectedUseOfThis::OtherAsStaticVariable" is not in camel caps format

    {
        static $a;

        return 'Ok';
    }

    /**
     * @return string
     */
    public function OtherAsGlobalVariable()

Method name "UnexpectedUseOfThis::OtherAsGlobalVariable" is not in camel caps format

    {
        global $a;

        return 'Ok';
    }

    /**
     * @return string
     */
    public function unsetOther()
    {
        $a = 1;
        unset($a);

        return 'Ok';
    }
}
?>
----------------------------
[
    {
        "type":"unexpected_use.this",
        "message":"Method thisAsArgument can not have a parameter named \"this\".",
        "file":"UnexpectedUseOfThis.php",
        "line":9
    },
    {
        "type":"unexpected_use.this",
        "message":"Catch block can not have a catch variable named \"this\".",
        "file":"UnexpectedUseOfThis.php",
        "line":21
    },
    {
        "type":"unexpected_use.this",
        "message":"Foreach loop can not use a value variable named \"this\".",
        "file":"UnexpectedUseOfThis.php",
        "line":31
    },
    {
        "type":"unexpected_use.this",
        "message":"Can not declare a static variable named \"this\".",
        "file":"UnexpectedUseOfThis.php",
        "line":41
    },
    {
        "type":"unexpected_use.this",
        "message":"Can not declare a global variable named \"this\".",
        "file":"UnexpectedUseOfThis.php",
        "line":51
    },
    {
        "type":"unexpected_use.this",
        "message":"Can not unset $this.",
        "file":"UnexpectedUseOfThis.php",
        "line":61
    }
]