Architektura to nie bzdura

Architektura to nie bzdura

DDD-WRO
Paweł Szulc – paul.szulc@gmail.com


DDD-WRO
https://twitter.com/paulszulc


DDD-WRO
https://twitter.com/paulszulc
http://rabbitonweb.com

Cel prezentacji

Nie jest celem znalezienie złotego środka – bo
taki nie istnieje
●

Cel prezentacji

Nie jest celem znalezienie złotego środka – bo
taki nie istnieje
● Nakłonić Was do dyskusji
●

Stack technologiczny

- Wicket
- Spring
- Hibernate/JPA

Przykłady

●

Aplikacja zarządzająca POI

Przykłady

● POI – ang. Point of Interest
●

Przykłady
Grupa

● Grupa
●

Przykłady
Grupa

● Grupa
● Podgrupa
●

Podgrupa

Przykłady
● Grupa
● Podgrupa
● POI: nazwa, położenie (x,y)
●

Grupa

Podgrupa

POI

Przykłady
● Grupa
● Podgrupa
● POI: nazwa, położenie (x,y)
● Użytkownik z prawami dostępu
●

Grupa

Podgrupa

POI

Mainstreamowa architektura

- nasz model to encje hibernetowe


- logika biznesowa zamknięta w warstwie
serwisowej


serwisowej
- kod zapytań do bazy danych przez obiekty typu
DAO


serwisowej
- kod zapytań do bazy danych przez obiekty typu
DAO
- encje wykorzystywane w warstwie widoku


Jak ją nazwać?


Jak ją nazwać?
"Model View Controler" ?


"Architektura trójwarstwowa" ?


"Entity-DAO-Service-View" ?


"Encje na twarz i pchasz"

Czemu nie używać, skoro działa?

●

Encja User.java

●

UserService.java

●

Encja User.java

●

UserService.java (List<User> findAll())

●

Encja User.java

●


●

UserServiceImpl.java

●

Encja User.java

●


●


public List<User> findAll() { userDao.findAll(); }

●

Encja User.java

●


●


●

UserDao.java

●

Encja User.java

●


●


●

UserDao.java

●

UserDaoImpl.java

Encja -> DAO -> Serwis
●

Single responsibility principle

●


●

8 tysięczniki

●


●

8 tysięczniki

●

Eksplozja klas

●


●

8 tysięczniki

●

Eksplozja klas

●

DAO – data access object?

●


●

8 tysięczniki

●

Eksplozja klas

user.getContest().getOwner()
●

public interface PoiDAO extends DAO<PoiEntity, Long> {
List<PoiEntity> findFiltered(PoiListingFilterWrapper
wrapper);
List<PoiEntity> findAllInSubgroup(SubgroupEntity subgroup);
List<PoiEntity> findTextFiltered(SubgroupEntity subgroup,
String textFilter);
List<PoiEntity> findTextFilteredOnStreet(StreetEntity street,
String textFilter);
List<PoiEntity> filterPoisByTextPhrase(String searchPhrase,
int start, int count);
List<PoiEntity> filterPoisWithTypeByTextPhrase(String filter,
int start, int maxAllowedResults);
....
}

●


●

8 tysięczniki

●

Eksplozja klas

●


●

Metody biznesowe?

public interface PoiDAO extends DAO<PoiEntity, Long> {
List<PoiEntity> findFiltered(PoiListingFilterWrapper
wrapper);
List<PoiEntity> findAllInSubgroup(SubgroupEntity subgroup);
List<PoiEntity> findTextFiltered(SubgroupEntity subgroup,
String textFilter);
List<PoiEntity> findTextFilteredOnStreet(StreetEntity street,
String textFilter);
List<PoiEntity> filterPoisByTextPhrase(String searchPhrase,
int start, int count);
List<PoiEntity> filterPoisWithTypeByTextPhrase(String filter,
int start, int maxAllowedResults);
void save(PoiEntity poi);
void update(PoiEntity poi);
}

Jako użytkownik z prawem
'aktualizator_lokalizacji' mogę zmieniać tylko i
wyłącznie wartość położenia danego POI.

PoiEntity poi =
poiService.find(poiId);

PoiEntity poi =
createForm(poi);

PoiEntity poi =
createForm(poi);
...
new Button("save") {

public void onSubmit() {
poiService.update(poi);
}
}

PoiEntity poi =
createForm(poi);
...

mailSender.send(body())

}
}

