C'era una domanda che confrontava PMD e CheckStyle . Tuttavia, non riesco a trovare una bella ripartizione delle differenze / somiglianze tra PMD e FindBugs. Credo che una differenza fondamentale sia che PMD funziona sul codice sorgente, mentre FindBugs funziona sui file bytecode compilati. Ma in termini di capacità, dovrebbe essere una scelta o / o o si completano a vicenda?
Risposte:
Sto usando entrambi. Penso che si completino a vicenda.
Come hai detto, PMD lavora sul codice sorgente e quindi trova problemi come: violazione delle convenzioni di denominazione, mancanza di parentesi graffe, controllo null fuori posto, lungo elenco di parametri, costruttore non necessario, interruzione mancante nell'interruttore, ecc. PMD ti parla anche di Cyclomatic complessità del tuo codice che trovo molto utile (FindBugs non ti parla della complessità Cyclomatic).
FindBugs funziona su bytecode. Ecco alcuni problemi che FindBugs trova che PMD non riesce a trovare: il metodo equals () fallisce sui sottotipi, il metodo clone può restituire null, confronto di riferimento di valori booleani, cast impossibile, int a 32 bit spostato di una quantità non compresa nell'intervallo 0-31, una raccolta che contiene se stessa, il metodo equals restituisce sempre true, un ciclo infinito, ecc.
Di solito ognuno di loro trova un diverso insieme di problemi. Usali entrambi. Questi strumenti mi hanno insegnato molto su come scrivere un buon codice Java.
La caratteristica migliore di PMD sono le sue regole XPath , in bundle con un Rule Designer per consentire di costruire facilmente nuove regole da esempi di codice (simili a RegEx e XPath GUI builder). FindBugs è più forte fuori dagli schemi, ma la costruzione di regole e modelli specifici del progetto è molto importante.
Ad esempio, ho riscontrato un problema di prestazioni che coinvolgeva 2 cicli for nidificati, risultando in un tempo di esecuzione O (n ^ 2), che potrebbe essere facilmente evitato. Ho usato PMD per costruire una query ad-hoc , per rivedere altre istanze di cicli for nidificati - // ForStatement / Statement // ForStatement. Ciò ha evidenziato altri 2 casi del problema. Questa non è una regola generica di sorta.
PMD è