[cdi-dev] Question about conversation.begin if transient

[Mark Struberg]

not using the CDI Conversations at all anymore. But when I did it was pretty verbose. And the isTransient() check is not the only thing a user must do. Thus once we start adding one convenience method, we will soon end up with tens of them...

just my humble opinion


LieGrue,
strub



----- Original Message -----
> From: Pete Muir <pmuir at redhat.com>
> To: Adrian Gonzalez <adr_gonzalez at yahoo.fr>
> Cc: "cdi-dev at lists.jboss.org" <cdi-dev at lists.jboss.org>
> Sent: Monday, December 19, 2011 10:13 AM
> Subject: Re: [cdi-dev] Question about conversation.begin if transient
> 
> Anyone else feel strongly about this?
> 
> On 18 Dec 2011, at 17:11, Adrian Gonzalez wrote:
> 
>>  Hi Pete, 
>>  It just seems to me this 2 line code snippet is written so often, it would 
> benefit to be a one code liner.
>> 
>> 
>>  De : Pete Muir <pmuir at redhat.com>
>>  À : Adrian Gonzalez <adr_gonzalez at yahoo.fr> 
>>  Cc : "cdi-dev at lists.jboss.org" <cdi-dev at lists.jboss.org> 
>>  Envoyé le : Vendredi 16 Décembre 2011 12h05
>>  Objet : Re: [cdi-dev] Question about conversation.begin if transient
>> 
>>  I don't personally see much wrong with the code snippet today, it seems 
> pretty clear…
>> 
>>  On 15 Dec 2011, at 16:24, Adrian Gonzalez wrote:
>> 
>>  > Hello,
>>  > 
>>  > This question has surely already been asked, but I'm looking over 
> and over the following code :
>>  >        if (conversation.isTransient()) {
>>  >            conversation.begin();
>>  >        }
>>  > 
>>  > Is it possible to create a single method in Conversation class to 
> handle this ?
>>  > Something like conversation.beginIfTransient() ? (of course with a 
> better name :) ).
>>  > 
>>  > WDYT ?
>>  > 
>>  > 
>>  > _______________________________________________
>>  > cdi-dev mailing list
>>  > cdi-dev at lists.jboss.org
>>  > https://lists.jboss.org/mailman/listinfo/cdi-dev
>> 
>> 
>> 
> 
> 
> _______________________________________________
> cdi-dev mailing list
> cdi-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/cdi-dev
>