https://travis-ci.com/github/Glusk/sveder/builds/188433311

Čeprav so šifre v preglednicah: <ImeZavoda>UrnCD.xlsx tipično cela števila, se lahko nekdo zatipka ali pa namerno (lahko da je tak dogovor znotraj zavoda) vnese niz.

Dodaj razred Sifra, ki bo znal prebrati šifro kot niz ali kot numerično vrednost. Primerjava šifer naj bo odslej primerjava nizov. Popravi tudi kodo tako, da vsepovsod, kjer pričakujemo šifro tipa Number, odslej lahko podamo tip Sifra.

Popravi tudi imena testnih datotek da bodo bolj dosledna in natančneje odsevala vsebino.

Treba je prenašati in brati razne Excelove datoteke.

https://travis-ci.com/github/Glusk/sveder/builds/184063395

Nekateri zavodi na svoji spletni strani objavijo podatke o urniku in čakalnih dobah kot Wordov dokument.

Primer: https://www.zzzs.si/zzzs/pao/pogizv.nsf/PoZZZSst/90255

Pri poskusu branja takšne datoteke Sveder vrže org.apache.poi.ooxml.POIXMLException, ki je tipa RuntimeException.

Bolje bi bilo uloviti to izjemo in jo ponovno vreči kot IOException.

Šifro lokacije (dodano v #79) je treba sestaviti na drugačen način.

Gre za napako v URL naslovih. Viri, na URL-jih http://... so bili premaknjeni na https://...

Povej, da lahko ordinacije vedno filtriramo ročno, da pa je smiselno zapakirati nekatere poizvedbe (tiste, za katere predvidevamo, da bodo pogoste) v svoj razred.

Dodaj funkcionalnost za branje .xls datotek z seznamom aktivnih zobozdravnikov. Iz datotek preberi naslednja polja:

• Šifra izvajalca
• Šifra zdravnika
• Doseganje povprečja

https://www.travis-ci.com/github/Glusk/sveder/builds/216215213

Razreda UrlOvoj v resnici sploh ne potrebujemo. Vsepovsod, kjer se pojavi, ga lahko nadomestimo z:

() -> new URL(/* url niz */)

Koda bo tako uporabna v več razredih.

Spletna stran zavoda, ki ne objavi preglednice lahko vrne:

• https://www.zzzs.si/zzzs/pao/pogizv.nsf/PoZZZSst/5940232 404 Not Found
• https://www.zzzs.si/zzzs/pao/pogizv.nsf/PoZZZSst/3096100 200 OK

V slednjem primeru se torej lahko zgodi, da bomo iskali ujemanje URL-ja preglednice z regularnim izrazom na spletni strani, ki ne vsebuje URL-ja preglednice. new RegularniIzraz(/**/).ujemanja().get() v tem primeru vrže IndexOutOfBoundsException.

Na Windows operacijskih sistemih privzeto kodiranje (Charset.defaultCharset()) ni Unicode, zato lahko knjižnica izpiše čudne nize.

https://stackoverflow.com/a/28470840/6367213

Razišči in dokumentiraj!

Pom je čistejši, če definiram verzije in konfiguracijo na enem mestu.

Datoteka: TestZdravstveniZavod.java...

Tole ni v redu, ker se ob vsakem poganjanju testa odpremo TCP povezavo do ZZZZS strežnika. Sploh pa je vsa logika, ki se tu testira, že testirana v TestUrlNaStrani.java.

// ....
@Test
public void vrnePreglednicoZavodaZgrajenegaIzZzzsStevilke() {
    assertDoesNotThrow(() ->
        new ZdravstveniZavod(6189896).preglednica()
    );
}

Dodaj:

// .... 
@Test
public void neMeceIzjemeCeNeNajdeStraniSPreglednico() {
    assertDoesNotThrow(() ->
        new ZdravstveniZavod(() -> new FileNotFoundException()).preglednica()
    );
}

najdeZdravnikaVPreglednici() - ta test ni potreben.

Podoben problem kot pri #57.

Dodaj orodje za statično analiziranje kode "checkstyle".

Zaenkrat zgleda kot da je to html 5 projekt. :P

Za dano:

• šifro zdravnika
• šifro dejavnosti

vrni:

{
  "ordinacija": {
    "id_izvajalec": "number",
    "zdravnik": {
      "id_zdravnik": "number",
      "ime_priimek": "string"
    },
    "id_dejavnost": "number",
    "doseganje_povprecja": "number",
  }
}

Testi se ne izvedejo: #20

Mogoče je problem v nenormiranih končnicah datotek.

Podatki, ki jih želimo izluščiti, so razpršeni po več različnih mestih. Iz nobenega posameznega vira ne dobimo vseh podatkov, ki bi si jih želeli.

V paketu com.github.glusk.sveder je preveč tipov. Vse kar v resnici potrebujemo je en tip, ki naj ima metodo json().

Namesto da poskušamo zgraditi drevesno strukturo različnih tipov, ki so definirani v paketu com.github.glusk.sveder, raje definirajmo posamezne preslikave. Na ta način razbijemo naš "scraper" na manjše kose, katere lahko enostavneje testiramo in vzdržujemo.

Preslikave

P1

Vhod

Seznam aktivnih zobzdravnikov (ZOB_dd_mm_yyyy.xls)

Izhod

{
  "ordinacija": {
    "sifra_izvajalca": "string",
    "zdravnik": {
      "sifra": "string",
      "ime_priimek": "string"
    },
    "dejavnost": {
      "sifra": "string",
      "naziv": "string"
    },
    "doseganje_povprecja": "number"
  },
  // ...
}

P2

Vhod

Šifra izvajalca, šifra dejavnosti, šifra zdravnika, šifra lokacije
Šifra izvajalca -> seznam zavodov
Vsaj en zavod (tipično, a ne vedno :/) objavi preglednico <Ime_zavoda>UrnČD.xlsx. V njej so podatki o urnikih
ordinacij na posamezni lokaciji.

Izhod

"urnik": {
  "razpored": [
    {
      "dan": "pon",
      "od": "string",
      "do": "string"
    },
    {
      "dan": "tor",
      "od": "string",
      "do": "string"
    },
    {
      "dan": "sre",
      "od": "string",
      "do": "string"
    },
    {
      "dan": "cet",
      "od": "string",
      "do": "string"
    },
    {
      "dan": "pet",
      "od": "string",
      "do": "string"
    },
    {
      "dan": "sob",
      "od": "string",
      "do": "string"
    },
  ],
  "opombe": "string"
}

Sveder trenutno ne podpira avtomatskega branja podatkov o lokacijah zobozdravnikov, ki so zaposleni pri gornjih izvajalcih!

Uporaben link: https://stackoverflow.com/questions/27749942/whats-the-recipe-for-running-mvn-integration-test-instead-of-mvn-test-with-trav

Združi razreda VrsticaUrnika in VrsticaPodrobnostiLokacije v enega.