[
https://issues.apache.org/jira/browse/LANG-165?focusedWorklogId=593780&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-593780
]
ASF GitHub Bot logged work on LANG-165:
---------------------------------------
Author: ASF GitHub Bot
Created on: 09/May/21 14:41
Start Date: 09/May/21 14:41
Worklog Time Spent: 10m
Work Description: garydgregory commented on a change in pull request #751:
URL: https://github.com/apache/commons-lang/pull/751#discussion_r628897878
##########
File path: src/main/java/org/apache/commons/lang3/StringUtils.java
##########
@@ -9638,6 +9638,79 @@ public static String wrapIfMissing(final String str,
final String wrapWith) {
return builder.toString();
}
+ /**
+ * Method that assembles all the numbers, form the passed string and
returns them as list.
+ * It is important to note here, is that bu 'number' method assume any
digit sequence, that
+ * can (but not necessary at all) contains dot within it (I mean just
plain old floats,
+ * something like 51.82)
+ *
+ * For example, you may pass a string "21.2 days 3 minutes 22 seconds".
For this particular string
+ * the result list of doubles will look like this : [21.2, 3.0, 22.0]
+ *
+ * if string contains invalid numbers (for example this string contains
+ * not valid number: "My height is 1234.23.13" This is invalid because it
+ * is not clear how to parse this part - 1234.23.13), {@link
NumberFormatException}
+ * will be thrown. Though if string will contain number, where right
+ * after second dot resides not a number, or, any other char, then this
+ * case will be considered as valid. For example, this string contains
+ * only valid numbers: "My pulse is 90.123. and weight is 78.2"
+ * In this case sequence "90.123." will be considered as "90.123", as well
as
+ * sequence "90." (imagine that there is no digit right after dot) will be
+ * considered as 90.0 double.
+ *
+ * @param stringThatContainsNumbers - string, that contains number or
several numbers.
+ * Not necessary integers, may be numbers
with float point.
+ * @return - list of numbers, that this particular string contains
+ *
+ * @throws NumberFormatException - see documentation clarification about
cases when thrown above
+ */
+ public static List<Double> extractNumbersFromString(String
stringThatContainsNumbers) {
Review comment:
The API name says Number but the return type says Double. They should
match.
But as soon as you type an API to Double, someone will want one for Integer,
Long, and so on.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 593780)
Time Spent: 40m (was: 0.5h)
> [lang] parseDate with TimeZone
> ------------------------------
>
> Key: LANG-165
> URL: https://issues.apache.org/jira/browse/LANG-165
> Project: Commons Lang
> Issue Type: Improvement
> Affects Versions: 2.1
> Environment: Operating System: All
> Platform: All
> Reporter: Bill Boland
> Priority: Minor
> Fix For: 2.2
>
> Time Spent: 40m
> Remaining Estimate: 0h
>
> I needed the ability to user a function like the
> org.apache.commons.lang.time.DateUtils.parseDate function but I needed to
> consider a different time zone when parsing the dates (assuming the format
> did
> not have the time zone as part of the input). This is needed for different
> clients to enter local date/time values on their browser and consider their
> defined time zone to convert this to the server/system time zone. I just
> thought an additional parameter to this function would make it more generic
> for this purpose while still keeping the current method signate which would
> use the tiem zone sensitive one with a null or default timezone value.
> Anyway, just thought I'd suggest it.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
