refactoring de plusieurs if-else-if-else .
Bonjour,
j'essaie de refactoriser le code que voici mais j'ai un peu de mal.
Code:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
|
public FigNode getFigNodeFor(
Object node, int x, int y,
Map styleAttributes) {
if (node == null) {
throw new IllegalArgumentException(
"A model element must be supplied");
}
FigNode figNode = null;
if (Model.getFacade().isAComment(node)) {
figNode = new FigComment();
} else if (Model.getFacade().isAStubState(node)) {
return new FigStubState();
} else if (Model.getFacade().isAAssociationClass(node)) {
figNode = new FigClassAssociationClass(node, x, y, 10, 10);
} else if (Model.getFacade().isAClass(node)) {
figNode = new FigClass(node, x, y, 10, 10);
} else if (Model.getFacade().isAInterface(node)) {
figNode = new FigInterface();
} else if (Model.getFacade().isAEnumeration(node)) {
figNode = new FigEnumeration();
} else if (Model.getFacade().isAStereotype(node)) {
figNode = new FigStereotypeDeclaration();
} else if (Model.getFacade().isADataType(node)) {
figNode = new FigDataType();
} else if (Model.getFacade().isAInstance(node)) {
figNode = new FigInstance();
} else if (Model.getFacade().isAModel(node)) {
figNode = new FigModel(node, x, y);
} else if (Model.getFacade().isASubsystem(node)) {
figNode = new FigSubsystem(node, x, y);
} else if (Model.getFacade().isAPackage(node)) {
figNode = new FigPackage(node, x, y);
} else if (Model.getFacade().isAAssociation(node)) {
figNode = new FigNodeAssociation();
} else if (Model.getFacade().isAActor(node)) {
figNode = new FigActor();
} else if (Model.getFacade().isAUseCase(node)) {
figNode = new FigUseCase();
} else if (Model.getFacade().isAPartition(node)) {
figNode = new FigPartition();
} else if (Model.getFacade().isACallState(node)) {
figNode = new FigCallState();
} else if (Model.getFacade().isAObjectFlowState(node)) {
figNode = new FigObjectFlowState();
} else if (Model.getFacade().isASubactivityState(node)) {
figNode = new FigSubactivityState();
} else if (Model.getFacade().isAClassifierRole(node)) {
figNode = new FigClassifierRole();
} else if (Model.getFacade().isAMessage(node)) {
figNode = new FigMessage();
} else if (Model.getFacade().isANode(node)) {
figNode = new FigMNode();
} else if (Model.getFacade().isANodeInstance(node)) {
figNode = new FigMNodeInstance();
} else if (Model.getFacade().isAComponent(node)) {
figNode = new FigComponent();
} else if (Model.getFacade().isAComponentInstance(node)) {
figNode = new FigComponentInstance();
} else if (Model.getFacade().isAObject(node)) {
figNode = new FigObject();
} else if (Model.getFacade().isAComment(node)) {
figNode = new FigComment();
} else if (Model.getFacade().isAActionState(node)) {
figNode = new FigActionState();
} else if (Model.getFacade().isAFinalState(node)) {
figNode = new FigFinalState();
} else if (Model.getFacade().isASubmachineState(node)) {
figNode = new FigSubmachineState();
} else if (Model.getFacade().isAConcurrentRegion(node)) {
figNode = new FigConcurrentRegion();
} else if (Model.getFacade().isASynchState(node)) {
figNode = new FigSynchState();
} else if (Model.getFacade().isACompositeState(node)) {
figNode = new FigCompositeState();
} else if (Model.getFacade().isAState(node)) {
figNode = new FigSimpleState();
} else if (Model.getFacade().isAPseudostate(node)) {
Object pState = node;
Object kind = Model.getFacade().getKind(pState);
if (Model.getPseudostateKind().getInitial().equals(kind)) {
figNode = new FigInitialState();
} else if (Model.getPseudostateKind().getChoice()
.equals(kind)) {
figNode = new FigBranchState();
} else if (Model.getPseudostateKind().getJunction()
.equals(kind)) {
figNode = new FigJunctionState();
} else if (Model.getPseudostateKind().getFork().equals(kind)) {
figNode = new FigForkState();
} else if (Model.getPseudostateKind().getJoin().equals(kind)) {
figNode = new FigJoinState();
} else if (Model.getPseudostateKind().getShallowHistory()
.equals(kind)) {
figNode = new FigShallowHistoryState();
} else if (Model.getPseudostateKind().getDeepHistory()
.equals(kind)) {
figNode = new FigDeepHistoryState();
}
}
if (figNode == null) {
throw new IllegalArgumentException(
"Failed to construct a FigNode for " + node);
}
setStyleAttributes(figNode, styleAttributes);
return figNode;
} |
Comme on peut le voir il y a un nombre important de if else if d'affiller que j'aimerais factoriser en utilisant plus la POO que cette horreur.
m'ais j'ai malheureusement du mal.
qqn pourrais me guider?
merci
a++
