Diskuze: Odstranění redundantního kódu
V předchozím kvízu, Test znalostí C# .NET online, jsme si ověřili nabyté zkušenosti z kurzu.
Zobrazeno 5 zpráv z 5.
//= Settings::TRACKING_CODE_B ?> //= Settings::TRACKING_CODE ?>
V předchozím kvízu, Test znalostí C# .NET online, jsme si ověřili nabyté zkušenosti z kurzu.
Vytvořil bych společné rozhraní (v tvém případě IFigurka), určil bych společné metody pro každou figurku a následně bych do každé třídy reprezentujíc figurku toto rozhraní implementoval.
Dávám příklad:
//Rohraní
interface IFigurka{
void AktualizujMozneUtoky(Situace situace, Sour sour); //Určení společné metody pro každou figurku
}
//Tvoje figurka
class Kral : IFigurka{
public void AktualizujMozneUtoky(Situace situace, Sour sour){
//Kód metody
}
}
//...a následně tvoje switch klazule:
//...
IFigurka figurka;
switch(figurkaChar){
case 'k': //Tento blok opakuj pro každou figurku, pokaždé vytvoříš instanci třídy tvé figurky, a přetypuješ ji na implementované rozhraní.
figurka = new Kral(...);
break;
//...
}
figurka.AktualizujMozneUtoky(situace, ...);
//Dále využívej metody, které sis v rozhraní nadefinoval.
//...
Příklad není samozřejmě úplný (nechtělo se mi implementovat každý detail, který je z tvého kódu patrný), ale myslím si, že jako ukázka to bohatě stačí.
Jinak, prosil bych vkládat kód pomocí editoru (tlačítko se symbolem kódu nad textovým políčkem); je to přehlednější.
Přesně takového případy
if a==1 udelej neco else if a==2 udelej neco jineho
atd se řeší
krásně polymorfismem.
Interface redundantní kód krásně zlikvidoval. Moc interface nepoužívám, protože si prostě neuvědomím, že se na určitou situaci krásně hodí. Budu si muset ještě projet pár praktických užití, ale už teď v něm vidím potenciál.
Interface se používá spíše pro definici určitého kontraktu, jakasi abstrakce.. Zde bych využil spíše polymorfismu a dědění, jelikož velmi pravděpodobně budou mít figurky velkou část kódu společnou a ta co společná není - tu jde extrahovat do samostatné metody, která bude overridnuta. Klasicka inner strategy. Interface zde povede napsal spíše k redundancí.
Zobrazeno 5 zpráv z 5.