Diskuze: Jak byste zkrátili tento kod? Nevím si rady
V předchozím kvízu, Test znalostí C# .NET online, jsme si ověřili nabyté zkušenosti z kurzu.
No přímo špatně na tom nic není ale ve všech těch podmínkách testuješ jestli je vybranej metr a něco. To je dost zbytečné a taky nepřehledné lepší způsob by byl použít jednu podmínku jestli je to metr nazačátku a potom pomocí switche testovat ostatní podmínky.
if (delka_comboBox1.SelectedItem.ToString() == "metr")
{
switch(delka_comboBox2.SelectedItem.ToString())
{
case "kilometr":
delka_vysledek_label.Text = Convert.ToString(delka / 1000);
break;
}
}a tímhle stylem pro zbytek. Dálé není vhodné používat diakritiku, maximálně v komentářích.
+20 Zkušeností
+2,50 Kč
sadlomaslox25:10.8.2015 19:03
hosi hosi stydte se 
public partial class Form1 : Form
{
public Form1()
{
InitializeComponent();
var data = new Ratio[]
{
new Ratio("metr", 1),
new Ratio("milimetr", 0.001m),
new Ratio("kilometr", 1000),
new Ratio("centimetr", 0.01m),
new Ratio("decimetr", 0.1m)
};
delka_comboBox1.Items.AddRange(data);
delka_comboBox2.Items.AddRange(data);
}
private void delka_button_Click(object sender, EventArgs e)
{
Ratio from = delka_comboBox1.SelectedItem as Ratio;
Ratio to = delka_comboBox2.SelectedItem as Ratio;
delka_vysledek_label.Text = (delka_numericUpDown1.Value * from.Ration / to.Ration).ToString();
}
}
public class Ratio
{
public string Name { get; private set; }
public decimal Ration { get; private set; }
public Ratio(string name, decimal ratio)
{
Name = name;
Ration = ratio;
}
public override string ToString()
{
return Name;
}
}Díky moc pjanusi, tohle mi moc pomohlo 
a sadlomaslox taky díky, ale tohle
je na mě ještě moc složitý
@ostrozan Díky moc za postřeh, měl jsem už vygumovanou hlavu a nevěděl,
kde to přepsat, díky!
ostrozan:11.8.2015 18:00
Já osobně se teda nestydím - tvoje řešení je jedno z několika které
se nabízí , ale vytasit ho na začátečníka je jako jít s kanónem na
vrabce tobě, mně a spoustě
dalších příjde jako triviální, ale věř mu, že mu z toho jde hlava kolem
a univerzálním typem var bych mu taky ještě hlavu nemotal.
Myslím, že i když by ho použil, tak ještě dlouhou dobu nebude vědět "vo
co gou".
A proto řešení Ondřeje považuju v tomto případě jako nejlepší.
Musím ale dodat, že tazatel by se samozřejmě měl časem dobrat i
takovýchto postupů a tys mu ukázal, jak že by to mnělo vypadat.
Když už jsem se do diskuse pustil, nedá mi to abyc ti (i dalším začátečníkům) nedoporučil ještě jednu věc i když je mimo téma:
Jestli se chceš tvorbou aplikací zabývat vážněji, pouštět se so složitějších projektů - co nejdřív přejdi z Windows Forms na WPF.
A proč?
- není to nic složitého - všechno, co ses do teď naučil využiješ - největší odlišností u WPF je, že vstupuje do hry značkovací jazyk (XAML) pro tvorbu formuláře,ale funguje tu stejně drag and drop z tooloxu jako u WF
- výhody jsou dobře popsané ve zdejších tutorialech tak jen ve zkratce: binding, relativní pozicování... ale i další, které v tutoriálu nejsou jako směrované události, nebo flow document mně osobně ulehčily hodně práce nemluvě o tom, že si ve WPF můžeš - třeba za pomoci flow documentu dovolit věci ve WF nemyslitelné.
- a nakonec WPF se už na tolik podobá jazyku JAVA , že naučit se ho je pak
hračka -
Zobrazeno 17 zpráv z 17.