[GitHub] [commons-math] aherbert commented on pull request #233: new method that takes string and extracts list of numbers

["aherbert (via GitHub)" <gi...@apache.org>]

aherbert commented on PR #233:
URL: https://github.com/apache/commons-math/pull/233#issuecomment-1616820363

   It is difficult to find a static class to host this method as it is not typically something performed in the math library, i.e. parsing numbers from within text is not within the current scope of math. It is not a mathematical operation or algorithm.
   
   Some legacy classes exist to perform String IO but the current trend in the math derived libraries for new code such as Numbers is to avoid IO conversions as this is locale specific and/or a user specific choice of format. Parsing of strings and formatting to strings delegates to the JDK methods, e.g. `double Double.parseDouble(String)` and `String Double.toString(double)`. Composite numbers are typically delimited strings of standard format numbers, see for example `o.a.c.numbers.complex.Complex` for `static Complex parse(String)` and `String toString()`. For the parsing, no regex is used and all identification of the number is left to `Double.parseDouble`. The commons method is solely responsible for cutting up the string using known delimiters.
   
   I think you may find some interest in the value of this type of method by writing to the dev mailing list. If you put `[math][lang][text]` in the subject heading and start a discussion on the utility of this functionality and the use cases you have, e.g. why are you stripping out numbers from strings but not supporting scientific notation; what mystery input strings are you wishing to decipher and use the numbers from?
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org