-
Enhancement
-
Resolution: Fixed
-
P4
-
None
-
b14
-
generic
-
generic
Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
---|---|---|---|---|---|---|
JDK-8361685 | 21.0.9-oracle | Kieran Farrell | P4 | Resolved | Fixed | b02 |
JDK-8362124 | 21.0.9 | Goetz Lindenmaier | P4 | Resolved | Fixed | master |
JDK-8361687 | 17.0.17-oracle | Kieran Farrell | P4 | Resolved | Fixed | b02 |
JDK-8362135 | 17.0.17 | Goetz Lindenmaier | P4 | Resolved | Fixed | master |
java.util.Currency.getInstance method throw IllegalArgumentException without any error message. As an enhancement a more meaningful error message could be produce.
Before :
boolean found = false;
if (currencyCode.length() != 3) {
throw new IllegalArgumentException();
}
After :
boolean found = false;
if (currencyCode.length() != 3) {
throw new IllegalArgumentException("Currency code not 3 characters length");
}
Before :
if (!found) {
OtherCurrencyEntry ocEntry = OtherCurrencyEntry.findEntry(currencyCode);
if (ocEntry == null) {
throw new IllegalArgumentException("Currency code not found");
}
...
}
After :
if (!found) {
OtherCurrencyEntry ocEntry = OtherCurrencyEntry.findEntry(currencyCode);
if (ocEntry == null) {
throw new IllegalArgumentException("Currency code not found");
}
...
}
As a comparable in java.time.format.DateTimeFormatter some error message are added in the exception message :
public TemporalAccessor parseBest(CharSequence text, TemporalQuery<?>... queries) {
Objects.requireNonNull(text, "text");
Objects.requireNonNull(queries, "queries");
if (queries.length < 2) {
throw new IllegalArgumentException("At least two queries must be specified");
}
- backported by
-
JDK-8361685 Provide more explicative error message parsing Currencies
-
- Resolved
-
-
JDK-8361687 Provide more explicative error message parsing Currencies
-
- Resolved
-
-
JDK-8362124 Provide more explicative error message parsing Currencies
-
- Resolved
-
-
JDK-8362135 Provide more explicative error message parsing Currencies
-
- Resolved
-
- relates to
-
JDK-8353713 Improve Currency.getInstance exception handling
-
- Resolved
-
- links to
-
Commit openjdk/jdk/3c8a6678
-
Commit(master) openjdk/jdk17u-dev/3cd75dd5
-
Commit(master) openjdk/jdk21u-dev/4e4f759f
-
Review openjdk/jdk/15458
-
Review(master) openjdk/jdk17u-dev/3734
-
Review(master) openjdk/jdk21u-dev/1958