-
Notifications
You must be signed in to change notification settings - Fork 19
HangszerUzlet - Kurai István AMIA3F #7
base: main
Are you sure you want to change the base?
Conversation
HangszerDBDataContext context = new HangszerDBDataContext(); | ||
HangszerTipusDAO hangszerTipus = new HangszerTipusDAO(); | ||
|
||
public readonly string readonlyNev = "Szintetizáror"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Readonly helyett célszerűbb konstansokra a const módosítót alkalmazni, főleg ha compile time tudott, hogy mi lesz az érték.
|
||
public async Task InsertHangszer(TextBox nev, ComboBox tipus, TextBox ar, DataGridView dataGridView) | ||
{ | ||
string netto = ar.Text; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ha az ár szövege nem szám, akkor a ToInt32() miatt ez szétfog szállni a fenébe.
public async Task InsertHangszer(TextBox nev, ComboBox tipus, TextBox ar, DataGridView dataGridView) | ||
{ | ||
string netto = ar.Text; | ||
double brutto = Convert.ToInt32(netto) + (Convert.ToInt32(netto) * 0.27); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Elég lenne egyszer is konvertálni, illetve már kapásból double-be.
|
||
namespace HangszerUzlet | ||
{ | ||
public class HangszerDbDAO |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mivel ez egy Data acces réteg akar lenni, ezért szerintem semmiképpen nem kéne tudnia a DAO rétegnek Windows forms típusokról, mert így lényegében UI függővé tetted az egészet. Tehát TextBox és Combobox helyett az értékeiket meg tartalmukat kellene beküldeni ennek a rétegnek, vagy legalább egy interféasz szegregációval leválasztani a UI-ról az egészet.
|
||
namespace HangszerUzlet.Controler | ||
{ | ||
class HangszerTipusDAO |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Szintén DAO akar lenni, de a UI-al egybe van kötve a típusok miatt.
} | ||
|
||
private void hangszerBindingSource_CurrentChanged(object sender, EventArgs e) | ||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Üresen hagyott metódusokat érdemes egy sorral megjelölni kommentben, hogy miért üres, mert így halott kódnak tűnik ez a metódus.
} | ||
|
||
private void comboBox1_SelectedIndexChanged(object sender, EventArgs e) | ||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Üresen hagyott metódusokat érdemes egy sorral megjelölni kommentben, hogy miért üres, mert így halott kódnak tűnik ez a metódus.
|
||
private void button1_Click(object sender, EventArgs e) | ||
{ | ||
//hangszerDbDAO.AddNewRowOffline(dataGridViewAkcios); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Üresen hagyott metódusokat érdemes egy sorral megjelölni kommentben, hogy miért üres, mert így halott kódnak tűnik ez a metódus.
|
||
namespace HangszerUzlet.Model | ||
{ | ||
public class HangszerModel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ez pl lehetett volna immutable...
|
||
namespace HangszerUzlet.Model | ||
{ | ||
class HangszerTipusModel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Szintén lehetett volna immutable
No description provided.