mboost-dp1

www.politi.dk

Polsagskode er sjusk

- Via Version2 - , redigeret af Net_Srak

Rigspolitiet måtte afbryde udviklingen af politiets nye elektroniske sagsstyringssystem Polsag, der blev udviklet af CSC og ScanJour. Afbrydelsen blev baseret på en rapport fra Globeteams gennemgang af koden af Polsag.

Det er denne gennemgang af koden af Polsag, som Version2 nu har fået aktindsigt i. Rapporten giver et generelt negativt billede af kvaliteten af koden. Det gælder blandt andet kompleksiteten af koden, manglende dokumentation og manglende test.

Globeteam skriver, at kildekoden er for kompleks, der gør koden svær at teste, fejlsøge og vedligeholde.

Rapport om kildekoden til Polsag, Globeteam skrev:

18% af kildekoden til POLSAG-klienten er svær at fejlsøge, teste og vedligeholde, fordi den er for kompleks, og heraf er 31% procent meget kompleks og dermed meget svær at fejlsøge, teste og vedligeholde…Heraf har 56% af kildekoden (20% af kildekoden til POLSAG-serverapplikationen) en alt for stor kompleksitet, hvilket betyder, at den må betegnes som værende meget svær at fejlsøge teste og vedligeholde.

Rapporten kommer også ind på, at niveauet for unit tests har været meget lavt. Normalt udføres der 2-3 unit tests per metode, men niveauet for Polsag har været meget lavere. Desuden gør organiseringen af kildekoden det svært at teste den. Globeteam har brugt værktøjer til statistisk analyse af kildekoden og fundet, at kildekoden ikke overholder den almindelige praksis for navngivning, strukturering og organisering.

Rapporten kritiserer også dokumentationen for Polsag, hvor kildekoden indeholder meget få kommentarer ligesom de forretningsmæssige krav og ønsker kan ikke udledes fra designdokumentet.

Til sidst kritiserer rapporten også valget af sprog. Polsag-klienten er baseret på Javascript, der ifølge Globeteam har medført en større kompleksitet end andre sprog. Desuden er valget af Microsofts .NET 2.0 også kritiseret, da der allerede i 2010 var kommet en version 4.0 af .NET.





Gå til bund
Gravatar #51 - arne_v
7. jan. 2013 20:13
#50

Din lommeregner fejler ikke noget.

:-)

Men man kan bruge mange timer, hvis man virkeligt vil.

:-)

Der blev allerede frigivet noget materiale i sommers.

Og vi havde også en tråd om det her på newz.dk.

Og en af de slemme ting var sammenblandingen mellem POLSAG og standard Captia.

* de havde fået kildekode til Captia og rettede i den når det var "nødvendig" (rettelserne skulle så ryge tilbage til ScanJour og puttes ind i Captia officielt)

* samme namespace brugt for både POLSAG og Captia klasser

* ingen adskillelse af POLSAG og Captia database tabeller
Gå til top

Opret dig som bruger i dag

Det er gratis, og du binder dig ikke til noget.

Når du er oprettet som bruger, får du adgang til en lang række af sidens andre muligheder, såsom at udforme siden efter eget ønske og deltage i diskussionerne.

Opret Bruger Login