Mailing-List: contact dev-help@commons.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Commons Developers List" <dev@commons.apache.org>
Received-SPF: pass (athena.apache.org: domain of sebbaz@gmail.com designates
 74.125.82.46 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CACZkXPwcRMJHgbSZPE7MojHYSpqMOUr9HL4WntgJq2KJm38yQQ@mail.gmail.com>
References: 
 <CACZkXPw4ipf6iPPkayGL10meJ_1a8zLNxJsDtgbPWD7osEXOVA@mail.gmail.com>
	<CAOGo0VbqwqHhYLVM5hGzG_iq2Oh=VKsMOtZ0zt5W7Jts7pitxw@mail.gmail.com>
	<CABLGb9wFkZEa3aEGa+nhtymdGH3Vy9e0E4r1_uZ=4ZQQ75dDSw@mail.gmail.com>
	<CACZkXPwcRMJHgbSZPE7MojHYSpqMOUr9HL4WntgJq2KJm38yQQ@mail.gmail.com>
Date: Tue, 22 Oct 2013 18:37:28 +0100
Message-ID: 
 <CAOGo0Vaz6xEbKOmAd6mfGb_jbQCcTTgTY3jsuORuoH0K88zmVw@mail.gmail.com>
Subject: Re: [lang] ImmutablePair is final
From: sebb <sebbaz@gmail.com>
To: Commons Developers List <dev@commons.apache.org>
Content-Type: text/plain; charset=ISO-8859-1

On 22 October 2013 18:33, Gary Gregory <garydgregory@gmail.com> wrote:
> On Tue, Oct 22, 2013 at 1:22 PM, Paul Benedict <pbenedict@apache.org> wrote:
>
>> If you can subclass, the class will likely be mutable somehow (accessing
>> protected or package-private data?) -- even introducing new variables
>> exclusive to the subclass. The "final" keyword is used well here.
>>
>
> Here is my use case for which I've cloned ImmutablePair into my own package
> (yuck):

Composition (rather than extension) would work better here surely?

> public final class ImmutableFooImmutableBarPair extends
> ImmutablePair<ImmutableFoo, ImmutableBar> {
>
>     private static final long serialVersionUID = 123L;
>
>     public ImmutableFooImmutableBarPair (final ImmutableFoo foo, final
> ImmutableBar bar) {
>         super(Validate.notNull(nameMatch), Validate.notNull(article));
>     }
>
>     public ImmutableFoo getImmutableFoo() {
>         return this.getValue();
>     }
>
>     public ImmutableBar getImmutableBar() {
>         return this.getKey();
>     }
> ...
> ImmutableFooImmutableBarPair pair = new ImmutableFooImmutableBarPair(myFoo,
> myBar);
> ...
> pair.getImmutableFoo();
> ...
> pair.getImmutableBar();
>
> Gary
>
>
>>
>> On Tue, Oct 22, 2013 at 12:15 PM, sebb <sebbaz@gmail.com> wrote:
>>
>> > On 22 October 2013 18:10, Gary Gregory <garydgregory@gmail.com> wrote:
>> > > Hi All:
>> > >
>> > > Is there any reason we would want to keep ImmutablePair final?
>> >
>> > To stop mutable subclasses from being created?
>> >
>> > BTW, it's unfortunate that the fields are public; they should have
>> > been private (there are public getters).
>> >
>> > > Gary
>> > >
>> > > --
>> > > E-Mail: garydgregory@gmail.com | ggregory@apache.org
>> > > Java Persistence with Hibernate, Second Edition<
>> > http://www.manning.com/bauer3/>
>> > > JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
>> > > Spring Batch in Action <http://www.manning.com/templier/>
>> > > Blog: http://garygregory.wordpress.com
>> > > Home: http://garygregory.com/
>> > > Tweet! http://twitter.com/GaryGregory
>> >
>> > ---------------------------------------------------------------------
>> > To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
>> > For additional commands, e-mail: dev-help@commons.apache.org
>> >
>> >
>>
>>
>> --
>> Cheers,
>> Paul
>>
>
>
>
> --
> E-Mail: garydgregory@gmail.com | ggregory@apache.org
> Java Persistence with Hibernate, Second Edition<http://www.manning.com/bauer3/>
> JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
> Spring Batch in Action <http://www.manning.com/templier/>
> Blog: http://garygregory.wordpress.com
> Home: http://garygregory.com/
> Tweet! http://twitter.com/GaryGregory

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org