Logiciel de paie

**bachirghazaouet** · 26/10/2019, 21h19

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 :

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

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.

**dourouc05** · 27/10/2019, 02h00

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.

**bachirghazaouet** · 27/10/2019, 18h39

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

**Guesset** · 28/10/2019, 10h08

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.

**bachirghazaouet** · 28/10/2019, 18h45

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

Logiciel de paie

Vue hybride

Discussions similaires

Partager

Partager