[2010–09–15] Třída Nette\Finder pro procházení adresářovou strukturou

Jestli to takhle půjde se všema novinkama v 1.0. Tak letos dooprvady ruším vánoce.

Ale bych jenom nechválil. Poslední dobou celkem hodně pracuju s obrázky. Jde něják specifikovat filter který by mě vrátil obrázek za takovýchto podmínek:

$basePath = DATA_DIR . "/images/foo";
$path = $basePath . ".gif";
if (!file_exists($path)) {
	$path = $basePath . ".png";
	if (!file_exists($path)) {
		$path = $basePath . ".jpg";
		if (!file_exists($path)) {
			$path = DATA_DIR . "/images/blank.png"; // 404 image
		}
	}
}
$image = Nette\Image::fromFile($path);

Editoval vrtak-cz (15. 9. 2010 15:29)

vrtak-cz napsal(a):

Jestli to takhle půjde se všema novinkama v 1.0. Tak letos dooprvady ruším vánoce.

Ale bych jenom nechválil. Poslední dobou celkem hodně pracuju s obrázky. Jde něják specifikovat filter který by mě vrátil obrázek za takovýchto podmínek:

$basePath = DATA_DIR . "/images/foo";
$path = $basePath . ".gif";
if (!file_exists($path)) {
	$path = $basePath . ".png";
	if (!file_exists($path)) {
		$path = $basePath . ".jpg";
		if (!file_exists($path)) {
			$path = DATA_DIR . "/images/blank.png"; // 404 image
		}
	}
}
$image = Nette\Image::fromFile($path);

A nechcem vrtat ale toto ide podla mna napisat aj krajsie a prehladnejsie:

$basePath = DATA_DIR . "/images/foo";
switch (TRUE) {
	case file_exists($path = $basePath . '.gif'):
		break;
	case file_exists($path = $basePath . '.png'):
		break;
	case file_exists($path = $basePath . '.jpg'):
		break;
	default:
		$path = DATA_DIR . "/images/blank.png";
		break;
}
$image = Nette\Image::fromFile($path);

Ako v podstate je to jedno, len tolko vnorenych if-ov mi pride hodne neprehladnych.

Já se ptal z důvodu jestli to umožňuje i takovéto „pokročilejší funkce“.

Projekty jako Doctrine2 a Symfony2 navzájem využívají své kódy a nevymýšlejí je znovu. Škoda, že se tento trend ještě nedostal k Nette – http://fabien.potencier.org/…d-your-files

Editoval hason (16. 9. 2010 23:23)

Trošku jsem si s Finderem pohrál a nejsem si jist, zda-li něco dělám špatně nebo je to bug, ale pořád mi vrací to co nechci. Jde mi o to, aby mi Finder vrátil všechny soubory dané složky, které jsou starší 20ti minut a začínají názvem „cropped_“.

Tady je můj testovací kód:

Debug::barDump(new DateTime('- 20 minutes'));
foreach (Finder::findFiles('cropped_*')->in(WWW_DIR.'/download/')->date('<', new DateTime('- 20 minutes')) as $file) {
	Debug::barDump($file->getFilename().' - '.date('Y-m-d H:i:s',$file->getMTime()));
}

Který mi vrací toto:
http://ukaz.at/vn
tj. všechny soubory, sice správně filtrované podle názvu, ale ne podle data.
Když změním < na > v metodě date tak dostavám totožný dump.

iguana007 napsal(a):

Trošku jsem si s Finderem pohrál a nejsem si jist, zda-li něco dělám špatně nebo je to bug

Ono je to takto:

Finder::findFiles('cropped_*') // nasleduji podminky pro soubory
	->in(WWW_DIR.'/download/') // nasleduji podminky pro prochazene adresare

takže tu podmínku „date“ uveď hned za findFiles.

hason napsal(a):

Projekty jako Doctrine2 a Symfony2 navzájem využívají své kódy a nevymýšlejí je znovu.

Já jsem si třeba tu část Doctrine2 která využívá Symfony2 přepsal tak aby využívala Nette. Takže Symfony2 v Nette projektu vůbec nemám (i přes to že mám Doctrine 2)… :-)

Editoval vrtak-cz (17. 9. 2010 9:19)

hason napsal(a):

Nechci dělat Davidovi advokáta (zvlášť když ho nepotřebuje), ale

