Skip to content
Snippets Groups Projects
Commit 68158d8f authored by Uwe Peuker's avatar Uwe Peuker
Browse files

Merge branch 'develop'

parents 92c359dd 5541787b
No related branches found
No related tags found
No related merge requests found
...@@ -30,6 +30,12 @@ de.bsvrz.sys.funclib.bitctrl.dua.AllgemeinerDatenContainer: ...@@ -30,6 +30,12 @@ de.bsvrz.sys.funclib.bitctrl.dua.AllgemeinerDatenContainer:
ausgegeben. Die Verwendung der Klasse in Set und Maps als Key ausgegeben. Die Verwendung der Klasse in Set und Maps als Key
führt aber zu undefiniertem Verhalten. führt aber zu undefiniertem Verhalten.
de.bsvrz.sys.funclib.bitctrl.dua.GanzZahl:
de.bsvrz.sys.funclib.bitctrl.dua.ufd.UmfeldDatenSensorWert:
de.bsvrz.sys.funclib.bitctrl.dua.MesswertMarkierung:
- equals ist implementiert, aber nicht "hashCode". Die equals-Methode wurde entfernt, das sie
nicht verwendet wird
- Obsolete SVN-Tags aus Kommentaren entfernt - Obsolete SVN-Tags aus Kommentaren entfernt
1.6.0 1.6.0
......
**********************************************************************************************************************
de.bsvrz.sys.funclib.bitctrl.dua:
de.bsvrz.sys.funclib.bitctrl.dua.AllgemeinerDatenContainer:
die Klasse sollte nicht mehr verwendet werden, weil der
API-Kontrakt für die Implementierung von equals und hashCode
verletzt wird und auch nicht sinnvoll implementiert werden kann.
Für die hashCode-Methode wird hier die hashCode-Methode der
Superklasse aufgerufen und eine entsprechende Fehlermeldung
ausgegeben. Die Verwendung der Klasse in Set und Maps als Key
führt aber zu undefiniertem Verhalten.
- Obsolete SVN-Tags aus Kommentaren entfernt
**********************************************************************************************************************
de.bsvrz.dua.aggrlve:
de.bsvrz.dua.aggrlve.AggregationsDatum
- überschreibt equals und nicht hashCode. Da equals nur die Supermethode aufruft, wurde die
Funktion hier entfernt
de.bsvrz.dua.aggrlve.AggregationsIntervall
- hashCode-Funktion entsprechend equals ergänzt
de.bsvrz.dua.aggrlve.tests.AggrLveTestBase
- der Member "_aggregationLVE" sollte nicht statisch sein, der er bei jedem Test neu initialisiert wird
- Obsolete SVN-Tags aus Kommentaren entfernt
- Obsolete inheritDoc-Kommentare entfernt
**********************************************************************************************************************
de.bsvrz.dua.dalve:
de.bsvrz.dua.dalve.analyse.AtgVerkehrsDatenKurzZeitAnalyseFs
de.bsvrz.dua.dalve.analyse.AtgVerkehrsDatenKurzZeitAnalyseMq
- die Klassen erweitern nicht mehr de.bsvrz.sys.funclib.bitctrl.dua.AllgemeinerDatenContainer
- equals und hashCode können nicht adäquat überschrieben werden, weil die Klasse änderbare Daten
repräsentiert und gleichzeitig als ClientDavReceiver in einer Map als Key eingesetzt wird
de.bsvrz.dua.dalve.stoerfall.fd4.ParameterFuerStoerfall
- die Klasse erweitern nicht mehr de.bsvrz.sys.funclib.bitctrl.dua.AllgemeinerDatenContainer
- equals und hashCode können nicht adäquat überschrieben werden, weil die Klasse änderbare Daten
repräsentiert. Die einzige von der ursprünglichen Basisklasse bereitgestellte Funktion "equals"
wird entsprechend Code-Analyse nicht genutzt
de.bsvrz.dua.dalve.analyse.DaAnalyseMessQuerschnitt#berechneVDifferenz(Data)
- null-Prüfung zu spät potentieller NPE
de.bsvrz.dua.dalve.prognose.PrognoseAttributParameter
- die Klasse sollte nicht in Sets oder Maps verwendet werden, weil equals überschrieben wurde, aber die Daten
änderbar sind, die Funktion hashCode wirft daher hier eine UnsupportedOperationException
de.bsvrz.dua.dalve.tests.DaLveTestBase
- der Member "_datenaufbereitungLVE" sollte nicht statisch sein, der er bei jedem Test neu initialisiert wird
- Javadoc für Java8-Kompatibilität korrigiert
- Obsolete SVN-Tags aus Kommentaren entfernt
- Obsolete inheritDoc-Kommentare entfernt
**********************************************************************************************************************
de.bsvrz.dua.daufd:
de.bsvrz.dua.daufd.UfdsKlassifizierungParametrierung
- im Konstruktor wurde fälschlicherweise zweimal der gleiche Parameter auf null geprüft und nicht der
gewünschte - potentielle NPE
de.bsvrz.dua.daufd.tp.Taupunkt.initialisiere(IVerwaltung)
- das verwendete Verwaltungsmodul kann potentiell mehrfach zugewiesen werden, was zu einem Bruch der Modulkette
führen würde. Wenn ein neues Modul zugewiesen werden soll wird eine IllegalStateEception geworfen
de.bsvrz.dua.daufd.tests.DuAUfdTestBase
- der Member "aufbereitungUFD" sollte nicht statisch sein, der er bei jedem Test neu initialisiert wird
- Javadoc für Java8-Kompatibilität korrigiert
- Obsolete inheritDoc-Kommentare entfernt
**********************************************************************************************************************
de.bsvrz.dua.fehlertls:
de.bsvrz.dua.fehlertls.tls.DeErfassungsZustand.Zustand
de.bsvrz.dua.fehlertls.tests.DeStatus
- equals wurde um entsprechende hashCode-Funktion ergänzt
- Javadoc für Java8-Kompatibilität korrigiert
- Obsolete SVN-Tags aus Kommentaren entfernt
- Obsolete inheritDoc-Kommentare entfernt
**********************************************************************************************************************
de.bsvrz.dua.guete:
de.bsvrz.dua.guete.GWert
- hashCode entsprechend equals-Funktion ergänzt
- Javadoc für Java8-Kompatibilität korrigiert
- Obsolete SVN-Tags aus Kommentaren entfernt
- Obsolete inheritDoc-Kommentare entfernt
**********************************************************************************************************************
de.bsvrz.sys.funclib.langfehlerlve:
de.bsvrz.dua.langfehlerlve.modell.ausw.DELzFhMessQuerschnitt.MQDatum
- Klasse hatte keine geeignete Hash-Funktion wurde aber in einem Set verwendet
de.bsvrz.dua.langfehlerlve.modell.online.Intervall
- hashCode entsprechend equals-Funktion ergänzt
de.bsvrz.dua.langfehlerlve.modell.ausw.DELzFhMessStelle
- Prüfung auf null-Object nach der Verwendung bei der Senderanmeldung
- Obsolete SVN-Tags aus Kommentaren entfernt
- Obsolete inheritDoc-Kommentare entfernt
**********************************************************************************************************************
de.bsvrz.dua.mwelve:
de.bsvrz.dua.mwelve.tests.DuAMweLveTestBase
- der Member "_messWertErsetzungLVE" sollte nicht statisch sein, der er bei jedem Test neu initialisiert wird
- Obsolete SVN-Tags aus Kommentaren entfernt
- Obsolete inheritDoc-Kommentare entfernt
**********************************************************************************************************************
de.bsvrz.dua.mweufd:
de.bsvrz.dua.mweufd.tests.DuAMweUfdTestBase
- der Member "_messwertErsetzungUFD" sollte nicht statisch sein, der er bei jedem Test neu initialisiert wird
de.bsvrz.dua.mweufd.tests.TestDuAMweUfd.DuaUfdLayout.setValues(SystemObject, Data, List<String>, int, String, boolean)
- Exception beim Auswerten der Daten wird ignoriert und nicht für den Test berücksichtigt?
- Obsolete SVN-Tags aus Kommentaren entfernt
- Obsolete inheritDoc-Kommentare entfernt
**********************************************************************************************************************
de.bsvrz.dua.plformal:
- Obsolete SVN-Tags aus Kommentaren entfernt
- Obsolete inheritDoc-Kommentare entfernt
**********************************************************************************************************************
de.bsvrz.dua.pllangufd:
de.bsvrz.dua.pllangufd.parameter.UfdsLangZeitPlPruefungsParameter
- die Klasse erweitert nicht mehr de.bsvrz.sys.funclib.bitctrl.dua.AllgemeinerDatenContainer
- equals und hashCode sind konform implementiert
de.bsvrz.dua.pllangufd.historie.HistPufferElement
- hashCode entsprechend equals-Funktion ergänzt
de.bsvrz.dua.pllangufd.tests.DuAPlLangUfdTestBase
- der Member "_pruefungLangzeitUfd" sollte nicht statisch sein, der er bei jedem Test neu initialisiert wird
- Javadoc für Java8-Kompatibilität korrigiert
- Obsolete inheritDoc-Kommentare entfernt
**********************************************************************************************************************
de.bsvrz.dua.plloglve:
de.bsvrz.dua.plloglve.plloglve.diff.AtgVerkehrsDatenDifferenzialKontrolleFs
de.bsvrz.dua.plloglve.plloglve.standard.AtgVerkehrsDatenKurzZeitIntervallPlLogisch
- die Klassen erweitert nicht mehr de.bsvrz.sys.funclib.bitctrl.dua.AllgemeinerDatenContainer
- equals und hashCode sind konform implementiert
de.bsvrz.dua.plloglve.plloglve.AbstraktDAVZeitEinzelDatum.equals(Object)
- in der equals-Methode wurde auf die falsche Klasse gecastet
- hashCode nicht implementiert, es wird jetzt eine UnsupportedOperationException geworfen
- Klasse auf Deprecated gesetzt, da scheinbar nicht verwendet
de.bsvrz.dua.plloglve.tests.DuAPlLogLveTestBase
- der Member "_pruefungLogischLVE" sollte nicht statisch sein, der er bei jedem Test neu initialisiert wird
- Javadoc für Java8-Kompatibilität korrigiert
- Obsolete SVN-Tags aus Kommentaren entfernt
- Obsolete inheritDoc-Kommentare entfernt
**********************************************************************************************************************
de.bsvrz.dua.pllogufd:
de.bsvrz.dua.pllogufd.grenz.UniversalAtgUfdsGrenzwerte.java
de.bsvrz.dua.pllogufd.testaufab.UniversalAtgUfdsAnstiegAbstiegKontrolle
de.bsvrz.dua.pllogufd.testdiff.UniversalAtgUfdsDifferenzialKontrolle
- die Klassen erweitert nicht mehr de.bsvrz.sys.funclib.bitctrl.dua.AllgemeinerDatenContainer
- equals und hashCode sind konform implementiert
de.bsvrz.dua.pllogufd.tests.DuAPlLogUfdTestBase
- der Member "_pruefungLogischUFD" sollte nicht statisch sein, der er bei jedem Test neu initialisiert wird
- Obsolete SVN-Tags aus Kommentaren entfernt
- Obsolete inheritDoc-Kommentare entfernt
**********************************************************************************************************************
de.bsvrz.dua.progglaette:
de.bsvrz.dua.progglaette.tests.DuAProgGlaetteTestBase
- der Member "_glaetteWarnungUndPrognose" sollte nicht statisch sein, der er bei jedem Test neu initialisiert wird
- Javadoc für Java8-Kompatibilität korrigiert
- Obsolete SVN-Tags aus Kommentaren entfernt
- Obsolete inheritDoc-Kommentare entfernt
**********************************************************************************************************************
...@@ -253,25 +253,6 @@ public class GanzZahl implements Comparable<GanzZahl> { ...@@ -253,25 +253,6 @@ public class GanzZahl implements Comparable<GanzZahl> {
return s; return s;
} }
/* FIXME equals und hashCode verletzt den API-Kontrakt und sollte
* hier entfernt bzw. überdacht werden. */
@Override
public boolean equals(final Object obj) {
boolean gleich = false;
if ((obj != null) && (obj instanceof GanzZahl)) {
final GanzZahl that = (GanzZahl) obj;
gleich = getWert() == that.getWert();
}
return gleich;
}
@Override
public int hashCode() {
throw new UnsupportedOperationException();
}
@Override @Override
public int compareTo(final GanzZahl that) { public int compareTo(final GanzZahl that) {
if (that == null) { if (that == null) {
......
...@@ -235,28 +235,6 @@ public class MesswertMarkierung implements Cloneable { ...@@ -235,28 +235,6 @@ public class MesswertMarkierung implements Cloneable {
this.veraendert = veraendert; this.veraendert = veraendert;
} }
/* FIXME equals und hashCode verletzt den API-Kontrakt und sollte
* hier entfernt bzw. überdacht werden. */
@Override
public boolean equals(final Object obj) {
boolean gleich = false;
if (obj instanceof MesswertMarkierung) {
final MesswertMarkierung that = (MesswertMarkierung) obj;
gleich = (nichtErfasst == that.nichtErfasst) && (implausibel == that.implausibel)
&& (interpoliert == that.interpoliert) && (formalMax == that.formalMax)
&& (formalMin == that.formalMin) && (logischMax == that.logischMax)
&& (logischMin == that.logischMin);
}
return gleich;
}
@Override
public int hashCode() {
throw new UnsupportedOperationException();
}
@Override @Override
public String toString() { public String toString() {
return (nichtErfasst ? "nErf " : Constants.EMPTY_STRING) + (formalMax ? "fMax " : Constants.EMPTY_STRING) return (nichtErfasst ? "nErf " : Constants.EMPTY_STRING) + (formalMax ? "fMax " : Constants.EMPTY_STRING)
......
...@@ -281,27 +281,6 @@ public class UmfeldDatenSensorWert implements Comparable<UmfeldDatenSensorWert> ...@@ -281,27 +281,6 @@ public class UmfeldDatenSensorWert implements Comparable<UmfeldDatenSensorWert>
return wertStr; return wertStr;
} }
/*
* FIXME equals und hashCode verletzt den API-Kontrakt und sollte hier
* entfernt bzw. überdacht werden.
*/
@Override
public boolean equals(final Object obj) {
boolean gleich = false;
if ((obj != null) && (obj instanceof UmfeldDatenSensorWert)) {
final UmfeldDatenSensorWert that = (UmfeldDatenSensorWert) obj;
gleich = getWert() == that.getWert();
}
return gleich;
}
@Override
public int hashCode() {
throw new UnsupportedOperationException();
}
@Override @Override
public int compareTo(final UmfeldDatenSensorWert that) { public int compareTo(final UmfeldDatenSensorWert that) {
if (that == null) { if (that == null) {
......
...@@ -53,8 +53,6 @@ public final class DUAUmfeldDatenMessStelle { ...@@ -53,8 +53,6 @@ public final class DUAUmfeldDatenMessStelle {
* statische Instanzen dieser Klasse. * statische Instanzen dieser Klasse.
*/ */
private static final Map<SystemObject, DUAUmfeldDatenMessStelle> instanzen = new HashMap<>(); private static final Map<SystemObject, DUAUmfeldDatenMessStelle> instanzen = new HashMap<>();
private static boolean initialized;
/** /**
* das Systemobjekt. * das Systemobjekt.
*/ */
...@@ -80,12 +78,9 @@ public final class DUAUmfeldDatenMessStelle { ...@@ -80,12 +78,9 @@ public final class DUAUmfeldDatenMessStelle {
if (messStellenObjekte == null) { if (messStellenObjekte == null) {
throw new NullPointerException("Menge der Umfelddaten-Messstellen ist <<null>>"); throw new NullPointerException("Menge der Umfelddaten-Messstellen ist <<null>>");
} }
if (initialized) {
Debug.getLogger().error("UFD-Modell darf nur einmal initialisiert werden");
return;
}
initialized = true; /* es wird immer nur ein ClientDavInterface bedient, die Initialisierung kann nur einmalig erfolgen. */
instanzen.clear();
for (final SystemObject mStObj : messStellenObjekte) { for (final SystemObject mStObj : messStellenObjekte) {
DUAUmfeldDatenMessStelle.instanzen.put(mStObj, new DUAUmfeldDatenMessStelle(dav, mStObj)); DUAUmfeldDatenMessStelle.instanzen.put(mStObj, new DUAUmfeldDatenMessStelle(dav, mStObj));
} }
...@@ -100,10 +95,6 @@ public final class DUAUmfeldDatenMessStelle { ...@@ -100,10 +95,6 @@ public final class DUAUmfeldDatenMessStelle {
* @return die statischen Instanzen dieser Klasse (ggf. leere Liste) * @return die statischen Instanzen dieser Klasse (ggf. leere Liste)
*/ */
public static Collection<DUAUmfeldDatenMessStelle> getInstanzen() { public static Collection<DUAUmfeldDatenMessStelle> getInstanzen() {
if (!initialized) {
throw new RuntimeException("DUAUmfeldDatenMessStelle wurde noch nicht initialisiert");
}
return DUAUmfeldDatenMessStelle.instanzen.values(); return DUAUmfeldDatenMessStelle.instanzen.values();
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment