10:24 a.m.
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 ?
5:05 a.m.
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(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/cdi-dev
11:11 a.m.
New subject: Re : Question about conversation.begin if transient
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(a)redhat.com>
À : Adrian Gonzalez <adr_gonzalez(a)yahoo.fr>
Cc : "cdi-dev(a)lists.jboss.org" <cdi-dev(a)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(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/cdi-dev
3:13 a.m.
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(a)redhat.com>
À : Adrian Gonzalez <adr_gonzalez(a)yahoo.fr>
Cc : "cdi-dev(a)lists.jboss.org" <cdi-dev(a)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(a)lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/cdi-dev
3:53 a.m.
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(a)redhat.com>
To: Adrian Gonzalez <adr_gonzalez(a)yahoo.fr>
Cc: "cdi-dev(a)lists.jboss.org" <cdi-dev(a)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(a)redhat.com>
> À : Adrian Gonzalez <adr_gonzalez(a)yahoo.fr>
> Cc : "cdi-dev(a)lists.jboss.org" <cdi-dev(a)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(a)lists.jboss.org
> > https://lists.jboss.org/mailman/listinfo/cdi-dev
>
>
>
_______________________________________________
cdi-dev mailing list
cdi-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/cdi-dev
3:55 p.m.
New subject: Re : Question about conversation.begin if transient
Thus once we start adding one convenience method, we will soon end up
with tens of them...
Youps... good point, didn't though about this. I should use CDI quite a bit more.
I think this closes this thread ;)
Thanks !
________________________________
De : Mark Struberg <struberg(a)yahoo.de>
À : Pete Muir <pmuir(a)redhat.com>
Cc : cdi-dev <cdi-dev(a)lists.jboss.org>
Envoyé le : Lundi 19 Décembre 2011 10h53
Objet : Re: [cdi-dev] Question about conversation.begin if transient
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(a)redhat.com>
To: Adrian Gonzalez <adr_gonzalez(a)yahoo.fr>
Cc: "cdi-dev(a)lists.jboss.org" <cdi-dev(a)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(a)redhat.com>
> À : Adrian Gonzalez <adr_gonzalez(a)yahoo.fr>
> Cc : "cdi-dev(a)lists.jboss.org" <cdi-dev(a)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(a)lists.jboss.org
> > https://lists.jboss.org/mailman/listinfo/cdi-dev
>
>
>
_______________________________________________
cdi-dev mailing list
cdi-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/cdi-dev
_______________________________________________
cdi-dev mailing list
cdi-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/cdi-dev
4751
days inactive
4755
days old
5 comments
3 participants
participants (3)
-
Adrian Gonzalez -
Mark Struberg -
Pete Muir