Jag sitter och skriver mitt examensarbete. Går ut på att kvalitetsgranska VB-kod. Skall alltså hitta punkter som bör granskas i ett VB-program. Skulle gärna vilja höra åsikter om vad ni tror kan vara viktigt att granska???

1. Kommentarer (Skall vara vettiga)
Man skall alltid kommentera trix som man gjort.
Man skall aldrig göra kommentarer på kod som förklarar sig själv.

Exempel: Denna rad skriver ut hej. Detta är en värdelös kommentar

2. Man skall alltid skriva så självförklarande kod som möjligt.

Exempel: Om man vill kolla om ett värde är sant kan man skriva

If värde then
'lite kod
End if

Detta försvårar läsning av koden.

Du bör alltid skriva:

If värde = annat värde then
'lite kod
end if

3. Om vi exvis fortsätter med If-satser förspråkar jag att man alltid
skriver ut hela "strukturen" typ så här

If A=B then
'lite kod
Else
End if

Skriva ut Else trots att det inte finns kod där. Gärna med en
kommentar då att vid Else så sker inget. (Detta underlättar om
man har stora nästlade If-satser)

4. I stora nästlade If-satser är det alltid bra att kommentera sin End If

If A=B then
'lite kod
End if 'A=B

5. En header skall alltid finnas i varje program, funktion, modul, klass

I klasser, formulär och kodmoduler skall det alltid framgå längst upp
följande saker.

1. Vad heter programmet, formuläret, klassen, modulen
2. Vad har den för avsikt
3. Vem har skapat den
4. När skapades den
5. Logg med datum för senaste åtgärd och vilken åtgärd som gjordes
6. Copyrightinfo

I en funktion/sub skall man alltid ange vad som gäller före det att
funktionen exekverats och efter det att den exekverats.

Exempel:

Public Sub times10(S as integer)
före: S är ett tal
efter: S är en faktor 10 större
S = S * 10
End Sub

Hoppas ni fått lite tips. OBSERVERA att detta är mina personliga åsikter
andra kanske har olika uppfattning och kanske till och med kommer att
gnälla på mina åsikter.

/peterh

Hej
Instämmer i peterh:s inlägg

Det får inte finnas någon odeklarerad varaiabel
Option Explicit skall användas.

Det skall finnas väl utvecklad ErrorHandler.

Programmet skall kunna användas utan mus i alla delar.

osv osv.

mvh
Sven

(Bortaget för att göra SvenPon glad)

Hej

Om det finns inmatningsfönster så bör man "Validera"(bekräfta) med Enter
Om det då finns ett nytt logiskt inmatningsfönster bör Cursorn
flytta dit.

mvh
Sven