Diskuze – Gascripe
ZpětUpozorňujeme, že diskuze pod našimi online kurzy jsou nemoderované a primárně slouží k získávání zpětné vazby pro budoucí vylepšení kurzů. Pro studenty našich rekvalifikačních kurzů nabízíme možnost přímého kontaktu s lektory a studijním referentem pro osobní konzultace a podporu v rámci jejich studia. Toto je exkluzivní služba, která zajišťuje kvalitní a cílenou pomoc v případě jakýchkoli dotazů nebo projektů.
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ří.
Zobrazeno 10 zpráv z 21.