Jedna metoda ve ve dvou třídách – .NET – Fórum – Programujte.com

Ahoj,

potřebuji ve dvou třídách vytvořit metodu public Task AnyName(CancellationToken cancel) která představuje dlouze a asynchronně běžící úlohu. Jak na to?

Pozn.: každá třída je potomkem jiné třídy takže dědičnost (na rozdíl od C++) nepřichází v úvahu. Zatím jsem narazil na pojem rozhraní, ale nevím, jestli je to vhodné pro tento případ.

hu

#1 hlucheucho
rozhraní je jenom definice metod, které se musí ve zděděných třídách udělat aby algoritmus fungoval jak má např při řešení problému "dependency injection (DI)"... to s tim nemá nic společnýho leda by si dělal DI asynchronně a to asi neděláš ...

udělat tu tvojí metodu se dá podle příkladu zde:

https://docs.microsoft.com/en-us/dotnet/api/system.threading.tasks.task?view=net-6.0

https://docs.microsoft.com/cs-cz/dotnet/standard/parallel-programming/task-based-asynchronous-programming

Pokud ti jde o to, aby ty instance byly "zaměnitelné", podobně jako by měly společného předka, tak s interface to jde.

interface INeco
{
    void Metoda();
}

class ChildA: BaseA, INeco
{
    //...
    public void Metoda() { ... }
}

class ChildB: BaseB, INeco
{
    //...
    public void Metoda() { ... }
}


ChildA ca = new ChildA();
ChildB cb = new ChildB();

INeco ia = ca;
INeco ib = cb;

#1 hlucheucho
Myslím, že nejvhodnější by bylo, (pokud stávající třídy nejsou zapečetěné a pokud nenarazíš na nějaký jiný problém) vytvořit novou třídu obsahující požadovanou metodu a tuto novou třídu přidat do dědiců obou tříd.

Jak už se tu psalo interface je pouze jakýsi vzor(předpis) jaké metody musí třída obsahovat. Například pokud je třída IComparable musí obsahovat public int CompareTo(object obj)

Ono jde spíš o to psát kód metody 1x.

hu

ještě taková drobnost nezapomeň do finalizeru třídy uvést ukončení vláken !!! a případně i uvolnění paměti nebo zajisti aby se vlákna ukončily samy

https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/finalizers

https://stackoverflow.com/questions/1076965/in-c-sharp-what-is-the-difference-between-a-destructor-and-a-finalize-method-in

Čekám, kdy někdo napíše: Tak to do jedné třídy napiš a do druhé zkopíruj.

Při ladění to asi budu muset upravit a pak už vtom bude akorát bordel.

hu

nekdy je lepší udělat 2 kopie než zmatkovat v jedné složité kopii ... neznáme okolnosti takže ti nepomužu ale ... ... jako nástřel asi bych možná volil tedy static class a static metodu .... tu jedinečnou ... ale to je otázka konkrétní implementace ...
 

Ono jde spíš o to psát kód metody 1x.

asi jedině tak jak jsem psal, musíš udělat dědice, který bude navíc obsahovat třídu obsahující to, co potřebuješ.

Ještě jeden nápad: Instance obou tříd jsou vloženy do nadřazené třídy. Vytvořit tu metodu v nadřezené třídě a parametrem jí předat objekt se kterým pracuje, něco jako public Task AnyName(Object my_object, CancellationToken cancel)

hu

#10 hlucheucho
tak ono záleží na tom, jestli potřebuješ s té metody přistupovat tam nebo tam.. nebo naopak..

Nadřazená třída má ty úlohy spouštět, ukončovat a řešit selhání. Akorát to odkázání se na třídu se kterou má pracovat jako Object - není to kostrbatý?

Ráno moudřejší večera.

hu

#12 hlucheucho tak to je skoro vždycky..

Zvítězila metoda nadřazené třídy a předání pracovní instance pomocí parametru. Vyskytl se problém, jak na Object volat metody instance pracovní třídy. Od použití Object jsem upustil a použil dynamic. Takže metoda nakonec vypadá takto 

public Task AnyName(dynamic my_object, CancellationToken cancel)

