Kritika programu (kodu) – .NET – Fórum – Programujte.com
 x   TIP: Přetáhni ikonu na hlavní panel pro připnutí webu
Reklama
Reklama

Kritika programu (kodu) – .NET – Fórum – Programujte.comKritika programu (kodu) – .NET – Fórum – Programujte.com

 

Hledá se programátor! Plat 1 800 € + bonusy (firma Boxmol.com)
marpit0
Stálý člen
7. 11. 2009   #1
-
0
-



using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace DPSR
{
public partial class Form1 : Form
{
double delka, sirka, tloustka;

public Form1()
{
InitializeComponent();
}

private void button1_Click(object sender, EventArgs e)
{
delka = double.Parse(textBox1.Text.ToString()) * 0.001;
sirka = double.Parse(textBox2.Text.ToString()) * 0.000001;
tloustka = double.Parse(textBox3.Text.ToString()) * 0.001; //načtení a převod na základní jednotky

RDPS rdps = new RDPS(delka, sirka, tloustka); //vyvolání
textBox4.Text = rdps.Odpor().ToString() + rdps.jednotka; //vypsání
}

private void button2_Click(object sender, EventArgs e)
{
textBox4.Clear();
}

private void linkLabel1_LinkClicked(object sender, LinkLabelLinkClickedEventArgs e)
{
System.Diagnostics.Process.Start("http://www.elmetr.ic.cz");
}
}

public class RDPS //RDPS
{
private double delka, sirka, tloustka, odpor;
public string jednotka;
private const double merny_odpor = 0.0000000178;

public RDPS(double delka, double sirka, double tloustka)
{
this.delka = delka;
this.sirka = sirka;
this.tloustka = tloustka;
}

public double Odpor()
{
odpor = merny_odpor * (delka / (sirka * tloustka));

if (odpor <= 0.001)
{
jednotka = " uR";
return odpor * 1000000; //uR
}

if (odpor < 1) //mR
{
jednotka = " mR";
return odpor * 1000;
}

if (odpor < 1000) //R
{
jednotka = " R";
return odpor;
}

if (odpor <= 1000)
{
odpor = odpor / 1000; //k
jednotka = " k";
return odpor;
}

else
{
MessageBox.Show("Odpor by byl větší než než 999.999k, což není možné...");
return 0;
}
}
}
}


Prosím o kritiku kodu.

Nahlásit jako SPAM
IP: 78.102.212.–
MarPit
Reklama
Reklama
7. 11. 2009   #2
-
0
-

Asi toho spoustu vynechám, ale napíšu, co mi vadí.

1. Název formuláře je Form1, bylo by dobrý ho nějak pojmenovat. Totéž pro prvky na formuláři.
2. Objevují se tam názvy proměnných, metod, apod. v češtině, což nemám moc rád.
3. Proč voláš ToString() u vlastnosti typu String?
4. Podtržítka v názvech proměnných, pro oddělení slov. Podle dohodnutých konvencí se v C# má používat camelCase (resp. PascalCase). U názvů konstant (i soukromých) je to pak PascalCase.
5. Mám pocit, že v místě, kde určuješ jednotku (uR, mR, atd), by měly být spíš "else if" bloky.
6. A možná by tam mohlo být pár komentářů.

Nahlásit jako SPAM
IP: 84.21.124.–
Dušan Janošík | web: djanosik.cz, @djanosik
marpit0
Stálý člen
8. 11. 2009   #3
-
0
-

To djanosik : Díky za komentář.
1, v tento program sloužil jen k vyzkoušení OOP v praxi, když dělám program tak to samozřejmě pojmenovávám, je to i přehlednější
3, String je jen u rdps.odpor, což je double, u jednotky, již ne

textBox4.Text = rdps.Odpor().ToString() + rdps.jednotka;

Nahlásit jako SPAM
IP: 78.102.212.–
MarPit
8. 11. 2009   #4
-
0
-

To marpit : Takže textBox1.Text.ToString() už tam není?

7. Porušuješ princip zapouzdření. Proměnná jednotka by v žádném případě neměla být public, ale měla by být zapouzdřena do vlastnosti, která by v tomto případě mohla být read-only. Navíc by ty jednotky mohly být řešeny nějakým výčtem (enum).

private string jednotka;

public string Jednotka
{
get { return this.jednotka; }
}

Nahlásit jako SPAM
IP: 84.21.124.–
Dušan Janošík | web: djanosik.cz, @djanosik
marpit0
Stálý člen
8. 11. 2009   #5
-
0
-

takže se mám příště radši proměnným public vyhýbat?
Na výčet se ještě podívám, zatím dík za rady

Nahlásit jako SPAM
IP: 78.102.212.–
MarPit
8. 11. 2009   #6
-
0
-

To marpit : Ano. Představ si, že bys tu třídu RDPS distribuoval dál jako nějakou Utilitku a někoho by napadlo do proměnné jednotka nacpat třeba "km". V tomhle případě by se nic nestalo, jen by byl výsledkem nesmysl, ale co když s tou proměnnou dál pracuješ a spoléháš na to, že tam mohou být jen tebou dané hodnoty. Mohlo by klidně dojít k chybě.

Naproti tomu, když ji zabalíš do vlastnosti, tak může být buď read-only (je tam jen blok get), pokud nechceš, aby ji kdokoliv z venku měnil a nebo můžeš v bloku set vkládanou hodnotu kontrolovat.

Nahlásit jako SPAM
IP: 84.21.124.–
Dušan Janošík | web: djanosik.cz, @djanosik
marpit0
Stálý člen
8. 11. 2009   #7
-
0
-

To djanosik : Díky za radu, tedka jsem se ještě díval na ten výčet enum, ale nedovedu si to moc v tomto případě představit.

Nahlásit jako SPAM
IP: 78.102.212.–
MarPit
Zjistit počet nových příspěvků

Přidej příspěvek

Toto téma je starší jak čtvrt roku – přidej svůj příspěvek jen tehdy, máš-li k tématu opravdu co říct!

Ano, opravdu chci reagovat → zobrazí formulář pro přidání příspěvku

×Vložení zdrojáku

×Vložení obrázku

Vložit URL obrázku Vybrat obrázek na disku
Vlož URL adresu obrázku:
Klikni a vyber obrázek z počítače:

×Vložení videa

Aktuálně jsou podporována videa ze serverů YouTube, Vimeo a Dailymotion.
×
 
Podporujeme Gravatara.
Zadej URL adresu Avatara (40 x 40 px) nebo emailovou adresu pro použití Gravatara.
Email nikam neukládáme, po získání Gravatara je zahozen.
-
Pravidla pro psaní příspěvků, používej diakritiku. ENTER pro nový odstavec, SHIFT + ENTER pro nový řádek.
Sledovat nové příspěvky (pouze pro přihlášené)
Sleduj vlákno a v případě přidání nového příspěvku o tom budeš vědět mezi prvními.
Reaguješ na příspěvek:

Uživatelé prohlížející si toto vlákno

Uživatelé on-line: 0 registrovaných, 39 hostů

Podobná vlákna

Kritika programu - výroky — založil Danstahr

Tetris - kritika — založil Matěj Andrle

Kritika webu — založil marpit

Kritika banneru — založil plasmo

Kritika webů — založil Zeleny_Muzik

 

Hostujeme u Českého hostingu       ISSN 1801-1586       ⇡ Nahoru Webtea.cz logo © 20032016 Programujte.com
Zasadilo a pěstuje Webtea.cz, šéfredaktor Lukáš Churý