You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Phil Steitz (JIRA)" <ji...@apache.org> on 2014/12/31 17:39:14 UTC
[jira] [Closed] (POOL-203) GenericKeyedObjectPool.ObjectDeque could
be better encapsulated
[ https://issues.apache.org/jira/browse/POOL-203?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Phil Steitz closed POOL-203.
----------------------------
> GenericKeyedObjectPool.ObjectDeque could be better encapsulated
> ---------------------------------------------------------------
>
> Key: POOL-203
> URL: https://issues.apache.org/jira/browse/POOL-203
> Project: Commons Pool
> Issue Type: Improvement
> Reporter: Sebb
> Priority: Minor
>
> GenericKeyedObjectPool.ObjectDeque is currently basically just a collection of Objects with getters.
> This necessarily exposes the implementation, and makes adding invariant checks a lot harder - e.g. ensuring createCount >=0.
> The suggestion is to replace the getters with functional methods, for example:
> objectDeque.getCreateCount().incrementAndGet() => objectDeque.createdEntry();
> objectDeque.getAllObjects().put(t, p); => objectDeque.addNew(t, p);
> objectDeque.getIdleObjects().addFirst(p)/addLast(p) => objectDeque.idle(p,getLifo())
> The new methods could include assertions for invariants, and adding debug would be a doddle.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)