search

Ohjelmistotuotanto-DoD / vinkr

MIT License
0 stars 3 forks source link

Vinkki-rajapinnan ja KirjaVinkki-luokan lisäys #18

Closed vmarttil closed 4 years ago

vmarttil commented 4 years ago

Vinkki-rajapinnassa on nyt määritelty kaikille vinkeille yhteiset julkiset metodit. KirjaVinkki-luokka sisältää näiden ja ISBN:n lisäksi vielä muutaman kirjoihin liittyvän perustiedon, joista kaikki muut paitsi otsikko (luokan sisäisesti "nimeke") ovat vapaaehtoisia, eli käyttöliittymässä niitä voidaan kysyä (ei vielä tarvi) mutta niihin ei ole pakko syöttää mitään. Tekijän nimeä käyttöliittymässä kysyttäessä olisi hyvä kertoa käyttäjälle, että se syötetään muodossa "Sukunimi, Etunimi", koska muuten useampiosaisten etu- ja sukunimien parsaaminen on menee hankalaksi.

codecov-io commented 4 years ago

Codecov Report

Merging #18 into master will increase coverage by 43.38%. The diff coverage is 94.95%.

Impacted file tree graph

@@              Coverage Diff              @@
##             master      #18       +/-   ##
=============================================
+ Coverage        50%   93.38%   +43.38%     
- Complexity        2       46       +44     
=============================================
  Files             1        4        +3     
  Lines             4      121      +117     
  Branches          0       12       +12     
=============================================
+ Hits              2      113      +111     
- Misses            2        6        +4     
- Partials          0        2        +2
Impacted Files Coverage Δ Complexity Δ
src/main/java/vinkr/App.java 0% <0%> (-50%) 0 <0> (-2)
src/main/java/vinkr/Vinkr.java 100% <100%> (ø) 3 <3> (?)
src/main/java/vinkr/TextUI.java 95% <95%> (ø) 10 <10> (?)
src/main/java/vinkr/vinkit/KirjaVinkki.java 98.59% <98.59%> (ø) 33 <33> (?)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update d98a460...bdb96d9. Read the comment docs.

pprepu commented 4 years ago

Vinkki-rajapinnassa on nyt määritelty kaikille vinkeille yhteiset julkiset metodit. KirjaVinkki-luokka sisältää näiden ja ISBN:n lisäksi vielä muutaman kirjoihin liittyvän perustiedon, joista kaikki muut paitsi otsikko (luokan sisäisesti "nimeke") ovat vapaaehtoisia,

Tällä hetkellä järjestelmään pystyy kuitenkin syöttämään kirjoja tyhjillä tiedoilla. Oisko tarkoitus, että ainakin nimekkeen (otsikon) täytyis olla jotain muuta kuin "" tai tyhjä stringi?