dependency injection class or interface

dependency injection class or interface
marta1995
  • Rejestracja:ponad 13 lat
  • Ostatnio:ponad 5 lat
  • Postów:155
0

Hej,

Zastanawiam się która praktyka (i dlaczego) jest lepsza:

  1. Wewnątrz przykładowego obiektu zadeklarować pole klasy Sample z adnotacją @Inject
  2. Wewnątrz przykładowego obiektu zadeklarować pole interfejsu SampleImpl z adnotacją @Inject
    Zakładam, ze SampleImpl implementuje Sample i ma dokładnie te same metody publiczne.
    Szukałam na SO, ale szukam prostej, klarownej interpretacji.

Dziękuje Panowie :)

Shalom
  • Rejestracja:około 21 lat
  • Ostatnio:około 3 lata
  • Lokalizacja:Space: the final frontier
  • Postów:26433
4

A jak będziesz kiedyś chciała podmienić SampleImpl na SampleImpl2? W ogóle generalnie stosuje się interfejsy a nie konkretne klasy, niezależnie od tego czy masz tam DI czy też nie. Dzięki temu masz kod powiązany jedynie kontraktami związanymi z dostarczanymi metodami, a nie powiązany z konkretnymi implementacjami. Taki kod potem dużo prościej zmieniać i rozwijać.

Wczoraj na przykład dane byly zapisywane w pliku, dziś w bazie danych a jutro klient sobie zażyczy opcji że w chmurze. A dla ciebie w całym kodzie to kwestia zaimplementowania kolejny raz interfejsu DataRepository, reszta kodu pozostaje bez zmian, bo polega tylko na tym interfejsie.

Oczywiście nie ma też co się dać zwariować. Jeśli wiesz na pewno że jakaś klasa na pewno nie będzie nigdy zmieniona to nie warto wydzielać interfejsu "na zapas". Niemniej użycie DI sugeruje że jednak ta zależność moze ulegać zmianie / być podmieniana za pomocą konfiguracji.


"Nie brookliński most, ale przemienić w jasny, nowy dzień najsmutniejszą noc - to jest dopiero coś!"
edytowany 1x, ostatnio: Shalom
ŁF
Ogólnie się z Tobą zgadzam, ale w praktyce... Ile razy zmieniałeś implementację interfejsu na coś kompletnie nowego? Bo ja nigdy, a mam za sobą ładnych kilka lat pracy z DI.
Shalom
@ŁF wbrew pozorom zdarza się. Dopiero co w pracy robiliśmy dość sporą zmianę z tym związaną, bo nagle zaczęliśmy wspierać 2 formaty plików a kod był pisany zgodnie z zasadą przecież ten format się nie zmieni ;) Widziałem też np. migracje zewnętrznego serwisu z SOAP na REST i wygodnie kiedy masz to opakowane w jakis ładny interfejs i piszesz sobie tylko nową implementacje.
marta1995
  • Rejestracja:ponad 13 lat
  • Ostatnio:ponad 5 lat
  • Postów:155
0

Rozumiem, że to jest tylko kwestia pozostawienia kodu łatwiejszego do rozbudowania?
Nie ma to żadnych uzasadnień wydajnościowych, optymalizacyjnych?
Dziękuje za wyjaśnienie :)

jarekr000000
Wydajnościowo nie ma to znaczenia mierzalnego w Javie. Znaczenie jest tylko w kontekście niepotrzebnego zaśmiecania kodu (popatrz poniżej).
jarekr000000
  • Rejestracja:ponad 8 lat
  • Ostatnio:5 minut
  • Lokalizacja:U krasnoludów - pod górą
  • Postów:4707
1
Shalom napisał(a):

A jak będziesz kiedyś chciała podmienić SampleImpl na SampleImpl2? W ogóle generalnie stosuje się interfejsy a nie konkretne klasy, niezależnie od tego czy masz tam DI czy też nie. Dzięki temu masz kod powiązany jedynie kontraktami związanymi z dostarczanymi metodami, a nie powiązany z konkretnymi implementacjami. Taki kod potem dużo prościej zmieniać i rozwijać.

Oczywiście nie ma też co się dać zwariować. Jeśli wiesz na pewno że jakaś klasa na pewno nie będzie nigdy zmieniona to nie warto wydzielać interfejsu "na zapas". Niemniej użycie DI sugeruje że jednak ta zależność moze ulegać zmianie / być podmieniana za pomocą konfiguracji.

Ogólnie się z tym nie zgadzam! (Pomijając nawet moje krytyczne zdanie na temat Springa... )

YAGNI

Jak masz jedną implementację to używasz bezpośrednio i koniec. Nie ma sensu bawić się w interfejsy (zwykle).

Jak się pojawia druga implementacja to wtedy dopiero warto wprowadzić interfejs (przecież to w XXI wieku proste).

Trudno też powiedzieć, że jakaś klasa nigdy nie będzie potrzebowała drugiej implementacji itp. software na ogół żyje, reguły się zmieniają. Klienci piją. Ale moment na wprowadzenie interfejsu jest właśnie i dopiero wtedy, kiedy się taka druga, trzecia implementacja pojawia. (Albo już wiesz na 100%, że się zaraz pojawi ).


jeden i pół terabajta powinno wystarczyć każdemu
Zobacz pozostały 1 komentarz
jarekr000000
@john_klamka - panie Trollu - pan nie przeszkadza, w dyskusji, jak pan nie ma pojęcia
john_klamka
@jarekr000000 panie 'Lubię generalizować do przesady' pan prowadzi jakoś tę dyskusję, jak pan ma pojęcie.
jarekr000000
@john_klamka to proszę bardzo - masz tekst powyżej i proszę się odnieść, z którą cześcia się nie zgadzasz. Możesz napisać dlaczego. Albo potrolluj gdzie indziej.
john_klamka
nah, nie chce mi się przepisywać z google, idę trolować gdzie indziej
jarekr000000
@john_klamka - tu z google - akurat gościa znam - bo się z nim kłócę na temat springa (to jeden z tych gości, który uważa, że spring się jeszcze do czegoś przydaje): https://dzone.com/articles/are-you-guilty-of-over-engineering

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.