[
https://issues.apache.org/jira/browse/ZOOKEEPER-2630?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16149115#comment-16149115
]
ASF GitHub Bot commented on ZOOKEEPER-2630:
-------------------------------------------
GitHub user tamaashu opened a pull request:
https://github.com/apache/zookeeper/pull/354
ZOOKEEPER-2630: Use interface type instead of implementation type whe…
Use interface type instead of implementation type when appropriate.
There are a couple of places in code base where we declare a field /
variable as implementation type (i.e. HashMap, HashSet) instead of interface
type (i.e. Map, Set), while in other places we do the opposite by declaring as
interface type. A quick check indicates that most if not all of these places
could be updated so we have a consistent style over the code base (prefer using
interface type), which is also a good coding style to stick per best practice.
Checked and fixed Set, Map and List interface usages.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tamaashu/zookeeper ZOOKEEPER-2630
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zookeeper/pull/354.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #354
----
commit c9ab1da4cfc8bc30e3a93df08e0d865532f9fb40
Author: Tamas Penzes <[email protected]>
Date: 2017-08-31T15:10:33Z
ZOOKEEPER-2630: Use interface type instead of implementation type when
appropriate.
Checked and fixed Set, Map and List interface usages.
----
> Use interface type instead of implementation type when appropriate.
> -------------------------------------------------------------------
>
> Key: ZOOKEEPER-2630
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2630
> Project: ZooKeeper
> Issue Type: Improvement
> Reporter: Michael Han
> Assignee: Tamas Penzes
> Priority: Trivial
> Labels: newbie, refactoring
>
> There are a couple of places in code base where we declare a field / variable
> as implementation type (i.e. HashMap, HashSet) instead of interface type
> (i.e. Map, Set), while in other places we do the opposite by declaring as
> interface type. A quick check indicates that most if not all of these places
> could be updated so we have a consistent style over the code base (prefer
> using interface type), which is also a good coding style to stick per best
> practice.
> See more info on https://github.com/apache/zookeeper/pull/102
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