PoiEntity poi =
createForm(poi);
...

mailSender.send(body())

}
}

// x i y nie mogą być ujemne

Encja na widoku, i co z tego?
●

Ważne pytanie dla encji...

●

Ważne pytanie dla encji – czy już zna Cię
PersistenceContext?

●

●

PersistenceContext?
Algorytm tworzenia widoku jak i sama jego
struktura oparta o encje

●

●

PersistenceContext?


Widok mocno związany ze strukturą bazy

●

●

PersistenceContext?





Skostniałość - dashboard

Dashboard
A

A
A
A

B

C

D

A

E

Dashboard
A

A
A
A

B

C

A

D

A1

A2

B1

E

B2

B3

C1

C2

C3

C4

E1

●

●

PersistenceContext?





Skostniałość – dashboard

●

●

PersistenceContext?



●


LazyInitializationException

PoiEntity poi = poiService.find(poiId);
poi.getConnectedPois();

STOP – This is Hibernate Police!

STOP – This is Hibernate Police!
A obywatel o Open Session In View
nie słyszał?

Open Session In View
Transaction.begin

Transaction.commit

REQUEST
Transaction.begin
Transaction.commit
Transaction.begin
Transaction.commit
Transaction.begin
Transaction.commit
RESPONSE


select * from t_pois where id = :poiId;


select * from t_pois where id = :poiId;
select * from t_pois where ....

List<PoiEntity> pois =
poiService.findForUser(user);
for(PoiEntity p : pois) {
}

}
select * from t_pois where user_id = :uId;

}
select * from t_pois where ...

}

●

●

PersistenceContext?






●


●

Testowalność

●

●

PersistenceContext?






●


●

Testowalność


"Testy już przechodzą, muszę jeszcze tylko przeklikać"

Zamknięcie logiki biznesowej

Warstwa prezentacji
Warstwa serwisowa
Warstwa bazodanowa


Warstwa prezentacji
Warstwa aplikacji
Warstwa serwisowa
Warstwa bazodanowa


Warstwa prezentacji
Warstwa aplikacji
Warstwa serwisowa
Warstwa bazodanowa

public interface UCListingPoisForGivenSubgroup {
List<PoiDto> list(Long subgroupId)
}


Warstwa prezentacji
Warstwa aplikacji
Warstwa serwisowa
Warstwa bazodanowa

public interface UserFromClientProvider { UserDto provide(); }


Warstwa prezentacji
Warstwa aplikacji
Warstwa serwisowa
Warstwa bazodanowa

public class UserProvider implements UserFromClientProvider {
public UserDto provide() { return userSession.getUser(); }
}
public interface UserFromClientProvider { UserDto provide(); }

Warstwa aplikacji – organizacja

●

Jest abstrakcją aplikacji

●


●

Aplikacja to zbiór funkcjonalności

●


●


●

Funkcjonalności zdefiniowane (w ten czy inny
sposób)

●


●


●

●

sposób)
Z natury bardzo 'proceduralne'

●


●


●

●
●

sposób)
Z natury bardzo 'proceduralne'
Przełożenie tych definicji na klasy, które
definiują dany konkretny wycinek
funkcjonalności

●

Jako użytkownik z prawem 'szczegóły poi' mogę wylistować
wszystkie POI w danej podgrupie


widzę nazwę, położenie oraz listę innych POI, z którym dane POI
jest powiązane

●



jest powiązane

public interface UCListingPoiDetails {
List<PoiDto> list(Long subgroupId);
boolean canList(Long subgroupId);
}

●



jest powiązane

}
public class PoiDto {
private final String name;
private final Coordinates coordinates;
private final List<PoiDto> connectedPois;
}

●



jest powiązane

}
@Data
public class PoiDto {
private final String name;
private final Coordinates coordinates;
private final List<PoiDto> connectedPois;
}

●



jest powiązane

ActionPossible canList(Long subgroupId);
}
public interface ActionPossible {
boolean isPossible();
String explainImpossible();
}

●



jest powiązane

