Logiciel de paie

Bonjour,

Etant débutant en Qt, autodidacte mais passionné, j'aimerai votre avis sur l'algorithme de calcul de paie que j'ai conçu pour une éventuelle amélioration, qui est fonctionnel et basé sur des formules. Les formules sont entrées au format texte dans un champ caractère et qui est interprété, On peut ajouter des conditions avec (condition: A,B), bon pour les calculs plafonnés ou autre. J'ai utilisé Qtscript pour cela.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 218 219 220 221 222 223 224 225 226 227 228 229 230 231

#include "calcul.h" #include "ui_calcul.h" #include <QString>   Calcul::Calcul(QWidget *parent) : QDialog(parent), ui(new Ui::Calcul) { ui->setupUi(this); }   Calcul::~Calcul() { delete ui; }   void Calcul::on_pushButton_clicked() {   ui->toscript->clear(); //QPlainTextEdit. ui->toresult->clear();//QPlainTextEdit. valeur.exec("delete from valeur"); QScriptValue result; QString formule, Scriptcode; //boucle sur le fichier des salariées. salarie.prepare(QString("select id_salarie,code_poste from salarie")); salarie.exec(); while (salarie.next()) { /* choix du code poste pou extraire le montant du salaire de base qui sera   mis dans la variable INDICE.*/   nomenclature.prepare( QString("select code_poste,indice from nomenclature where code_poste=:code")); nomenclature.bindValue(":code", salarie.value(1).toString()); nomenclature.exec(); nomenclature.first(); formule = "INDICE=" + nomenclature.value(1).toString() + ";"; ui->toscript->appendPlainText(formule.toUpper()); rubrique.prepare("select * from rubrique order by id_rubrique"); rubrique.exec();   // calcul de toutesles rubriques pour le salarié. while (rubrique.next()) { /*pour chaque mode de la rubrique faire un choix 0=rubrique directement calculée,   pour les modes 1,2,3,4 pointer dans le fichier des variables*/   int mod; mod = rubrique.value(3).toInt(); if (mod == 0) { formule = rubrique.value(4).toString() + ";"; ui->toscript->appendPlainText(formule.toUpper()); Scriptcode = ui->toscript->toPlainText(); result = engine.evaluate(Scriptcode); ui->toresult->appendPlainText(rubrique.value(1).toString() + "=" + result.toString());   //insertion dans le fichier des valeurs. valeur.prepare( "INSERT INTO VALEUR " "(id_salarie,id_rubrique,mois,annee,centre_cout,nombre,taux," "montant,sens," "affiche)" " VALUES (:id_sal,:id_rub,:mois,:annee,:cout,:nbre,:tx,:mont,:sens,:affiche)"); valeur.bindValue(":id_sal", salarie.value(0).toInt()); valeur.bindValue(":id_rub", rubrique.value(0).toInt()); valeur.bindValue(":mois", 10); valeur.bindValue(":annee", 2019); valeur.bindValue(":cout", 0); valeur.bindValue(":nbre", 0.00); valeur.bindValue(":tx", 0.00); valeur.bindValue(":mont", result.toNumber()); valeur.bindValue(":sens", rubrique.value(8).toString()); valeur.bindValue(":affiche", rubrique.value(9).toInt()); valeur.exec(); } if (mod == 1) { variable.prepare( "select * from variabl where id_salarie=:id_sal and id_rubrique=:id_rub "); variable.bindValue(":id_sal", salarie.value(0).toString()); variable.bindValue(":id_rub", rubrique.value(0).toString()); variable.exec(); variable.first();   QString formule = "NOMBRE=" + variable.value(4).toString() + ";"; ui->toscript->appendPlainText(formule.toUpper());   formule = rubrique.value(4).toString() + ";"; ui->toscript->appendPlainText(formule.toUpper()); Scriptcode = ui->toscript->toPlainText(); result = engine.evaluate(Scriptcode); ui->toresult->appendPlainText(rubrique.value(1).toString() + "=" + result.toString());   // Insertion dans le fichier valeur. valeur.prepare( "INSERT INTO VALEUR " "(id_salarie,id_rubrique,mois,annee,centre_cout,nombre,taux," "montant,sens," "affiche)" " VALUES (:id_sal,:id_rub,:mois,:annee,:cout,:nbre,:tx,:mont,:sens,:affiche)"); valeur.bindValue(":id_sal", salarie.value(0).toInt()); valeur.bindValue(":id_rub", rubrique.value(0).toInt()); valeur.bindValue(":mois", 10); valeur.bindValue(":annee", 2019); valeur.bindValue(":cout", 0); valeur.bindValue(":nbre", variable.value(4).toDouble()); valeur.bindValue(":tx", 0.00); valeur.bindValue(":mont", result.toNumber()); valeur.bindValue(":sens", rubrique.value(8).toString()); valeur.bindValue(":affiche", rubrique.value(9).toInt()); valeur.exec(); }   if (mod == 2) { variable.prepare(QString( "select * from variabl where id_salarie=:id_sal and id_rubrique=:id_rub ")); variable.bindValue(":id_sal", salarie.value(0).toString()); variable.bindValue(":id_rub", rubrique.value(0).toString()); variable.exec(); variable.first();   formule = "TAUX=" + variable.value(5).toString() + ";"; ui->toscript->appendPlainText(formule.toUpper()); Scriptcode = ui->toscript->toPlainText();     formule = rubrique.value(4).toString() + ";"; ui->toscript->appendPlainText(formule.toUpper()); Scriptcode = ui->toscript->toPlainText(); result = engine.evaluate(Scriptcode); ui->toresult->appendPlainText(rubrique.value(1).toString() + "=" + result.toString());   // Insertion dans le fichier valeur. valeur.prepare( "INSERT INTO VALEUR " "(id_salarie,id_rubrique,mois,annee,centre_cout,nombre,taux," "montant,sens," "affiche)" " VALUES (:id_sal,:id_rub,:mois,:annee,:cout,:nbre,:tx,:mont,:sens,:affiche)"); valeur.bindValue(":id_sal", salarie.value(0).toInt()); valeur.bindValue(":id_rub", rubrique.value(0).toInt()); valeur.bindValue(":mois", 10); valeur.bindValue(":annee", 2019); valeur.bindValue(":cout", 0); valeur.bindValue(":nbre", 0); valeur.bindValue(":tx", variable.value(5).toDouble()); valeur.bindValue(":mont", result.toNumber()); valeur.bindValue(":sens", rubrique.value(8).toString()); valeur.bindValue(":affiche", rubrique.value(9).toInt()); valeur.exec(); } if (mod == 3) { variable.prepare(QString( "select * from variabl where id_salarie=:id_sal and id_rubrique=:id_rub ")); variable.bindValue(":id_sal", salarie.value(0).toString()); variable.bindValue(":id_rub", rubrique.value(0).toString()); variable.exec(); variable.first();   formule = "NOMBRE=" + variable.value(4).toString() + ";"; ui->toscript->appendPlainText(formule.toUpper()); formule = "TAUX=" + variable.value(5).toString() + ";"; ui->toscript->appendPlainText(formule.toUpper());     formule = rubrique.value(4).toString() + ";"; ui->toscript->appendPlainText(formule.toUpper()); Scriptcode = ui->toscript->toPlainText(); result = engine.evaluate(Scriptcode); ui->toresult->appendPlainText(rubrique.value(1).toString() + "=" + result.toString());   // Insertion dans le fichier valeur. valeur.prepare( "INSERT INTO VALEUR " "(id_salarie,id_rubrique,mois,annee,centre_cout,nombre,taux," "montant,sens," "affiche)" " VALUES (:id_sal,:id_rub,:mois,:annee,:cout,:nbre,:tx,:mont,:sens,:affiche)"); valeur.bindValue(":id_sal", salarie.value(0).toInt()); valeur.bindValue(":id_rub", rubrique.value(0).toInt()); valeur.bindValue(":mois", 10); valeur.bindValue(":annee", 2019); valeur.bindValue(":cout", 0); valeur.bindValue(":nbre", variable.value(4).toDouble()); valeur.bindValue(":tx", variable.value(5).toDouble()); valeur.bindValue(":mont", result.toNumber()); valeur.bindValue(":sens", rubrique.value(8).toString()); valeur.bindValue(":affiche", rubrique.value(9).toInt()); valeur.exec(); }   if (mod == 4) { variable.prepare(QString( "select * from variabl where id_salarie=:id_sal and id_rubrique=:id_rub ")); variable.bindValue(":id_sal", salarie.value(0).toString()); variable.bindValue(":id_rub", rubrique.value(0).toString()); variable.exec(); variable.first(); formule = "MONTANT=" + variable.value(6).toString(); ui->toscript->appendPlainText(formule.toUpper());   formule = rubrique.value(4).toString() + ";"; ui->toscript->appendPlainText(formule.toUpper()); Scriptcode = ui->toscript->toPlainText(); result = engine.evaluate(Scriptcode); ui->toresult->appendPlainText(rubrique.value(1).toString() + "=" + result.toString()); // Insertion dans le fichier valeur. valeur.prepare( "INSERT INTO VALEUR " "(id_salarie,id_rubrique,mois,annee,centre_cout,nombre,taux," "montant,sens," "affiche)" " VALUES (:id_sal,:id_rub,:mois,:annee,:cout,:nbre,:tx,:mont,:sens,:affiche)"); valeur.bindValue(":id_sal", salarie.value(0).toInt()); valeur.bindValue(":id_rub", rubrique.value(0).toInt()); valeur.bindValue(":mois", 10); valeur.bindValue(":annee", 2019); valeur.bindValue(":cout", 0); valeur.bindValue(":nbre", 0); valeur.bindValue(":tx", 0); valeur.bindValue(":mont", result.toNumber()); valeur.bindValue(":sens", rubrique.value(8).toString()); valeur.bindValue(":affiche", rubrique.value(9).toInt()); valeur.exec(); } } } }   #

