c# SOLID

JA
  • Rejestracja:ponad 5 lat
  • Ostatnio:ponad 4 lata
  • Postów:18
0

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)

Miang
  • Rejestracja:ponad 7 lat
  • Ostatnio:4 minuty
  • Postów:1756
3

co robi ta funkcja i dlaczego się tak nazywa?

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

dzisiaj programiści uwielbiają przepisywać kod z jednego języka do drugiego, tylko po to by z projektem nadal stać w miejscu ale na nowej technologii
Zobacz pozostały 1 komentarz
JA
Ustawia numer grupy dla studenta dla tego nazywa się ustaw numer grupy (przynajmniej taki był zamysł). Przyjmuje ona obiekt typu Groupe i przypisuje do lokalnej wartości Group obiekt typu group, jak mniemam po referencji, ale to chyba dobrze w tym przypadku. Choć teraz jak o tym myślę Grade źle działa yyy w ogóle nie będzie śmigał bo no jest fatalnie zaimplementowany wywali wyjątek - do poprawy (już się za to biorę).
JA
No jak by co poprawiłem ale grade'a i teraz już chyba jest okay wrzuciłęm poprawioną wersje na gita hmnn ale z tą grupą ^^ nadal nie wiem co nie tak panowie :D jakiś przytępy jestem w tej materii.
Miang
Funkcja się nazywa UstawNumer a przypisujesz nie numer ale cały obiekt. Nazwa klasy Groupe ma być(z błędem) po niemiecku czy co?
JA
Okay rozumiem, poprawiłem na SetGroupe, dziękuję że zwróciłeś uwagę masz racje :)
JA
  • Rejestracja:ponad 5 lat
  • Ostatnio:ponad 4 lata
  • Postów:18
0

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!

Zobacz pozostałe 4 komentarze
somekind
Wprowadzenie jakiejś klasy czy interfejsu bazowego dla Student i Teacher nie będzie miało żadnego związku z SRP.
Miang
niewprowadzanie za to będzie miało coś wspólnego z YAGNI ;)
JA
Bardzo interesujące jest, to co tu piszecie. Zastanawiam się tylko jaki z tego wnisoek/morał, w takim razie. Bo o ile dobrze rozumie, to kwestia dyskusyjna czy należy to robić czy nie, a jeżeli chce aby kod był zbudowany zgodnie z zasadami SOLID wcale nie muszę rozbijać tego na jeszcze jedną klasę, choć jak to zrobię będzie to decyzja w pewnym stopniu zasadna? (rozbicie teacher i student z którego powstanie klasa/ interfejs posiadający właściwości Imię, Nazwisko). Inne pytanie co w takim razie z pomysłem na osobną klasę która odpowiada za tworzenie obiektów?
somekind
Przede wszystkim ten kod niczego nie robi, nie zawiera żadnej logiki, to tylko obiekty przechowujące dane. A zasady SOLID dotyczą pisania kodu, który coś robi.
JA
Rozumiem, dzięki :)
KA
  • Rejestracja:ponad 7 lat
  • Ostatnio:ponad 5 lat
  • Postów:30
2

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

JA
  • Rejestracja:ponad 5 lat
  • Ostatnio:ponad 4 lata
  • Postów:18
0
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ć.

Kopiuj
        private string _firstName;
        public string FirstName { get
        {
                return _firstName;
        }
         set
        {
               firstName=value;
        }
        }
        public Student(string firstName)
        {    
            FirstName = firstName;
        }
edytowany 2x, ostatnio: somekind
mad_penguin
mad_penguin
  • Rejestracja:prawie 11 lat
  • Ostatnio:prawie 4 lata
  • Lokalizacja:Rzeszów
2

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

Miang
wykładowca najwyraźniej uczy ludzi zaczynając od podstaw bo to nie bootkamp
JA
Miang może mieć racje :) bo często, dochodzi do etapu gdy mówi, a teraz wam pokażę na co macie się przestawić np. tak było przy wprowadzaniu LINQu, wcześniejsze przykłady oparte na działaniu na tablicach zamienione na operacje na kolekcjach przy użyciu gotowych zapytań. Ale dzięki @mad_penguin za link na pewno przejrzę podejrzewam że bardzo wartościowe =)
somekind
@Miang: Podstawy są takie, że używa się automatic properties, a to, że dawno temu ich nie było, to ciekawostka historyczna, o której można co najwyżej napomknąć.
Miang
@Jarver więc szacun dla gościa, dzięki takiemu tłumaczeniu poradzisz sobie jak trafisz w pracy na starszy kod
somekind
Moderator
  • Rejestracja:ponad 17 lat
  • Ostatnio:dzień
  • Lokalizacja:Wrocław
1

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.

JA
Rozumiem, nie ma co przypisywać przez metodę właściwości jakiejś wartości która przypisze ją do konkretnego już pola, jeżeli może już to zrobić metoda w kodzie bezpośrednio (nazwana z sensem, wskazująca że przypisujemy/ odczytujemy wartość do/z tego miejsca gdzie chcemy. Super dzięki @somekind za radę! :)

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.