Prilisna slozitost FormContainer::setValues + ::getValues

juzna.cz

Studoval jsem tyto dve funkce docela dlouho, abych pochopil jak funguji (link) a ta implementace je docela drsna! Je nejaky duvod, proc je to takto slozite udelane? Proc to neni treba rekurzivne, treba takto ?

Rekurzivni verze mi prijde o mnoho srozumitelnejsi. Je tam neco, co nevidim? Nebo se jedna o nejakou vykonovou optimalizaci?

juzna.cz

Zkusil jsem napsat patch a krasne mi funguje. Co vy na to?

Jan Tvrdík

Předpokládám, že se jednalo o výkonovou optimalizaci. Zkoušel jsi to změřit?

juzna.cz

Zkusil jsem udelat test case s obrovskym mnozstvim vnorenych formularu a prvku, kde jsem vychazel z tests/Forms/Forms.namingContainer.001.phpt.

Moje implementace: 12.5 sekundy

Originalni implementace: 13.0 sekundy

Tam tedy rozdil prakticky neni

Edit: muj test je zde

Editoval juzna.cz (19. 3. 2011 15:11)

Patrik Votoček

a paměť?

juzna.cz

Spotreba pameti je uplne stejna (v mem test case si obe implementace vezmou 37MB)

David Grudl

To je z historických důvodů, kdy místo if ($control instanceof FormContainer) tam bylo if ($control instanceof INamingContainer), tj. objekt, který neměl setValues(). Tohle to už dnes neplatí, takže by šlo použít rekurzivní volání.

juzna.cz

Tak v tom pripade tedy prosim o merge

David Grudl

Říkám si, že podporu pro jiné kontainery bych asi vyhodil, tj. tyto řádky https://github.com/…4e2b07c17c68#…

juzna.cz

Tot otazka, nechtel jsem do toho zasahovat protoze si nejsem jisty, jak a zda se to pouziva. Ja jsem to teda nikdy driv nepouzil a na tuto moznost jsem narazil az pri psani tohoto patche.