udg-propro-spring-2020 / projecte-2020-a3

projecte-2020-a3 created by GitHub Classroom
0 stars 1 forks source link

Error testing i UI #25

Closed miqueldedo closed 4 years ago

miqueldedo commented 4 years ago

Bones, Porto un parell de dies treballant amb la validació del format d'entrada dels fitxers JSON. He creat una excepció per a aquest tipus d'errors. També, he afegit una carpeta (file_error_tests) on hi he penjat diverses configuracions diferents i l'explicació dels errors que conté cadascuna.

També estic intentant millorar l'interfície d'usuari i ho tinc quasi tot preparat per a només haver-me de centrar en generar el tauler i el control de peces. Però, hi ha un dubte que no sé com arreclar. Com ho haig de fer per a mostrar caràcters especials (accents, ç i d'altres)? Per exemple, se'm veu així: image

miquelbofill commented 4 years ago

Miquel,

No puc provar-ho perquè dóna error de compilació:

UIChess.java:557: error: missing return statement
    }
    ^

M'agradaria provar-ho en algun sistema Unix. En teoria ho tens gravat tot en format UTF-8 correctament. Per tant sospito que és cosa del Windows.

miqueldedo commented 4 years ago

Que estrany. A mi no em passava. Ara ja he avançat bastant i la versió que hi ha ara hauria de funcionar correctament.

miquelbofill commented 4 years ago

Miquel,

Ara compila i executa però no aconsegueixo fer que surti aquesta finestra. Pots indicar la seqüència de passos necessària?

miqueldedo commented 4 years ago

Sí, ara t'ho explico. Abans, però, hauries de tornar a fer un pull ja que al mostrar el pop up no feia un showAndWait() sinó un wait() i el programa se'm tancava. Ara en teoria hauria d'anar bé.

Simplement el que faig és, al fitxer configuration.json hi modifico el nombre de columnes o files a un valor no vàlid (v < 4 o v > 16).

Una altre manera, sense haver de modificar res del codi és: Obrir l'aplicació -> Preconfigura una partida ->Afegir fitxer -> Esculls qualsevol dels fitxers config_error_x.json -> selecciones el mode Jugador vs Jugador. Si fas servir el 4 no mostrarà cap error, ja que hi ha un moviment repetit (la resta del fitxer està bé!) i carregarà igualment.

Quan s'ha sel·leccionat el mode, es carrega el fitxer de configuració i es valida. En aquests casos, detectarà un error i mostrarà el pop up. En tancar el pop up, es tanca l'applicació.

miquelbofill commented 4 years ago

He fet el que dius i simplement es tanca el programa. Veig que el darrer commit és de fa 5 hores, cosa que em fa pensar que no has pujat els canvis.

miqueldedo commented 4 years ago

Ara sí que hauria d'anar. Disculpa.

miquelbofill commented 4 years ago
Screenshot 2020-05-11 at 19 22 20

Com m'imaginava, a mi em surt bé. És problema del joc de caràcters del Windows. Deixa-ho així amb UTF-8, que és l'estàndard de Java i dels sistemes Unix.

Desconec si hi ha cap truc per canviar el joc de caràcters "on the fly", però jo de tu ara no hi perdria temps.

miqueldedo commented 4 years ago

Carai, es veu millor que a Windows.

miquelbofill commented 4 years ago

Ull, no et deixis impressionar. La qualitat de les captures de pantalla és directament proporcional a la resolució. He fet la captura en una pantalla 4K. Amb Windows també es veuria bé donat el cas.

miquelbofill commented 4 years ago

Atès que ho esteu fent gairebé en anglès, no em fa res que la interfície també sigui en anglès. Llavors no tindríeu el problema dels caràcters especials. Com vulgueu.

miqueldedo commented 4 years ago

Tant el mode gràfic com el de consola?

I un dubte sobre el Doxygen, que ara estic mirant com es veu la documentació, veig que m'exporta els comentaris de dins de la funció. He estat remenant i no soc capaç de solucionar-ho. Em mostra el següent: Error Doxygen

miquelbofill commented 4 years ago

Tant el mode gràfic com el de consola?

Sí, per mi sí.

I un dubte sobre el Doxygen, que ara estic mirant com es veu la documentació, veig que m'exporta els comentaris de dins de la funció. He estat remenant i no soc capaç de solucionar-ho.

Tots els comentaris amb /// o /** ... */ surten a la documentació. Si no vols que surtin has de fer servir els comentaris estàndard // o /* ... */.

miqueldedo commented 4 years ago

Ja ho he traduit tot a anglès i el Doxygen ja no em mostra els comentaris. Perfecte!