We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
module-constraint
Description:
Some of the Java classes in constraint module, namely:
constraint
Constants
ErrorUtils
ConstraintCompilerPluginUtils
CompilerPluginTestUtils
CompilerPluginTestConstants
appear to be utility classes (i.e. all of the members and methods are static). Generally speaking it is a good practice to:
static
private
ModuleUtils
final
There are benefits of doing this:
Suggested Labels:
module/constraint,hacktoberfest
Suggested Assignees:
@vil02
Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:
#4894, #4901, #4902, #4910, #4911, #4918, #4919, #4921, #4923, #4926, #4927.
The text was updated successfully, but these errors were encountered:
module-grpc
module-mime
module-tcp
module-os
module-task
module-ftp
module-soap
Successfully merging a pull request may close this issue.
Description:
Some of the Java classes in
constraint
module, namely:Constants
,ErrorUtils
,Constants
,ConstraintCompilerPluginUtils
,CompilerPluginTestUtils
,CompilerPluginTestConstants
,appear to be utility classes (i.e. all of the members and methods are
static
). Generally speaking it is a good practice to:private
(like inModuleUtils
) - because it does not make sense to have objects of such classes,final
- because it does not make sense to derive any class from them.There are benefits of doing this:
Suggested Labels:
module/constraint,hacktoberfest
Suggested Assignees:
@vil02
Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:
#4894, #4901, #4902, #4910, #4911, #4918, #4919, #4921, #4923, #4926, #4927.
The text was updated successfully, but these errors were encountered: