Witam!
Jako, że uczę się programowania od kilku miesięcy, chciałbym prosić bardziej doświadczone osoby o uwagi na temat tego co jest niedopuszczalne w tym projekcie oraz na co powinienem zwrócić uwagę.
A projekt to mini programik, który wyświetla zegarek binarny.
1.Nie ma sensu wydzielać katalogów względem struktura, klasa czy na co można tam jeszcze wpaść - nazwy powinny odzwierciedlać zadanie, a nie implementację.
public int Ones
{
get
{
return _ones;
}
set
{
_ones = value;
}
}
Oezu, ale masturbacja. Nie ma potrzeby robić takich kwiatków pseudoakcesorowych w C# - po prostu zrób sobie public int Ones; i w metodach także odnoś się do tego pola. To samo w przypadku reszty klas.
3.Brak komentarzy - do czego służy klasa Number? Co odzwierciedla Ones oraz Tens?
4.Wszystkie katalogi są nazwane z wykorzystaniem liczb mnogich, poza katalogiem img. Minus dla jednolitości.
5.Kiedyś nazywanie prywatnych pól zaczynając od _ było modne, teraz już nie wykorzystuje się tego na rzecz odnoszenia się przez this..
Generalnie nie ma najgorzej jak na pierwszy projekt - ale to i owo masz do poprawy ;-)
Oezu, ale masturbacja. Nie ma potrzeby robić takich kwiatków pseudoakcesorowych w C# - po prostu zrób sobie public int Ones; i w metodach także odnoś się do tego pola. To samo w przypadku reszty klas.
Po prostu można zrobić skróconą formę dla właściwości: public int Ones { get; set; }. Jeśli kiedyś zajdzie potrzeba, to się zmodyfikuje do bardziej skomplikowanej postaci setter albo getter. Gołe publiczne pola to raczej nie.
Kiedyś nazywanie prywatnych pól zaczynając od _ było modne, teraz już nie wykorzystuje się tego na rzecz odnoszenia się przez this..
Zależy, AFAIR w standardowej konfiguracji VS2015 się "czepia" odwołania przez this i proponuje jego usunięcie.