KohanaPHP » Kohana v3.x

Sam, what is $fruit here?
https://github.com/kohana/core/blob/705d5fc5d95af4ce466dc37f46d5ec902a18c4b4/classes/kohana/response.php#L350
And why not use foreach instead of while?

list/while is faster than foreach().

Jeremy, I see, thanks for the reply.

Nevertheless

while (list($_key, $_value) = each($fruit))

while (list($_key, $_value) = each($key))

Probably :)

Jeremy Bush wrote:

list/while is faster than foreach().

Actually I don't think that's true. I haven't benchmarked it in a while, but last time I looked foreach() was actually faster (compared to using a while/list/each/reset combination). It does depend on what you do with the array in the foreach/loop, but most of the time foreach will be faster. However, the difference is normally so small it really doesn't matter though. I normally use foreach() for this kind of stuff because I find it easier to use.

Sorry, I didn't mean faster. it uses less memory in some situations where a foreach() would consume more memory and not let it go.

I used to have a link about it, but I seem to have lost it.

Anyway, this isn't the place for this debate :)

I agree with Isaiah. IMO code readability is more important than speed (futhermore when spped gain is doubtful), so I suggest change to foreach

Isaiah DeRose-Wilson wrote:

Jeremy Bush wrote:

list/while is faster than foreach().

Actually I don't think that's true. I haven't benchmarked it in a while, but last time I looked foreach() was actually faster (compared to using a while/list/each/reset combination).

foreach is better every time if it's a iteration over all elements

The so called benchmarks that supposedly prove otherwise I found most use very pointless tests such as empty bodies or operations that don't really access data (ie. make use of the loop). Each time I would benchmark against a real world example and not some cute code, foreach would come out on top. I am not sure (bad memory) if this was one of them, but at one point PHP devs have come forward to debunk a lot of these so called "better coding" practices, a lot of which had become so widespread google even had them as "recommended practices".

Anyway, foreach should be favored, if nothing else, because of readability.

Jeremy Bush wrote:

Sorry, I didn't mean faster. it uses less memory in some situations where a foreach() would consume more memory and not let it go.

Do you mean as in "foreach consumes more memory"? Last time I researched into this, I found if you have speed or memory problems in the loop consider using a more specialized data type (not that array is bad). See: http://matthewturland.com/2010/05/20/new-spl-features-in-php-5-3/ for some charts.

Or do you mean "foreach is bugged"? Do you have some kind of code that produces this? Or some bug report of this being a memory leak? Does this happen when you write:

foreach ($list as &$value)

foreach ($list as $value)

The reason for while/each in this instance is because each() produces such a horrible interface for the foreach loop. I personally believe in this instance, the syntax used is easier to read that foreach - therefore it will remain. The incorrect variable splurge however has been amended ;) apologies for that obvious error.

Please keep this discussion on topic.