dag7dev / common-voice-tool Goto Github PK

Aggiungere il "Cerca frasi duplicate" nelle funzioni

As title so we can gather help from other communities

NON bisogna aggiungere il punto se la frase termina con:

• " " spazio
• " " doppio spazio
• "!"/"?" punto esclamativo/interrogativo (NON HO VERIFICATO SE LO METTA O NO IN QUESTA CONDIZIONE)
• ";" punto e virgola (NON HO VERIFICATO SE LO METTA O NO IN QUESTA CONDIZIONE)

@jotaro-sama ha chiesto di localizzare in Italiano, vogliamo fare una cosa figa con gettext o ci accontentiamo di un accrocco fatto a mano?

Se il comando digitato non esiste BISOGNA restituire un errore.
Invece non viene catturato l'errore e fa selezionare il file normalmente.

Non funziona il RANGE.
Il conteggio dei caratteri superiori funziona bene e restituisce anche correttamente la stringa "Caratteri superiori a quelli consentiti", ma se è troppo corta non restituisce la stringa corretta: restituisce, infatti, la stessa stringa del superamento della soglia max :) correggere

Ho effettuato parecchi test e ho trovato alcune cose da migliorare. Ma, in generale, il programma riesce a gestire quasi tutto :)
|| Esito OK significa che è stato gestito correttamente || Esito NO significa che non è stato gestito ||
VERSIONE TOOL: UNKNOWN BETA

• Scelta lingua:
  • Inserito un valore differente da quello indicato: Esito OK -> Restituisce messaggio "File not found! Exiting..." -> Io mostrerei un messaggio di tipo "Language file not found! Exiting..." specificando che è il file di lingua a non essere stato trovato :)
• range (o chkLen):
  • Inserito valore stringa alfanumerico: Esito OK
  • Inserito valore decimale: Esito NO -> causa errore successivo nel controllo poiché si aspetta un valore numerico intero (Errore restituito nel controllo successivo "Attesa espressione intera")
• Scelta file:
  • Inserito file insesistente: Esito OK
• trim: Esito NO -> NON toglie gli spazi a fine riga e, inoltre, toglie tutti i primi spazio.
  Esempio:
  (Originale)

"Prova esempio frase numero uno ciao ciao ciao ciao  "
(Risultato)
"Provaesempio frase numero uno ciao ciao ciao nbsp;"

• ac: Esito OK -> Suggerimento: Quando viene mostrato "Aggiungo..." aggiungere anche un [OK] se viene aggiunto correttamente, ottenendo, in questo modo: "Aggiungo... [OK]"
• no-empy: Esito NO -> STESSO PROBLEMA DI TRIM -> NON CANCELLA LE RIGHE VUOTE E RIMUOVE I PRIMI SPAZI
  • Suggerimento: RINOMINARE LA FUNZIONE IN "noEmpty" -> bisogna seguire una regola di stile. O adotti a tutti il "-" oppure metti le maiuscole. Fare un minestrone confonde parecchio.
• capitalize: Esito OK
  • Ho provato anche a mettere differenti caratteri per verificare che trasformasse in maiuscolo ogni cosa e funziona correttamente. Anche le lettere accentate funzionano bene.
• chkPoint: Esito OK

Inoltre aggiungere il numero di versione, e non solo il tipo :)

Attenzione! Possono esserci anche più doppi spazi, anche dopo averne sostituiti.
Esempio:

Ciao    Ciao

(Ha 4 doppi spazi)
Se tu fai un replaceAll da "  " a " " otterresti

Ciao  Ciao

Ma c'è ancora un doppio spazio!
Quindi deve ripetere l'operazione finché la ricerca di dopi spazi risulta pari a 0 risultati

visualizza ancora "no-empty" piuttosto che "noEmpty"

**update:
Non solo viene visualizzato male, ma se si inserisce "-noEmpty" restituisce l'errore di funzione inesistente.