public class SubGroupPoisListPanel {
@Autowired private UCListingPoiDetails uc;
...
public SubGroupPoisListPanel(Long subgroupId) {
ActionPossible canList = uc.canList();
if(canList.isPossible() {
add(createList(uc.list(subgroupId));
} else {
String reason = canList.explainImpossible();
add(new Label("info", reason);
}
}

●

Jako użytkownik z prawem 'aktualizator_lokalizacji' mogę
zmieniać tylko i wyłącznie wartość położenia danego POI

public interface UCUpdatingPoiCoordinates {
void update(Long poiId, Coordinates c);
ActionPossible canUpdate();
ActionPossible canUpdateWithArgs(
Long poiId,
Coordinates c);
}

●


@Autowired private UCUpdatingPoiCoordinates uc;
Button update = new Button("update") {
ActionPossible withArgs =
uc.canUpdateWithArgs(poiId, coords);
if(withArgs.isPossible())
uc.update(poiId, coords)
else
userSession.warn(withArgs.explainImpossible());
}
}
update.setVisible(uc.canUpdate().isPossible());

●


@Autowiredprivate UCUpdatingPoiCoordinates uc;
ActionPossible withArgs =
uc.canUpdateWithArgs(poiId, coords);
if(withArgs.isPossible())
uc.update(poiId, coords)
else
userSession.warn(withArgs.explainImpossible());
}
}

●


...
}
}

●


...
}
}
ActionPossible canUpdate = uc.canUpdate();
update.setEnabled(canUpdate.isPossible());
if(!canUpdate.isPossible()) {
String reason = canUpdate.explainImpossible();
update.add(new TooltTip(reason));
}

Problem jednego stack'a

ID

Nazwa

Data utworzenia

Stan

1

Konkurs 1

05.07.2013

Nowy

2

Konkurs 2

01.06.2013

Zamknięty

3

Konkurs 3

15.06.2013

Uruchomiony

@Entity public class Contest {
@Id private Long id;
private String name;
private LocalDate creationDate;
@Enumerated private State state;

ID

Nazwa

Data utworzenia

Stan

1

Konkurs 1

05.07.2013

Nowy

2

Konkurs 2

01.06.2013

Zamknięty

3

Konkurs 3

15.06.2013

Uruchomiony

private String name;
private LocalDate creationDate;
@Enumerated private State state;
public Contest(String name) {
this.name = name;
this.creationDate = new LocalDate();
this.state = State.NOWY;
}

ID

Nazwa

Data utworzenia

Stan

1

Konkurs 1

05.07.2013

Nowy

2

Konkurs 2

01.06.2013

Zamknięty

3

Konkurs 3

15.06.2013

Uruchomiony

...
public void start() { .. }
public void close() { .. }
public boolean isAnnual() { .. }
}

ID

Nazwa

Data utworzenia

Stan

1

Konkurs 1

05.07.2013

Nowy

2

Konkurs 2

01.06.2013

Zamknięty

3

Konkurs 3

15.06.2013

Uruchomiony

...
}
okej..
Chce taki panel

ID

Nazwa

Data utworzenia

Stan

1

Konkurs 1

05.07.2013

Nowy

2

Konkurs 2

01.06.2013

Zamknięty

3

Konkurs 3

15.06.2013

Uruchomiony

...
public String getName() {..}
public LocalDate getCreationDate() {..}
public State getState() {..}
}
ID

Nazwa

Data utworzenia

Stan

1

Konkurs 1

05.07.2013

Nowy

2

Konkurs 2

01.06.2013

Zamknięty

3

Konkurs 3

15.06.2013

Uruchomiony

Rozwiązania
●

CQRS

●

CQRS dla ubogich :)

Rozwiązania
●

CQRS

●

CQRS dla ubogich :)
●

Rozróżnienie na Command i Query

Rozwiązania
●

CQRS

●

CQRS dla ubogich :)
●
●

●

Obiekty modelu (User, Poi, Subgroup) i serwisy
(UserCreator) używane do obsługi poleceń
Specjalne serwisy typu Query używane do obsługi
zapytań

Rozwiązania
●

CQRS

●

CQRS dla ubogich :)
●
●

●

●

Obiekty modelu (User, Poi, Subgroup) i serwisy
(UserCreator) używane do obsługi poleceń
Specjalne serwisy typu Query używane do obsługi
zapytań
Query niczym Wyrocznia

ActionPossible canList();
}

@Service
public class DefaultUCListingPoiDetails
implements UCListingPoiDetails {
@Autowired
private ClientFromUserProvider userProvider;
public ActionPossible canList() {
UserDto user = userProvider.provide();
if(user != null) {
return user != null && user.hasRight(AR.POI_DETAILS);
}
return impossible("User not logged in");
}
public List<PoiDto> list(Long subgroupId) {..}

@Service
@Autowired
private UserAuthorization userAuthorization;
public ActionPossible canList() {
return userAuthorization.hasRight(AR.POI_DETAILS);
}
}

