DaanMooij\AdventOfCode\Year2018\Day1

Complexity

Total Complexity

11

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	calculateFrequency()	0	7	2
A	setChanges()	0	3	1
A	getFrequencyReachedTwice()	0	19	4
A	getChanges()	0	3	1
A	solve()	0	5	1
A	loadInput()	0	7	2

<?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)) {

It seems like $file can also be of type false; however, parameter $handle of feof() does only seem to accept resource, maybe add an additional type check?

            $this->changes[] = intval(fgets($file));

It seems like $file can also be of type false; however, parameter $handle of fgets() does only seem to accept resource, maybe add an additional type check?

        }
        fclose($file);

It seems like $file can also be of type false; however, parameter $handle of fclose() does only seem to accept resource, maybe add an additional type check?

    }

    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;
    }
}