stephabauva / alyra Goto Github PK

Waaaa

T'es allé beaucoup plus loin que ce que j'attendais. L'utilisation d'un tableau pour les workflow, accounts, proposals en js est super bien vue, pour boucler et verifier que tout passe. Ton code est parfaitement organisé et descriptif. Tu as meme fait le cas d'égalité.

Tu as du passer beaucoup de temps dessus. C'est vraiment un super travail. Bravo.

10/10

admin =owner ? tu n'as pas de fonction pour set l'admin, du coup ça sert a rien de le lancer en constructeur, autant utiliser le modifier onlyOwner

event whitelisted fait doublon avec le registered voter, et il est au milieux de plusieurs declaration de variable (et non utilisé)

Si tu as besoin d'indexer tes proposals, autant utiliser un tableau Proposal[] proposals

on aligne les variables de meme types ensemble (pour une meilleure allocation mémoire)

C'est bien d'avoir automatisé le changement de status (mais faut faire gaffe, c'est pas forcément toujours pertinent)

addproposal emet un évènement qui devrait etre: emit ProposalRegistered(proposalsCount-1);

Tes getters et quelques fonctions peuvent etre external pour couter moins cher

8/10