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 ;-)