Hello. Le mar. 29 janv. 2019 à 00:14, Eric Barnhill <ericbarnh...@gmail.com> a écrit : > > Fraction already has a toString() method which should cover VALJO concerns > by representing the instance in one specific way.
It has 2 different outputs (suppress the fraction bar and denominator when it is 1). Not sure that's very robust: Expecting a "/" as part of the representation will make parsing easier (noting that class is still missing the parse/valueOf method). > The FractionFormat classes allow for options beyond this such as proper > fractions or region-specific versions. IMO it's out of scope for a low-level component, and at least until we have an actual use-case. Locale-specific input/output is a can of worms that should be handled by text-oriented libraries. Having output (e.g. error messages) differ from locale to locale is a very bad idea (in a low-level component[1]), and so is the capacity (of a low-level component) to truncate data that might be needed by the caller. [Those two things are the purpose of the "NumberFormat" family of classes.] The Javadoc of the "...FractionFormat" classes is also badly out-of-sync since most methods refer to "complex" (?), witnessing the extremely low usage. Best regards, Gilles > > It doesn't seem to me like it violates VALJO principles to have an > auxiliary class that takes care of these alternate cases. On the contrary > from a VALJO perspective it seems like a nice idea to have encapsulated > them in an auxiliary class structure. > > Eric > [1] Application developers can customize at will from the return values of "getNumerator()" and "getDenominator()" methods. > On Sat, Jan 26, 2019 at 12:11 PM Gilles Sadowski <gillese...@gmail.com> > wrote: > > > Le sam. 26 janv. 2019 à 17:24, Gary Gregory <garydgreg...@gmail.com> a > > écrit : > > > > > > On Sat, Jan 26, 2019 at 10:19 AM Gilles Sadowski <gillese...@gmail.com> > > > wrote: > > > > > > > Le sam. 26 janv. 2019 à 14:01, Gary Gregory <garydgreg...@gmail.com> a > > > > écrit : > > > > > > > > > > Are we talking about formatting [numbers] specific classes or JRE > > > > classes? > > > > > > > > They are classes that aim to customize the output from classes in > > > > [Numbers], > > > > (specifically, the way to display a {{Fraction}} or {{BigFraction}} > > > > object). > > > > > > > > > > Well, then that code does not belong in [text] since it requires [number] > > > classes. > > > > Not what I meant. > > People wanting custom formatting of a fraction can get the parts that > > define it (i.e. numerator and denominator) and call whatever they want > > (e.g. something which might be in [Text]) in order to craft a string > > representation of those numbers. > > Following ValJO (even if "BigFraction" is not one, strictly speaking), > > we want *one* way to represent the contents of the instance. > > > > Regards, > > Gilles > > > > > > > > Gary > > > > > > > > > > The classes provide accessors to the numerator and denominator which > > can be > > > > used by outside code to display the fraction as it wishes. > > > > > > > > Side note: Similar formatting classes in Commons Math are more of a > > > > nuisance > > > > than anything, e.g. displaying small numbers as "0" (in exception > > > > messages), > > > > because the default is to provide 6 decimal digits, thus discarding > > > > all significant > > > > information. > > > > > > > > Gilles > > > > > > > > > > > > > > Gary > > > > > > > > > > On Sat, Jan 26, 2019 at 7:14 AM Gilles Sadowski < > > gillese...@gmail.com> > > > > > wrote: > > > > > > > > > > > Hi. > > > > > > > > > > > > In reference to the current changes in module > > > > "commons-numbers-fraction" > > > > > > (on the "fraction-dev"), my opinion is that the formatting classes > > > > should > > > > > > be > > > > > > removed.[1] > > > > > > At the level of a math component, it's safer to stick to a single, > > > > > > locale-independent format.[2] > > > > > > > > > > > > Regards, > > > > > > Gilles > > > > > > > > > > > > [1] Rationale is that pretty-printing is not the purpose of the > > library > > > > > > (better > > > > > > leave that to [Text], or a dedicated module). > > > > > > [2] There is a pending issue (NUMBERS-88) that suggests > > hard-coding the > > > > > > format. > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > > > > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > > > > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > > For additional commands, e-mail: dev-h...@commons.apache.org > > > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org