You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ofbiz.apache.org by "Pawan Verma (JIRA)" <ji...@apache.org> on 2019/05/25 08:37:00 UTC
[jira] [Created] (OFBIZ-11064) Use Groovy Truth instead of
UtilValidate Class in Groovy
Pawan Verma created OFBIZ-11064:
-----------------------------------
Summary: Use Groovy Truth instead of UtilValidate Class in Groovy
Key: OFBIZ-11064
URL: https://issues.apache.org/jira/browse/OFBIZ-11064
Project: OFBiz
Issue Type: Improvement
Components: ALL COMPONENTS
Affects Versions: Trunk
Reporter: Pawan Verma
As we all know, Groovy is a powerful language with great built-in functions. Groovy Truth[1] is one of them, which is not used properly in our code base. We have used UtilValidate Class to validate arguments for Empty or NotEmpty, which can easily be done in groovy with built-in functionality.
Current Code: if (UtilValidate.isNotEmpty(locations)) \{ ... }
Groovy Built-in Code: if (locations) \{ ... }
IMO, We should use this Groovy Truth feature instead of UtilValidate Class. Please let me know your thoughts on this. Thanks!
[1] - [http://groovy-lang.org/semantics.html#Groovy-Truth]
We need to careful for some points while we change this:
Like:
maxRetry = 0
if (!maxRetry) {
// Not set, use a default
maxRetry = -1
}
Because groovy evaluates zero to be false, it wouldn't be possible to set
maxRetry to zero. So it's best not to use groovy truth for null-checks on
numbers in some cases.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)