From 327bce947abf3d52b72d538ea8f22444eb7cd6d3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=85smund=20V=C3=A5ge=20Fannemel?= <34712686+asmfstatoil@users.noreply.github.com> Date: Mon, 6 Jun 2022 17:52:48 +0200 Subject: [PATCH] refact: inherit getfluid from interface --- .../ProcessEquipmentBaseClass.java | 10 +--------- .../ProcessEquipmentInterface.java | 20 ++++++++++--------- .../reservoir/SimpleReservoir.java | 6 ------ .../processSystem/ProcessModuleBaseClass.java | 6 ------ 4 files changed, 12 insertions(+), 30 deletions(-) diff --git a/src/main/java/neqsim/processSimulation/processEquipment/ProcessEquipmentBaseClass.java b/src/main/java/neqsim/processSimulation/processEquipment/ProcessEquipmentBaseClass.java index 2bcbded5bc..13e2057fe1 100644 --- a/src/main/java/neqsim/processSimulation/processEquipment/ProcessEquipmentBaseClass.java +++ b/src/main/java/neqsim/processSimulation/processEquipment/ProcessEquipmentBaseClass.java @@ -47,20 +47,12 @@ public ProcessEquipmentBaseClass(String name) { super(name); } - - /** {@inheritDoc} */ @Override public SystemInterface getThermoSystem() { return null; } - /** {@inheritDoc} */ - @Override - public SystemInterface getFluid() { - return getThermoSystem(); - } - /** {@inheritDoc} */ @Override public void displayResult() {} @@ -98,7 +90,7 @@ public void setRegulatorOutSignal(double signal) {} @Override public void setController(ControllerDeviceInterface controller) { this.controller = controller; - hasController = true; + hasController = controller == null; } /** diff --git a/src/main/java/neqsim/processSimulation/processEquipment/ProcessEquipmentInterface.java b/src/main/java/neqsim/processSimulation/processEquipment/ProcessEquipmentInterface.java index bcadc52190..1aab7b31eb 100644 --- a/src/main/java/neqsim/processSimulation/processEquipment/ProcessEquipmentInterface.java +++ b/src/main/java/neqsim/processSimulation/processEquipment/ProcessEquipmentInterface.java @@ -88,31 +88,33 @@ public interface ProcessEquipmentInterface extends SimulationInterface, java.io. /** *

- * getThermoSystem. + * getFluid. *

* * @return a {@link neqsim.thermo.system.SystemInterface} object */ - public SystemInterface getThermoSystem(); + default public SystemInterface getFluid() { + return getThermoSystem(); + } /** *

- * getMassBalance. + * getThermoSystem. *

* - * @param unit a {@link java.lang.String} object - * @return a double + * @return a {@link neqsim.thermo.system.SystemInterface} object */ - public double getMassBalance(String unit); + public SystemInterface getThermoSystem(); /** *

- * getFluid. + * getMassBalance. *

* - * @return a {@link neqsim.thermo.system.SystemInterface} object + * @param unit a {@link java.lang.String} object + * @return a double */ - public SystemInterface getFluid(); + public double getMassBalance(String unit); /** *

diff --git a/src/main/java/neqsim/processSimulation/processEquipment/reservoir/SimpleReservoir.java b/src/main/java/neqsim/processSimulation/processEquipment/reservoir/SimpleReservoir.java index fd31b20aea..5d3127f2df 100644 --- a/src/main/java/neqsim/processSimulation/processEquipment/reservoir/SimpleReservoir.java +++ b/src/main/java/neqsim/processSimulation/processEquipment/reservoir/SimpleReservoir.java @@ -83,12 +83,6 @@ public SystemInterface getReservoirFluid() { * public StreamInterface getOilOutStream() { return oilOutStream; } */ - /** {@inheritDoc} */ - @Override - public SystemInterface getFluid() { - return thermoSystem; - } - /** *

* addGasProducer. diff --git a/src/main/java/neqsim/processSimulation/processSystem/ProcessModuleBaseClass.java b/src/main/java/neqsim/processSimulation/processSystem/ProcessModuleBaseClass.java index fe897466fa..ca3360a858 100644 --- a/src/main/java/neqsim/processSimulation/processSystem/ProcessModuleBaseClass.java +++ b/src/main/java/neqsim/processSimulation/processSystem/ProcessModuleBaseClass.java @@ -128,12 +128,6 @@ public SystemInterface getThermoSystem() { return null; } - /** {@inheritDoc} */ - @Override - public SystemInterface getFluid() { - return getThermoSystem(); - } - /** *

* setSpecification.