@Service
@Autowired
public ActionPossible canList() { .. }
}

@Service
@Autowired

public List<PoiDto> list(Long subgroupId) {
}
}

@Service
@Autowired

checkThat(canList());
}
}

@Service
@Autowired
@Autowired
private ListingPoiDetailsQuery query;
checkThat(canList());
return query.execute(subgroupId);
}
}

public interface UCUpdatingPoiCoordinates {
void update(Long poiId, Coordinates c);
ActionPossible canUpdate();
ActionPossible canUpdateWithArgs(
Long poiId,
Coordinates c);
}

@Service
public class DefaultUCUpdatingPoiCoordinates
implements UCUpdatingPoiCoordinates {
public ActionPossible canUpdate() {..}
ActionPossible canUpdateWithArgs(Long poiId,
Coordinates c) {..}
public void update(Long poiId, Coordinates c) {..}
}

@Service

Coordinates c) {..}
public void update(Long poiId, Coordinates c) {
}
}

@Service

Coordinates c) {..}
checkThat(canUpdate());
}
}

@Service

Coordinates c) {..}
checkArguments(canUpdateWithArgs(poiId,c));
}
}

@Service
@Autowired PoiFinder poiFinder;
Coordinates c) {..}
poiFinder.find(poiId).updateCoordinates(c);
}
}

@Service
@Autowired PoiFinder poiFinder;
@Autowired MailSender mailSender;
Coordinates c) {..}
poiFinder.find(poiId).updateCoordinates(c);
mailSender.send(createContent());
}

Warstwa Application
●

przejrzyste API zorientowane na przypadki
użycia

Warstwa Application
●

●

użycia
testowalność

Warstwa Application
●

użycia

●

testowalność

●

"głupi" widok, wiele warstw prezentacji

Warstwa Application
●

użycia

●

testowalność

●


●

wejście w projekt

Warstwa Application
●

użycia

●

testowalność

●


●

wejście w projekt

●

polyglot programmer

Warstwa Application
●

użycia

●

testowalność

●


●

wejście w projekt

●

polyglot programmer

●

chodliwe funkcjonalności niemalże za darmo

Warstwa Application
●

użycia

●

testowalność

●


●

wejście w projekt

●

polyglot programmer

●

●

Excel export

Warstwa Application
●

użycia

●

testowalność

●


●

wejście w projekt

●

polyglot programmer

●

●

Excel export

●

Aktywność użytkownika

@Retention(RetentionPolicy.RUNTIME)
@Target(ElementType.METHOD)
public @interface Audit {
public String value() default "";
}

@Aspect
@Configurable
public class AuditAspect {
@Autowired
private AuditLogger auditLogger;
@Around("@annotation(audit)")
public Object around(ProceedingJoinPoint joinPoint,
Audit audit) throws Throwable {
DateTime start = DateTime.now();
Object proceed = joinPoint.proceed();
long executionTime = DateTime.now().getMillis() start.getMillis();
auditLogger.log(getActionName(joinPoint),
audit.value(),getParameters(joinPoint),
start, executionTime);
}

Obiekty Modelu
●

Encja jest obiektem reprezentującym model

Obiekty Modelu
●


●

Gettery/settery defacto wymagane

Obiekty Modelu
●


●


●

Metody typu add, remove

Obiekty Modelu
●


●


●


●

Metody "biznesowe" (run(), isRunning(),
close())

Obiekty Modelu
●


●


●


●

●

close())
Troche bardziej skomplikowana encja może
mieć nawet i kilka tysięcy linii kodu

Obiekty Modelu
●


●


●


●

●

●

close())
Troche bardziej skomplikowana encja może
mieć nawet i kilka tysięcy linii kodu
Encja nie jest ani strukturą danych, ani
obiektem (metoda canRun() obok getState())

Pomysł...
●

Niech encja reprezentuje tylko i wyłącznie
schemat relacyjny w świecie obiektowym

Pomysł...
●

●

Encje jedynie jako reprezentacje tabel

Pomysł...
●


●


●

Encja anemiczna

Pomysł...
@Entity
public class UserEntity {
}

Pomysł...
@Entity
private String login;
@OneToMany private Set<RoleEntity> roles;
}

