Diskusia – Gascripe
SpäťUpozorňujeme, že diskusie pod našimi online kurzami sú nemoderované a primárne slúžia na získavanie spätnej väzby pre budúce vylepšenie kurzov. Pre študentov našich rekvalifikačných kurzov ponúkame možnosť priameho kontaktu s lektormi a študijným referentom pre osobné konzultácie a podporu v rámci ich štúdia. Toto je exkluzívna služba, ktorá zaisťuje kvalitnú a cielenú pomoc v prípade akýchkoľvek otázok alebo projektov.
David Hartinger:27.9.2012 12:41
Těší mě, že máš zájem se zlepšit 
První věc, teď jsem si teprve všiml toho VAR. Proboha, tohle slouží k něčemu úplně jinému, C# je silně typovaný jazyk, typy používej, tohle je opravdu špatné.
Podívejme se třeba na funkci EditSectionName:
private void EditSectionName()
{
if (SectionViewer.SelectedNode != null)
{
var doc = Interaction.InputBox("Zadej nové jméno sekce.", "Change Name Dialog", Path.GetFileName(actualPath));
var di = new DirectoryInfo(actualPath);
if (!String.IsNullOrEmpty(doc))
{
var act = di.Parent.FullName + @"\" + doc;
if (act != actualPath + @"\" + doc)
di.MoveTo(NoExistName(act));
}
SectionViewerActualize();
}
}Pracuješ tam s nějakou vizuální komponentou, předpokládám, že tohle je v kódu formu. To je správně. Ale tato část:
var di = new DirectoryInfo(actualPath);
if (!String.IsNullOrEmpty(doc))
{
var act = di.Parent.FullName + @"\" + doc;
if (act != actualPath + @"\" + doc)
di.MoveTo(NoExistName(act));
}je již přeci logika. Ta bu se neměla míchat se vzhledovou částí aplikace. Uděláš si nějakou třídu, moc nevím jak to přesně funguje, ale tady děláš něco se sekcemi, tak třeba SectionManager? Tam dáš tuto metodu (ty var jsem dal pryč):
public function rename(string doc)
// btw proč se jmenuje nové jméno sekce doc?
{
DirectoryInfo di = new DirectoryInfo(actualPath);
if (!String.IsNullOrEmpty(doc))
{
string act = di.Parent.FullName + @"\" + doc;
if (act != actualPath + @"\" + doc)
di.MoveTo(NoExistName(act));
}
}Actualpath se asi používá častěji, bude to prostě atribut té třídy SectionManager. No a teď tohle zavoláš v té zobrazovací části:
private void EditSectionName()
{
if (SectionViewer.SelectedNode != null)
{
string doc = Interaction.InputBox("Zadej nové jméno sekce.", "Change Name Dialog", Path.GetFileName(actualPath));
sectionManager.rename(doc);
SectionViewerActualize();
}
}Všimni si, že zobrazovací část říká co se má dělat, ale již tam není jak se to má dělat. Logika je schována v další třídě, kam patří.
Zatiaľ nikto nevložil komentár - buď prvý!