[ http://issues.apache.org/jira/browse/MATH-60?page=comments#action_12442653 ] Luc Maisonobe commented on MATH-60: -----------------------------------
doesn't the answers provided by Phil Steitz suit you ? This issue is marked as closed, do you want to reopen it ? > [math] Function math.fraction.ProperFractionFormat.parse(String, > ParsePosition) return illogical result > ------------------------------------------------------------------------------------------------------- > > Key: MATH-60 > URL: http://issues.apache.org/jira/browse/MATH-60 > Project: Commons Math > Issue Type: Bug > Affects Versions: 1.1 Final > Environment: Operating System: other > Platform: Other > Reporter: nhung.nnguyen > > Hello, > I find illogical returned result from function "Fraction parse(String source, > ParsePostion pos)" (in class ProperFractionFormat of the Fraction Package) of > the Commons Math library. Please see the following code segment for more > details: > " > ProperFractionFormat properFormat = new ProperFractionFormat(); > result = null; > String source = "1 -1 / 2"; > ParsePosition pos = new ParsePosition(0); > //Test 1 : fail > public void testParseNegative(){ > > String source = "-1 -2 / 3"; > ParsePosition pos = new ParsePosition(0); > Fraction actual = properFormat.parse(source, pos); > assertNull(actual); > } > // Test2: success > public void testParseNegative(){ > > String source = "-1 -2 / 3"; > ParsePosition pos = new ParsePosition(0); > Fraction actual = properFormat.parse(source, pos); // return Fraction 1/3 > assertEquals(1, source.getNumerator()); > assertEquals(3, source.getDenominator()); > } > " > Note: Similarly, when I passed in the following inputs: > input 2: (source = 1 2 / -3, pos = 0) > input 3: ( source = -1 -2 / 3, pos = 0) > Function "Fraction parse(String, ParsePosition)" returned Fraction 1/3 (means > the result Fraction had numerator = 1 and denominator = 3)for all 3 inputs > above. > > I think the function does not handle parsing the numberator/ denominator > properly incase input string provide invalid numerator/denominator. > Thank you! -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]