c# SOLID

Hej mam ogromną prośbę, jeżeli ktoś kto miałby czas i chęci ocenić moje wypociny byłbym bardzo wdzięczny. Dopiero się uczę i że tak powiem dostałem od wykładowcy na uczelni zadanko polegające na stworzeniu struktury klas: uczelnia -> profesor/student/grupa/przedmiot/ocena iii chyba tyle, ma być to zgodne z standardem SOLID. No i tyle w temacie trochę o nim poczytałem i postarałem się coś złożyć do kupy prosiłbym o krytykę, informacje co mógłbym poprawić itd. itp. :) Z góry dziękuję i przepraszam za mój brak doświadczenia, wiedzy i głupotę.

Tutaj link do repozytorium https://github.com/Jarverr/University (jak by co gita też używałem tylko hmn dla siebie, więc jakby coś było niedostępne, albo niewidoczne cokolwiek, dajcie znać, bo nie miąłem nigdy styczności w ramach współpracy tak żęby korzystać z gita.
Pozdrawiam Michał
(w załączniku dorzucam moje zrozumienia zasad SOLID jak by co, tak żeby hmnn nie wiem... może komuś się przyda)

co robi ta funkcja i dlaczego się tak nazywa?

        public void SetNumberOfGroup(Groupe group)
        {
            Group = group;
        }

Okay =) dopisuje jakby ktoś był kiedyś zainteresowany na przyszłość, to co obecnie jest na gitcie jest w porządku (tak mi się wydaje), wykładowca też powiedział że spoko, z drobnym wyjątkami - interfejs do usunięcia bo jest bez sensu i deklarowanie obiektów klas do przepisania tak żeby była osobna klasa która by je deklarowała i przypisywała żeby nie trzeba było to robić w programie (it makes seanse, takiej bardziej flexible). Na razie tego nie poprawię, może soon. No a to takie info jakby ktoś był zainteresowany pozdrowionka!

Mnie tylko zastanawia po co robic getery i setery dla properties? Po to masz {get; set;} by tego nie robić. Zalatuje mi Java

Kadzio napisał(a):

Mnie tylko zastanawia po co robic getery i setery dla properties? Po to masz {get; set;} by tego nie robić. Zalatuje mi Java

Hmmn no nie do końca rozumiem pytanie :D. Ogólnie to robię te propertiesy z myśla o zamysle taki co w sumie ostatecznie (jeszcze) nie wdrożyłem w projekcie na gitcie - że w konstruktorze przypisuje do właściwości wartość a włąściwość przypisuje ją przez setera do pola prywatnego coś jak ten tutaj przykład poniżej. A czemu tak? :D No wykładowca na wykładzie mówił że to hmn standard? Że tak się powinno to pisać.

        private string _firstName;
        public string FirstName { get
        {
                return _firstName;
        }
         set
        {
               firstName=value;
        }
        }
        public Student(string firstName)
        {    
            FirstName = firstName;
        }

Wykładowca najwyraźniej przespał ostatnie 12 lat rozwoju języka.
https://www.google.com/amp/s/gunnarpeipman.com/csharp-automatic-properties/amp/

Można mieć metody ustawiające oraz czytające wartości pól, tylko wtedy wypadałoby je nazywać mądrzej niż tylko używając prefiksów Get oraz Set. Można mieć też właściwości służące temu samemu, jeśli obiekt to proste DTO.
Ale jeśli mamy obie te rzeczy, to zalatuje jakimś rozdwojeniem jaźni prowadzącym do tworzenia kodu spaghetti.