====== Checklist Qualità Codice ======

Verifiche per convenzioni di nomenclatura, design delle funzioni, principio DRY e SSOT.

<note>Questa checklist integra le [[.:code-konventionen|Convenzioni di Codice]] con punti di verifica concreti.</note>

===== Convenzioni di Nomenclatura =====

==== Pattern Namespace (Stile MS) ====

<code>
[ ] Nomi unit seguono WvdS.<Categoria>.<Componente>.pas
[ ] Rispecchia la struttura Microsoft .NET con prefisso WvdS.
[ ] Esempio: WvdS.UI.Controls.Button, WvdS.System.Logging
</code>

==== Gerarchia Classi (Stile Borland) ====

<code>
[ ] Base astratta usa prefisso TWvdSCustom*
[ ] Base pubblica usa prefisso TWvdSBase*
[ ] Classi concrete usano prefisso TWvdS*
[ ] Pattern: TWvdSCustomXxx -> TWvdSBaseXxx -> TUserXxx
</code>

==== Prefissi Tipo ====

^ Categoria ^ Prefisso ^ Esempio ^
| Classi | ''TWvdS*'' | ''TWvdSRouter'', ''TWvdSButton'' |
| Interfacce | ''IWvdS*'' | ''IWvdSActionResult'', ''IWvdSBindable'' |
| Eccezioni | ''EWvdS*'' | ''EWvdSRoutingException'' |
| Record | ''TWvdS*'' | ''TWvdSBuildResult'' |
| Enum | ''TWvdS*'' | ''TWvdSProjectType'' |
| Callback | ''TWvdS*'' | ''TWvdSBuildCallback'' |

==== Nomenclatura Generale ====

<code>
[ ] Nomi descrittivi (nessuna singola lettera tranne cicli)
[ ] Nessuna abbreviazione tranne quelle conosciute (URL, HTTP, JSON)
[ ] Interfacce hanno GUID per container DI
[ ] Campi privati: prefisso F* (FValue, FCount)
[ ] Parametri: prefisso A* (AIndex, APath)
</code>

===== Design Funzioni =====

==== Linee Guida Dimensioni ====

<code>
[ ] 12-24 righe ideale
[ ] Massimo 40 righe (oltre dividere)
[ ] Profondità massima annidamento: 3
</code>

==== Linee Guida Parametri ====

<code>
[ ] Massimo 5-7 parametri
[ ] Con più parametri: usare Record/Classe
[ ] Parametri opzionali alla fine
</code>

==== Single Responsibility ====

<code>
[ ] Ogni funzione ha UN compito
[ ] Il nome funzione descrive completamente il compito
[ ] Se serve "e" nel nome -> dividere
</code>

==== Naming Verb-First ====

<code>
[ ] Get* - Recuperare valore
[ ] Set* - Impostare valore
[ ] Calculate* - Eseguire calcolo
[ ] Process* - Eseguire elaborazione
[ ] Validate* - Eseguire validazione
[ ] Create* - Creare oggetto
[ ] Parse* - Analizzare input
[ ] Format* - Formattare output
</code>

===== Principio DRY =====

<code>
[ ] Nessun codice duplicato (più di 3 righe identiche -> estrarre)
[ ] Nessun pattern copia-incolla
[ ] Logica comune in funzioni helper
[ ] Costanti invece di letterali ripetuti
</code>

==== Esempio: Correggere Violazione DRY ====

<code pascal>
(* VIETATO - Codice duplicato *)
procedure ProcessUserA(const AUser: TUser);
begin
  if not ValidateEmail(AUser.Email) then
    raise EValidationError.Create('Invalid email');
  if Length(AUser.Name) > 64 then
    raise EValidationError.Create('Name too long');
  (* ... ulteriore elaborazione *)
end;

procedure ProcessUserB(const AUser: TUser);
begin
  if not ValidateEmail(AUser.Email) then
    raise EValidationError.Create('Invalid email');
  if Length(AUser.Name) > 64 then
    raise EValidationError.Create('Name too long');
  (* ... altra elaborazione *)
end;

