+1 for the first, because -3 lines of code -1 variable, but really either
are good.

>From my iPhone

On Mar 15, 2013, at 10:35 PM, Joseph Crawford <codeb...@gmail.com> wrote:

I personally would disagree with you on that one.  I think the single
return point is more confusing.  Though that is more of a preference.

The single return path leads to extra processing that would not be required.
On Mar 15, 2013 9:12 PM, "Robert Stoll" <rst...@tutteli.ch> wrote:

> I would say the first version is ok but only because your function is
> small enough.****
>
> If your function gets bigger and bigger, then it is better to have only
> one exit point  in terms of readability.****
>
> ** **
>
> *Von:* talk-boun...@lists.nyphp.org [mailto:talk-boun...@lists.nyphp.org]
> *Im Auftrag von *Joseph Crawford
> *Gesendet:* Freitag, 15. März 2013 22:15
> *An:* NYPHP Talk
> *Betreff:* Re: [nyphp-talk] style question: returning from a function
> while you're in a foreach****
>
> ** **
>
> The only reason would be poorly designed code where the return value could
> change based on multiple separate conditionals.  I have seen it in some
> legacy code.****
>
> On Mar 15, 2013 5:50 PM, "Anthony Ferrara" <ircmax...@gmail.com> wrote:
> >
> > The first is fine. There's no reason not to do it...
> >
> >
> > On Fri, Mar 15, 2013 at 5:30 PM, Joseph Crawford <codeb...@gmail.com>
> wrote:
> >>
> >> There are a few things to learn from here.
> >>
> >> First it is not only a styling thing.  In the first one you have
> several return
> >> statements and in the second you only have 1 return statement.
> >>
> >> It will depend on the code logic but I tend to return when I have the
> value I need rather
> >> than assign it to a variable and have it continue to process through
> the rest of the function.
> >>
> >> If you only return after the entire function is complete but your known
> case was the first check in
> >> the function you are "over" processing and will only add time to your
> sites load time.
> >>
> >> Thanks,
> >> Joseph Crawford
> >>
> >> On Mar 15, 2013, at 5:08 PM, David Mintz wrote:
> >>
> >>> function whatever(Array $array) {
> >>>
> >>>     foreach ($array as $key => $value) {
> >>>         if  ($something)  {
> >>>              return true;
> >>>         }
> >>>     }
> >>>     return false;
> >>> }
> >>>
> >>> Is there any reason  -- style, legibility, whatever -- not to do the
> above? Or should you do something like
> >>>
> >>> function whatever(Array $array)  {
> >>>
> >>>    $return = false;
> >>>     foreach ($array as $key => $value) {
> >>>         if  ($something)  {
> >>>              $return = true;
> >>>              break;
> >>>         }
> >>>     }
> >>>     return $return;
> >>> }
> >>>
> >>> Thanks.
> >>>
> >>>
> >>> --
> >>> David Mintz
> >>> http://davidmintz.org/
> >>> Fight for social equality:
> >>> http://socialequality.com/
> >>>
> >>> _______________________________________________
> >>> New York PHP User Group Community Talk Mailing List
> >>> http://lists.nyphp.org/mailman/listinfo/talk
> >>>
> >>> http://www.nyphp.org/show-participation
> >>
> >>
> >>
> >> _______________________________________________
> >> New York PHP User Group Community Talk Mailing List
> >> http://lists.nyphp.org/mailman/listinfo/talk
> >>
> >> http://www.nyphp.org/show-participation
> >
> >
> >
> > _______________________________________________
> > New York PHP User Group Community Talk Mailing List
> > http://lists.nyphp.org/mailman/listinfo/talk
> >
> > http://www.nyphp.org/show-participation****
>
> _______________________________________________
> New York PHP User Group Community Talk Mailing List
> http://lists.nyphp.org/mailman/listinfo/talk
>
> http://www.nyphp.org/show-participation
>
_______________________________________________
New York PHP User Group Community Talk Mailing List
http://lists.nyphp.org/mailman/listinfo/talk

http://www.nyphp.org/show-participation
_______________________________________________
New York PHP User Group Community Talk Mailing List
http://lists.nyphp.org/mailman/listinfo/talk

http://www.nyphp.org/show-participation

Reply via email to