Advertisement
Not a member of Pastebin yet?
Sign Up,
it unlocks many cool features!
- Indentazione : 6/10
- Nomenclatura : 7/10
- Interpolazione di stringhe non necessaria.
- I for sulle linee 168 e 227 non è molto intelligente nè elegante.
- L'assenza di warnings (ma vedo che comunque hai rispettato i canoni imposti dal sopracitato flag)
- Poi...
- [code]
- foreach (@posti_sed_spett1) {
- print;
- print " ";
- }
- [/code]
- e
- [code]
- foreach (@posti_sx_spett2) {
- print;
- print " ";
- }
- [/code]
- e ancora
- [code]
- foreach (@posti_sed_spett3) {
- print;
- print " ";
- }
- [/code]
- o usi un local per quella variabile globale o (ancora meglio) non la usi proprio
- Ci sono, inoltre, da constatare altre piccole imprecisioni come l'assenza di delimitatori intorno ai filehandle, l'utilizzo di croak per gli errori che avrebbe reso la segnalazione di quest'ultimi più efficace e l'assenza della documentazione dello script.
- Non mi sembra ci sia altro se ho tralasciato qualcosa fatemelo notare...
- Voto totale: 6.5 (volendo essere abbastanza largo di maniche a mio avviso)
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement