Discussion :

[chrtophe]

Le code, même si il n'est pas propre, est efficace : on y échappe pas. Quand à savoir si la somme juste est prélevée, donc si l'algorithmie est correcte, il faudrait déjà pouvoir comprendre le calcul, avec des taux communaux, départementaux, des abattements obligatoires, facultatifs, calculé sur la moyenne des valeurs locatives de la commune, si on y a droit : une vraie usine à gaz un truc d'énarque.

Quand à reprendre celui-ci, la taxe d'habitation devrait être supprimée, enfin c'est pas sûr, on sait pas trop. J'aimerais autant que ma part de pizza (pour ne pas dire pizzo) ne soit pas employée à refaire une taxe usine à gaz dopnt la suppression est envisagée, mais c'est beau de rêver.

[byrautor]

Ce qui caractérise la valeur d'un code est sa redondance.
ce mot est honni des littéraires qui ne font pourtant que l'utiliser plus qu'abondamment.
Pour ce que j'en ai appris : c'est la faculté d'un langage d'exprimer la même "chose" de différentes façons pour parvenir au même résultat !
C'était la difficulté du latin qui a provoqué (entre autres) l'apparition du français.
Revenez à la tirade célèbre du Bourgeois Gentilhomme :
Marquise vos beaux yeux me font mourir d'amour
D'amour vos beaux yeux marquise me font mourir
Vos beaux yeux d'amour marquise me font mourir
etc..etc ..
Je crois qu'en anglais il n'y a pas la même souplesse d'expression blue sky # sky blue
Bref, deux programmeurs, avec le même langage ne produiront pas le même code
d'où la difficulté de les relire et de les comparer (les lignes de codes ; pas les programmeurs, automatiques ou humains)
Vive la diversité !

[koala01]

Salut,

Je n'ai pas tout lu, ni dans le code source, ni dans les commentaires de cette discussion...

Mais quand je lis (lignes 54 à 58 de TH-7KBNC.C)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
                         /*=========*/
                         /*   main  */
                         /*=========*/

int th_7kbnc(s_bne1 * p_bne1, s_bne2 * p_bne2, s_bns * p_bns)

je ne peux m'empêcher de me demander si c'est une blague!!!

Au vu du commentaire, je me serais plutôt attendu à -- je sais pas, moi -- une fonction int main(int argc, char ** argv) par exemple

[c_majed]

la méthode cherche_const est juste une blague en soit!!!

[Pyramidev]

À propos de la fonction int cherche_const(short annee,s_cons * * p_cons) de TH-7KARC.C, je rebondis là-dessus :

Pour revenir au quart de l'appli moche que je n'ai pas refactoré : va expliquer à un financier que tu vas prendre 6 mois pour permettre la maintenance facile d'un composant qui a zéro maintenance depuis 20 ans, et que comme personne ne comprend ce que ça fait, le risque que ça ne marche plus est fort.

Avec le code actuel, la fonction cherche_const doit être remise à jour chaque année.

En effet, si on regarde le code source de la fonction cherche_const, on voit qu'elle définit un tableau de 4 objets de type s_cons, chacun correspondant à une année parmi 2017, 2016, 2015 et 2014. Ensuite, à partir du paramètre annee, elle recherche l'objet correspondant. Si elle le trouve, alors elle édite le pointeur *p_cons pointé par p_cons pour que *p_cons pointe vers l'objet trouvé.
Cette fonction cherche_const fait 787 lignes et joue le rôle d'une mini-base de données. Toutes les valeurs contenues dans ces objets de type s_cons sont écrites en dur dans le code source.

Cette fonction doit évoluer chaque année, puisque chaque objet de type s_cons correspond à une seule année.

Normalement, ce genre d'information devrait être centralisé dans un fichier dont le format est facile à parser (par exemple XML ou JSON) et tous les programmes qui en auront besoin, dont ce programme en C, devraient lire ce fichier. Cela éviterait de perdre du temps chaque année à recopier bêtement les valeurs en dur dans le code source avec les risques d'étourderie que cela entraîne.

[koala01]

Faudrait presque prendre contact avec le ministère concerné pour leur proposer de réécrire ce code

[Marco46]

Si seulement ils savaient utiliser git et animer une communauté t'aurais forcément quelqu'un pour proposer une PR gratos ...

[chrtophe]

Pour les aider à nous détrousser ? JAMAIS!!!

[el_slapper]

Pour les aider à nous détrousser ? JAMAIS!!!

Déjà dit ailleurs, mais ce détroussage est bien pratique quand on commence à avoir besoin de la toute puissance de l'état(soucis de santé, par exemple).

(.../...)

