feat: implementing binary_converter #308
Conversation
There was a problem hiding this comment.
In linea generale è ok. Prova ad incapsulare il codice che fa l'operazione di conversione in una funzione apparte, passandogli i parametri che hai preso in input.
Dal punto di vista della semantica del codice avrebbe più senso usare un nome tipo "decimal_input" o qualcos'altro a tua scelta. Se utilizzi il termine "primo valore" può essere fuorviante perché idealmente chiunque si aspetterebbe di dover aggiungere un secondo o un terzo valore. Ovviamente su un codice di queste dimensioni è una sciocchezza capire il significato ma è sempre bene pensare anche a chi, da esterno, legge il nostro codice. Ti tornerà utile questo approccio quando scriverai da te o lavorerai su progetti più grandi.
La pipeline segna un errore nello stile, credo possa essere qualcosa dovuto agli spazi a fine riga (dipende dalla config che ha dato Stefano credo). Prova a vedere se è questo ed eventualmetne correggi.
Prendi inoltre la buona abitudine di scrivere qualche riga di commento per la descrizione della PR dove spieghi brevemente cosa fa il tuo codice
Grazie dei tuoi preziosi suggerimenti! |
No description provided.