carlospata
commented
2 years ago Ciao @Helias, ho provato a inserire tutto quello che hai scritto tra cui pylint, di cui il test falliva anche perchè ad esempio mi segnalava come errore la riga vuota, ma personalmente la uso per un fatto di leggibilità. Comunque uno dei problemi che ho riscontrato è stata la comunicazione tra i vari file e le varie classi che spesso davano errore -> "Circular Import" (ex: class1 importa class2 che a sua volta importa class1). Volevo capire se c'è un modo pulito per risolvere questo problema oppure in questo caso è sbagliata la progettazione del codice.
Helias
Da due studenti del primo anno non mi aspettavo un progetto simile.
Non avrei alcun appunto da fare anche perchè non mi aspetto tanta professionalità da studenti della triennale.
Bravissimi! 🚀 🥇
Se vi siete divertiti e vi va di migliorare tecnicamente questo progetto al fine di imparare potreste:
Una cosa che si impara solo col tempo e "masterando" il lingauggio di programmazione python è la qualità del codice, su quella avrei degli appunti da fare che saranno probabilmente simili anche a pylint, ma non mi aspetto da nessuno di scrivere codice di qualità (nemmeno da me stesso 😁 ). Avete scritto tante funzioni con
if, elife deireturn, in realtà quando in un if è scritto un return si può scrivere direttamenteifal posto dielifper migliorare leggibilità e forse un po' di efficienza nel codice, inoltre, nella funzionebasal_metabolism(è presente una variabile res che si potrebbe omettere, al posto dell'assegnazioneres =è possibile mettere direttamentereturn ...in modo da risparmiare una variabile e quindi della memoria.Comunque, tutte queste ottimizzazioni si possono fare automaticamente usando pylint come linter, il vostro progetto mi è piaciuto tantissimo, ma sappiate che si può sempre imparare e migliorare.
⭐ ⭐ ⭐ ⭐ ⭐ 🚀