PHPSA\Compiler\Statement\TryCatchSt

Complexity

Total Complexity

5

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

69.23%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	compile()	0	16	5

<?php
/**
 * @author Patsura Dmitry https://github.com/ovr <talk@dmtry.me>
 */

namespace PHPSA\Compiler\Statement;

use PHPSA\CompiledExpression;
use PHPSA\Context;

class TryCatchSt extends AbstractCompiler
{
    protected $name = '\PhpParser\Node\Stmt\TryCatch';

    /**
     * @param \PhpParser\Node\Stmt\TryCatch $statement
     * @param Context $context
     * @return CompiledExpression
     */
    public function compile($statement, Context $context)
    {
        foreach ($statement->stmts as $stmt) {
            \PHPSA\nodeVisitorFactory($stmt, $context);
        }

        foreach ($statement->catches as $stmt) {
            \PHPSA\nodeVisitorFactory($stmt, $context);
        }

        if (count($statement->finallyStmts) > 0) {
            foreach ($statement->finallyStmts as $stmt) {

The expression $statement->finallyStmts of type null|array<integer,object<PhpParser\Node>> is not guaranteed to be traversable. How about adding an additional type check?

                \PHPSA\nodeVisitorFactory($stmt, $context);
            }
        }
    }
}