1. srovnáváš syntax a nevidíš rozdíl v implementaci? Samozřejmě, rozdíl v implementaci je na úrovni zdrojových kódů. A jestli po letmém seznámení se se zdrojáky David zjistil, že Symfony vyhledává neefektivně, tak mu můžeš buď věřit, nebo to nastudovat pořádně a pak polemizovat.
2. jestliže existují volně použitelné knihovny na stejné činnosti které podporuje Nette, tak super, můžeš je vesele používat. Ale vůbec nevidím důvod kritizovat, že se v Nette vynalézá kolo – ono každé to kolo je trošku jiné a to Netťácké by mělo do zbytku frameworku pasovat jak prdýlka na nočník. A jedna z výhod Nette je, že všechny kódy vychází z jedné hlavy. (Např. na Zendu se mi nelíbilo, že to byl takový slepenec všech všemožných knihoven z různých zdrojů. Chyběla tam jednotná štábní kultura už na úrovni syntaxe.)

Zdravím, měl bych podnět … callback modifikátory. Příklad: najdu všechny .jpg obrázky v určitém adresáři a chci je všechny zobrazit na webu. Jenže cesta na disku <> url, takže by se mi hodilo předat Finderu modifikátor, který by to pěkně upravil a Finder by vracel hotové url obrázků ;)

Ono by možná stačilo přidat funkci toArray(), aby se na to daly použít array_map a array_filter funkce.

Honza Marek napsal(a):

Ono by možná stačilo přidat funkci toArray(), aby se na to daly použít array_map a array_filter funkce.

jestli to byla reakce na mě – tak samozřejmě to by šlo, nicméně, to už není o tom, jak to udělat, ale jak pěkně to udělat, a pěkné to není

<?php
$files = Finder::findFiles('*')->in(..)->toArray();
$files = array_filter($files, function($val) {
	return preg_match('#^\d{4}_\d{2}_\d{2}.log$#', $val->getFilename());
});

# je trosku prasarna oproti

$files = Finder::findFiles('#^\d{4}_\d{2}_\d{2}.log$#')->in(..);
?>

HosipLan napsal(a):

nevidím problém…

$finder = Finder::findFiles('*.log')->filter(function($file) {
        return String::match($file->getFilename(), '#^\d{4}_\d{2}_\d{2}.log$#');
});

No tohle je easy, ale já bych potřeboval např. něco takového:

$finder = Finder::findFiles('*.jpg', '*.gif', '*.png')->modifier(function($file) {
        return preg_replace("#\.(jpg|gif|png)#", "_small.\\1", $file->getFilename());
});

pesu: co takhle?

$maleobrazky = array_map(function($file) {
        return String::replace($file->getFilename(), "#\.(jpg|gif|png)#", "_small.\\1");
}, iterator_to_array(Finder::findFiles('*.jpg', '*.gif', '*.png')));

dump($maleobrazky);

hrach: tvoje řešení je samozřejmě hezčí, to je bez debaty :)

Editoval HosipLan (9. 11. 2010 11:21)

hrach napsal(a):

Feature request – doprogramovat filtrovani pomoci regexpu.

Maličkost, jen vymysli syntax, rozlišení skutečných regulárů od ořezaných. (v nich lze #^\d{4}_\d{2}_\d{2}.log$#' také zadat, sice ukecaněji [0-9][0-9][0-9][0-9]_[0-9][0-9]_[0-9][0-9].log, ale pro mnohé uživatele srozumitelněji)

David Grudl napsal(a):
Maličkost, jen vymysli syntax, rozlišení skutečných regulárů od ořezaných

Já vím, že to je právě ten zádrhel, možná by šlo:
Finder::findFilesRegexp(..)

David Grudl napsal(a):

hrach napsal(a):

Víš, v čem je zádrhel, ale stejně sem napíšeš: „Feature request – doprogramovat filtrovani pomoci regexpu.“ Vnímáš to taky jako buzeraci?

pesu napsal(a):

nepotřebuješ modifikátor, stačí

foreach (Finder.... as $file) {
	$url = String::replace($file->getFilename(), "#\.(jpg|gif|png)#", "_small.\\1");
}

Nemá smysl API Finderu zaplácávat něčím, co má triviální řešení.

No to ano, to jistě triviální řešení má. Akorát je tam o jednu iteraci navíc a pak když to chci dostat do šablony, tak to musím někam ukládat, předávat atd. Asi nějak takto:

foreach (Finder.... as $file) {
	$urls[] = String::replace($file->getFilename(), "#\.(jpg|gif|png)#", "_small.\\1");
}
$this->template->urls = $urls;

nebo

$m = new FinderModifier;
$m->setRewrite(....); //momentální nápad

$this->template->urls = Finder::find("*.jpg")->in($path)->modifier($m);

Modifier pak můžu mít v celém projektu jeden a používat ho můžu vícekrát, může dokonce mít na starosti víc věcí. Jinak mi není moc jasné, proč když jsou ve Finderu filtry by tam nemohly být modifiery … filtrování v extra iteraci je přece úplně stejně triviální a přesto jim to výsostné právo býti ve Finderu upřeno není.

David Grudl napsal(a):
Víš, v čem je zádrhel, ale stejně sem napíšeš: „Feature request – doprogramovat filtrovani pomoci regexpu.“ Vnímáš to taky jako buzeraci?

lol :) přemýšlím, kdo se blbě vyspal. nene, přečetl jsem si ve zdrojáku tvé todo, a toto mělo být takové to „+1“, jako že by se to hodilo. nic víc, nic míň. nevím co je v té větě buzeračního.

