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 is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to