ça, c'est différent. La maintenance est à faire de toutes façons. Dans ce cas, je suis bien d'accord : à la prochaine maintenance, autant faire propre. Sauf si changer un fichier de référence est plus couteux que changer un programme(ne pas rigoler, j'ai vu ça, chez un grand compte financier - ils avaient tellement blindé les fichiers de paramètres que pour changer une table de paramètres, le délai était de 3 semaines. Je pouvais livrer une valeur en dur dans un programme en quelques heures. Quand des contraintes aberrantes de ce type sont en vigueur, ne pas s'étonner de choix techniques qui paraissent aberrants.....)

Non, ce que je disais, c'est que si la fonction marche et n'a pas besoin de ce genre de maintenance, alors ce n'est pas la priorité de la refactoriser. Si il faut tout le temps repasser dessus(ce qui semble être le cas de ton exemple), autant repasser dessus de manière définitive, on est d'accord. J'ai fait ça un certain nombre de fois. Mais l'occasion fait le larron.

[koala01]

Déjà dit ailleurs, mais ce détroussage est bien pratique quand on commence à avoir besoin de la toute puissance de l'état(soucis de santé, par exemple).

Le problème, c'est que le détroussage permet aussi de payer des salaires mirobolant à un nombre bien trop élevé de politicards qui ne le méritent pas forcément.

Je ne dis pas ici que tous sont incompétents / corrompus / des voleurs / what ever. Je dis juste qu'il y en a trop, et que certains (qui font partie d'une voir de plusieurs des catégories citées) ont trouvé la "bonne planque".

Je dis "juste" que, si "les politiciens" (allez, certains policiticiens, pour éviter les généralités qui seraient forcément fausses) étaient "un peu plus raisonnables" dans le nombre de niveau qu'ils ont créé pour -- trop souvent -- permettre de "pousser leurs potes à une bonne place", et / ou vis-à-vis des avantages qu'ils s'octroient --durant leur mandat ou à l'issue de celui-ci -- "les finances" pourraient être "un peu plus raisonnable" sur les montants réclamés de toutes parts et / ou que ces montants pourraient être utilisés "de manière plus judicieuse ou plus utiles".

[Marco46]

Sympa non ?

Vous la sentez la puissance de l'économie collaborative ?

On croit rêver sérieux

[chrtophe]

Le problème, c'est que le détroussage permet aussi de payer des salaires mirobolant à un nombre bien trop élevé de politicards qui ne le méritent pas forcément.

C'est ce que je pense, je ne suis pas contre l"impôt, mais contre sa mauvaise utilisation :
- payer un rapport à 200000 €
- construire un pont qui ne sert à rien, qui sera probablement détruit un jour toujours à nos frais - exemple concret dans ma ville.

La taxe d'habitation a augmenté de 15% en moyenne je pense ces dernières années, j'ai pas 15% de service en plus, c'est plutôt l'inverse. Et je trouve que c'est un impôt injuste, devoir payer pour habiter quelque part... d'autant plus injuste que dans certains cas, les habitants de ville dites riches payent moins de taxe d’habitation que les habitant de villes dites pauvres.

[halaster08]

La taxe d'habitation a augmenté de 15% en moyenne je pense ces dernières années, j'ai pas 15% de service en plus, c'est plutôt l'inverse.

Ces dernières années l'Etat s'est déchargé de beaucoup de choses sur les collectivité locales (décentralisation), sans leur donner plus de budget (ou en tout cas pas le budget nécessaire, ce qui permettait à l'Etat d'avoir un budget "à l'équilibre"), ils n'ont pas le choix.

c'est un impôt injuste, devoir payer pour habiter quelque part...

C'est pas pour habiter que tu payes, mais pour des trucs locaux comme l'entretien des routes par exemple.

d'autant plus injuste que dans certains cas, les habitants de ville dites riches payent moins de taxe d’habitation que les habitant de villes dites pauvres.

Là par contre je te rejoins complètement.

[chrtophe]

Ces dernières années l'Etat s'est déchargé de beaucoup de choses sur les collectivité locales (décentralisation), sans leur donner plus de budget

oui, ils ont baissé les impôts pour augmenter les taxes. Pour le transfert de charges ok, mais qu"on ne dise pas dans ce cas que les impôts ont baissé.

C'est pas pour habiter que tu payes, mais pour des trucs locaux comme l'entretien des routes par exemple.

Bien sûr. Sur le principe, oui, sauf que dans le concret, il y a de plus en plus de nids de poule sur les routes.


Cette taxe est en cours de suppression de toute façon (du moins si ça change pas).

[koala01]

Cette taxe est en cours de suppression de toute façon (du moins si ça change pas).

Et quand bien même elle serait supprimée, ils vont en trouver deux autres pour la remplacer, et cela reviendra encore 50% plus cher