Code Review
/
nonrtric.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Remove code smells in PolicyController
[nonrtric.git]
/
policy-agent
/
src
/
main
/
java
/
org
/
oransc
/
policyagent
/
repository
/
Ric.java
diff --git
a/policy-agent/src/main/java/org/oransc/policyagent/repository/Ric.java
b/policy-agent/src/main/java/org/oransc/policyagent/repository/Ric.java
index
4169150
..
e50a98c
100644
(file)
--- a/
policy-agent/src/main/java/org/oransc/policyagent/repository/Ric.java
+++ b/
policy-agent/src/main/java/org/oransc/policyagent/repository/Ric.java
@@
-25,6
+25,9
@@
import java.util.HashMap;
import java.util.Map;
import java.util.Vector;
import java.util.Map;
import java.util.Vector;
+import lombok.Getter;
+import lombok.Setter;
+
import org.oransc.policyagent.clients.A1Client.A1ProtocolType;
import org.oransc.policyagent.configuration.RicConfig;
import org.oransc.policyagent.clients.A1Client.A1ProtocolType;
import org.oransc.policyagent.configuration.RicConfig;
@@
-33,8
+36,11
@@
import org.oransc.policyagent.configuration.RicConfig;
*/
public class Ric {
private final RicConfig ricConfig;
*/
public class Ric {
private final RicConfig ricConfig;
+
private RicState state = RicState.UNDEFINED;
private Map<String, PolicyType> supportedPolicyTypes = new HashMap<>();
private RicState state = RicState.UNDEFINED;
private Map<String, PolicyType> supportedPolicyTypes = new HashMap<>();
+ @Getter
+ @Setter
private A1ProtocolType protocolVersion = A1ProtocolType.UNKNOWN;
/**
private A1ProtocolType protocolVersion = A1ProtocolType.UNKNOWN;
/**
@@
-50,16
+56,16
@@
public class Ric {
return ricConfig.name();
}
return ricConfig.name();
}
- public Ric
State state
() {
- return
state
;
+ public Ric
Config getConfig
() {
+ return
this.ricConfig
;
}
}
- public
void setState(RicState newState
) {
-
state = newS
tate;
+ public
synchronized RicState getState(
) {
+
return this.s
tate;
}
}
- public
RicConfig getConfig(
) {
-
return this.ricConfig
;
+ public
synchronized void setState(RicState state
) {
+
this.state = state
;
}
/**
}
/**
@@
-67,8
+73,8
@@
public class Ric {
*
* @return a vector containing the nodes managed by this Ric.
*/
*
* @return a vector containing the nodes managed by this Ric.
*/
- public
Vector
<String> getManagedElementIds() {
- return
ricConfig.managedElementIds(
);
+ public
synchronized Collection
<String> getManagedElementIds() {
+ return
new Vector<>(ricConfig.managedElementIds()
);
}
/**
}
/**
@@
-77,7
+83,7
@@
public class Ric {
* @param managedElementId the node name to check.
* @return true if the given node is managed by this Ric.
*/
* @param managedElementId the node name to check.
* @return true if the given node is managed by this Ric.
*/
- public boolean isManaging(String managedElementId) {
+ public
synchronized
boolean isManaging(String managedElementId) {
return ricConfig.managedElementIds().contains(managedElementId);
}
return ricConfig.managedElementIds().contains(managedElementId);
}
@@
-86,7
+92,7
@@
public class Ric {
*
* @param managedElementId the node to add.
*/
*
* @param managedElementId the node to add.
*/
- public void addManagedElement(String managedElementId) {
+ public
synchronized
void addManagedElement(String managedElementId) {
if (!ricConfig.managedElementIds().contains(managedElementId)) {
ricConfig.managedElementIds().add(managedElementId);
}
if (!ricConfig.managedElementIds().contains(managedElementId)) {
ricConfig.managedElementIds().add(managedElementId);
}
@@
-97,7
+103,7
@@
public class Ric {
*
* @param managedElementId the node to remove.
*/
*
* @param managedElementId the node to remove.
*/
- public void removeManagedElement(String managedElementId) {
+ public
synchronized
void removeManagedElement(String managedElementId) {
ricConfig.managedElementIds().remove(managedElementId);
}
ricConfig.managedElementIds().remove(managedElementId);
}
@@
-106,12
+112,12
@@
public class Ric {
*
* @return the policy types supported by this Ric in an unmodifiable list.
*/
*
* @return the policy types supported by this Ric in an unmodifiable list.
*/
- public Collection<PolicyType> getSupportedPolicyTypes() {
- return
supportedPolicyTypes.values(
);
+ public
synchronized
Collection<PolicyType> getSupportedPolicyTypes() {
+ return
new Vector<>(supportedPolicyTypes.values()
);
}
}
- public Collection<String> getSupportedPolicyTypeNames() {
- return
supportedPolicyTypes.keySet(
);
+ public
synchronized
Collection<String> getSupportedPolicyTypeNames() {
+ return
new Vector<>(supportedPolicyTypes.keySet()
);
}
/**
}
/**
@@
-119,14
+125,14
@@
public class Ric {
*
* @param type the policy type to support.
*/
*
* @param type the policy type to support.
*/
- public void addSupportedPolicyType(PolicyType type) {
+ public
synchronized
void addSupportedPolicyType(PolicyType type) {
supportedPolicyTypes.put(type.name(), type);
}
/**
* Removes all policy type as supported by this Ric.
*/
supportedPolicyTypes.put(type.name(), type);
}
/**
* Removes all policy type as supported by this Ric.
*/
- public void clearSupportedPolicyTypes() {
+ public
synchronized
void clearSupportedPolicyTypes() {
supportedPolicyTypes.clear();
}
supportedPolicyTypes.clear();
}
@@
-137,12
+143,12
@@
public class Ric {
*
* @return true if the given type is supported by this Ric, false otherwise.
*/
*
* @return true if the given type is supported by this Ric, false otherwise.
*/
- public boolean isSupportingType(String typeName) {
+ public
synchronized
boolean isSupportingType(String typeName) {
return supportedPolicyTypes.containsKey(typeName);
}
@Override
return supportedPolicyTypes.containsKey(typeName);
}
@Override
- public String toString() {
+ public
synchronized
String toString() {
return Ric.class.getSimpleName() + ": " + "name: " + name() + ", state: " + state + ", baseUrl: "
+ ricConfig.baseUrl() + ", managedNodes: " + ricConfig.managedElementIds();
}
return Ric.class.getSimpleName() + ": " + "name: " + name() + ", state: " + state + ", baseUrl: "
+ ricConfig.baseUrl() + ", managedNodes: " + ricConfig.managedElementIds();
}
@@
-152,7
+158,7
@@
public class Ric {
*/
public static enum RicState {
/**
*/
public static enum RicState {
/**
- * The agent view of the agent may be inconsistent
+ * The agent view of the agent may be inconsistent
.
*/
UNDEFINED,
/**
*/
UNDEFINED,
/**
@@
-160,18
+166,8
@@
public class Ric {
*/
IDLE,
/**
*/
IDLE,
/**
- * The Ric states are recovered
+ * The Ric states are recovered
.
*/
RECOVERING
}
*/
RECOVERING
}
-
- public A1ProtocolType getProtocolVersion() {
- return protocolVersion;
- }
-
- public void setProtocolVersion(A1ProtocolType version) {
- protocolVersion = version;
-
- }
-
}
}