ConoscereLinux / DeskApp

Utility per gestire, rinominare e indicizzare file
GNU General Public License v3.0
5 stars 23 forks source link

Feature/parser #19

Closed FrGermano closed 1 year ago

FrGermano commented 1 year ago

Ho creato solo ora un pull request poichè mi è venuto il sospetto che senza questa azione il codice da me scritto non sia visibile, spero di sbagliarmi poichè ho eseguito git push ogni volta che ho effettuato modifiche.

ZompaSenior commented 1 year ago

Ciao Germano, innanzi tutto scusa per l'incredibile ritardo con cui ti rispondo.

Allora, buona l'idea di utilizzare una classe, però ti chiederei di dare un occhio al modulo segnaposto ch ho creato nel repository, dove ho già inserito una classe vuota in cui mettere il tuo codice (la trovi qui), ma questo non è un grosso problema, lo posso fare io dopo quando integro.

Diciamo che dobbiamo dividerci i compiti e limitarci solo a quella parte (con eventuale codice di test), quindi la parte dell'option_manager dovrebbe essere quella di leggere le opzioni da riga di comando (hai fatto implementato qualcosa di simile) e memorizzare le opzioni raccolte nella classe, poi sarà un altro pezzo di programma ch andrà ad utilizzarle.

L'utilizzo da riga di comando, però, non deve essere interattivo, ma deve funzionare come un comando classico, tipo il type in Windows o il cat in Linux (solo per fare esempi, non dobbiamo implementare le stesse funzioni).

Ti consiglio di guardare la libreria standard di Python argparser che ha già tutte le funzioni che ti servono.

Resto a disposizione per ulterior chiarimenti, e cerco di rispondere prima questa volta.

ZompaSenior commented 1 year ago

Ciao, ho visto l'aggiornamento: domani sera aggiustiamo ed integriamo.

ZompaSenior commented 1 year ago

Per adesso ti respingo la Pull Request.

Verifica le istruzioni nella Issue.

Quando hai implemento la nuova versione fanne un'altra.

Se hai bisogno di chiarimenti, scrivi pure in risposta alla Issue.