Bezpečnější foreach v latte

Honza Marek

V PHP mě irituje ta vlastnost foreache, že dokáže přepsat dříve definovanou proměnnou, která se nešťastnou shodou okolností jmenuje stejně jako proměnná, do které foreach ukládá procházené prvky.

$name = 'name';

foreach (array('a', 'b', 'c') as $name) {

}

echo $name; // vypíše c místo name

To vede k občasným záhadným chybám v šablonách. Nemohlo by tedy latte předchozí proměnnou (pokud existuje) před foreachem zazálohovat a pak jí navrátit původní obsah?

David Grudl

Taky jsem na to párkrát narazil, ale nejsem si vědom žádného non-WTF způsobu, jak to řešit.

Foowie

Možná by se mohla proměnná po ukončení cyklu alespoň unsetnout.

Honza Marek

DG: vyparsovat z „expression as $a ⇒ $b“ názvy proměnných za as, kouknout na ně v get_defined_vars. Pak by možná stačilo vyhodit laděnku s varováním a nemuselo by se nijak čarovat se zálohováním původních hodnot.

Foowie: To je k ničemu. Pokud mi foreach něco omylem přepíše nebo omylem smaže, to vyjde na stejno.

Filip Procházka

Mohlo by to vyhazovat třeba notice…

Tomáš Kolinger

V případě přepsání vyhodit laděnku…

To zálohování je víc WTF, než samotná vlastnost PHP. Hlavně když si na to člověk zvykl.

paranoiq

Foowie napsal(a):

Možná by se mohla proměnná po ukončení cyklu alespoň unsetnout.

někdy je ale účelem, aby poslední hodnota zůstala v proměnné

mkoubik

paranoiq: V tom případě si ji můžeš před cyklem unsetnout, nebo ještě lépe pojmenovat si ji tak, aby s ničím nekolidovala.

David Grudl

Honza Marek napsal(a):

DG: vyparsovat z „expression as $a ⇒ $b“ názvy proměnných za as, kouknout na ně v get_defined_vars. Pak by možná stačilo vyhodit laděnku s varováním a nemuselo by se nijak čarovat se zálohováním původních hodnot.

Jako že by

{foreach $cokoliv as $item}...{/foreach}
{foreach $cokoliv as $item}...{/foreach}

Vyhodilo varování, že si mám v každém cyklu vymyslet jinou proměnou? Olalá!

juzna.cz

Fixing magic with more magic :)
Taky se mi parkrat stalo, ze jsem kvuli tomuto zuril a chtel to zmenit. Ale jak se clovek uklidni, dojde mu, ze by vznikly daleko vetsi WTF a ze by v budoucnu asi zuril jeste vic.

(toto aspon kazdemu dojde, pze je to tak v PHP; ale kdo by pak mohl tusit, jak se chova latte, ktere do detailu nezna)

Spise bych to resil nejakou statickou analyzou na urovni IDE

paranoiq

@mkoubik: wtf?

Milo

// Kontrolovat jedinečnost $item, na konci cyklu unset($item),
// nebo spustit v limited scope.
{foreach $items as $item}...{/foreach}

// Nic nekontrolovat
{!foreach $items as $item}...{/foreach}

jtousek

Milo napsal(a):

// Kontrolovat jedinečnost $item, na konci cyklu unset($item),
// nebo spustit v limited scope.
{foreach $items as $item}...{/foreach}

// Nic nekontrolovat
{!foreach $items as $item}...{/foreach}

Když už tak kvůli zpětné kompatibilitě raději naopak.

mkoubik

paranoiq: Po skončení cyklu by ti tam ta proměnná normálně zůstala. Nicméně jak nad tím přemýšlím, tak by to bylo velký WTF.

paranoiq

pořád nerozumím na co vlastně reaguješ :]

k čemu unsetování před cyklem, když se stejně v cyklu přepíše?
jak „pojmenovat aby nekolidovala“? tahle debata je právě o tom jak onu kolizi rozumě detekovat (a asi to nejde)

@jtousek: to ale odporuje konvenci vykřičníku ve zbytku Latte

mkoubik

Reagoval jsem na:

někdy je ale účelem, aby poslední hodnota zůstala v proměnné

Navrhovaná úprava (vyhazovaní notice) by tohle nerozbila (pokud jsi tu proměnnou nepoužil už předtím).

// tady $item neexistuje
foreach ($items as $item) { ... }
// tady $item obsahuje poslední položku, no problem

Honza Marek

David Grudl napsal(a):

Jako že by
{foreach $cokoliv as $item}...{/foreach}
{foreach $cokoliv as $item}...{/foreach}
Vyhodilo varování, že si mám v každém cyklu vymyslet jinou proměnou? Olalá!

No tak pak se to bude muset kontrolovat jen proti $template->getParameters()… #tonoucisesteblachyta

uestla

Nedá mi to nereagovat:

Myslím, že tohle už by bylo až moc, mohlo by to dokonce býti i na škodu.

PHP holt takové je, a faktu, že iterační proměnné jsou po doběhnutí skriptu „obsazené“ hodnotami z poslední iterace, může někdo využívat.

Pak je jakákoli machinace (jako udělat iterační proměnné lokální jen pro vnitřek cyklu, kontrola předchozí obsazenosti proměnné se stejným jménem, atd.) vyloženě nežádoucí.

Editoval uestla (12. 6. 2012 0:02)