Bumper tidsserie-versjon, og noen små forenklinger (#274)

* Bumper tidsserie-versjon, og noen små forenklinger

* final på klasser

Author: tendestad

innsyn/src/main/java/no/nav/k9/innsyn/Søknadsammenslåer.java

@@ -20,58 +20,61 @@
 import no.nav.k9.søknad.ytelse.psb.v1.tilsyn.TilsynPeriodeInfo;
 import no.nav.k9.søknad.ytelse.psb.v1.tilsyn.Tilsynsordning;
 
-public class Søknadsammenslåer {
-    
+public final class Søknadsammenslåer {
+
+    private Søknadsammenslåer() {
+    }
+
     public static Søknad kunPleietrengendedata(Søknad søknad) {
         if (!(søknad.getYtelse() instanceof PleiepengerSyktBarn)) {
             throw new IllegalArgumentException("Kun PSB-søknader er støttet.");
         }
         final PleiepengerSyktBarn ytelse = (PleiepengerSyktBarn) søknad.getYtelse();
-        
+
         final Map<Periode, TilsynPeriodeInfo> periodeMap = hentUtTilsynsperioder(ytelse);
         final PleiepengerSyktBarn resultatYtelse = new PleiepengerSyktBarn();
         resultatYtelse.medTilsynsordning(new Tilsynsordning().medPerioder(periodeMap));
-        
+
         return new Søknad().medYtelse(resultatYtelse);
     }
 
     private static Map<Periode, TilsynPeriodeInfo> hentUtTilsynsperioder(final PleiepengerSyktBarn ytelse) {
-        /* 
+        /*
          * Advarsel: Data som returneres her deles på tvers av søkere. Ikke legg inn
          *           flere felter i mappingen hvis de ikke skal deles.
-         *           
+         *
          *           Av denne grunn brukes ikke standard copy-contructor for Map/Tilsynordning
          */
         return ytelse.getTilsynsordning()
                 .getPerioder()
                 .entrySet()
                 .stream()
                 .map(e -> Map.entry(
-                    new Periode(e.getKey()),
-                    new TilsynPeriodeInfo().medEtablertTilsynTimerPerDag(e.getValue().getEtablertTilsynTimerPerDag())
+                        new Periode(e.getKey()),
+                        new TilsynPeriodeInfo().medEtablertTilsynTimerPerDag(e.getValue().getEtablertTilsynTimerPerDag())
                 ))
                 .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
     }
-    
+
     public static Søknad slåSammen(Søknad tidligereSøknad, Søknad nySøknad) {
         if (!(tidligereSøknad.getYtelse() instanceof PleiepengerSyktBarn) || !(nySøknad.getYtelse() instanceof PleiepengerSyktBarn)) {
             throw new IllegalArgumentException("Kun PSB-søknader er støttet.");
         }
 
-        final PleiepengerSyktBarn s1Ytelse = (PleiepengerSyktBarn) tidligereSøknad.getYtelse();
-        final PleiepengerSyktBarn s2Ytelse = (PleiepengerSyktBarn) nySøknad.getYtelse();
-        
+        final PleiepengerSyktBarn s1Ytelse = tidligereSøknad.getYtelse();
+        final PleiepengerSyktBarn s2Ytelse = nySøknad.getYtelse();
+
         final PleiepengerSyktBarn ytelse = new PleiepengerSyktBarn();
         ytelse.medSøknadsperiode(slåSammenSøknadsperioder(s1Ytelse, s2Ytelse));
-        
+
         /* Bruk pleietrengendeAktørId fra PsbSøknadsinnhold fremfor: */
         ytelse.medBarn(new Barn()
-            .medNorskIdentitetsnummer(NorskIdentitetsnummer.of("00000000000"))
+                .medNorskIdentitetsnummer(NorskIdentitetsnummer.of("00000000000"))
         );
 
         ytelse.medArbeidstid(Arbeidstidsammenslåer.slåSammenArbeidstid(s1Ytelse, s2Ytelse));
         ytelse.medTilsynsordning(Tilsynsammenslåer.slåsammen(s1Ytelse, s2Ytelse));
-        
+
         final Søknad s = new Søknad()
                 .medSøknadId(SøknadId.of("generert"))
                 .medVersjon("1.0.0.")
@@ -86,10 +89,10 @@ private static Map<Periode, TilsynPeriodeInfo> hentUtTilsynsperioder(final Pleie
     private static List<Periode> slåSammenSøknadsperioder(final PleiepengerSyktBarn s1Ytelse, final PleiepengerSyktBarn s2Ytelse) {
         final LocalDateTimeline<Boolean> t1 = TidsserieUtils.toLocalDateTimeline(s1Ytelse.getSøknadsperiodeList());
         final LocalDateTimeline<Boolean> t2 = TidsserieUtils.toLocalDateTimeline(s2Ytelse.getSøknadsperiodeList());
-        final List<Periode> søknadsperioder = TidsserieUtils.tilPeriodeList(t1
+        LocalDateTimeline<Boolean> harSøknadTidslinje = t1
                 .union(t2, StandardCombinators::coalesceRightHandSide)
                 .disjoint(TidsserieUtils.toLocalDateTimeline(s2Ytelse.getTrekkKravPerioder()))
-                .compress());
-        return søknadsperioder;
+                .compress();
+        return TidsserieUtils.tilPeriodeList(harSøknadTidslinje);
     }
 }

innsyn/src/main/java/no/nav/k9/innsyn/søknadsammeslåer/Arbeidstidsammenslåer.java

@@ -26,10 +26,10 @@ public class Arbeidstidsammenslåer {
 
         final List<Arbeidstaker> arbeidstakerliste = slåSammenArbeidstakerArbeidstid(s2Ytelse, a1, a2);
         sammenslåttArbeidstid.medArbeidstaker(arbeidstakerliste);
-        
+
         final ArbeidstidInfo frilanserArbeidstidInfo = slåSammenFrilanserArbeidstid(s2Ytelse, a1, a2);
         sammenslåttArbeidstid.medFrilanserArbeidstid(frilanserArbeidstidInfo);
-        
+
         final ArbeidstidInfo selvstendigNæringsdrivendeArbeidstidInfo = slåSammenSelvstendigNæringsdrivendeArbeidstidInfo(s2Ytelse, a1, a2);
         sammenslåttArbeidstid.medSelvstendigNæringsdrivendeArbeidstidInfo(selvstendigNæringsdrivendeArbeidstidInfo);
 
@@ -42,7 +42,7 @@ public class Arbeidstidsammenslåer {
         final ArbeidstidInfo selvstendigNæringsdrivendeArbeidstidInfo = slåSammenArbeidstidInfo(s2Ytelse, t1, t2);
         return selvstendigNæringsdrivendeArbeidstidInfo;
     }
-    
+
     private static ArbeidstidInfo slåSammenFrilanserArbeidstid(PleiepengerSyktBarn s2Ytelse, final Arbeidstid a1, final Arbeidstid a2) {
         final LocalDateTimeline<ArbeidstidPeriodeInfo> t1 = lagArbeidstidTidslinje(a1.getFrilanserArbeidstidInfo().orElse(null));
         final LocalDateTimeline<ArbeidstidPeriodeInfo> t2 = lagArbeidstidTidslinje(a2.getFrilanserArbeidstidInfo().orElse(null));
@@ -78,28 +78,26 @@ public class Arbeidstidsammenslåer {
         return new ArbeidstidInfo(new ArbeidstidInfo().medPerioder(periodeMap));
     }
 
-    @SuppressWarnings("unchecked")
     private static LocalDateTimeline<ArbeidstidPeriodeInfo> hentTidslinje(final Map<Object, LocalDateTimeline<ArbeidstidPeriodeInfo>> arbeidstakertid1, Object arbeidstakerIdent) {
         final LocalDateTimeline<ArbeidstidPeriodeInfo> tidslinje = arbeidstakertid1.get(arbeidstakerIdent);
         if (tidslinje == null) {
-            return LocalDateTimeline.EMPTY_TIMELINE;
+            return LocalDateTimeline.empty();
         }
         return tidslinje;
     }
 
     private static Map<Object, LocalDateTimeline<ArbeidstidPeriodeInfo>> byggTidslinjeMap(Arbeidstid arbeidstid) {
         return arbeidstid.getArbeidstakerList().stream()
-            .map(a -> {
-                final LocalDateTimeline<ArbeidstidPeriodeInfo> tidslinje = lagArbeidstidTidslinje(a.getArbeidstidInfo());
-                return Map.entry(hentArbeidsgiverIdent(a),tidslinje);
-            })
-            .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
+                .map(a -> {
+                    final LocalDateTimeline<ArbeidstidPeriodeInfo> tidslinje = lagArbeidstidTidslinje(a.getArbeidstidInfo());
+                    return Map.entry(hentArbeidsgiverIdent(a), tidslinje);
+                })
+                .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
     }
 
-    @SuppressWarnings("unchecked")
     private static LocalDateTimeline<ArbeidstidPeriodeInfo> lagArbeidstidTidslinje(ArbeidstidInfo arbeidstidInfo) {
         if (arbeidstidInfo == null) {
-            return LocalDateTimeline.EMPTY_TIMELINE;
+            return LocalDateTimeline.empty();
         }
         return SøknadsammenslåerUtils.lagTidslinje(arbeidstidInfo.getPerioder());
     }

innsyn/src/main/java/no/nav/k9/innsyn/søknadsammeslåer/SøknadsammenslåerUtils.java

@@ -7,44 +7,37 @@
 import no.nav.fpsak.tidsserie.LocalDateSegment;
 import no.nav.fpsak.tidsserie.LocalDateTimeline;
 import no.nav.fpsak.tidsserie.StandardCombinators;
+import no.nav.k9.søknad.TidsserieUtils;
 import no.nav.k9.søknad.felles.type.Periode;
 import no.nav.k9.søknad.ytelse.psb.v1.PleiepengerSyktBarn;
 
 public final class SøknadsammenslåerUtils {
 
     private SøknadsammenslåerUtils() {
-        
     }
-    
-    
+
     static <T> Map<Periode, T> slåSammenOgHåndterTrukkedeKrav(PleiepengerSyktBarn s2Ytelse, final LocalDateTimeline<T> t1, final LocalDateTimeline<T> t2) {
         final LocalDateTimeline<T> kombinertInformasjon = t1.union(t2, StandardCombinators::coalesceRightHandSide);
         final LocalDateTimeline<T> resultat = kombinertInformasjon.disjoint(trukkedeKravTidslinje(s2Ytelse)).compress();
-        final Map<Periode, T> periodeMap = resultat.stream().map(p -> {
-                return Map.entry(new Periode(p.getFom(), p.getTom()), p.getValue());
-            })
-            .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
-        return periodeMap;
+        return resultat.stream()
+                .map(p -> Map.entry(new Periode(p.getFom(), p.getTom()), p.getValue()))
+                .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
     }
-    
-    @SuppressWarnings("unchecked")
+
     static <T> LocalDateTimeline<T> lagTidslinje(Map<Periode, T> perioder) {
         if (perioder == null) {
-            return LocalDateTimeline.EMPTY_TIMELINE;
+            return LocalDateTimeline.empty();
         }
         final List<LocalDateSegment<T>> segments = perioder
                 .entrySet()
                 .stream()
-                .map(e -> {
-                    return new LocalDateSegment<T>(e.getKey().getFraOgMed(), e.getKey().getTilOgMed(), e.getValue());
-                })
+                .map(e -> new LocalDateSegment<T>(e.getKey().getFraOgMed(), e.getKey().getTilOgMed(), e.getValue()))
                 .collect(Collectors.toList());
         return new LocalDateTimeline<>(segments);
     }
-    
 
     private static LocalDateTimeline<Boolean> trukkedeKravTidslinje(PleiepengerSyktBarn s1Ytelse) {
-        return new LocalDateTimeline<Boolean>(s1Ytelse.getTrekkKravPerioder().stream().map(p -> new LocalDateSegment<>(p.getFraOgMed(), p.getTilOgMed(), Boolean.TRUE)).collect(Collectors.toList()));
+        return TidsserieUtils.toLocalDateTimeline(s1Ytelse.getTrekkKravPerioder());
     }
 
 }

innsyn/src/main/java/no/nav/k9/innsyn/søknadsammeslåer/Tilsynsammenslåer.java

@@ -12,15 +12,14 @@ public class Tilsynsammenslåer {
         final LocalDateTimeline<TilsynPeriodeInfo> t1 = lagTidslinje(s1Ytelse.getTilsynsordning());
         final LocalDateTimeline<TilsynPeriodeInfo> t2 = lagTidslinje(s2Ytelse.getTilsynsordning());
         tilsyn.medPerioder(SøknadsammenslåerUtils.slåSammenOgHåndterTrukkedeKrav(s2Ytelse, t1, t2));
-        
+
         // Defensiv kopiering av Tilsynsordning for å få nye Periode- og TilsynPeriodeInfo-objekter:
         return new Tilsynsordning(tilsyn);
     }
 
-    @SuppressWarnings("unchecked")
     private static LocalDateTimeline<TilsynPeriodeInfo> lagTidslinje(Tilsynsordning tilsynsordning) {
         if (tilsynsordning == null) {
-            return LocalDateTimeline.EMPTY_TIMELINE;
+            return LocalDateTimeline.empty();
         }
         return SøknadsammenslåerUtils.lagTidslinje(tilsynsordning.getPerioder());
     }

pom.xml

@@ -110,7 +110,7 @@
             <dependency>
                 <groupId>no.nav.fpsak.tidsserie</groupId>
                 <artifactId>fpsak-tidsserie</artifactId>
-                <version>2.5.5</version>
+                <version>2.6.3</version>
             </dependency>
         </dependencies>
     </dependencyManagement>

soknad/src/main/java/no/nav/k9/søknad/TidsserieUtils.java

@@ -7,24 +7,22 @@
 import no.nav.fpsak.tidsserie.LocalDateTimeline;
 import no.nav.k9.søknad.felles.type.Periode;
 
-public class TidsserieUtils {
+public final class TidsserieUtils {
+    private TidsserieUtils() {
+    }
+
     public static List<Periode> tilPeriodeList(LocalDateTimeline<?> t) {
-        return t.stream().map(l -> new Periode(l.getFom(), l.getTom())).collect(Collectors.toList());
+        return t.stream()
+                .map(l -> new Periode(l.getFom(), l.getTom()))
+                .collect(Collectors.toList());
     }
 
     public static LocalDateTimeline<Boolean> toLocalDateTimeline(List<Periode> perioder) {
-        return new LocalDateTimeline<Boolean>(perioder
+        return new LocalDateTimeline<>(perioder
                 .stream()
-                .map(no.nav.k9.søknad.TidsserieUtils::mapLocalDateSegment)
+                .map(periode -> new LocalDateSegment<>(periode.getFraOgMed(), periode.getTilOgMed(), true))
                 .collect(Collectors.toList()))
                 .compress();
     }
 
-    private static LocalDateSegment<Boolean> mapLocalDateSegment(Periode periode) {
-        return new LocalDateSegment<Boolean>(
-                periode.getFraOgMed(),
-                periode.getTilOgMed(),
-                true);
-    }
-
 }