(* CORRETTO - Logica comune estratta *)
procedure ValidateUser(const AUser: TUser);
begin
  if not ValidateEmail(AUser.Email) then
    raise EValidationError.Create(rsInvalidEmail);
  if Length(AUser.Name) > MAX_NAME_LENGTH then
    raise EValidationError.Create(rsNameTooLong);
end;

procedure ProcessUserA(const AUser: TUser);
begin
  ValidateUser(AUser);
  (* ... ulteriore elaborazione *)
end;
</code>

===== Verifica SSOT =====

<note warning>**Single Source of Truth** - Nessuna copia locale di codice comune!</note>

==== Usare Librerie Common ====

<code>
[ ] API NodeJS via ~/sources/common/web/nodejs/
    - NodeJS.FS, NodeJS.Path, NodeJS.ChildProcess
[ ] API VSCode via ~/sources/common/web/vscode/
    - VSCode.Window, VSCode.Commands, VSCode.Workspace
[ ] WvdS Core via ~/sources/common/core/
    - WvdS.System.Logging, WvdS.VSCode.Security
</code>

==== Vietato ====

<code>
[ ] Nessuna chiamata require() diretta nelle Extension
[ ] Nessun duplicato di unit Common nelle Extension
[ ] Nessuna implementazione wrapper locale
</code>

==== Esempio: Violazione SSOT ====

<code pascal>
(* VIETATO - require() locale *)
function FileExists(const APath: string): Boolean;
begin
  asm
    var fs = require('fs');  (* Violazione SSOT! *)
    Result = fs.existsSync(APath);
  end;
end;

(* CORRETTO - Libreria Common *)
uses
  NodeJS.FS;

function FileExists(const APath: string): Boolean;
begin
  Result := ExistsSync(APath);  (* Da NodeJS.FS *)
end;
</code>

===== Separation of Concerns =====

<code>
[ ] Logica UI solo in strato UI (extension_main, *Dialog)
[ ] Logica business solo in strato Service (*Service)
[ ] Strutture dati solo in strato Model (*Models)
[ ] Services non chiamano API VSCode direttamente
[ ] Models non contengono logica
</code>

==== Architettura a Strati ====

<code>
+-------------------------------------------------+
|  UI Layer (extension_main.pas, *Dialog.pas)     |
|  -> API VSCode, Interazione Utente              |
+-------------------------------------------------+
|  Service Layer (*Service.pas)                   |
|  -> Logica Business, Validazione                |
+-------------------------------------------------+
|  Model Layer (*Models.pas)                      |
|  -> Record, Tipi, Enum                          |
+-------------------------------------------------+
|  Infrastructure (NodeJS.*, VSCode.*, WvdS.*)    |
|  -> API Esterne, Astrazione Piattaforma         |
+-------------------------------------------------+
</code>

===== Checklist Rapida da Copiare =====

<code>
Checklist Review (Qualità Codice):

Naming:
- [ ] Nomi unit: WvdS.<Categoria>.<Componente>.pas
- [ ] Classi: prefisso TWvdS*
- [ ] Interfacce: prefisso IWvdS*
- [ ] Nomi descrittivi (nessuna abbreviazione)

Funzioni:
- [ ] 12-24 righe ideale, max 40
- [ ] Max 5-7 parametri
- [ ] Single Responsibility
- [ ] Naming Verb-First

DRY:
- [ ] Nessun codice duplicato
- [ ] Logica comune estratta
- [ ] Costanti invece di letterali

SSOT:
- [ ] Librerie Common usate
- [ ] Nessuna chiamata require() locale
- [ ] Nessun duplicato di unit Common

SoC:
- [ ] UI solo in strato UI
- [ ] Logica solo in strato Service
- [ ] Services senza chiamate VSCode dirette
</code>

===== Vedi anche =====

  * [[.:qualitaetssicherung|Panoramica Garanzia Qualità]]
  * [[.:code-konventionen|Convenzioni di Codice]]
  * [[.:audit-core|Checklist Core]]
  * [[.:extension-entwicklung|Sviluppo Extension]]