justagoldeneye / tiralabra-nonogram

CircleCI stopped working again after adding JavaFX functionality to the program. The tests still function correctly on my computer. After doing a little bit of research online, it seems likely that the problem is at the CircleCI's end.

Nonogram solver

Eemeli Leppä

Weekly reports

• Weekly report 1
• Weekly report 2
• Weekly report 3
• Weekly report 4
• Weekly report 5
• Weekly report 6
• Extra work hour log

Documentation

• Definition document
• Testing document
• Implementation document
• Manual

tiralabra-nonogram's Issues

Vertaisarvio

17.4.2020, 16:10

Alku näyttää sinänsä hyvältä vaikka ohjelma tarvitsee vielä paljon työtä. Runko on kuitenkin valmis varsinaista ratkaisijaa varten.

Rowsolverista olisi varmaan helpompi luoda yleinen instanssi, jolle annat seuraavaksi ratkaistavan rivin erikseen. Tämä nyt saattaa olla ajatuksena jo muutenkin.

NumberRow luokan metodeissa (lookNumber, changeNumber) olisi varmaan hyvä olla tarkastuksia pyynnöille. Toistaiseksi ei ilmeisesti tule out of bound viittauksia, mutta saattaisi helpottaa ongelmien löytämistä tulevaisuudessa ja on aika nopeaa toteuttaa.

Itse yritin luoda ratkaisun etsintää rajaavia ehtoja omaan ohjelmaani, tässä saattaisi ohjelmaa nopeuttaa seuraavien rivien tai sarakkeiden merkitseminen tarkastettavaksi. Käsittääkseni nonogramit pitkälti ratkaistaan pomppimalla vuorotellen riveillä ja sarakkeilla, uuden tiedon avatessa seuraavaa tietoa ns. ristiin. Ohjelma ei taida kyllä vielä olla ihan näin pitkällä.

Kokonaisuudessaan metodit on rajattu sopivan mittaisiksi ja metodien kommentointia on jo kohtuullisesti.