rarouš.weblog - komentáře k článku

Komentář k článku Vracet List je špatné

Michal — Sat, 01 Aug 2009 09:10:27 GMT

Tato strategie je vhodna tehdy, pokud potrebujes do budoucna vnitrni implementaci skryt nebo se ocekavaji vetsi zmeny ve vnitrni implementaci (anebo je nezadouci, aby mohl konzument s datama snadno manipulovat).

Pokud se pracuje s treba vysledkem dotazu, seznamem uzivatelu podle pevnych kriterii (a zmeny v kolekci nemaji dopad kamkoliv jinam), pak je IList/List/kolecke apod. prijemne zjednoduseni zivota na obou stranach.

Ale dobry clanek, spouste lidi tebou prezentovana situace nedochazi a to je spatne. A obecne by API melo spise skryvat, nez odkryvat ;-)

Komentář k článku Vracet List je špatné

Aleš Roubíček — Sat, 01 Aug 2009 09:32:53 GMT

[1] Michal: Ani u výsledků dotazů není důvod List/IList zveřejňovat. Dnes už neni situace, kde by nestačilo IEnumerable a LINQ. :) A pokud jako konzument ten List opravdu nutně potřebuješ, pak má pěkný konstruktor, který ho z IEnumerable vytvoří.

Ad API: To neni o skrývání, ale o usměrňování.

Komentář k článku Vracet List je špatné

Tomáš Jecha — Sat, 01 Aug 2009 17:29:16 GMT

Nebylo by lepší podědit od ReadOnlyCollection a implementovat mu metody na přidání a odebrání rolí? Třída User jej pak bude nabízet vlastností Roles. Proč přetahovat logiku kolekce přímo do objektu uživatele?

Komentář k článku Vracet List je špatné

Aleš Roubíček — Sat, 01 Aug 2009 18:04:02 GMT

[3] Tomáš Jecha: Ano, nebylo. User totiž nemá logiku kolekce. To si myslíš jen díky tomu, že znáš implementační detaily. Ty se ovšem mohou v průběhu času měnit (a v praxi se mění).

V prvním případě, s takovou změnou budeš mít hodně práce. Všude, kde pracuješ s listem rolí user.Roles, budeš muset udělat změnu. Pokud tvůj kód konzumuje i někdo jiný, máš zaděláno na BC a ty většinou nikdo rád nemá. V druhém případě takovým nepříjemnostem předcházíš.

Navíc je snad pochopitelné, že když chci přidat uživateli roli user.AddRole, tak to neznamená, že chci přidal doli do kolekce, která patří uživateli user.Roles.Add. BTW přečti si v článku odkazovaný Demeterův zákon, tam je přesně popsáno, proč má odpověď na tvou první otázku je kladná.

Komentář k článku Vracet List je špatné

Tomáš Jecha — Sat, 01 Aug 2009 18:37:36 GMT

[4] Aleš Roubíček: O to jde, mě se právě onen „Demeterův zákon“ moc nelíbí. Seznam rolí je jednou seznam a schovávat ho za metody do rodičovského objektu mi přijde jako krok zjednodušení objektového návrhu za cenu ztráty obecného přístupu ke kolekci. Podle mě se vhodnost použití obou přístupů značně liší případ od případu.

Komentář k článku Vracet List je špatné

Aleš Roubíček — Sat, 01 Aug 2009 20:49:07 GMT

[5] Tomáš Jecha: Pozor na to, seznam rolí, rozhodně nemusí být kolekce rolí!

public class User {
  private string _roles;

  public User() {
    _roles = String.Empty;
  }

  public void AddRole(Role role) {
    if (String.IsNullOrEmpty(_roles) == false) {
      _roles += " ";
    }
    _roles += role.Name;
  }

  public void RemoveRole(Role role) {
    _roles.
      Replace(role.Name, String.Empty).
      Replace("  ", " ");
  }

  public bool IsInRole(Role role) {
    return _role.Contains(role.Name);
  }
}

Tato implementace nepoužívá žádnou kolekci a je stejně validní jako implememntace s listem rolí. Je to jen implementační detail. Objektový návrh by měl být co nejjednoduší. O to přece jde, dělat věci jednoduše, intuitivně, ne něco lámat přes koleno. Jak už jsem psal v minulé reakci, jde o to přiřadit roli uživateli, ne o to přidat záznam do nějaké kolekce.

Můj příklad byl možná špatný v tom, že jsem publikoval vlastnost IEnumerable<Role> Roles. V podstatě by měly bohatě statčit ony tři metody AddRole, RemoveRole a IsInRole. Nic víc…

Komentář k článku Vracet List je špatné

Augi — Sat, 01 Aug 2009 21:19:26 GMT

Pěkný. Vracet IEnumerable<T> jsem se naučil už dávno. Předtim jsem měl ještě nějaký techtle-mechtle s ReadOnlyCollection, ale ty jaksi nedopadly. Vracet IEnumerable<T> má taky tu výhodu, že v metodě můžeme vesele yieldovat ;-) A ještě bych vypíchl nedoceněné použití ToArray() – dokáže to vyřešit pár základních problémů – multi-threading a úpravu kolekce během enumerace.

Komentář k článku Vracet List je špatné

Aleš Roubíček — Sat, 01 Aug 2009 22:01:28 GMT

