From 9c8a85bfa8630ea8c14c80bf0a2cbfbec7a23275 Mon Sep 17 00:00:00 2001 From: Oli B Date: Tue, 30 Apr 2024 21:43:33 +0200 Subject: [PATCH] getFeldSparte() aus API wieder entfernt --- .../main/java/gdv/xport/satz/Datensatz.java | 20 +++------------ lib/src/main/java/gdv/xport/satz/Satz.java | 25 ++++--------------- .../test/java/gdv/xport/satz/SatzTest.java | 4 +-- .../java/gdv/xport/satz/xml/SatzXmlTest.java | 2 +- 4 files changed, 11 insertions(+), 40 deletions(-) diff --git a/lib/src/main/java/gdv/xport/satz/Datensatz.java b/lib/src/main/java/gdv/xport/satz/Datensatz.java index 1a7194b4..c2449981 100644 --- a/lib/src/main/java/gdv/xport/satz/Datensatz.java +++ b/lib/src/main/java/gdv/xport/satz/Datensatz.java @@ -302,7 +302,7 @@ public boolean hasWagnisart() { * @since 18.04.2018 */ public boolean hasKrankenFolgeNr() { - return this.getSatzart() == 220 && this.getFeldSparte().get().toInt() == 20 + return this.getSatzart() == 220 && this.getSparte() == 20 && (this.hasFeld(Bezeichner.FOLGE_NR_ZUR_LAUFENDEN_PERSONEN_NR_UNTER_NR_LAUFENDE_NR_TARIF) || this.hasFeld(Bezeichner.FOLGE_NR_ZUR_LAUFENDEN_PERSONEN_NR_UNTER_NR_BZW_LAUFENDEN_NR_TARIF)); } @@ -315,8 +315,7 @@ public boolean hasKrankenFolgeNr() { * @since 30.06.2021 */ public boolean hasBausparenArt() { - Optional feldSparte = this.getFeldSparte(); - return this.getSatzart() == 220 && feldSparte.isPresent() && feldSparte.get().toInt() == 580 + return this.getSatzart() == 220 && hasSparte() && getSparte() == 580 && (this.hasFeld(Bezeichner.ART_580)); } @@ -346,10 +345,7 @@ public boolean hasArt() { * Gets the sparte feld. * * @return die Sparte als Feld - * @deprecated seit 7.1 durch getFeldSparte() ersetzt - * // TODO: mit v9 entsorgen */ - @Deprecated @JsonIgnore public NumFeld getSparteFeld() { return getFeldSparte().get(); @@ -366,17 +362,7 @@ public boolean hasSparte() { return getFeldSparte().isPresent(); } - /** - * Diese Methode dient als Ersatz fuer die getSparte()- und hasSparte()- - * Implementierung, die durch den Mehrdeutigkeit von "Sparte" in die Irre - * fuehren koennen. - * - * @return Optional.empty() fuer Vor- und Nachsatz, ansonsten Sparte-Feld - * an Byte-Adresse 11 - * @since 7.1 - */ - @JsonIgnore - public Optional getFeldSparte() { + private Optional getFeldSparte() { ByteAdresse adresseSparte = ByteAdresse.of(11); if (hasFeld(adresseSparte)) { Feld feld = getFeld(adresseSparte); diff --git a/lib/src/main/java/gdv/xport/satz/Satz.java b/lib/src/main/java/gdv/xport/satz/Satz.java index c6eda519..b2edbbd9 100644 --- a/lib/src/main/java/gdv/xport/satz/Satz.java +++ b/lib/src/main/java/gdv/xport/satz/Satz.java @@ -724,19 +724,18 @@ public SatzTyp getSatzTyp() { if (StringUtils.isNotEmpty(this.gdvSatzartName)) { return SatzTyp.of(this.gdvSatzartName); } else { - Optional sparte = getFeldSparte(); - if (sparte.isPresent()) { + if (hasSparte()) { if (this.hasWagnisart() && this.getWagnisart().matches("\\d")) { - return SatzTyp.of(this.getSatzart(), sparte.get().toInt(), + return SatzTyp.of(this.getSatzart(), getSparte(), Integer.parseInt(this.getWagnisart())); } else if (this.hasKrankenFolgeNr() && this.getKrankenFolgeNr().matches("\\d")) { - return SatzTyp.of(this.getSatzart(), sparte.get().toInt(), + return SatzTyp.of(this.getSatzart(), getSparte(), Integer.parseInt(this.getKrankenFolgeNr())); } else if (this.hasBausparenArt() && this.getBausparenArt().matches("\\d")) { - return SatzTyp.of(this.getSatzart(), sparte.get().toInt(), + return SatzTyp.of(this.getSatzart(), getSparte(), Integer.parseInt(this.getBausparenArt())); } else { - return SatzTyp.of(this.getSatzart(), sparte.get().toInt()); + return SatzTyp.of(this.getSatzart(), getSparte()); } } else { return SatzTyp.of(this.getSatzart()); @@ -832,20 +831,6 @@ public int getSparte() { this.toShortString() + " hat kein Feld \"Sparte\" an Pos 11 in den Kopfdaten!"); } - /** - * Diese Methode dient als Ersatz fuer die getSparte()- und hasSparte()- - * Implementierung, die durch den Mehrdeutigkeit von "Sparte" in die Irre - * fuehren koennen. - * - * @return Optional.empty() fuer Vor- und Nachsatz, ansonsten Sparte-Feld - * an Byte-Adresse 11 - * @since 7.1 - */ - @JsonIgnore - public Optional getFeldSparte() { - return Optional.empty(); - } - /** * Liefert den Inhalt des Wagnisart-Felds. Vorher sollte allerdings mittels * {@link #hasWagnisart()} geprueft werden, ob der Satz ein Wagnisart-Feld diff --git a/lib/src/test/java/gdv/xport/satz/SatzTest.java b/lib/src/test/java/gdv/xport/satz/SatzTest.java index 41a1c2b5..67dedf1e 100644 --- a/lib/src/test/java/gdv/xport/satz/SatzTest.java +++ b/lib/src/test/java/gdv/xport/satz/SatzTest.java @@ -562,8 +562,8 @@ public void testHasSparteAsProdukt() { assertEquals(580, bausparen.getSparte()); assertEquals(satzTyp, bausparen.getSatzTyp()); for (Teildatensatz tds : bausparen.getTeildatensaetze()) { - assertTrue(tds.getFeldSparte().isPresent()); - assertEquals(580, tds.getFeldSparte().get().toInt()); + assertTrue(tds.hasSparte()); + assertEquals(580, tds.getSparte()); assertEquals(satzTyp, tds.getSatzTyp()); } } diff --git a/lib/src/test/java/gdv/xport/satz/xml/SatzXmlTest.java b/lib/src/test/java/gdv/xport/satz/xml/SatzXmlTest.java index f200eced..36245cd8 100644 --- a/lib/src/test/java/gdv/xport/satz/xml/SatzXmlTest.java +++ b/lib/src/test/java/gdv/xport/satz/xml/SatzXmlTest.java @@ -130,7 +130,7 @@ public void testGetSatzart() { */ @Test public void testGetSparte() { - NumFeld sparte = satz100.getFeldSparte().get(); + NumFeld sparte = satz100.getSparteFeld(); assertEquals(3, sparte.getAnzahlBytes()); assertEquals(11, sparte.getByteAdresse()); assertEquals("Sparte", sparte.getBezeichnung());