Pomysł...
@Entity
Long getId() {..}, void setId(Long id) {..}
Long getLogin() {..}, void setLogin(String lo) {..}
Set<RoleEntity> getRoles() {..},
void setRoles(Set<RoleEntity> roles) {..}
}

Pomysł...
@Entity
Long getId() {..}, void setId(Long id) {..}
Long getLogin() {..}, void setLogin(String lo) {..}
Set<RoleEntity> getRoles() {..},
void setRoles(Set<RoleEntity> roles) {..}
void addRole(RoleEntity role) {..}
void removeRole(RoleEntity role) {..}
}

Pomysł...
@Entity
@Data
void addRole(RoleEntity role) {..}
void removeRole(RoleEntity role) {..}
}

Pomysł - obiekt biznesowy
●


●


●

Encja anemiczna

●


●


●

Encja anemiczna

●

Obiekt biznesowy – reprezentant modelu

●


●


●

Encja anemiczna

●



na początku można o nim myśleć jako wrapper na
encję

●


●


●

Encja anemiczna

●





encję
udostępnia zestaw metod biznesowych
pozwalających operować na konkretnym modelu,
który reprezentuje (i tylko na nim)

●


●


●

Encja anemiczna

●







encję
udostępnia zestaw metod biznesowych
pozwalających operować na konkretnym modelu,
który reprezentuje (i tylko na nim)
nie istnieje "niezapisany"

@Configurable
public abstract class BusinessObject<T extends
Identifiable<K>, K> implements Serializable {
protected T entity;
private Class<T> clazz;
@PersistenceContext private EntityManager entityManager;
public BusinessObject(T entity, Class<T> clazz) {
this.entity = checkNotNull(entity);
this.clazz = clazz;
}
public T attached() {
return entityManager().find(clazz, entity.getId());
}
public K getId() { return attached().getId(); }
public boolean equals(Object o) { .. }
public int hashCode() {..}

@Configurable
public class User
extends BusinessObject<UserEntity, Long> {
public User(UserEntity entity) {
super(entity, UserEntity.class);
}
}

@Configurable
public class User
extends BusinessObject<UserEntity, Long> {
public User(UserEntity entity) {
super(entity, UserEntity.class);
}
public void addRole(Role role) {
attached().addRole(role.attached());
attached().setLastModificationDate(new DateTime());
}
}

@Service
public class UserCreator {
@PersistenceContext
private EntityManager entityManager;
@Transactional
public User create(final String login) {
UserEntity entity = new UserEntity();
entity.setLogin(login);
entity.setCreationDate(new DateTime());
entityManager.persist(entity);
return new User(entity);
}
}

@Configurable
public class Group
extends BusinessObject<GroupEntity, Long> {
public Group(GroupEntity entity) {.. }
public Subgroup addSubgroup(String name) {
SubgroupEntity se = new SubgroupEntity();
se.setName(name);
attached().addSubgroup(se);
entityManager.persist(se);
return new Subgroup(se);
}
}

@Service
public class UserFinder {
@PersistenceContext
private EntityManager entityManager;
@Transactional(readOnly = true)
public User find(Long id) {
UserEntity entity =
entityManager.find(UserEntity.class, id);
return entity == null ? null : new User(entity);
}
}

Obiekty biznesowe
●

Tylko metody związane z operacjami na modelu

●

Podobiekty (realizujące zakres funkcjonalności)

finish

launch
new

running

closed

public class Contest
extends BusinessObject<ContestEntity, Long> {
public boolean isLaunchable() {..}
public void launch() {..}
public boolean isFinishable() {..}
public void finish() {..}
...
}

public class Contest
public ContestLifecycle lifecycle() {
return new ContestLifecycle(attached());
}
}
public class ContestLifecycle
public boolean isLaunchable() {..}
public void launch() {..}
public boolean isFinishable() {..}
public void finish() {..}
}

public class NewContest
public RunningContest launch() {..}
}
public class RunningContest
public ClosedContest finish() {..}
}
public class ClosedContest
}

Obiekty biznesowe
●

Tylko metody związane z operacjami na modelu

●

Podobiekty (realizujące zakres funkcjonalności)

●

Testowalność

Architektura to nie bzdura

Recommandé

Recommandé

Contenu connexe

Similaire à Architektura to nie bzdura

Similaire à Architektura to nie bzdura (20)

Plus de Pawel Szulc

Plus de Pawel Szulc (20)

Architektura to nie bzdura