Day1::getFrequencyReachedTwice() - Code Metrics - Inspection of "refactor 2018 day 1" - daanmooij/advent-of-code-php - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 8c822c...e5da76 )

by Daan

created 2020-12-09 00:14 UTC

Day1::getFrequencyReachedTwice() A

↳ Parent: Day1

Complexity

Conditions	4
Paths	4

Size

Total Lines	19
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
eloc	10
c	1
b	0
f	0
nc	4
nop	1
dl	0
loc	19
rs	9.9332

<?php

namespace DaanMooij\AdventOfCode\Year2018;

use DaanMooij\AdventOfCode\Day;

class Day1 implements Day
{
    /**
     * @var array
     */
    private array $changes;

    public function loadInput(): void
    {
        $file = fopen(__DIR__ . '/input/day-1.txt', "r");
        while (!feof($file)) {
        while (!feof(/** @scrutinizer ignore-type */ $file)) {
            $this->changes[] = intval(fgets($file));
            $this->changes[] = intval(fgets(/** @scrutinizer ignore-type */ $file));
        }
        fclose($file);
        fclose(/** @scrutinizer ignore-type */ $file);
    }

    public function solve()
    {
        $result = $this->calculateFrequency();

        printf('The resulting frequency is: %s', $result);
    }

    public function setChanges(array $changes): void
    {
        $this->changes = $changes;
    }

    public function getChanges(): array
    {
        return $this->changes;
    }

    /**
     * @param int $frequency -> The starting frequency
     * @return int
     */
    public function calculateFrequency(int $frequency = 0): int
    {
        foreach ($this->changes as $change) {
            $frequency += $change;
        }

        return $frequency;
    }

    /**
     * @param int $frequency -> The starting frequency
     * @return int
     */
    public function getFrequencyReachedTwice(int $frequency = 0): int
    {
        $memory = [$frequency => 1];

        $found = false;
        while (!$found) {
            foreach ($this->changes as $change) {
                $frequency += $change;

                if (isset($memory[$frequency])) {
                    $found = true;
                    break;
                }

                $memory[$frequency] = 1;
            }
        }

        return $frequency;
    }
}


daanmooij / advent-of-code-php

Push — master ( 8c822c...e5da76 )

Day1::getFrequencyReachedTwice() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like