Le virgole finali in Perl sono una cattiva pratica?

Question

Oggi ero in una riunione Webex che mostrava il mio schermo con un codice Perl che ho scritto. Il mio capo mi ha detto improvvisamente mentre tutti gli altri guardavano e sentivano che dovevo rimuovere le virgole finali dal mio hash e dalle strutture dell'array perché è una cattiva pratica. Ho detto che non pensavo che fosse una cattiva pratica in Perl, ma lui ha insistito e mi ha fatto cancellare quelle virgole solo per mostrare il mio copione durante la riunione.

Continuo a pensare che non sia una cattiva pratica in Perl, ma posso sbagliarmi. In realtà li trovo piuttosto convenienti e una buona pratica perché mi impediscono di aggiungere nuovi elementi e di dimenticare di aggiungere la virgola corrispondente nel processo.

Ma, mi piacerebbe davvero sapere se è una buona o cattiva pratica ed essere in grado di mostrarlo al mio capo (se ha torto) con buoni argomenti e anche buone fonti per i miei argomenti.

Quindi, è una cattiva pratica lasciare le virgole finali?

Questo è un esempio:

my $hash_ref = { 
    key1 => 'a', 
    key2 => 'b', 
    key3 => 'c', 
}; 

my $array_ref = [ 
    1, 
    2, 
    3, 
]; 

Answer 1

È una buona pratica avere la virgola finale. Larry l'ha aggiunto a causa di un errore comune che ha visto fare ai programmatori. Quando hanno aggiunto elementi a una lista (o qualunque sia stata definita dalla loro lingua), hanno dimenticato il carattere separatore. Perl consente la virgola finale di proposito. Non è una stranezza o un effetto collaterale di qualcos'altro.

Ciò che è una cattiva pratica, tuttavia, sta distragendo un incontro pieno di persone con qualcosa che il tuo capo avrebbe potuto correggere in seguito. A meno che l'incontro non sia stato specificamente una revisione del codice, il tuo capo ha perso un sacco di tempo. Ho sempre desiderato che, per partecipare a una videoconferenza, dovessi inserire la tua compensazione per minuto in modo che un contatore potesse apparire sullo schermo di tutti per mostrare quanti soldi venivano sprecati. Spendere un paio di centinaia di dollari per guardarti togliere le virgole su un programma di lavoro ridurrebbe quelle sciocchezze.

Answer 2

L'Apache mod_perl coding style document dice che questo

Ogni volta che si crea un elenco o una matrice, sempre aggiungere una virgola dopo l'ultimo elemento. La ragione per fare questo è che è molto probabile che i nuovi articoli verranno aggiunti alla fine della lista in futuro. Se manca la virgola e questo non viene notato, si verificherà un errore.

Ciò che il tuo manager potrebbe aver pensato è che fare la stessa cosa in C è non standard e non portatile, tuttavia non ci sono scuse per il suo comportamento straordinario.

Answer 3

Quindi la pagina PBP a cui fa riferimento Miller sostiene che è più semplice riordinare l'elenco tagliando e incollando le righe; il documento di stile mod_perl collegato da Borodin sostiene di evitare un errore di sintassi momentaneo quando si aggiungono cose.

Molto più significativo di entrambi, a mio parere, è che se si dispone sempre di una virgola finale e si aggiunge una linea, la differenza mostra solo la riga aggiunta e le linee esistenti rimangono invariate. Questo rende la ricerca della colpa migliore e rende le differenze più leggibili.

Answer 4

Preferisco virgole in testa, anche se so che è piuttosto impopolare e sembra irritare il dislessico. Anche io non sono stato in grado di trovare un'opzione perltidy per questo. Risolve anche il problema del cambio di riga (tranne che per la prima riga, ma di solito non è quello che viene modificato nella mia esperienza), e adoro il modo in cui le virgole si allineano in colonne ordinate. Funziona anche in linguaggi che sono agnostici nello spazio bianco ma non amano elencare virgole tra virgolette sulle liste in modo abbastanza preciso. Penso di aver imparato questo modello mentre lavoravo con javascript ...

my $hash_ref = 
    { key1 => 'a' 
    , key2 => 'b' 
    , key3 => 'c' 
    }; 

my $array_ref = 
    [ 1 
    , 2 
    , 3 
    ]; 

Answer 5

È davvero una buona pratica e menzionata anche nel famoso PBP.

V'è in realtà una politica per la perlcritic che mi fa sempre: https://metacpan.org/pod/Perl::Critic::Policy::CodeLayout::RequireTrailingCommas