Finder: prohodit prořadí argumentů?

před 9 lety

David Grudl
Nette Core | 6848
+
0
-

V diskusi pod článkem o Nette\Finder zazněly názory, že by bylo lepší prohodit pořadí „co hledat & kde“ na „kde hledat & co“. Tedy místo

foreach (Finder::findFiles('*.txt')->in($dir) as $key => $file) {
}

psát (třeba)

foreach (Finder::findIn($dir)->files('*.txt') as $key => $file) {
}

Sloveso „find“ by mi v DSL chybělo, proto nechci prosté Finder::in(...) nebo Finder::from(...).

Možnosti jsou v zásadě čtyři:

  • nechat API tak, jak je (rozhodně je nevnímám jako jediné správné)
  • obrátit pořadí, tj. přidat statické metody findIn a findFrom a nahradit findFiles() a findDirectories() za files() a directories()
  • ponechat jako statickou jen find() bez parametrů a použít např. Finder::find()->files(...)
  • věc rozvolnit a povolit obě (nebo obě tři) formy zápisu

Uvnitř je třída navržená flexibilně a úpravy by byly jen povrchové. Dříve, než se k tomu vyjádříte, zkuste si napsat pár příkladů, třeba pro běžná vyhledávání. Tohle se skutečně nedá rozhodnout, pokud si to nevyzkoušíte.

před 9 lety

Vyki
Člen | 391
+
0
-

Možnosti 3 a 4 bych nevolil. Ta třetí zbytečně zápis prodlužuje, čtvrtá se mi nelíbí, protože by v tom byl guláš, 2 možnosti zápisu by byly asi ještě v pohodě, ale tři se mi zdá už moc. Nejvíce se mi zamlouvá dvojka.

před 9 lety

jtousek
Člen | 956
+
0
-

stávající zápis +1

před 9 lety

arron
Člen | 462
+
0
-

Jenom jsem to videl vedle sebe a jsem pro obratit poradi.

před 9 lety

jansfabik
Člen | 193
+
0
-

Já bych byl radši pro tu obrácenou variantu, ale Finder::findIn se mi moc nelíbí a Finder::in taky moc ne.

David Grudl napsal(a):
Sloveso „find“ by mi v DSL chybělo, proto nechci prosté Finder::in(...) nebo Finder::from(...).

Co takhle Search::in(), příp. Search::forFiles() a Search::forDirectories()?

foreach (Search::in($dir)->forFiles('*.txt') as $key => $file)

Mě se to takhle krásně píše, protože se to tak i čte: „Hledej v $dir soubory *.txt.“

před 9 lety

xTrip
Člen | 15
+
0
-

Mě se to takhle krásně píše, protože se to tak i čte: „Hledej v $dir soubory *.txt.“

Mě se krásně píše tak jak to je včíl. „Hledej *.txt v $dir“ – nejdříve říkám CO CHCI a pak teprve KDE to mám najít.

před 9 lety

22
Člen | 1484
+
0
-

tak tak, do vyhledavace taky pisu napred co a pak kde a dalsi priklad z praxe je array_search(co, kde)

Editoval 22 (22. 9. 2010 10:13)

před 9 lety

toka
Člen | 251
+
0
-

Jsem pro zachování stávajícího zápisu. Čistě teoreticky, v případě velkého zájmu, proč neumožnit obě varianty?

před 9 lety

Filip Procházka
Moderator | 4693
+
0
-

přesně, vždycky když používám nějakou interní funkci v php tak si říkám „co kde“ abych nemusel koukat na argumenty, protože mám děravou paměť. Nechci si pamatovat další vyjímku…

//edit: na druhou stranu Nette má většinou opačnou konvenci, třeba String::match ← kde co

Editoval HosipLan (22. 9. 2010 10:31)

před 9 lety

pekelnik
Člen | 468
+
0
-

Jsem též pro zachování stávajícího stavu. Přijde mi velmi elegantní a navrch také použitelné.

před 9 lety

Mesiah
Člen | 242
+
0
-

současný zápis mi příjde zcela přirozený, když jsem si četl na phpfashion komentáře, tak jsem si říkal, že by to mohlo být taky dobry, ale takhle naživo… no.. ne.

Další co se mi fakt líbí je Finder::find()->files(...) ještě kdyby to bylo fluent, tak by každá strana mohla psát „dotazy“ jak by si přála :)

před 9 lety

hanakus
Člen | 22
+
0
-

Určitě ne více způsobů, byl by v tom bordel.