---

En vous en remerciant par avance.

:salut:

Quelques commentaires rapides.
- Qt Script, ce n'est pas vraiment une bonne idée, vu que ça sera bientôt supprimé de Qt (au plus tard, Qt 6). Tu peux regarder du côté de JavaScript ou bien tout coder en C++ (ou lorgner du côté de Python avec PyQt, Qt for Python / PySide).
- Tu fais beaucoup trop de choses dans ta méthode on_pushButton_clicked. En fait, tu devrais plutôt décomposer ton code pour avoir une classe (ou quelques fonctions) qui font du calcul, rien que du calcul, puis une ou plusieurs classes pour l'intégration de ce code avec l'interface graphique (sans le moindre calcul). Toutes les parties relatives à la base de données pourraient aussi être extraites dans une classe annexe.
- Deux cents lignes pour une seule fonction, c'est vraiment beaucoup beaucoup trop. Mieux vaut viser les trente lignes par fonction, mais surtout faire une seule chose par fonction (et par classe). Ca t'aidera vraiment pour te replonger dans ton code après quelque temps.

Bonsoir,

Je vous remercie pour vos conseils, qui j'en suis sur, me seront d'une grande utilité. Je vais faire en sorte de refondre le code. Quand au maintien du code,
à l'avenir il n'y aura aucun changement sauf minim. L'algorithme a été conçu de tel sorte qu'il réponde à n'apporte quelle rubrique de paie.
Cordialement

Bonjour,

Les langages de scripts embarqués dans un programme sont des facilités à utiliser marginalement. S'ils sont utilisés systématiquement, c'est la pire des choses.

D'une part cela traduit que le champ fonctionnel n'est pas réellement défini et qu'il sera toujours possible de corriger le tir sans nouvelle version (avec tous les risques qui vont avec) voire même directement par l'utilisateur.

D'autre part, ils ne sont pas très efficaces car ils ne bénéficient pas d'un environnement d'exécution optimisé pour eux (à la différence d'un langage de script classique avec ses caches, sa compilation JIT etc.)

Les grands projets étatiques qui ont échoué en avait tous.

Salutations.

Bonsoir,

L'utilisation de javascript est ici, particulier au calcul de la paie. Ce que vous dites est vrai J'aurai pu utiliser un parser d'expression pour avoir le même résultat .
L'avantage avec cette méthode, c'est de ne plu toucher au code. J'avais écris la même chose en clipper, en Delphi puis en windev. Depuis des années le code n'a pas changé quelque soit la rubrique de paie.
En tous cas je vous remercie pour vos conseils.
Cordialement