Advertisement
Guest User

Untitled

a guest
Jun 23rd, 2017
54
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
text 1.00 KB | None | 0 0
  1. Indentazione : 6/10
  2. Nomenclatura : 7/10
  3. Interpolazione di stringhe non necessaria.
  4. I for sulle linee 168 e 227 non è molto intelligente nè elegante.
  5. L'assenza di warnings (ma vedo che comunque hai rispettato i canoni imposti dal sopracitato flag)
  6.  
  7. Poi...
  8.  
  9. [code]
  10. foreach (@posti_sed_spett1) {
  11. print;
  12. print " ";
  13. }
  14. [/code]
  15.  
  16. e
  17.  
  18. [code]
  19. foreach (@posti_sx_spett2) {
  20. print;
  21. print " ";
  22. }
  23. [/code]
  24.  
  25. e ancora
  26.  
  27. [code]
  28. foreach (@posti_sed_spett3) {
  29. print;
  30. print " ";
  31. }
  32. [/code]
  33.  
  34. o usi un local per quella variabile globale o (ancora meglio) non la usi proprio
  35.  
  36. 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.
  37.  
  38. Non mi sembra ci sia altro se ho tralasciato qualcosa fatemelo notare...
  39.  
  40. Voto totale: 6.5 (volendo essere abbastanza largo di maniche a mio avviso)
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement