Carno\Net\Routing\Table::reset()

Complexity

Conditions	5
Paths	6

Size

Total Lines	29
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    public function reset(Endpoint ...$presents) : array
    {
        $changes = [];

        $current = [];

        foreach ($presents as $endpoint) {
            $current[$epn = $endpoint->id()] = $endpoint;

            // if exists
            if (isset($this->current[$epn])) {
                unset($this->current[$epn]);
                continue;
            }

            // mark join
            $changes[] = new Changed(Events::JOIN, $endpoint);
        }

        if ($this->current) {

The expression $this->current of type Carno\Net\Endpoint[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            foreach ($this->current as $endpoint) {
                // mark leave
                $changes[] = new Changed(Events::LEAVE, $endpoint);
            }
        }

        $this->current = $current;

        return $changes;
    }