feat: Implementation Risiko

[NuccioDe1]

No description provided.

[Picred]

Evita di usare variabili globali perchè sono "rischiose" e ci si può accedere da qualsiasi punto del codice. Piuttosto potresti creare questi array nel main e passarli alla funzione compare, tipo:

// Definisco la costante della dimensione degli array
#define DIM 3
void compare (int* attacker_Dices, int* defender_Dices){
    //...
}

//...

int main(){
    int* attacker_Dices = new int[DIM];
    int* defender_Dices = new int[DIM];
    //...
    compare(attacker_Dices, defender_Dices);
}

E ovviamente si dovrebbe adattare il codice con la costante definita DIM che incrementa la leggibilità del codice.

[Picred]

Potresti eliminare questo for e utilizzare il precedente per fare le operazioni sul secondo array visto che esse sono indipendenti fra loro, quindi evitare una serie di cicli.

[Picred]

Consiglio di mettere l'istruzione di return del main e di rimuovere eventuali spazi bianchi (o ritorni a capo) che allungano il codice, quindi migliorare l'indentazione per migliorare la leggibilità del codice.

[Picred]

Potresti mettere attacker_Dices[i] = rand() % 6; piuttosto che lasciare esplicitamente 5+1