Gilles created NUMBERS-123:
------------------------------

             Summary: "BigFraction(double)" is unnecessary
                 Key: NUMBERS-123
                 URL: https://issues.apache.org/jira/browse/NUMBERS-123
             Project: Commons Numbers
          Issue Type: Improvement
          Components: fraction
            Reporter: Gilles
            Assignee: Gilles
             Fix For: 1.0


Constructor {{BigFraction(double value)}} is only called from the {{from(double 
value)}} method.
 Actually, this constructor is misleading as it is indeed primarily a 
conversion from which appropriate {{numerator}} and {{denominator}} fields are 
computed; those could be set by
 the "direct" constructor {{BigFraction(BigInteger num, BigInteger den)}}.

Moreover, the private field {{ZERO}} goes through this conversion code whereas 
it could constructed "directly", e.g. using {{of(0)}}. Similarly for field 
{{ONE}}.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to