Skip to content

Commit

Permalink
Add sum() method in FractionCollection
Browse files Browse the repository at this point in the history
  • Loading branch information
tazorax committed Dec 11, 2018
1 parent f50326a commit 4da9694
Show file tree
Hide file tree
Showing 2 changed files with 39 additions and 0 deletions.
18 changes: 18 additions & 0 deletions src/Arithmetic/FractionCollection.php
Original file line number Diff line number Diff line change
Expand Up @@ -77,4 +77,22 @@ public function simplify()

return $buffer;
}

/**
* Get sum
*
* @return Fraction
*/
public function sum()
{
$result = new Fraction();
$simplified_collection = $this->simplify();

foreach ($simplified_collection->fractions as $fraction) {
$result->numerator += $fraction->numerator;
$result->denominator = $fraction->denominator;
}

return $result;
}
}
21 changes: 21 additions & 0 deletions tests/Arithmetic/FractionCollectionTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -61,4 +61,25 @@ public function testSimplify() {
$this->assertEquals([$f1, $f2, $f3, $f4, $f5], $fc2->fractions);

}

/**
*
*/
public function testSum() {
$fc1 = new FractionCollection();

$fc1->addFraction(new Fraction(1, 2));
$fc1->addFraction(new Fraction(1, 4));
$fc1->addFraction(new Fraction(1, 8));
$fc1->addFraction(new Fraction(1, 8));
$fc1->addFraction(new Fraction(4, 256));

$this->assertEquals(5, count($fc1->fractions));

$expected_sum = new Fraction(65, 64);
$f = $fc1->sum();

$this->assertEquals($expected_sum, $f);

}
}

0 comments on commit 4da9694

Please sign in to comment.