{"id":27060,"date":"2015-03-30T17:07:28","date_gmt":"2015-03-30T16:07:28","guid":{"rendered":"https:\/\/artigianodelsoftware.wordpress.com\/?p=27060"},"modified":"2015-03-30T17:07:28","modified_gmt":"2015-03-30T16:07:28","slug":"fisheye-crucible-esempio-di-una-code-review","status":"publish","type":"post","link":"https:\/\/artigianodelsoftware.it\/blog\/2015\/03\/30\/fisheye-crucible-esempio-di-una-code-review\/","title":{"rendered":"FishEye & Crucible – Esempio di una Code Review"},"content":{"rendered":"
\"image_pdf\"<\/a><\/a><\/div>
\n

Code Review<\/h1>\n<\/blockquote>\n

In questo post, vedremo un esempio di utilizzo di FishEye e di Crucible, per realizzare una Code Review.<\/p>\n

\"\"<\/a><\/p>\n

\n

Definizioni<\/h1>\n<\/blockquote>\n

Partiamo, come sempre, dalle definizioni. La prima definizione che segnalo \u00e8 quella di wikipedia<\/a>, che ritengo abbastanza completa (in inglese). \u00a0Una Code Review fondamentalmente \u00e8 una analisi critica del codice, il cui obbiettivo \u00e8 quello di determinare eventuali problemi o errori o possibili punti in cui si pu\u00f2 migliorare il codice.\"\"<\/p>\n

Mi permetto di evidenziare la parola\u00a0eventuali<\/strong><\/em> , in quanto non \u00e8 detto che quanto rilevato in una Code Review sia effettivamente un errore. Infatti (faccio appello alla mia esperienza), la soluzione adottata potrebbe essere dovuta alla situazione del momento o ad un particolare workaround adottato per poter risolvere il problema in una determinata emergenza. L’analisi deve essere critica ma deve essere fatta con giudizio e sopratutto con la …. testa :-D.<\/p>\n

 <\/p>\n

\n

\u00a0Iniziamo<\/h1>\n<\/blockquote>\n

Gi\u00e0 nei precedenti post dedicati a FishEye<\/a>, abbiamo visto come \u00e8 possibile poter navigare il codice direttamente da interfaccia Web.<\/p>\n

\"\"<\/a><\/p>\n

 <\/p>\n

I primi passi per poter iniziare una code Review sono i seguenti. Primo passo in assoluto \u00e8 quello di creare un progetto all’interno di Crucible<\/em><\/strong>. Premessa: Occorre disporre dei privilegi di amministratore \ud83d\ude42<\/p>\n

Dal Cog\u00a0<\/em>men\u00f9, come indicato in figura:<\/p>\n

\"Cog\"<\/p>\n

selezionare\u00a0Administration<\/em>. Quindi, nella sezione\u00a0Project Settings<\/em>, selezionare\u00a0Projects<\/em>. Quindi Dare New Project.<\/em><\/p>\n

\"\"<\/a><\/p>\n

 <\/p>\n

Se si conosce come definire i progetti sotto\u00a0JIRA<\/em><\/strong>, allora risulter\u00e0 molto semplice gestire i progetti sotto Crucible<\/em><\/strong>. La logica \u00e8 la medesima. Si definisce una KEY<\/em>, utilizzata per numerare le segnalazioni di Code Review.<\/p>\n

Una volta definito il progetto, possiamo andare ad iniziare la Code Review. Come? Semplicemente andando a spulciarci il codice e andando a controllarlo.<\/p>\n

\"\"<\/a><\/p>\n

 <\/p>\n

Semplicemente andiamo alla riga di codice che … attira la nostra curiosit\u00e0 \ud83d\ude42 e con un semplice click del mouse, andiamo ad inserire il nostro commento. Il codice non sar\u00e0 modificato: tutte queste informazioni saranno inserite nel db di FishEye<\/em><\/strong>. In questo modo possiamo iniziare una discussione che pu\u00f2 terminare con una richiesta di intervento, esattamente come viene fatto per JIRA<\/strong><\/em>.<\/p>\n

\"\"<\/a><\/p>\n

Da queste richieste \u00e8 possibile andare a generare la richiesta di intervento sotto JIRA. Se abbiamo eseguito le operazioni di\u00a0Application Link<\/em> tra JIRA<\/em> <\/strong>e FishEye<\/strong><\/em>, allora possiamo anche eseguire questa operazione, assegnando la modifica del codice.<\/p>\n

\"\"<\/a><\/p>\n

 <\/p>\n

Tutte le attivit\u00e0 sotto\u00a0Crucible<\/strong><\/em> sono comunque tracciate:<\/p>\n

\"\"<\/a><\/p>\n

 <\/p>\n

Ogni singolo commento viene tracciato e, in caso di necessit\u00e0, \u00e8 possibile risalire all’iter che ha portato alla modifica oppure alla mancata modifica del sorgente.<\/p>\n

\"\"<\/a><\/p>\n

 <\/p>\n

Tramite apposite Dashboard, abbiamo la possibilit\u00e0 di avere sempre la situazione completa delle Reviews in corso, quali chiuse, quali sono sfociate in segnalazioni JIRA<\/strong><\/em>, etc. \ud83d\ude42<\/p>\n

\n

Conclusioni<\/h1>\n<\/blockquote>\n

Abbiamo visto come lavorare sotto Crucible<\/em>, come gestire una Code Review e come poter sfruttare al meglio\u00a0FishEye<\/em> e come intergrarlo con i vari applicativi della\u00a0Atlassian<\/em>. Nei prossimi post andremo a visionare altre funzionalit\u00e0.<\/p>\n

\n

Riferimenti<\/h1>\n<\/blockquote>\n

Manualistica:<\/p>\n