Le principe de base de programmation est de rester « DRY » (Don’t Repeat Yourself). Le code ci-dessus se repete pour rien.

« Outre la surabondance de variables causant une lecture difficile du code et une difficulté supplémentaire pour le maintenir ».

Le code ci-dessus est encore moins lisible qu’une longue liste de variables et bien plus dangeureux. Si d’autres developpeurs travaillent sur ce projet ils n’auraient pu imaginer qu’un « petit malin » s’est amuse a declarer des variables en plein milieu de la fonction.

« créant des risques de confusion, d’effets de bord non prévus et donc des risques de dysfonctionnement, de comportements non prévus ou alors un plantage intégral de l’application. »

Ce qui correspond exactement au code presente ci-dessus.

Si vous voulez voir a quoi un bon code C ressemble, jetez un coup d’oeil sur la source de Quake3. (https://github.com/id-Software/Quake-III-Arena). Vous pourrez voir qu’il doit y’avoir maximum 6 declaration de variables dans une fonction.