[7] Augi: Přesně o tom bude zítřejší spot, pokud nebude velká kocovina :)

Komentář k článku Vracet List je špatné

Rene Stein — Sat, 01 Aug 2009 22:20:21 GMT

Jen rychla poznamka z MDA. Tento postup je hezky, ale v prazi rychle narazis na limity. Podle me jde v clanku o „skolni“ priklad – na prvni pohled pekne reseni ukazujici vyhody, ale trochu nucene a nepresvedcive.

Demeteruv zakon – slovo zakon je nestastne, jedna se spis o uzitecne doporuceni, jak kontrolovat, ze implementace metod nevede ke vzrustajicimu zatizeni tridy dalsimi nesouvisejicimi tridami, a tim i k likvidaci jednoho z hlavnich zaklinadel OOP, tedy znovupouzitelnosti tridy, „cerne skrinky“, ktera nema prilis predpokladu o ostatnch tridach v okoli.
Co kdyz trida uzivatel bude obsahovat i kolekci vlastnich prav. Bude vystavena kolekce? Objekt, na ktery je delegova sprava prav? Nebo ty vystavis dalsi (treba jen delegujic na spravce pravi) metody AddRight, RemoveRight, HasRight? Rozhrani sice bude pekne, ale po case se pri tomto postupu zacnes potykat se zasadou"rozhrani tridy by melo byt jednoduche a intuitivni, nemely byste mit ve tride prilis mnoho metod". Neni tedy potreba jeste odkazat na princip separatnich rozhrani tridy (ISG)? A jsme schopni „rozseka“ rozhrani tridy User tak, aby jednotliva specializovana rozhrani slouzila ruznym klientum?

Tim chci jen rici, ze navrh je (neprekvapive) kompromisem mezi ruznymi zakony a principy a ze napsat „kontroverzni“ i clanek, jak jsi zminoval na Twitteru, se da vzdy nejlepe pojetim nejakeho principu jakozto axiomu vsech axiomu a kladiva s totalnim narokem na ubiti vsech problemu. ;)

List(T) pro typove kolekce nepouzivam. Mam vlastni typove kolekce zalozene na Collection(T).

Omlouvam se za preklepy, na MDA se ale komentar do tohoto boxu pise spatne. ;)

Komentář k článku Vracet List je špatné

stej — Mon, 03 Aug 2009 10:05:15 GMT

„Ale, jak teď zjistim, kolik rolí má uživatel přiřazených, když IEnumerable nemá vlastnost Count ani Length?“ using System.Linq;

Osobně se ztotožňuju s IEnumerable.Count() is a Code Smell

Komentář k článku Vracet List je špatné

Aleš Roubíček — Mon, 03 Aug 2009 10:27:42 GMT

[10] stej: Samozřejmě, v tamtom případě, ano. Jenže to, co popisuju já, negeneruje dotaz do databáze. :) Pokud programátor, netuší, co jeho kód dělá, měl by přemýšlet o jiném zaměstnání.

Komentář k článku Vracet List je špatné

Steve — Sat, 29 Aug 2009 20:33:36 GMT

[11] Aleš Roubíček: Kdychom to měli vzít důsledně, tak na msdn se o IEnumerable píše „Exposes the enumerator, which supports a simple iteration over a (non-)generic collection.“ => IEnumerable (schválně) „neexposuje“ nic o počtu prvků.

Dogmatické řešení dovedené do krajností by podle mě mělo vypadat tak, že třída User bude mít ještě vlastnost RolesCount.

Btw. a co třeba RolesCollection implementující IList<> (ICollection<>) a vnitřně využivající List<> pomocí skládání? :-P

Jinak taky vždy vracím IEnumerable<>, ale teprve díky tomuhle článku a komentářům pod ním, jsem o tom začal hlouběji přemýšlet.

Komentář k článku Vracet List je špatné

Aleš Roubíček — Sun, 30 Aug 2009 08:31:48 GMT

[12] Steve: Díky za reakci. :)

Já jsem s odstupem času došel k tomu, že příklad s uživatelskými rolemi, nebyl zrovn nejšťastnější. :) Lepší by byla ukázka ve stylu person.Wallet.Remove(), kde Wallet je IList of Money vs. person.PayBill(). :) Myslím, že šahat na peněženku si jen tak někdo nenechá. A pak je Demeterův zákon stravitelnější.

Ano RolesCount by šlo implementovat, ale opravdu potřebuje uživatel znát, kolik rolí má?

Komentář k článku Vracet List je špatné

Steve — Sun, 30 Aug 2009 23:16:57 GMT

[13] Aleš Roubíček: Souhlasím: peněženka dává větší smysl, tam je to pěkně vidět.

Ad potřebnost RolesCount: záleží na tom. Když bych např. chtěl v UI administrace umožnit editaci seznamu rolí přiřazených danému uživateli, tak by IEnumerable<Role> i RolesCount (resp. Linq) přišli vhod pro jejich výpis, stránkování atd.

Taky díky za reakci :)

Komentář k článku Vracet List je špatné

zsery — Wed, 21 Sep 2011 11:35:13 GMT

Připadá mi zvláštní : Když vracíte List<T>, znamená to, že říkáte všechno o vnitřní implementaci. Naopak si myslím, že tím říkáme jen a jen že to „vylejeme“ jako List<T>.