pesu napsal(a):

No to ano, to jistě triviální řešení má. Akorát je tam o jednu iteraci navíc a pak když to chci dostat do šablony, tak to musím někam ukládat, předávat atd. Asi nějak takto:

Modifikuj přímo v šabloně {$file|image}

hrach napsal(a):

jaký je názor na ten findFilesRegexp()?

Tři nové funkce findRegExp, findFilesRegexp a findDirectoriesRegExp by asi fakt skončily jako nepoužitý pull request.

to vypadá hrozně :)

Co by možná nebylo od věci tak udělat třídu RegExp jako je String, php obdobu k javascriptové verzi.

$files = Finder::find(new RegExp('~^\d{4}_\d{2}_\d{2}.log$~i'))->in(...);

• dalo by se to rozlišit globálně ve FW
• vyčlenily by se metody ze String
• hezčí api
• možnost udělat zkratku jako mají callbacky

$regexp = new RegExp('~(a+b)~i');
$result = $regexp->matchAll('baaabaaabaaa');

$results = regexp('~(a+b)~i')->matchAll('baaabaaabaaa');

Když teď nad tím tak přemýšlím,
nechce se mi mazat co už jsem napsal, ale uvědomil jsem si jak hrozně to vypadá (především ta zkratka)

Takže pokud něco takového tak spíš asi jenom takový jednoduchý kontejner, prostě typ regulární výraz pomocí třídy

namespace Nette;

use Nette;

class RegExp extends Object
{
	private $regexp;


	public function __construct($regexp)
	{
		// if not string throw new InvalidArgumentException
		// + možnost automatického doplnění delimiterů, ikdyž to je asi moc magické...

		$this->regexp = $regexp;
	}


	public function __toString()
	{
		return $this->regexp;
	}
}

Tak HosipLanovo řešení je elegantnější :) Také by se mohl vytvořit Interface:

interface IStringFinder
{
    public function __construct( $pattern );
    public function match( $text );
    public function matchAll( $text );
    ...
}

class RegExp    implements IStringFinder...
class WildChars implements IStringFinder...

Potřeboval jsem obohatit API o třídění. Takto jsem se s tím vypořádal:

<?php
class Finder extends \Nette\Finder
{
    private $order;

    /**
     * Sets the order comparison function
     * @param callback $order
     * @return Finder
     */
    public function order($order)
    {
        $this->order = $order;
        return $this;
    }


    public function orderByName()
    {
        $this->order = function($f1, $f2) {
            return \strcasecmp($f1->getFilename(), $f2->getFilename());
        };
        return $this;
    }


    public function orderByMTime()
    {
        $this->order = function($f1, $f2) {
            return $f2->getMTime() - $f2->getMTime();
        };
        return $this;
    }


    /**
     * Returns iterator.
     * @return \Iterator
     */
    public function getIterator()
    {
        $iterator = parent::getIterator();
        if ($this->order === NULL) {
            return $iterator;
        }

        $iterator = new \ArrayIterator(\iterator_to_array($iterator));
        $iterator->uasort($this->order);

        return $iterator;
    }

}
?>

Asi bych si je náhodně vyfiltroval a pak vzal prvních pět prvků výsledku.

vybrat všechny a pak použít array_rand()

tak ty klíče použij. od toho jsou