hu

#14 hlucheucho
Na dynamic jsem narazil poprvé. :)

Dobré v případě, že typ objektu zjistíš až za běhu. Přetypování na dynamic je implicitní. Metodu pak voláš jakoby byla metodou třídy dynamic. Takže pak to může vypadat třeba takto 

public Task AnyName(dynamic my_object, CancellationToken cancel)
{
   my_object.object_method();
}

//volani
Task t1 = AnyName(instance_class1, cancel_token);
Task t2 = AnyName(instance_class2, cancel_token);

hu

#16 hlucheucho
Díky za info už jsem se na to i trochu díval. Pro tvůj případ se to asi hodí, sám ale nemám představu nač bych to využil...

#1 hlucheucho
Můžeš tu metodu dát do samostatné třídy a provést kompozici do těch dvou tříd.

#18 Kit
Myslím, že něco v tomto smyslu bylo napsáno ve #4.

hu

#19 hlucheucho
Ne, tam byla navržena dědičnost - místo toho navrhuji kompozici.

Neřeší to volání vnořené metody, opět by došlo na dynamic. Vytváření další třídy jen kvůli jedné metodě je dost nešikovné. Navíc když nadřazená třída existuje, obsahuje všechny instance tříd pro které se to řeší a má jim poskytovat "all inclusive".

hu

#12 hlucheucho
Ta nadřazená třída by tu metodu mohla injektovat do těch instancí.

#22 Kit
Udělej nějakou ukázku, prosím.

#23 BDS
Nemá náhodou C# delegáty přesně na tohle?

#24 Kit
Navrhuješ s tvého pohledu nejlepší řešení, píšeš o kompozici a injektování. Docela by mě zajímalo to provedení... Uniká mě rozdíl mezi tvým a HU řešením. Jeho řešení si dokážu představit, tvé ne.

#25 BDS
Nevím, jestli by někoho zajímalo mé řešení v PHP.

Nevím ani, zda ty objekty mají společného předka či nikoli, viz první příspěvek.

#26 Kit
Pozn.: každá třída je potomkem jiné třídy takže dědičnost (na rozdíl od C++) nepřichází v úvahu. Zatím jsem narazil na pojem rozhraní, ale nevím, jestli je to vhodné pro tento případ.

Nemají společného předka.

hu

Možná má představu, že obě moje třídy budou mít delegáta a jeho prostřednictvím budou volat metodu nadřazené třídy.

hu

#28 hlucheucho
Nevím sice přesně, jak funguje delegát (vzor Visitor?), ale možná by se na to hodil.

#29 Kit
delegát je v podstatě to samé jako v c++ ukazatel na funkci. Navíc může ukazovat na víc metod. 

#30 BDS
Tak to by sedělo. V různých jazycích se tomu říká různě, ale například Java to nemá, ani to nepotřebuje. JS to nazývá callback. V parametrech se předají hodnoty ke zpracování a nemusí se kvůli tomu dělat pochybné accessory.

Jinak s tím rozhraním (interface) si nebyl vůbec daleko. Záleží teda v jaké verzi C# to píšeš ale od osmičky už je dostupná default implementaci metody v rozhraní.

https://devblogs.microsoft.com/dotnet/default-implementations-in-interfaces/

Tj. by ti stačilo v obou třídách implementovat nějaké rozhraní, které by mělo definovanou tvou metodu.

#32 Matrix17
My se tady dohadujeme, kam tu implementaci umístit tak, aby nevznikla duplicita. Dávat ji do rozhraní mi připadá jako docela hloupé.

to co asi hledáte je generická třída :)

https://www.tutorialsteacher.com/csharp/csharp-generics

jednoduše voláte generiku ... v obou větvích, kde na vrvholu stromu je nějaký interface

#33 Kit
Já to naprosto chápu ale přesně kvůli tomu to je přidaný do Javy i C#. To jestli to je nebo není pěkné je věc zcela jiná. O tom se dá hádat roky. Nicméně je to pro případy, když bys tím rozbil public API a nebo i tento případ.

Ale má to i svá omezení viz. můj koment.