Problème d'algorithme java

Bonjour,

Je développe une application de gestion horaire.



Dans un premier temps, on m'a demandé de gérer les horaires par personne et cette méthode fonctionne très bien.

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 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 256 257 258 259 260 261 262 263 264 265 266 267 268 269 270 271 272 273 274 275 276 277 278 279 280 281 282 283 284 285 286 287 288 289 290 291 292 293 294 295 296 297 298 299 300 301 302 303 304 305 306 307 308 309 310 311 312 313 314 315 316 317 318 319 320 321 322 323 324 325 326 327 328 329 330 331 332 333 334 335 336 337 338 339 340 341 342 343 344 345 346 347 348 349 350 351 352 353 354 355 356 357 358 359 360 361 362 363 364 365 366 367 368 369 370 371 372 373 374 375 376 377 378 379 380 381 382 383 384 385 386

@Override public List<Moisdisp> getMoisDispByIdPersonneAndAnneePasCivilComptage(Integer idPersonne, String anneeDebut, String AnneeFin) { List<Moisdisp> moisDips = new ArrayList<Moisdisp>(); String queryString = " select moisdisp.idMois, moisdisp.idPers, moisdisp.nomMois, moisdisp.annee, moisdisp.totalTravMois, moisdisp.nbCannuel, " +" moisdisp.nbCt, moisdisp.nbFerie, moisdisp.nbMaladie, moisdisp.nbCancien, moisdisp.nbRecup, " +" moisdisp.nbExcep, moisdisp.nbFamil, moisdisp.totalTravAfaire, moisdisp.ecart, moisdisp.nbHeureSuppPaye, " +" moisdisp.nbFormation, moisdisp.nbRecupFerie, moisdisp.nbDelegation, moisdisp.nbIntemperie, moisdisp.nbSolidarite, moisdisp.nbEnfantMalade, moisdisp.nbNonJustif," + "moisdisp.nbSoldeCPaye, moisdisp.nbSoldeCAnciennete, moisdisp.nbSoldeCTrimestriel " +" from Moisdisp as moisdisp " +" where moisdisp.idPers = ? " +" and (moisdisp.nomMois = 01 or moisdisp.nomMois = 02 or moisdisp.nomMois = 03 or moisdisp.nomMois = 04 or moisdisp.nomMois = 05 ) " +" and moisdisp.annee = ? " +" or ( moisdisp.nomMois = 06 or moisdisp.nomMois = 07 or moisdisp.nomMois = 8 or moisdisp.nomMois = 9 or moisdisp.nomMois = 10 or moisdisp.nomMois = 11 or moisdisp.nomMois = 12 ) " +" and moisdisp.annee = ? " +" and moisdisp.idPers = ? " +" ORDER BY moisdisp.annee , moisdisp.nomMois ASC ";     logger.debug("getMoisDispByIdPersonneAndAnneeNonCivile : query : "+queryString);               Object[] params = {idPersonne, AnneeFin , anneeDebut, idPersonne};   int ecartConstate = 0; int dureeDeTravAnneeFait = 0; int dureeDeTravAnneeAfaire = 0 ; int heureSuppAreporte = 0;   int nbCA = 0; int nbCT = 0; int nbFer = 0; int nbCMal = 0; int nbCAnc = 0; int nbCRecup = 0; int nbCExc = 0; int nbCParent = 0; int nbCFormation = 0 ; int nbCRecupFerie = 0 ; int nbCDelegation = 0; int nbCIntemperie = 0;   int nbCSolidarite = 0; int nbCEnfantMalade = 0; int nbCNonJustif = 0;   int nbSoldeCPaye = 0; int nbSoldeCAnciennete = 0; int nbSoldeCTrimestriel = 0; int idPersonneOld = 0 ; Moisdisp moisDis = new Moisdisp(); Iterator results = this.getHibernateTemplate().find(queryString, params).iterator(); while ( results.hasNext() ) {     Object[] row = (Object[]) results.next();   Integer idMois = (Integer) row[0]; moisDis.setIdMois(idMois);     Integer idPers = (Integer) row[1]; moisDis.setIdPers(idPers);   if(idPersonne != idPersonneOld && idPersonneOld != 0){   }   String mois = (String) row[2]; logger.debug("nom mois du moisDisp : "+mois); moisDis.setNomMois(mois);   String anneeTrouv = (String) row[3]; moisDis.setAnnee(anneeTrouv);     String totalTravMois = (String)row[4]; moisDis.setTotalTravMois(totalTravMois);   logger.debug("totalTravMois !!!!!! : "+totalTravMois); //Pour additionner le total qui doit être par an   String valeurDureeJourAdecoupe = ":"; if(!totalTravMois.contains("Report")){ if(totalTravMois != null){ String [] totalTravMoisTemps = totalTravMois.split(valeurDureeJourAdecoupe); String totalTravMoisheure = totalTravMoisTemps[0]; totalTravMoisheure = totalTravMoisheure.replace(" ","");   String totalTravMoismin = totalTravMoisTemps[1]; totalTravMoismin = totalTravMoismin.replace(" ","");   int iTotalTravMoisHeure = Integer.parseInt(totalTravMoisheure); int iTotalTravMoisMin = Integer.parseInt(totalTravMoismin);   int dureeDeTravFait = (iTotalTravMoisHeure * 60) + iTotalTravMoisMin;   dureeDeTravAnneeFait += dureeDeTravFait; logger.debug("dureeDeTravAnneeFait :"+dureeDeTravAnneeFait); }   }     Integer nbCannuel = (Integer) row[5]; logger.debug("nbCannuel :"+nbCannuel); if(nbCannuel != null && !"".equals(nbCannuel) && !"0".equals(mois)){ nbCA += nbCannuel; moisDis.setNbCannuel(nbCA); }   Integer nbCt = (Integer) row[6]; if(nbCt != null && !"0".equals(mois)){ nbCT += nbCt; moisDis.setNbCt(nbCT); }   Integer nbFerie = (Integer) row[7]; if(nbFerie != null && !"0".equals(mois)){ nbFer += nbFerie; moisDis.setNbFerie(nbFer); }   Integer nbMaladie = (Integer) row[8]; if(nbMaladie != null && !"0".equals(mois)){ nbCMal += nbMaladie; moisDis.setNbMaladie(nbCMal); }   Integer nbCancien = (Integer) row[9]; if(nbCancien != null && !"0".equals(mois)){ nbCAnc += nbCancien; moisDis.setNbCancien(nbCAnc); }   Integer nbRecup = (Integer) row[10]; if(nbRecup != null && !"0".equals(mois)){ nbCRecup += nbRecup; moisDis.setNbRecup(nbCRecup); }     Integer nbExcep = (Integer) row[11]; if(nbExcep != null && !"0".equals(mois)){ nbCExc += nbExcep; moisDis.setNbExcep(nbCExc); }     Integer nbFamil = (Integer) row[12]; if(nbFamil != null && !"0".equals(mois)){ nbCParent += nbFamil; moisDis.setNbFamil(nbCParent); }     Integer nbFormation = (Integer) row[17]; if(nbFormation != null && !"0".equals(mois)){ nbCFormation += nbFormation; moisDis.setNbFormation(nbCFormation); logger.debug("nbCFormation : "+nbCFormation); }       Integer nbRecupFerie = (Integer) row[16]; if(nbRecupFerie != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCRecupFerie += nbRecupFerie; moisDis.setNbRecupFerie(nbCRecupFerie); logger.debug("nbCRecupFerie : "+nbCRecupFerie); }     Integer nbDelegation = (Integer) row[18]; if(nbDelegation != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCDelegation += nbDelegation; moisDis.setNbDelegation(nbCDelegation); logger.debug("nbCDelegation : "+nbCDelegation); }   Integer nbIntemperie = (Integer) row[19]; if(nbIntemperie != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCIntemperie += nbIntemperie; moisDis.setNbIntemperie(nbCIntemperie); logger.debug("nbCIntemperie : "+nbCIntemperie); }   Integer nbSolidarite = (Integer) row[20]; if(nbSolidarite != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCSolidarite += nbSolidarite; moisDis.setNbSolidarite(nbCSolidarite); logger.debug("nbCSolidarite : "+nbCSolidarite); }   Integer nbEnfantMalade = (Integer) row[21]; if(nbEnfantMalade != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCEnfantMalade += nbEnfantMalade; moisDis.setNbEnfantMalade(nbCEnfantMalade); logger.debug("nbCEnfantMalade : "+nbCEnfantMalade); }   Integer nbNonJustif = (Integer) row[22]; if(nbNonJustif != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCNonJustif += nbNonJustif; moisDis.setNbNonJustif(nbCNonJustif); logger.debug("nbCNonJustif : "+nbCNonJustif); }   Integer nbSoldePaye = (Integer) row[23]; if(nbSoldePaye != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbSoldeCPaye += nbSoldePaye; moisDis.setNbSoldeCPaye(nbSoldeCPaye); logger.debug("nbSoldeCPaye : "+nbSoldeCPaye); }     Integer nbSoldeAnc = (Integer) row[24]; if(nbSoldeAnc != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbSoldeCAnciennete += nbSoldeAnc; moisDis.setNbSoldeCAnciennete(nbSoldeCAnciennete); logger.debug("nbSoldeCAnciennete : "+nbSoldeCAnciennete); }   Integer nbSoldeTri = (Integer) row[25]; if(nbSoldeTri != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbSoldeCTrimestriel += nbSoldeTri; moisDis.setNbSoldeCTrimestriel(nbSoldeCTrimestriel); logger.debug("nbSoldeCTrimestriel : "+nbSoldeCTrimestriel); }   String heureTotalAfaire = (String)row[13]; moisDis.setTotalTravAfaire(heureTotalAfaire); logger.debug("heureTotalAfaire :"+heureTotalAfaire); //Pour additionner le total des heures a réalisée qui doit être par an if( !"".equals(heureTotalAfaire) && heureTotalAfaire != null){ String [] heureTotalAfaireTemps = heureTotalAfaire.split(valeurDureeJourAdecoupe); String heureTotalAfaireHeure = heureTotalAfaireTemps[0];   String heureTotalAfaireMin = heureTotalAfaireTemps[1];   heureTotalAfaireHeure = heureTotalAfaireHeure.replace(" ",""); heureTotalAfaireMin = heureTotalAfaireMin.replace(" ","");     int iheureTotalAfaireHeure = Integer.parseInt(heureTotalAfaireHeure); int iheureTotalAfaireMin = Integer.parseInt(heureTotalAfaireMin);   int dureeDeTravAfaire = (iheureTotalAfaireHeure * 60) + iheureTotalAfaireMin;   dureeDeTravAnneeAfaire += dureeDeTravAfaire; logger.debug("dureeDeTravAnneeAfaire :"+dureeDeTravAnneeAfaire); }         String heureSupp = (String)row[15]; moisDis.setNbHeureSuppPaye(heureSupp); logger.debug("heureSupp : "+heureSupp); //Pour effectue le calcul avec les heure supp et les écart if(heureSupp != null){ String [] heureSupTemps = heureSupp.split(valeurDureeJourAdecoupe); String heureSupHeure = heureSupTemps[0]; String heureSupMin = heureSupTemps[1];   heureSupHeure = heureSupHeure.replace(" ",""); heureSupMin = heureSupMin.replace(" ","");     int iheureSuppHeure = Integer.parseInt(heureSupHeure); int iheureSuppMin = Integer.parseInt(heureSupMin);       heureSuppAreporte = heureSuppAreporte + (iheureSuppHeure * 60) + iheureSuppMin;   //heureSuppAreporte += heureSuppAreporte; logger.debug("heureSuppAreporte :"+heureSuppAreporte); }     //On transmforme l'ecart en min String ecart = (String)row[14]; moisDis.setEcart(ecart); logger.debug("ecart :"+ecart);   int valeurNegative = ecart.indexOf("-");   int iEcartHeure = 0 ; int iEcartMin = 0 ;   int ecartMin = 0 ;   if(valeurNegative == -1){ logger.debug("***** Absence du tiret VALEUR POSITIVE "); String [] ecartTemps = ecart.split(valeurDureeJourAdecoupe); String ecartHeure = ecartTemps[0]; String ecartMinin = ecartTemps[1];     ecartHeure = ecartHeure.replace(" ",""); ecartMinin = ecartMinin.replace(" ","");     iEcartHeure = Integer.parseInt(ecartHeure); iEcartMin = Integer.parseInt(ecartMinin);   ecartMin = (iEcartHeure * 60) + iEcartMin;   logger.debug("ecartMin POSITIF : "+ecartMin);   }else{ logger.debug("Presence du tiret VALEUR NEGATIVE ");   String [] ecartTemps = ecart.split(valeurDureeJourAdecoupe); String ecartHeure = ecartTemps[0]; ecartHeure = ecartHeure.replace("-",""); ecartHeure = ecartHeure.replace(" ","");   String ecartMinin = ecartTemps[1]; ecartMinin = ecartMinin.replace(" ","");     iEcartHeure = Integer.parseInt(ecartHeure); iEcartHeure = 0 - iEcartHeure;   iEcartMin = Integer.parseInt(ecartMinin);   ecartMin = (iEcartHeure * 60) - iEcartMin;     logger.debug("ecartMin NEGATIVE : "+ecartMin);   }           ecartConstate += ecartMin;   //String ecartAenregistrer = this.getTotalHours(intEcartAenregistrer); //moisDis.setEcart(ecartAenregistrer);     }     ecartConstate = ecartConstate - heureSuppAreporte; logger.debug("** ecartConstate ** : "+ecartConstate);   String dureeTravailRealise = getTotalHours(dureeDeTravAnneeFait); moisDis.setTotalTravMois(dureeTravailRealise); logger.debug("!! dureeTravailRealise !!" + dureeTravailRealise);     String dureeTravailArealise = getTotalHours(dureeDeTravAnneeAfaire); logger.debug("!! dureeTravailArealise !!" + dureeTravailArealise); moisDis.setTotalTravAfaire(dureeTravailArealise);   String ecartConstateHeure = getTotalHours(ecartConstate); moisDis.setEcart(ecartConstateHeure); logger.debug("!! ecartConstateHeure !! " + ecartConstateHeure);     moisDips.add(moisDis);   return moisDips;     }

---

Néanmoins, dans un deuxième temps, on m'a demandé de gérer les horaires des personnes par service. Et là, je suis bloquée. :oops::oops::oops:

Grace à de l'aide, j'ai réussi à trouver la bonne requête pour trouver les bonnes informations. Toutefois, je suis bloqué par un problème d'algorithme très con, très con.

Je n'arrive pas à calculer les horaires par personne. C'est à dire que la requête me retourne tous les horaires des personnes du service concernée, mais que je n'arrive pas à calculer les horaires par personnes.

Pourriez vous m'aider s'il vous plait :oops:

Voici la fonction en question.

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 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 256 257 258 259 260 261 262 263 264 265 266 267 268 269 270 271 272 273 274 275 276 277 278 279 280 281 282 283 284 285 286 287 288 289 290 291 292 293 294 295 296 297 298 299 300 301 302 303 304 305 306 307 308 309 310 311 312 313 314 315 316 317 318 319 320 321 322 323 324 325 326 327 328 329 330 331 332 333 334 335 336 337 338 339 340 341 342 343 344 345 346 347 348 349 350 351 352 353 354 355 356 357 358 359 360 361 362 363 364 365 366 367 368 369 370 371 372 373 374 375 376 377 378 379 380 381 382 383 384 385 386 387 388 389 390 391 392 393

public List<Moisdisp> getMoisDispByIdUniteSoldeVacanceComptage(Integer unite, String anneeDebut, String AnneeFin) { List<Moisdisp> moisDips = new ArrayList<Moisdisp>();   String queryString = " select moisdisp.idMois, moisdisp.idPers, moisdisp.nomMois, moisdisp.annee, moisdisp.totalTravMois, moisdisp.nbCannuel, " +" moisdisp.nbCt, moisdisp.nbFerie, moisdisp.nbMaladie, moisdisp.nbCancien, moisdisp.nbRecup, " +" moisdisp.nbExcep, moisdisp.nbFamil, moisdisp.totalTravAfaire, moisdisp.ecart, moisdisp.nbHeureSuppPaye, moisdisp.nbFormation, moisdisp.nbRecupFerie, " +" moisdisp.nbDelegation, moisdisp.nbIntemperie, moisdisp.nbSolidarite, moisdisp.nbEnfantMalade, moisdisp.nbNonJustif, moisdisp.nbSoldeCPaye, moisdisp.nbSoldeCAnciennete, " + "moisdisp.nbSoldeCTrimestriel, personne.nom " + "from Moisdisp AS moisdisp, Personne as personne, Uniteservice as uniteservice, Unite as unite " + "where moisdisp.idPers = personne.idPersonne and personne.service.idService = uniteservice.service.idService " + "and uniteservice.unite.idUnite = unite.idUnite " + "and unite.idUnite = ? " + " AND ( (moisdisp.annee = ? AND moisdisp.nomMois > 5)OR (moisdisp.annee = ? AND moisdisp.nomMois < 6) )" + " and moisdisp.nomMois !=0 " + "ORDER BY moisdisp.idPers, moisdisp.annee, moisdisp.nomMois ASC";         logger.debug("getMoisDispByIdUniteSoldeVacance : query : "+queryString);     Object[] params = {unite, anneeDebut, AnneeFin};     int idPersonneOld = 0;     int ecartConstate = 0; int dureeDeTravAnneeFait = 0; int dureeDeTravAnneeAfaire = 0 ; int heureSuppAreporte = 0;   int nbCA = 0; int nbCT = 0; int nbFer = 0; int nbCMal = 0; int nbCAnc = 0; int nbCRecup = 0; int nbCExc = 0; int nbCParent = 0; int nbCFormation = 0 ; int nbCRecupFerie = 0 ; int nbCDelegation = 0; int nbCIntemperie = 0;   int nbCSolidarite = 0; int nbCEnfantMalade = 0; int nbCNonJustif = 0;   int nbSoldeCPaye = 0; int nbSoldeCAnciennete = 0; int nbSoldeCTrimestriel = 0;   Integer idPers = 0;   Moisdisp moisDis = new Moisdisp(); Iterator results = this.getHibernateTemplate().find(queryString, params).iterator(); while ( results.hasNext() ) {     Object[] row = (Object[]) results.next();   Integer idMois = (Integer) row[0]; moisDis.setIdMois(idMois);     idPers = (Integer) row[1]; moisDis.setIdPers(idPers);       String mois = (String) row[2]; logger.debug("nom mois du moisDisp : "+mois); moisDis.setNomMois(mois);   String anneeTrouv = (String) row[3]; moisDis.setAnnee(anneeTrouv);     String totalTravMois = (String)row[4]; moisDis.setTotalTravMois(totalTravMois);   logger.debug("totalTravMois !!!!!! : "+totalTravMois); //Pour additionner le total qui doit être par an   String valeurDureeJourAdecoupe = ":"; if(!totalTravMois.contains("Report")){ if(totalTravMois != null){ String [] totalTravMoisTemps = totalTravMois.split(valeurDureeJourAdecoupe); String totalTravMoisheure = totalTravMoisTemps[0]; totalTravMoisheure = totalTravMoisheure.replace(" ","");   String totalTravMoismin = totalTravMoisTemps[1]; totalTravMoismin = totalTravMoismin.replace(" ","");   int iTotalTravMoisHeure = Integer.parseInt(totalTravMoisheure); int iTotalTravMoisMin = Integer.parseInt(totalTravMoismin);   int dureeDeTravFait = (iTotalTravMoisHeure * 60) + iTotalTravMoisMin;   dureeDeTravAnneeFait += dureeDeTravFait; logger.debug("dureeDeTravAnneeFait :"+dureeDeTravAnneeFait); }   }     Integer nbCannuel = (Integer) row[5]; logger.debug("nbCannuel :"+nbCannuel); if(nbCannuel != null && !"".equals(nbCannuel) && !"0".equals(mois)){ nbCA += nbCannuel; moisDis.setNbCannuel(nbCA); }   Integer nbCt = (Integer) row[6]; if(nbCt != null && !"0".equals(mois)){ nbCT += nbCt; moisDis.setNbCt(nbCT); }   Integer nbFerie = (Integer) row[7]; if(nbFerie != null && !"0".equals(mois)){ nbFer += nbFerie; moisDis.setNbFerie(nbFer); }   Integer nbMaladie = (Integer) row[8]; if(nbMaladie != null && !"0".equals(mois)){ nbCMal += nbMaladie; moisDis.setNbMaladie(nbCMal); }   Integer nbCancien = (Integer) row[9]; if(nbCancien != null && !"0".equals(mois)){ nbCAnc += nbCancien; moisDis.setNbCancien(nbCAnc); }   Integer nbRecup = (Integer) row[10]; if(nbRecup != null && !"0".equals(mois)){ nbCRecup += nbRecup; moisDis.setNbRecup(nbCRecup); }     Integer nbExcep = (Integer) row[11]; if(nbExcep != null && !"0".equals(mois)){ nbCExc += nbExcep; moisDis.setNbExcep(nbCExc); }     Integer nbFamil = (Integer) row[12]; if(nbFamil != null && !"0".equals(mois)){ nbCParent += nbFamil; moisDis.setNbFamil(nbCParent); }     Integer nbFormation = (Integer) row[17]; if(nbFormation != null && !"0".equals(mois)){ nbCFormation += nbFormation; moisDis.setNbFormation(nbCFormation); logger.debug("nbCFormation : "+nbCFormation); }       Integer nbRecupFerie = (Integer) row[16]; if(nbRecupFerie != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCRecupFerie += nbRecupFerie; moisDis.setNbRecupFerie(nbCRecupFerie); logger.debug("nbCRecupFerie : "+nbCRecupFerie); }     Integer nbDelegation = (Integer) row[18]; if(nbDelegation != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCDelegation += nbDelegation; moisDis.setNbDelegation(nbCDelegation); logger.debug("nbCDelegation : "+nbCDelegation); }   Integer nbIntemperie = (Integer) row[19]; if(nbIntemperie != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCIntemperie += nbIntemperie; moisDis.setNbIntemperie(nbCIntemperie); logger.debug("nbCIntemperie : "+nbCIntemperie); }   Integer nbSolidarite = (Integer) row[20]; if(nbSolidarite != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCSolidarite += nbSolidarite; moisDis.setNbSolidarite(nbCSolidarite); logger.debug("nbCSolidarite : "+nbCSolidarite); }   Integer nbEnfantMalade = (Integer) row[21]; if(nbEnfantMalade != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCEnfantMalade += nbEnfantMalade; moisDis.setNbEnfantMalade(nbCEnfantMalade); logger.debug("nbCEnfantMalade : "+nbCEnfantMalade); }   Integer nbNonJustif = (Integer) row[22]; if(nbNonJustif != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCNonJustif += nbNonJustif; moisDis.setNbNonJustif(nbCNonJustif); logger.debug("nbCNonJustif : "+nbCNonJustif); }   Integer nbSoldePaye = (Integer) row[23]; if(nbSoldePaye != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbSoldeCPaye += nbSoldePaye; moisDis.setNbSoldeCPaye(nbSoldeCPaye); logger.debug("nbSoldeCPaye : "+nbSoldeCPaye); }     Integer nbSoldeAnc = (Integer) row[24]; if(nbSoldeAnc != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbSoldeCAnciennete += nbSoldeAnc; moisDis.setNbSoldeCAnciennete(nbSoldeCAnciennete); logger.debug("nbSoldeCAnciennete : "+nbSoldeCAnciennete); }   Integer nbSoldeTri = (Integer) row[25]; if(nbSoldeTri != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbSoldeCTrimestriel += nbSoldeTri; moisDis.setNbSoldeCTrimestriel(nbSoldeCTrimestriel); logger.debug("nbSoldeCTrimestriel : "+nbSoldeCTrimestriel); }     String nomPersonne = (String) row [26]; moisDis.setNomPersonne(nomPersonne);   String heureTotalAfaire = (String)row[13]; moisDis.setTotalTravAfaire(heureTotalAfaire); logger.debug("heureTotalAfaire :"+heureTotalAfaire); //Pour additionner le total des heures a réalisée qui doit être par an if( !"".equals(heureTotalAfaire) && heureTotalAfaire != null){ String [] heureTotalAfaireTemps = heureTotalAfaire.split(valeurDureeJourAdecoupe); String heureTotalAfaireHeure = heureTotalAfaireTemps[0];   String heureTotalAfaireMin = heureTotalAfaireTemps[1];   heureTotalAfaireHeure = heureTotalAfaireHeure.replace(" ",""); heureTotalAfaireMin = heureTotalAfaireMin.replace(" ","");     int iheureTotalAfaireHeure = Integer.parseInt(heureTotalAfaireHeure); int iheureTotalAfaireMin = Integer.parseInt(heureTotalAfaireMin);   int dureeDeTravAfaire = (iheureTotalAfaireHeure * 60) + iheureTotalAfaireMin;   dureeDeTravAnneeAfaire += dureeDeTravAfaire; logger.debug("dureeDeTravAnneeAfaire :"+dureeDeTravAnneeAfaire); }         String heureSupp = (String)row[15]; moisDis.setNbHeureSuppPaye(heureSupp); logger.debug("heureSupp : "+heureSupp); //Pour effectue le calcul avec les heure supp et les écart if(heureSupp != null){ String [] heureSupTemps = heureSupp.split(valeurDureeJourAdecoupe); String heureSupHeure = heureSupTemps[0]; String heureSupMin = heureSupTemps[1];   heureSupHeure = heureSupHeure.replace(" ",""); heureSupMin = heureSupMin.replace(" ","");     int iheureSuppHeure = Integer.parseInt(heureSupHeure); int iheureSuppMin = Integer.parseInt(heureSupMin);       heureSuppAreporte = heureSuppAreporte + (iheureSuppHeure * 60) + iheureSuppMin;   //heureSuppAreporte += heureSuppAreporte; logger.debug("heureSuppAreporte :"+heureSuppAreporte); }     //On transmforme l'ecart en min String ecart = (String)row[14]; moisDis.setEcart(ecart); logger.debug("ecart :"+ecart);   int valeurNegative = ecart.indexOf("-");   int iEcartHeure = 0 ; int iEcartMin = 0 ;   int ecartMin = 0 ;   if(valeurNegative == -1){ logger.debug("***** Absence du tiret VALEUR POSITIVE "); String [] ecartTemps = ecart.split(valeurDureeJourAdecoupe); String ecartHeure = ecartTemps[0]; String ecartMinin = ecartTemps[1];     ecartHeure = ecartHeure.replace(" ",""); ecartMinin = ecartMinin.replace(" ","");     iEcartHeure = Integer.parseInt(ecartHeure); iEcartMin = Integer.parseInt(ecartMinin);   ecartMin = (iEcartHeure * 60) + iEcartMin;   logger.debug("ecartMin POSITIF : "+ecartMin);   }else{ logger.debug("Presence du tiret VALEUR NEGATIVE ");   String [] ecartTemps = ecart.split(valeurDureeJourAdecoupe); String ecartHeure = ecartTemps[0]; ecartHeure = ecartHeure.replace("-",""); ecartHeure = ecartHeure.replace(" ","");   String ecartMinin = ecartTemps[1]; ecartMinin = ecartMinin.replace(" ","");     iEcartHeure = Integer.parseInt(ecartHeure); iEcartHeure = 0 - iEcartHeure;   iEcartMin = Integer.parseInt(ecartMinin);   ecartMin = (iEcartHeure * 60) - iEcartMin;     logger.debug("ecartMin NEGATIVE : "+ecartMin);   }           ecartConstate += ecartMin;   //String ecartAenregistrer = this.getTotalHours(intEcartAenregistrer); //moisDis.setEcart(ecartAenregistrer);     }     ecartConstate = ecartConstate - heureSuppAreporte; logger.debug("** ecartConstate ** : "+ecartConstate);   String dureeTravailRealise = getTotalHours(dureeDeTravAnneeFait); moisDis.setTotalTravMois(dureeTravailRealise); logger.debug("!! dureeTravailRealise !!" + dureeTravailRealise);     String dureeTravailArealise = getTotalHours(dureeDeTravAnneeAfaire); logger.debug("!! dureeTravailArealise !!" + dureeTravailArealise); moisDis.setTotalTravAfaire(dureeTravailArealise);   String ecartConstateHeure = getTotalHours(ecartConstate); moisDis.setEcart(ecartConstateHeure); logger.debug("!! ecartConstateHeure !! " + ecartConstateHeure);       idPersonneOld = idPers;   moisDips.add(moisDis);   return moisDips;     }

---

Je pense que je dois rajouter une condition tel que cela mais je ne sais pas où ....:oops::oops::oops::oops:

Code:

---

1 2 3 4 5

if(idPers != idPersonneOld && idPersonneOld != 0){     }

---

Je vous remercie infiniment

Bonjour,

J'ai pas eu le temps de tout lire mais pour

Citation:

---

Je n'arrive pas à calculer les horaires par personne. C'est à dire que la requête me retourne tous les horaires des personnes du service concernée, mais que je n'arrive pas à calculer les horaires par personnes.

---

As-tu essayé une requête dans le genre:

Code:

---

select ..., sum(nom_colonne_horraire), ... from ... where id_service=2 group by id_personne

---

.

A+.

Merci beaucoup pour votre réponse.

La requête est bonne, le soucis c'est dans le traitement des résultats.

Je pense qu'il me manque juste une conditions que je ne sais pas où mettre.

Il faudrait que j'arrive à créer un objet personne à chaque fois que j'ai une nouvelle personne qui apparaît dans la liste (cf l'image) et que les calcules se refont pour chaque nouvelle personne.

Citation:

---

Envoyé par leara500

Merci beaucoup pour votre réponse.

La requête est bonne, le soucis c'est dans le traitement des résultats.

Je pense qu'il me manque juste une conditions que je ne sais pas où mettre.

Il faudrait que j'arrive à créer un objet personne à chaque fois que j'ai une nouvelle personne qui apparaît dans la liste (cf l'image) et que les calcules se refont pour chaque nouvelle personne.

---

Bonjour leara500,

Ce que andry.aime est entrain de te dire à mon avis, c'est que tu peux utiliser une requête SQL simple pour retourner la liste des personnes selon les conditions qu'on te demande.

Avec une requête telle que l'a écrit andry.aime, tu n'as pas besoin d'un algorithme à faire dans ton appli en java, ton SGBD va te faire la somme des horaires et te regrouper tous ça par service et personnes.

Code:

---

select ..., sum(nom_colonne_horraire), ... from ... where conditions group by id_service, id_personne

---

Merci beaucoup de vos réponses. Le soucis, c'est qu'il y a plein de calculs qui sont gérer en Java... C'est pour cela que j'ai besoin de trouver la bonne condition pour effectuer les calcules.

J'aimerai que tant que c'est la meme personne, les calculs de fassent. Lorsque l'on change de personne, on refait les calculs en créant une nouvelle personne... :oops::(

Bonjour,

Tu peux utiliser un Map

Code:

---

1 2 3 4 5 6 7 8 9

Map<Integer, Double> sum = new HashMap<Integer, Double>(); while ( results.hasNext() ) { ... if(sum.containsKey(idPers)){ sum.put(idPers, nbHeure+sum.get(idPers)); }else{ sum.put(idPers, nbHeure); } }

---

A+.

Merci infiniment pour votre réponse mais je n'ai rien compris TToTT YoY :oops::oops::oops::oops: Je l'avoue, j'ai honte, mais j'ai toujours eu du mal avec les Map.

Il n'existe pas une condition toutes simple pour faire ceux que je souhaite, c'est à dire tant que les idPersonnes sont identique, créer un objet personne, et y mettre le total des résultats ?

Pièce jointe 412171



J'ai cru qu'en rajoutant cette condition

Code:

---

1 2	if((idPers == idPersonneOld)||(idPersonneOld==0)){

---

cela aurait pu marché mais ce n'est pas le cas. En effet, cette requête me renvoi 28 réponses qui concernent 4 personne différentes. La fonction m'a créé 7 personne au lieu de 4

Auriez vous d'autre idée s'il vous plait. Je vous en remercie d'avance...

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 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 256 257 258 259 260 261 262 263 264 265 266 267 268 269 270 271 272 273 274 275 276 277 278 279 280 281 282 283 284 285 286 287 288 289 290 291 292 293 294 295 296 297 298 299 300 301 302 303 304 305 306 307 308 309 310 311 312 313 314 315 316 317 318 319 320 321 322 323 324 325 326 327 328 329 330 331 332 333 334 335 336 337 338 339 340 341 342 343 344 345 346 347 348 349 350 351 352 353 354 355 356 357 358 359 360 361 362 363 364 365 366 367 368 369 370 371 372 373 374 375 376 377 378 379 380 381 382 383 384 385 386 387 388 389 390 391 392 393 394 395 396

Iterator results = this.getHibernateTemplate().find(queryString, params).iterator(); while ( results.hasNext() ) {     Object[] row = (Object[]) results.next();   Integer idMois = (Integer) row[0];             int idPers = (Integer) row[1];     if((idPers == idPersonneOld)||(idPersonneOld==0)){       System.out.println("idPersonneOld : "+idPersonneOld); System.out.println("idPers : "+idPers);     int nbMoisCompte = 0;   Moisdisp moisDis = new Moisdisp();   moisDis.setIdMois(idMois); moisDis.setIdPers(idPers);     int ecartConstate = 0; int dureeDeTravAnneeFait = 0; int dureeDeTravAnneeAfaire = 0 ; int heureSuppAreporte = 0;   int nbCA = 0; int nbCT = 0; int nbFer = 0; int nbCMal = 0; int nbCAnc = 0; int nbCRecup = 0; int nbCExc = 0; int nbCParent = 0; int nbCFormation = 0 ; int nbCRecupFerie = 0 ; int nbCDelegation = 0; int nbCIntemperie = 0;   int nbCSolidarite = 0; int nbCEnfantMalade = 0; int nbCNonJustif = 0;   int nbSoldeCPaye = 0; int nbSoldeCAnciennete = 0; int nbSoldeCTrimestriel = 0;               String mois = (String) row[2]; logger.debug("nom mois du moisDisp : "+mois); moisDis.setNomMois(mois);   String anneeTrouv = (String) row[3]; moisDis.setAnnee(anneeTrouv);     String totalTravMois = (String)row[4]; moisDis.setTotalTravMois(totalTravMois);   logger.debug("totalTravMois !!!!!! : "+totalTravMois); //Pour additionner le total qui doit être par an   String valeurDureeJourAdecoupe = ":"; if(!totalTravMois.contains("Report")){ if(totalTravMois != null){ String [] totalTravMoisTemps = totalTravMois.split(valeurDureeJourAdecoupe); String totalTravMoisheure = totalTravMoisTemps[0]; totalTravMoisheure = totalTravMoisheure.replace(" ","");   String totalTravMoismin = totalTravMoisTemps[1]; totalTravMoismin = totalTravMoismin.replace(" ","");   int iTotalTravMoisHeure = Integer.parseInt(totalTravMoisheure); int iTotalTravMoisMin = Integer.parseInt(totalTravMoismin);   int dureeDeTravFait = (iTotalTravMoisHeure * 60) + iTotalTravMoisMin;   dureeDeTravAnneeFait += dureeDeTravFait; logger.debug("dureeDeTravAnneeFait :"+dureeDeTravAnneeFait); }   }     Integer nbCannuel = (Integer) row[5]; logger.debug("nbCannuel :"+nbCannuel); if(nbCannuel != null && !"".equals(nbCannuel) && !"0".equals(mois)){ nbCA += nbCannuel; moisDis.setNbCannuel(nbCA); }   Integer nbCt = (Integer) row[6]; if(nbCt != null && !"0".equals(mois)){ nbCT += nbCt; moisDis.setNbCt(nbCT); }   Integer nbFerie = (Integer) row[7]; if(nbFerie != null && !"0".equals(mois)){ nbFer += nbFerie; moisDis.setNbFerie(nbFer); }   Integer nbMaladie = (Integer) row[8]; if(nbMaladie != null && !"0".equals(mois)){ nbCMal += nbMaladie; moisDis.setNbMaladie(nbCMal); }   Integer nbCancien = (Integer) row[9]; if(nbCancien != null && !"0".equals(mois)){ nbCAnc += nbCancien; moisDis.setNbCancien(nbCAnc); }   Integer nbRecup = (Integer) row[10]; if(nbRecup != null && !"0".equals(mois)){ nbCRecup += nbRecup; moisDis.setNbRecup(nbCRecup); }     Integer nbExcep = (Integer) row[11]; if(nbExcep != null && !"0".equals(mois)){ nbCExc += nbExcep; moisDis.setNbExcep(nbCExc); }     Integer nbFamil = (Integer) row[12]; if(nbFamil != null && !"0".equals(mois)){ nbCParent += nbFamil; moisDis.setNbFamil(nbCParent); }     Integer nbFormation = (Integer) row[17]; if(nbFormation != null && !"0".equals(mois)){ nbCFormation += nbFormation; moisDis.setNbFormation(nbCFormation); logger.debug("nbCFormation : "+nbCFormation); }       Integer nbRecupFerie = (Integer) row[16]; if(nbRecupFerie != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCRecupFerie += nbRecupFerie; moisDis.setNbRecupFerie(nbCRecupFerie); logger.debug("nbCRecupFerie : "+nbCRecupFerie); }     Integer nbDelegation = (Integer) row[18]; if(nbDelegation != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCDelegation += nbDelegation; moisDis.setNbDelegation(nbCDelegation); logger.debug("nbCDelegation : "+nbCDelegation); }   Integer nbIntemperie = (Integer) row[19]; if(nbIntemperie != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCIntemperie += nbIntemperie; moisDis.setNbIntemperie(nbCIntemperie); logger.debug("nbCIntemperie : "+nbCIntemperie); }   Integer nbSolidarite = (Integer) row[20]; if(nbSolidarite != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCSolidarite += nbSolidarite; moisDis.setNbSolidarite(nbCSolidarite); logger.debug("nbCSolidarite : "+nbCSolidarite); }   Integer nbEnfantMalade = (Integer) row[21]; if(nbEnfantMalade != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCEnfantMalade += nbEnfantMalade; moisDis.setNbEnfantMalade(nbCEnfantMalade); logger.debug("nbCEnfantMalade : "+nbCEnfantMalade); }   Integer nbNonJustif = (Integer) row[22]; if(nbNonJustif != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCNonJustif += nbNonJustif; moisDis.setNbNonJustif(nbCNonJustif); logger.debug("nbCNonJustif : "+nbCNonJustif); }   Integer nbSoldePaye = (Integer) row[23]; if(nbSoldePaye != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbSoldeCPaye += nbSoldePaye; moisDis.setNbSoldeCPaye(nbSoldeCPaye); logger.debug("nbSoldeCPaye : "+nbSoldeCPaye); }     Integer nbSoldeAnc = (Integer) row[24]; if(nbSoldeAnc != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbSoldeCAnciennete += nbSoldeAnc; moisDis.setNbSoldeCAnciennete(nbSoldeCAnciennete); logger.debug("nbSoldeCAnciennete : "+nbSoldeCAnciennete); }   Integer nbSoldeTri = (Integer) row[25]; if(nbSoldeTri != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbSoldeCTrimestriel += nbSoldeTri; moisDis.setNbSoldeCTrimestriel(nbSoldeCTrimestriel); logger.debug("nbSoldeCTrimestriel : "+nbSoldeCTrimestriel); }     String nomPersonne = (String) row [26]; moisDis.setNomPersonne(nomPersonne);   String heureTotalAfaire = (String)row[13]; moisDis.setTotalTravAfaire(heureTotalAfaire); logger.debug("heureTotalAfaire :"+heureTotalAfaire); //Pour additionner le total des heures a réalisée qui doit être par an if( !"".equals(heureTotalAfaire) && heureTotalAfaire != null){ String [] heureTotalAfaireTemps = heureTotalAfaire.split(valeurDureeJourAdecoupe); String heureTotalAfaireHeure = heureTotalAfaireTemps[0];   String heureTotalAfaireMin = heureTotalAfaireTemps[1];   heureTotalAfaireHeure = heureTotalAfaireHeure.replace(" ",""); heureTotalAfaireMin = heureTotalAfaireMin.replace(" ","");     int iheureTotalAfaireHeure = Integer.parseInt(heureTotalAfaireHeure); int iheureTotalAfaireMin = Integer.parseInt(heureTotalAfaireMin);   int dureeDeTravAfaire = (iheureTotalAfaireHeure * 60) + iheureTotalAfaireMin;   dureeDeTravAnneeAfaire += dureeDeTravAfaire; logger.debug("dureeDeTravAnneeAfaire :"+dureeDeTravAnneeAfaire); }         String heureSupp = (String)row[15]; moisDis.setNbHeureSuppPaye(heureSupp); logger.debug("heureSupp : "+heureSupp); //Pour effectue le calcul avec les heure supp et les écart if(heureSupp != null){ String [] heureSupTemps = heureSupp.split(valeurDureeJourAdecoupe); String heureSupHeure = heureSupTemps[0]; String heureSupMin = heureSupTemps[1];   heureSupHeure = heureSupHeure.replace(" ",""); heureSupMin = heureSupMin.replace(" ","");     int iheureSuppHeure = Integer.parseInt(heureSupHeure); int iheureSuppMin = Integer.parseInt(heureSupMin);       heureSuppAreporte = heureSuppAreporte + (iheureSuppHeure * 60) + iheureSuppMin;   //heureSuppAreporte += heureSuppAreporte; logger.debug("heureSuppAreporte :"+heureSuppAreporte); }     //On transmforme l'ecart en min String ecart = (String)row[14]; moisDis.setEcart(ecart); logger.debug("ecart :"+ecart);   int valeurNegative = ecart.indexOf("-");   int iEcartHeure = 0 ; int iEcartMin = 0 ;   int ecartMin = 0 ;   if(valeurNegative == -1){ logger.debug("***** Absence du tiret VALEUR POSITIVE "); String [] ecartTemps = ecart.split(valeurDureeJourAdecoupe); String ecartHeure = ecartTemps[0]; String ecartMinin = ecartTemps[1];     ecartHeure = ecartHeure.replace(" ",""); ecartMinin = ecartMinin.replace(" ","");     iEcartHeure = Integer.parseInt(ecartHeure); iEcartMin = Integer.parseInt(ecartMinin);   ecartMin = (iEcartHeure * 60) + iEcartMin;   logger.debug("ecartMin POSITIF : "+ecartMin);   }else{ logger.debug("Presence du tiret VALEUR NEGATIVE ");   String [] ecartTemps = ecart.split(valeurDureeJourAdecoupe); String ecartHeure = ecartTemps[0]; ecartHeure = ecartHeure.replace("-",""); ecartHeure = ecartHeure.replace(" ","");   String ecartMinin = ecartTemps[1]; ecartMinin = ecartMinin.replace(" ","");     iEcartHeure = Integer.parseInt(ecartHeure); iEcartHeure = 0 - iEcartHeure;   iEcartMin = Integer.parseInt(ecartMinin);   ecartMin = (iEcartHeure * 60) - iEcartMin;     logger.debug("ecartMin NEGATIVE : "+ecartMin);   }           ecartConstate += ecartMin;   //String ecartAenregistrer = this.getTotalHours(intEcartAenregistrer); //moisDis.setEcart(ecartAenregistrer);                 ecartConstate = ecartConstate - heureSuppAreporte; logger.debug("** ecartConstate ** : "+ecartConstate);   String dureeTravailRealise = getTotalHours(dureeDeTravAnneeFait); moisDis.setTotalTravMois(dureeTravailRealise); logger.debug("!! dureeTravailRealise !!" + dureeTravailRealise);     String dureeTravailArealise = getTotalHours(dureeDeTravAnneeAfaire); logger.debug("!! dureeTravailArealise !!" + dureeTravailArealise); moisDis.setTotalTravAfaire(dureeTravailArealise);   String ecartConstateHeure = getTotalHours(ecartConstate); moisDis.setEcart(ecartConstateHeure); logger.debug("!! ecartConstateHeure !! " + ecartConstateHeure);       idPersonneOld = idPers; nbMoisCompte +=1 ;   moisDips.add(moisDis);   System.out.println("idPersonneOld : "+idPersonneOld); System.out.println("idPers : "+idPers); System.out.println("nbMoisCompte : "+nbMoisCompte);     }           }

---

Bonjour,

Citation:

---

Envoyé par leara500

Je l'avoue, j'ai honte, mais j'ai toujours eu du mal avec les Map.

---

:fleche: https://java.developpez.com/faq/java...-dictionnaires

Citation:

---

Envoyé par leara500

Il n'existe pas une condition toutes simple pour faire ceux que je souhaite, c'est à dire tant que les idPersonnes sont identique, créer un objet personne, et y mettre le total des résultats ?

---

La solution que j'ai proposé est déjà simple. Quand l'id existe déjà, ça fait l'addition avec la valeur existante, sinon on ajoute dans la Map. A la fin de la boucle tu auras pour chaque id, la somme des heures dans le Map.

A+.

Boujour,

Citation:

---

Il faudrait que j'arrive à créer un objet personne à chaque fois que j'ai une nouvelle personne qui apparaît dans la liste (cf l'image) et que les calcules se refont pour chaque nouvelle personne.

---

D'après ce que j'ai compris, tu veux avoir un récapitulatif pour chaque personne.
Est ce que ce traitement concerne toutes les personnes ou bien uniquement les personnes qui apparaissent dans ta tables horaires?

Bon il faut dire que peu importe le choix que tu adoptes (requête ou bien calcul cote java ) il faut que tu travail ton algorithme pour ne pas perdre de temps à boucler inutilement pour atteindre ton objectif.

Premièrement , un objet personne ne peut pas contenir un bilan de toutes les données de la base.
Donc il faut créer un objet que tu pourras appeler userRecap si tu veux qui contiendra une référence à la ressource , un mois et une année et l'ensemble des données fédérées.Ta clé primaire sera donc une clé composée pour pourvoir a chaque fois récupérer un seul élément (je ne sais même pas si c'est utile car je ne connais pas tes tables ni ta BD).

Bref je pense qu'on va un peu s’éloigner ou bien compliquer les choses mais cela peut représenter pour toi un nouveau chantier de reflexion.

Il faut , je pense , tout simplement boucler sur les personnes et faire les traitements.

Code:

---

1 2 3

for (Personne pers: listDePersonnes) { //Faire tes traitements et tes calculs }

---

Pour chaque personne il faudra récupérer de ton résultat les données à calculer et à sauvegarder dans ton objet userRecap (voir plus haut).

Le soucis a traiter est que tu va devoir créer des fonctions pour récupérer les données de chaque personne ou bien exécuter ta requête dans la boucle (trop d'aller-retour vers ta base donc déconseillé) ou alors,utiliser la force des map qui sont sous forme de <cle,valeur> pour une utilisation simple donc dans ton cas tu aura <cle,listdevaleurs> et donc l'avantage est que tu pourra accéder aux valeurs de la personne sur laquelle tu boucle grâce à la clé.

Voir cours sur les map.

Comme te l'a très bien expliqué @andry.aime tu pourras faire ce que tu veux (si tu veux une somme c'est bon sinon une liste de sommes).

Donc première chose changer ton approche.
Deuxième chose ,aller consulter les cours dont tu as besoin et ne pas avoir peur des Map qui sont très simples à utiliser.
Troisième chose avancer un peu et revenir poster dans ta discussion.

Au passage, l'idée de @andry.aime est très bonne aussi: boucler sur le résultat de ta requête, à chaque itération, faire les calculs et en sortant de la boucle(itération) stocker les valeurs dans une Map dont la clé est l'id de la personne et l'année(concaténation ou objet) donc si tu recroise dans une autre itération la même personne, alors tu ajoute tes calculs dans la Map à l'indice souhaité.

Attention: je vois que tu as des mois donc a toi de consulter les specs: si le recap est annuel alors pas grave sinon il faudra modifier ta clé en utilisant une concatenation de l'id personne ,l'année et le mois pour avoir des données correctes.


J’espère que ce message n'est pas hors sujet et pourras t'offrir une piste de reflexion.

Bon courage

Bonjour,

Citation:

---

cela aurait pu marché mais ce n'est pas le cas. En effet, cette requête me renvoi 28 réponses qui concernent 4 personne différentes. La fonction m'a créé 7 personne au lieu de 4

---

Le fait de comparer les idpersonne de cette manière fait que tu auras plus de personnes que prévu car tu compare les id successifs dans ton résultat.

SI tu as TOTO TOTO TITI TOTO ALORS tu auras 3 personnes créées au lieu de deux ce qui est logique donc le test est inutile à part si tu as un résultat trié.

Je ne sais pas si ma reponse est juste car ton code a changé entre tes deux post :JE NE VOIS PLUS DE

Code:

---

return moisDisp

---

Bon courage

Merci infiniment pour votre aide.

@johnlobs> Les résultats des personnes sont triés. Donc ça va. ouf ;)

J'ai lu la documentation et j'avoue que j'ai eu du mal à tout comprendre...:oops:

j'ai fait :

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 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 256 257 258 259 260 261 262 263 264 265 266 267 268 269 270 271 272 273 274 275 276 277 278 279 280 281 282 283 284 285 286 287 288 289 290 291 292 293 294 295 296 297 298 299 300 301 302 303 304 305 306 307 308 309 310 311 312 313 314 315 316 317 318 319 320 321 322 323 324 325 326 327 328 329 330 331 332 333 334 335 336 337 338 339 340 341 342 343 344 345 346 347 348 349 350 351 352 353 354 355 356 357 358 359 360 361 362 363 364 365 366 367 368 369 370 371 372 373 374 375 376 377 378 379 380 381 382 383 384 385 386 387 388 389 390 391 392 393 394 395 396 397 398 399 400 401 402 403 404 405 406 407 408 409 410 411 412 413 414 415 416 417 418 419 420 421 422 423 424 425 426 427 428 429 430 431 432 433 434 435 436 437 438 439 440 441 442 443 444 445 446 447 448 449 450 451 452 453 454 455 456 457 458 459 460 461 462 463

public List<Moisdisp> getMoisDispByIdUniteSoldeVacanceComptage(Integer unite, String anneeDebut, String AnneeFin) { List<Moisdisp> listMoisDipsParPersonne = new ArrayList<Moisdisp>(); StringBuffer qString = new StringBuffer();       if(unite != 6){ qString.append(" select moisdisp.idMois, moisdisp.idPers, moisdisp.nomMois, moisdisp.annee, moisdisp.totalTravMois, moisdisp.nbCannuel, "); qString.append(" moisdisp.nbCt, moisdisp.nbFerie, moisdisp.nbMaladie, moisdisp.nbCancien, moisdisp.nbRecup, "); qString.append(" moisdisp.nbExcep, moisdisp.nbFamil, moisdisp.totalTravAfaire, moisdisp.ecart, moisdisp.nbHeureSuppPaye, moisdisp.nbFormation, moisdisp.nbRecupFerie, "); qString.append(" moisdisp.nbDelegation, moisdisp.nbIntemperie, moisdisp.nbSolidarite, moisdisp.nbEnfantMalade, moisdisp.nbNonJustif, moisdisp.nbSoldeCPaye, moisdisp.nbSoldeCAnciennete, "); qString.append("moisdisp.nbSoldeCTrimestriel, personne.nom "); qString.append("from Moisdisp AS moisdisp, Personne as personne, Uniteservice as uniteservice, Unite as unite "); qString.append("where moisdisp.idPers = personne.idPersonne and personne.service.idService = uniteservice.service.idService "); qString.append("and uniteservice.unite.idUnite = unite.idUnite "); qString.append("and unite.idUnite = ? "); qString.append(" AND ( (moisdisp.annee = ? AND moisdisp.nomMois > 5)OR (moisdisp.annee = ? AND moisdisp.nomMois < 6) ) "); qString.append(" and moisdisp.nomMois !=0 "); qString.append("ORDER BY moisdisp.idPers, moisdisp.annee, moisdisp.nomMois ASC"); }else{ qString.append(" select moisdisp.idMois, moisdisp.idPers, moisdisp.nomMois, moisdisp.annee, moisdisp.totalTravMois, moisdisp.nbCannuel, "); qString.append(" moisdisp.nbCt, moisdisp.nbFerie, moisdisp.nbMaladie, moisdisp.nbCancien, moisdisp.nbRecup, "); qString.append(" moisdisp.nbExcep, moisdisp.nbFamil, moisdisp.totalTravAfaire, moisdisp.ecart, moisdisp.nbHeureSuppPaye, moisdisp.nbFormation, moisdisp.nbRecupFerie, "); qString.append(" moisdisp.nbDelegation, moisdisp.nbIntemperie, moisdisp.nbSolidarite, moisdisp.nbEnfantMalade, moisdisp.nbNonJustif, moisdisp.nbSoldeCPaye, moisdisp.nbSoldeCAnciennete, "); qString.append("moisdisp.nbSoldeCTrimestriel, personne.nom "); qString.append("from Moisdisp AS moisdisp, Personne as personne, Uniteservice as uniteservice, Unite as unite "); qString.append("where moisdisp.idPers = personne.idPersonne and personne.service.idService = uniteservice.service.idService "); qString.append("and uniteservice.unite.idUnite = unite.idUnite "); qString.append(" AND ( (moisdisp.annee = ? AND moisdisp.nomMois > 5)OR (moisdisp.annee = ? AND moisdisp.nomMois < 6) ) "); qString.append(" and moisdisp.nomMois !=0 "); qString.append("ORDER BY moisdisp.idPers, moisdisp.annee, moisdisp.nomMois ASC");   }             Iterator results =null;   if(unite != 6){ logger.debug("***monthParam != 0"); Object[] params = {unite, anneeDebut, AnneeFin}; results = this.getHibernateTemplate().find(qString.toString(), params).iterator(); }else{ logger.debug("*DebutPeriode != null ****"); Object[] params = {anneeDebut, AnneeFin}; results = this.getHibernateTemplate().find(qString.toString(), params).iterator();   }   int idPersonneOld = 0;         //int idPers = 0; Moisdisp moisDipsPersonne = new Moisdisp(); ArrayList<HashMap> listeMap = new ArrayList<HashMap>();   Map<Integer, Moisdisp> sum = new HashMap<Integer, Moisdisp>(); //Iterator results = this.getHibernateTemplate().find(queryString, params).iterator(); while ( results.hasNext() ) {     Object[] row = (Object[]) results.next();   Integer idMois = (Integer) row[0];   int idPers = (Integer) row[1];   /*if(idPers != idPersonneOld && idPersonneOld != 0){ moisDipsPersonne.setListe(listeMap); listMoisDipsParPersonne.add(moisDipsPersonne); } */     int nbMoisCompte = 0;   Moisdisp moisDis = new Moisdisp();   moisDis.setIdMois(idMois); moisDis.setIdPers(idPers);     int ecartConstate = 0; int dureeDeTravAnneeFait = 0; int dureeDeTravAnneeAfaire = 0 ; int heureSuppAreporte = 0;   int nbCA = 0; int nbCT = 0; int nbFer = 0; int nbCMal = 0; int nbCAnc = 0; int nbCRecup = 0; int nbCExc = 0; int nbCParent = 0; int nbCFormation = 0 ; int nbCRecupFerie = 0 ; int nbCDelegation = 0; int nbCIntemperie = 0;   int nbCSolidarite = 0; int nbCEnfantMalade = 0; int nbCNonJustif = 0;   int nbSoldeCPaye = 0; int nbSoldeCAnciennete = 0; int nbSoldeCTrimestriel = 0;               String mois = (String) row[2]; logger.debug("nom mois du moisDisp : "+mois); moisDis.setNomMois(mois);   String anneeTrouv = (String) row[3]; moisDis.setAnnee(anneeTrouv);     String totalTravMois = (String)row[4]; moisDis.setTotalTravMois(totalTravMois);   logger.debug("totalTravMois !!!!!! : "+totalTravMois); //Pour additionner le total qui doit être par an   String valeurDureeJourAdecoupe = ":"; if(!totalTravMois.contains("Report")){ if(totalTravMois != null){ String [] totalTravMoisTemps = totalTravMois.split(valeurDureeJourAdecoupe); String totalTravMoisheure = totalTravMoisTemps[0]; totalTravMoisheure = totalTravMoisheure.replace(" ","");   String totalTravMoismin = totalTravMoisTemps[1]; totalTravMoismin = totalTravMoismin.replace(" ","");   int iTotalTravMoisHeure = Integer.parseInt(totalTravMoisheure); int iTotalTravMoisMin = Integer.parseInt(totalTravMoismin);   int dureeDeTravFait = (iTotalTravMoisHeure * 60) + iTotalTravMoisMin;   dureeDeTravAnneeFait += dureeDeTravFait; logger.debug("dureeDeTravAnneeFait :"+dureeDeTravAnneeFait); }   }     Integer nbCannuel = (Integer) row[5]; logger.debug("nbCannuel :"+nbCannuel); if(nbCannuel != null && !"".equals(nbCannuel) && !"0".equals(mois)){ nbCA += nbCannuel; moisDis.setNbCannuel(nbCA); }   Integer nbCt = (Integer) row[6]; if(nbCt != null && !"0".equals(mois)){ nbCT += nbCt; moisDis.setNbCt(nbCT); }   Integer nbFerie = (Integer) row[7]; if(nbFerie != null && !"0".equals(mois)){ nbFer += nbFerie; moisDis.setNbFerie(nbFer); }   Integer nbMaladie = (Integer) row[8]; if(nbMaladie != null && !"0".equals(mois)){ nbCMal += nbMaladie; moisDis.setNbMaladie(nbCMal); }   Integer nbCancien = (Integer) row[9]; if(nbCancien != null && !"0".equals(mois)){ nbCAnc += nbCancien; moisDis.setNbCancien(nbCAnc); }   Integer nbRecup = (Integer) row[10]; if(nbRecup != null && !"0".equals(mois)){ nbCRecup += nbRecup; moisDis.setNbRecup(nbCRecup); }     Integer nbExcep = (Integer) row[11]; if(nbExcep != null && !"0".equals(mois)){ nbCExc += nbExcep; moisDis.setNbExcep(nbCExc); }     Integer nbFamil = (Integer) row[12]; if(nbFamil != null && !"0".equals(mois)){ nbCParent += nbFamil; moisDis.setNbFamil(nbCParent); }     Integer nbFormation = (Integer) row[17]; if(nbFormation != null && !"0".equals(mois)){ nbCFormation += nbFormation; moisDis.setNbFormation(nbCFormation); logger.debug("nbCFormation : "+nbCFormation); }       Integer nbRecupFerie = (Integer) row[16]; if(nbRecupFerie != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCRecupFerie += nbRecupFerie; moisDis.setNbRecupFerie(nbCRecupFerie); logger.debug("nbCRecupFerie : "+nbCRecupFerie); }     Integer nbDelegation = (Integer) row[18]; if(nbDelegation != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCDelegation += nbDelegation; moisDis.setNbDelegation(nbCDelegation); logger.debug("nbCDelegation : "+nbCDelegation); }   Integer nbIntemperie = (Integer) row[19]; if(nbIntemperie != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCIntemperie += nbIntemperie; moisDis.setNbIntemperie(nbCIntemperie); logger.debug("nbCIntemperie : "+nbCIntemperie); }   Integer nbSolidarite = (Integer) row[20]; if(nbSolidarite != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCSolidarite += nbSolidarite; moisDis.setNbSolidarite(nbCSolidarite); logger.debug("nbCSolidarite : "+nbCSolidarite); }   Integer nbEnfantMalade = (Integer) row[21]; if(nbEnfantMalade != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCEnfantMalade += nbEnfantMalade; moisDis.setNbEnfantMalade(nbCEnfantMalade); logger.debug("nbCEnfantMalade : "+nbCEnfantMalade); }   Integer nbNonJustif = (Integer) row[22]; if(nbNonJustif != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbCNonJustif += nbNonJustif; moisDis.setNbNonJustif(nbCNonJustif); logger.debug("nbCNonJustif : "+nbCNonJustif); }   Integer nbSoldePaye = (Integer) row[23]; if(nbSoldePaye != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbSoldeCPaye += nbSoldePaye; moisDis.setNbSoldeCPaye(nbSoldeCPaye); logger.debug("nbSoldeCPaye : "+nbSoldeCPaye); }     Integer nbSoldeAnc = (Integer) row[24]; if(nbSoldeAnc != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbSoldeCAnciennete += nbSoldeAnc; moisDis.setNbSoldeCAnciennete(nbSoldeCAnciennete); logger.debug("nbSoldeCAnciennete : "+nbSoldeCAnciennete); }   Integer nbSoldeTri = (Integer) row[25]; if(nbSoldeTri != null && !"0".equals(mois)){ logger.debug("Cas du mois Zero"); nbSoldeCTrimestriel += nbSoldeTri; moisDis.setNbSoldeCTrimestriel(nbSoldeCTrimestriel); logger.debug("nbSoldeCTrimestriel : "+nbSoldeCTrimestriel); }     String nomPersonne = (String) row [26]; moisDis.setNomPersonne(nomPersonne);   String heureTotalAfaire = (String)row[13]; moisDis.setTotalTravAfaire(heureTotalAfaire); logger.debug("heureTotalAfaire :"+heureTotalAfaire); //Pour additionner le total des heures a réalisée qui doit être par an if( !"".equals(heureTotalAfaire) && heureTotalAfaire != null){ String [] heureTotalAfaireTemps = heureTotalAfaire.split(valeurDureeJourAdecoupe); String heureTotalAfaireHeure = heureTotalAfaireTemps[0];   String heureTotalAfaireMin = heureTotalAfaireTemps[1];   heureTotalAfaireHeure = heureTotalAfaireHeure.replace(" ",""); heureTotalAfaireMin = heureTotalAfaireMin.replace(" ","");     int iheureTotalAfaireHeure = Integer.parseInt(heureTotalAfaireHeure); int iheureTotalAfaireMin = Integer.parseInt(heureTotalAfaireMin);   int dureeDeTravAfaire = (iheureTotalAfaireHeure * 60) + iheureTotalAfaireMin;   dureeDeTravAnneeAfaire += dureeDeTravAfaire; logger.debug("dureeDeTravAnneeAfaire :"+dureeDeTravAnneeAfaire); }         String heureSupp = (String)row[15]; moisDis.setNbHeureSuppPaye(heureSupp); logger.debug("heureSupp : "+heureSupp); //Pour effectue le calcul avec les heure supp et les écart if(heureSupp != null){ String [] heureSupTemps = heureSupp.split(valeurDureeJourAdecoupe); String heureSupHeure = heureSupTemps[0]; String heureSupMin = heureSupTemps[1];   heureSupHeure = heureSupHeure.replace(" ",""); heureSupMin = heureSupMin.replace(" ","");     int iheureSuppHeure = Integer.parseInt(heureSupHeure); int iheureSuppMin = Integer.parseInt(heureSupMin);       heureSuppAreporte = heureSuppAreporte + (iheureSuppHeure * 60) + iheureSuppMin;   //heureSuppAreporte += heureSuppAreporte; logger.debug("heureSuppAreporte :"+heureSuppAreporte); }     //On transmforme l'ecart en min String ecart = (String)row[14]; moisDis.setEcart(ecart); logger.debug("ecart :"+ecart);   int valeurNegative = ecart.indexOf("-");   int iEcartHeure = 0 ; int iEcartMin = 0 ;   int ecartMin = 0 ;   if(valeurNegative == -1){ logger.debug("***** Absence du tiret VALEUR POSITIVE "); String [] ecartTemps = ecart.split(valeurDureeJourAdecoupe); String ecartHeure = ecartTemps[0]; String ecartMinin = ecartTemps[1];     ecartHeure = ecartHeure.replace(" ",""); ecartMinin = ecartMinin.replace(" ","");     iEcartHeure = Integer.parseInt(ecartHeure); iEcartMin = Integer.parseInt(ecartMinin);   ecartMin = (iEcartHeure * 60) + iEcartMin;   logger.debug("ecartMin POSITIF : "+ecartMin);   }else{ logger.debug("Presence du tiret VALEUR NEGATIVE ");   String [] ecartTemps = ecart.split(valeurDureeJourAdecoupe); String ecartHeure = ecartTemps[0]; ecartHeure = ecartHeure.replace("-",""); ecartHeure = ecartHeure.replace(" ","");   String ecartMinin = ecartTemps[1]; ecartMinin = ecartMinin.replace(" ","");     iEcartHeure = Integer.parseInt(ecartHeure); iEcartHeure = 0 - iEcartHeure;   iEcartMin = Integer.parseInt(ecartMinin);   ecartMin = (iEcartHeure * 60) - iEcartMin;     logger.debug("ecartMin NEGATIVE : "+ecartMin);   }           ecartConstate += ecartMin;     ecartConstate = ecartConstate - heureSuppAreporte; logger.debug("** ecartConstate ** : "+ecartConstate);   String dureeTravailRealise = getTotalHours(dureeDeTravAnneeFait); moisDis.setTotalTravMois(dureeTravailRealise); logger.debug("!! dureeTravailRealise !!" + dureeTravailRealise);     String dureeTravailArealise = getTotalHours(dureeDeTravAnneeAfaire); logger.debug("!! dureeTravailArealise !!" + dureeTravailArealise); moisDis.setTotalTravAfaire(dureeTravailArealise);   String ecartConstateHeure = getTotalHours(ecartConstate); moisDis.setEcart(ecartConstateHeure); logger.debug("!! ecartConstateHeure !! " + ecartConstateHeure);       if(sum.containsKey(idPers)){ sum.put(idPers, moisDis); System.out.println("moisDis solde CPAYE : "+moisDis.getNbSoldeCPaye()); }else{   Moisdisp moisDis2 = new Moisdisp(); sum.put(idPers, moisDis2); System.out.println("moisDis2 solde **** : "+moisDis2.getNbSoldeCPaye()); }     idPersonneOld = idPers; nbMoisCompte +=1 ;   listMoisDipsParPersonne.add(moisDis);           System.out.println("idPersonneOld : "+idPersonneOld); System.out.println("idPers : "+idPers); System.out.println("nbMoisCompte : "+nbMoisCompte);     }               System.out.println("moisDips.size() : "+listMoisDipsParPersonne.size()); return listMoisDipsParPersonne;     }

---

Néanmoins

Code:

---

1 2 3 4 5 6 7 8 9 10 11

if(sum.containsKey(idPers)){ sum.put(idPers, moisDis); System.out.println("moisDis solde CPAYE : "+moisDis.getNbSoldeCPaye()); }else{   Moisdisp moisDis2 = new Moisdisp(); sum.put(idPers, moisDis2); System.out.println("moisDis2 solde CPAYE **** : "+moisDis2.getNbSoldeCPaye()); }

---

me renvoie toujours une valeur nulle U_U

Bonjour leara500,

Je vois que le problème n'est pas réglé.

Pour mieux avancer je te conseille de réduire le code que tu donne dans la discussion pour permettre au gens de mieux te répondre. Pour cela tu devrais enlever tous les calcul car je pense que cette partie ne pose pas de problème.

Laisses deux ou trois valeurs a calculer pour nous permettre de rapidement parcourir ton code et aller directement à l'essentiel.

Je n'ai pas eu le temps de lire tout le code mais je te conseille de travailler avec les mêmes objets que tu initialise en entrant dans chaque itération.
A la fin de tes calculs tu dois:

SI l'objet existe ALORS
-récupérer de ta Map l'objet à l'indice voulu.
-additionner les nouvelles valeurs aux anciennes
-ajouter l'objet modifié
SINON
-ajouter l'objet à la Map avec l'indice voulu.

Concernant le test d'existance, il vaut mieu travailler avec les key qu'avec contains qui est plus adaptée pour les List.
tu peux le faire a travers une boucle :aie: mais tu peux aussi aller voir de ce cote et te pencher sur containsKey ;)

https://docs.oracle.com/javase/7/doc.../util/Map.html

regardes aussi les autres fonctions de récupération des valeurs V (moisDisp dans ton cas pour additionner les nouvelles valeurs)

Ne pas oublier les test sur la nullité et isEmpty aussi

Ajouter des blocs try/catch pour intercepter les exceptions

Voila

Bon courage

Déjà, tu as mal pris mon exemple, à aucun moment tu fais de l'addition (johnlobs a donné une explication de plus). Ensuite, tu n'as jamais affecté la valeur récupéré de NbSoldeCPaye depuis la requête dans moisDis, moisDis.getNbSoldeCPaye() retournera toujours un null et il faut faire attention avec, comme disait johnlobs.

A+.

Bonjour,

Je reviens vous voir, toujours pour mon problème d'algorithme....

Finalement, je n'ai pas utilisé les Map car, honnêtement je n'y comprend rien :oops:

Enfin bref, si je viens vous voir, si pour vous dire que je suis toujours confronté à un problème d'algo mais à un autre niveau.

En effet, j'ai réussi à calculer les informations par personne (ouf).

Mais (car oui il y a un mais) il ne fonctionne correctement que pour la première personne. Pour les autres personnes, il oublie de comptabiliser le 1er mois à chaque fois:oops: T0T (c'est à dire le mois de mai de l'année n) Par contre, il comptabilise tout le reste

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

if (!listMoisDipsParPersonne.isEmpty()){ int personneCourante = 0; int personneCouranteOld = 0;   int sumTotalCongePaye = 0; int sumTotalSoldeCongePaye = 0; int sumTotalNbAncien = 0; int sumTotalNbCongeParental = 0 ; int sumTotalNbCt = 0 ; int sumNbDelegation = 0; int sumNbEnfantMalade = 0; int sumNbExcept = 0 ; int sumNbFamil = 0 ; int sumNbFerie = 0; int sumNbFormation = 0; int sumNbMaladie = 0 ; int sumNbRecuperation = 0 ; int sumNbRecupFerie = 0;   int nbMois = 0;   Moisdisp moisDis = new Moisdisp();   Iterator<Moisdisp> iter = listMoisDipsParPersonne.iterator(); while (iter.hasNext()) { Moisdisp moisdisplay = (Moisdisp) iter.next();   personneCourante = moisdisplay.getIdPers();   if (personneCourante == personneCouranteOld || personneCouranteOld == 0 ){   if(moisdisplay.getNbCannuel() != null) sumTotalCongePaye += moisdisplay.getNbCannuel(); if(moisdisplay.getNbSoldeCPaye() != null) sumTotalSoldeCongePaye += moisdisplay.getNbSoldeCPaye(); if(moisdisplay.getNbCancien() != null) sumTotalNbAncien += moisdisplay.getNbCancien(); if(moisdisplay.getNbCongeParental() != null) sumTotalNbCongeParental+= moisdisplay.getNbCongeParental(); if(moisdisplay.getNbCt() != null) sumTotalNbCt += moisdisplay.getNbCt(); if(moisdisplay.getNbDelegation() != null) sumNbDelegation += moisdisplay.getNbDelegation(); if(moisdisplay.getNbEnfantMalade() != null) sumNbEnfantMalade += moisdisplay.getNbEnfantMalade(); if(moisdisplay.getNbExcep() != null) sumNbExcept += moisdisplay.getNbExcep(); if(moisdisplay.getNbFamil() != null) sumNbFamil += moisdisplay.getNbFamil(); if(moisdisplay.getNbFerie() != null) sumNbFerie += moisdisplay.getNbFerie(); if(moisdisplay.getNbFormation() != null) sumNbFormation += moisdisplay.getNbFormation(); if(moisdisplay.getNbMaladie() != null) sumNbMaladie += moisdisplay.getNbMaladie(); if(moisdisplay.getNbRecup() != null) sumNbRecuperation += moisdisplay.getNbRecup(); if(moisdisplay.getNbRecupFerie() != null) sumNbRecupFerie += moisdisplay.getNbRecupFerie();       nbMois +=1;     moisDis.setIdPers(moisdisplay.getIdPers());   moisDis.setNbHeureSuppPaye(moisdisplay.getNbHeureSuppPaye());   moisDis.setNomPersonne(moisdisplay.getNomPersonne()); moisDis.setPrenomPersonne(moisdisplay.getPrenomPersonne());   moisDis.setNbCannuel(sumTotalCongePaye); moisDis.setNbSoldeCPaye(sumTotalSoldeCongePaye); moisDis.setNbCancien(sumTotalNbAncien); moisDis.setNbCongeParental(sumTotalNbCongeParental); moisDis.setNbCt(sumTotalNbCt); moisDis.setNbDelegation(sumNbDelegation); moisDis.setNbEnfantMalade(sumNbEnfantMalade); moisDis.setNbExcep(sumNbExcept); moisDis.setNbFamil(sumNbFamil); moisDis.setNbFerie(sumNbFerie); moisDis.setNbFormation(sumNbFormation); moisDis.setNbMaladie(sumNbMaladie); moisDis.setNbRecup(sumNbRecuperation); moisDis.setNbRecupFerie(sumNbRecupFerie);   }   if ((personneCouranteOld != personneCourante && personneCouranteOld != 0) || !iter.hasNext()){ listMoisDipsParPersonne2.add(moisDis); moisDis = new Moisdisp();   sumTotalCongePaye = 0; sumTotalSoldeCongePaye = 0;   sumTotalNbAncien = 0; sumTotalNbCongeParental = 0 ; sumTotalNbCt = 0 ; sumNbDelegation = 0; sumNbEnfantMalade = 0; sumNbExcept = 0 ; sumNbFamil = 0 ; sumNbFerie = 0; sumNbFormation = 0; sumNbMaladie = 0; sumNbRecuperation = 0 ; sumNbRecupFerie = 0;   dureeDeTravAnneeAfaire = 0; dureeDeTravAnneeFait = 0 ; dureeDeTravAnneeEffective = 0; dureeDeTravAnneeDimanche = 0; dureeDeTravAnneeFerie = 0; dureeDeTravAnneeReposCompensateur = 0; dureeDeTravAnneeMaladieMois = 0; dureeDeTravAnneeHeureSansSolde = 0; nbMois = 0;     }   personneCouranteOld = personneCourante ; } }

---

Pièce jointe 414089



Comme vous pouvez le voir, la personne 1 a bien les 10 mois qui sont comptabilité. Or, pour la personne 2, il y a 7 mois qui sont comptabilisé au lieu des 8 présents.... Et ainsi de suite pour les autres personnes....

Je vous remercie infiniment pour votre aide

Bonjour,

Je n'ai rien compris à ce que tu veux faire, mais pire, je n'ai rien compris à ton code qui est trop complexe, et qui pour moi n'est pas maintenable.

Néanmoins, j'ai vu que tu utilises Hibernate, et je pense que ton probleme est lié à une mauvaise maîtrise d'Hibernate et/ou une mauvaise connaissance des concepts d'Hibernate.

Il faut d'abord comprendre que Hibernate est un ORM.

C'est un framework qui va te permettre de faire le lien entre un modèle objet, au sein de ton application, et un modèle relationnel au sein de ta base de données.

Pour que l'on puisse d'aider, il faudrait d'abord que tu nous donne l'UML (Côté Java).
Si j'ai bien compris, "Moisdisp" est un Objet côté Hibernate qui mappe une table en BDD.

La requête permet à Hibernate d'aller chercher en BDD les données, et de mapper pour récupérer directement les objets.

Dans ton cas, il y a un vrai travail de simplification de requête à faire.

Pour comprendre, soit une classe Personne ayant comme attribut id, nom et prénom.

Tu écris une requête comme:

Code:

---

select pers.id, pers.nom, pers.prenom from Personne pers

---

Franchement, ça ne sert à rien, vu que Personne est mappé avec la BDD.

Tu peux écrire si tu veux être précis:

Code:

---

select pers from Personne pers

---

Ou simplement:

Code:

---

from Personne

---

Hibernate va te chercher la liste des personnes en BDD et te donnera la liste d'objet Java.

Si tu cheche une personne par id:

Code:

---

select pers from Personne pers where pers.id = ?

---

Il en est de meme pour les jointure.
Normalement, elles sont déjà dans ton modèle Java.

Un moment, tu écris:

Code:

---

moisdisp.idPers = personne.idPersonne

---

Franchement, si le modèle est bien fait, côté Java, on doit avoit un lien entre la classe Moisdisp et Personne.
On a donc pas, normalement besoin de spécifier la personne dans la requête.
Et on sélectionne simplement par:

Code:

---

moisdisp.personne.idPers = ?

---

Code:

---

personne.service.idService = uniteservice.service.idService and uniteservice.unite.idUnite = unite.idUnite and unite.idUnite = ?

---

Est remplaçable par:

Code:

---

personne.service.idService = ? and personne.service.unite.idUnit = ?

---

ect...

Bonjour,

D'après ce que je vois, tu perdras toujours une personne à cause de ta condition.

On prends un exemple simple:

Nous avons TOTO TOTO TITI TITI
Au debut ,premiere itération: personneOld = 0 donc calcul
deuxieme iteration : TOTO = TOTO donc calcul
troisieme itération : TOTO != TITI et personneOld != 0 donc RIEN
Quatrième itération : TITI = TITI DONC CALCUL

tu perds donc un calcul.
J'ai pas vraiment lu ton code car trop long mais je pense que c'est bien cela le problème.

Bon courage.

Citation:

---

Envoyé par PhilippeGibault

Bonjour,

Je n'ai rien compris à ce que tu veux faire, mais pire, je n'ai rien compris à ton code qui est trop complexe, et qui pour moi n'est pas maintenable.

Néanmoins, j'ai vu que tu utilises Hibernate, et je pense que ton probleme est lié à une mauvaise maîtrise d'Hibernate et/ou une mauvaise connaissance des concepts d'Hibernate.

Il faut d'abord comprendre que Hibernate est un ORM.

C'est un framework qui va te permettre de faire le lien entre un modèle objet, au sein de ton application, et un modèle relationnel au sein de ta base de données.

Pour que l'on puisse d'aider, il faudrait d'abord que tu nous donne l'UML (Côté Java).
Si j'ai bien compris, "Moisdisp" est un Objet côté Hibernate qui mappe une table en BDD.

La requête permet à Hibernate d'aller chercher en BDD les données, et de mapper pour récupérer directement les objets.

Dans ton cas, il y a un vrai travail de simplification de requête à faire.

Pour comprendre, soit une classe Personne ayant comme attribut id, nom et prénom.

Tu écris une requête comme:

Code:

---

select pers.id, pers.nom, pers.prenom from Personne pers

---

Franchement, ça ne sert à rien, vu que Personne est mappé avec la BDD.

Tu peux écrire si tu veux être précis:

Code:

---

select pers from Personne pers

---

Ou simplement:

Code:

---

from Personne

---

Hibernate va te chercher la liste des personnes en BDD et te donnera la liste d'objet Java.

Si tu cheche une personne par id:

Code:

---

select pers from Personne pers where pers.id = ?

---

Il en est de meme pour les jointure.
Normalement, elles sont déjà dans ton modèle Java.

Un moment, tu écris:

Code:

---

moisdisp.idPers = personne.idPersonne

---

Franchement, si le modèle est bien fait, côté Java, on doit avoit un lien entre la classe Moisdisp et Personne.
On a donc pas, normalement besoin de spécifier la personne dans la requête.
Et on sélectionne simplement par:

Code:

---

moisdisp.personne.idPers = ?

---

Code:

---

personne.service.idService = uniteservice.service.idService and uniteservice.unite.idUnite = unite.idUnite and unite.idUnite = ?

---

Est remplaçable par:

Code:

---

personne.service.idService = ? and personne.service.unite.idUnit = ?

---

ect...

---

Je suis tout à fait en phase avec ce commentaire, si ce que tu veux faire est trop compliqué à faire c'est que peut être tu ne t'y prends pas de la bonne manière, la manière dont tu veux le faire en terme de maintenabilité et d'évolution c'est n'est pas top, ça peut marché sur le coup, mais dans 2 mois, il se pourrait que tu ne comprendras plus ce que tu avais fais à plus forte raison une autre personne :calim2:

PhilippeGibault> La difficulté est que je récupère fait par une autre personne et qu'effectivement, cette application est difficilement maintenable. Sauf que je n'ai pas le temps d'un refaire une autre YYoYY


johnlobs > Merci infiniment de votre réponse. Le problème vient effectivement de là...

Auriez vous une solution pour modifier ma condition ?

J'ai de nouveau simplifié mon code.

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

if (!listMoisDipsParPersonne.isEmpty()){ int personneCourante = 0; int personneCouranteOld = 0;   int sumNbFamil = 0; int sumNbFerie = 0; int sumNbFormation = 0; int sumNbMaladie = 0;   Moisdisp moisDis = new Moisdisp();   Iterator<Moisdisp> iter = listMoisDipsParPersonne.iterator(); while (iter.hasNext()) { Moisdisp moisdisplay = (Moisdisp) iter.next();   personneCourante = moisdisplay.getIdPers();   if (personneCourante == personneCouranteOld || personneCouranteOld == 0 ){     if(moisdisplay.getNbFamil() != null) sumNbFamil += moisdisplay.getNbFamil(); if(moisdisplay.getNbFerie() != null) sumNbFerie += moisdisplay.getNbFerie(); if(moisdisplay.getNbFormation() != null) sumNbFormation += moisdisplay.getNbFormation(); if(moisdisplay.getNbMaladie() != null) sumNbMaladie += moisdisplay.getNbMaladie(); nbMois +=1;   moisDis.setNbFamil(sumNbFamil); moisDis.setNbFerie(sumNbFerie); moisDis.setNbFormation(sumNbFormation); moisDis.setNbMaladie(sumNbMaladie);       }   if ((personneCouranteOld != personneCourante && personneCouranteOld != 0) || !iter.hasNext()){ listMoisDipsParPersonne2.add(moisDis); moisDis = new Moisdisp();   sumNbFamil = 0 ; sumNbFerie = 0; sumNbFormation = 0; sumNbMaladie = 0;   nbMois = 0;     }   personneCouranteOld = personneCourante ; } } }

---

Je vous remercie par avance de votre aide....

Bonjour,

Ça me rassure que ma réponse t'ait indiquer la source de ton problème.
Je peux te donner la solution pour resoudre ce probleme mais etant donné que je ne connais pas tout ton algo il faudra que tu traite la suite.

D'après ce que j'ai compris, tu as mis la condition uniquement pour pouvoir additionner dans les mêmes variables :

Ta condition revient a dire : si c'est la première personne ou bien si c'est la même personne fais les calculs.

Ce qui est faux car tu dois scinder cette condition car ça ne devrait pas être le même traitement mais plutôt:

SI c'est la premiere personne ou bien si c'est une personne differente alors nouvelle variable et nouveau calcul
SINON additionner aux ancienne variables.

Ce qui correspond a:

Code:

---

1 2 3 4 5 6 7 8 9 10

if(personneCouranteOld == 0 || personneCourante != personneCouranteOld) { // initialiser les variables et affecter les valeurs actuelles sumNbFamil = 0; if(moisdisplay.getNbFamil() != null) sumNbFamil += moisdisplay.getNbFamil(); }else{ // additionner les valeurs au variables if(moisdisplay.getNbFamil() != null) sumNbFamil += moisdisplay.getNbFamil(); // etc ..... }

---

J'ai mis ce code non optimisé pour que tu comprennes mais ce code reviendrait a faire:

Code:

---

1 2 3 4 5

if(personneCouranteOld == 0 || personneCourante != personneCouranteOld) sumNbFamil = 0; //etc ...... //Faire les calculs dans tous les cas if(moisdisplay.getNbFamil() != null) sumNbFamil = moisdisplay.getNbFamil(); //etc ......

---

Voila mais ATTENTION il faut que tu sauvegarde tes valeurs dans un objet (d'où la proposition de Map ) à part si tu traite la chose differemment car le problème est que tu dois savoir lorsque tu passe à une autre personne pour ne pas perdre tes calculs mais je pense que ce point est deja traité vu que tu arrive a afficher par personne.

Tu peux meme ajouter un flag dans le if pour savoir que tu es passée à une autre personne et donc qu'il faut stocker les calculs et pour cela tu dois separer ton if en deux if comme cela:
SI c'est la premiere personne ALORS on initaialise la variable
SINON SI on change de personne ALORS on stocke d’abord les résultats puis on réinitialise.

Bon courage :D

Vu de très loin, je crois qu'il y a un problème de modélisation.

Surtout qu'en general, on considère que la majorité du calcul doit être fait en BDD.

D'abord côté BDD:
Je suis persuadé qu'il y a un lien (Toujours vu de loin) entre Moisdisp et Personne.
La colonne idPers de la table Moisdisp ne peut pas prendre n'importe quoi comme valeur. Elle ne peut prendre que un identifiant dans la table personne (Colonne idPersonne de la Table Personne).
La colonne idPers de la table Moisdisp est donc une clé étrangère. Je pense en plus (toujours vu de loin) que ce lien est fonctionnel. Et fonctionnellement, il est représenté par une clé étrangère.

Côté UML/Hibernate/Java
Il y a un lien entre la classe Moisdisp et Personne. A un Moisdisp correspond 1 Personne et à une Personne correspond 0-N Moisdisp.
Ce qui fait que dans la classe Personne, il n'y a pas de reference idPers, il n'y a pas d'attribut idPers.
Au contraire, il y a dans la classe Moisdisp un attribut Personne, le mapping est fait à l'aide d'annotation Hibernate, à partir de la clé étrangère en BDD.
Attention à utiliser un chargement "LAZY" histoire de ne pas plomber les performances.

En faisant ces deux modifications, tu poseras mieux le problème et de fait, tu fera un code plus simple, plus lisible et plus maintenable.

La conclusion est que quand on travaille avec la BDD/Hibernate, il faut toujours avoir sous les yeux un MCD/UML diagramme de classe.

Bonjour @PhilippeGibault,

D’après ce qu'elle nous a communiqué comme infos, la table moisDisp est une table qui n'a pas été créée par elle et je pense perso que cette table est déjà mappée et je suppose que son mapping est correct;

Code:

---

personneCourante = moisdisplay.getIdPers();

---

Cela se confirme bien par le code suivant qui nous montre clairement que la clé étrangère existe bel et bien dans cette table.

Maintenant si tu regarde bien la photo qu'elle a joint à son message , tu constate que dans cette photo il y a deux parties .
La première montre la table moisdisp dans laquelle on a pour chaque personne tous les mois et un recap des conges (par exemple pour ce mois).

La deuxième partie de la photo contient un petit tableau avec 4 lignes (car 4 personnes) et dans la ligne le total des congés(par exemple si on prend la colonne congés payés) pour cette personne à l'année ET je pense que c'est sur cette partie qu'elle travaille.

Son problème est donc algorithmique et non pas architectural(je pense je n'ai pas vu ses classes ni son mapping).

Donc pour mieux cerner le problème, elle charge les données de la table moisDisp et elle veut fédérer les infos dans une seule ligne pour chaque personne.

Elle peux comme tu as tres bien dis passer directement par requête et charger les données calculées grâce au fonctions d'agrégation sql .Elle peut aussi créer une vue pour ce travaille est directement passer par un select sur la vue(car les données de la vue ne change pas je pense vu que ce sont des congés etc qui, le mois etant fini ne changeront pas ) comme elle peut créer une vue matérialisée si elle est sous oracle ou encore le faire par alimentation de la table à partir de la vue+ procedure + triggers ETC.....ce qui lui permettra d'avoir des donnée à jour(sait on jamais).

Maintenant le problème comme l'indique le titre est d'ordre algorithmique et plus exactement un simple test pour pouvoir boucler et calculer ,stocker les bonnes valeurs aux bons endroits.

Je suis d'accord avec toi concernant le cote BDD.
Nous lui avons aussi proposé d'utiliser les map pour faciliter son stockage et ses accès à ces données fédérées si elle veut le faire cote java.

Mais sa question ne concernait que le test qui lui pose problème car ses résultats sont faux ( tuples non traités).

Voila j’espère ne pas être hors sujet ou bien à cote de la plaque :mouarf:

Bon courage

Sincèrement, connaissant très bien Hibernate (et pas trop mal les BDD), je ne suis vraiment pas d'accord.

Je pense qu'il y a vraiment un problème de modélisation en BDD et/ou un mauvais mapping Hibernate.

Effectivement, on a :

Code:

---

int personneCourante = 0;

---

Or, si on avait une bonne modélisation et un bon mapping Hibernate, on devrait avoir:

Code:

---

Personne personneCourante;

---

On récupèrerait:

Code:

---

personneCourante = moisdisplay.getCourantePersonne();

---

De fait, on aurait un chargement EAGER (et non pas LAZY comme je l'ai dit précédement).

Ce mauvais mapping, lié peut-être à une mauvaise modélisation (ayant pour cause une clé étrangère dans le cas présent) fait que l'on a du mal à comprendre le problème, et que l'on a du mal à avoir un code propre, lisible et maintenable.

Pire, on fait une requête HQL vraiment indigeste.

Si il y avait une bonne modélisation de la base de données et/ou surtout un bon mapping Hibernate, je suis persuade que le problème serait mieux posé, et surtout qu'on le résoudrait avec seulement une requête HQL.

On n'aurait pas de boucle FOR/WHILE (Hibernate donnant directement la réponse) et on éviterait un code qui est vraiment une usine à gaz et qui va transformer la personne qui va le maintenir en veritable psychopathe.

Bonjour,

Citation:

---

Je pense qu'il y a vraiment un problème de modélisation en BDD et/ou un mauvais mapping Hibernate.

---

Je suis d'accord sur le fait que le mapping peut ne pas être bon mais nous n'avons pas à "penser" car nous n'avons pas assez de matière pour trancher.

Citation:

---

on éviterait un code qui est vraiment une usine à gaz et qui va transformer la personne qui va le maintenir en veritable psychopathe.

---

Tout à fait d'accord mais il faut savoir que c'est le cas d'un très grand nombre d'application(même avec un bon mapping) qui rendent les personnes qui maintiennent nerveux.
On ne peut malheureusement pas corriger tout le code et refaire les mapping car il y a souvent une forte corrélation entre modules et toucher à un module a souvent des répercussion sur d'autres et génère des régression un peu partout (réalité vs utopie).
Ce que l'on fait souvent ,c'est relancer de nouveaux devs en parallele aux maintenances (sujet à traiter ailleurs).

De toutes les facons, la personne concernée n'est plus la car elle a eu la réponse a sa question et s'est éclipsée sans taguer la discussion.

Merci pour ton analyse pertinente @PhilippeGibault.

A bientot...

Merci infiniment de vos réponses

Citation:

---

une usine à gaz et qui va transformer la personne qui va le maintenir en véritable psychopathe.

---

C'est déjà le cas... Quoiqu'il en soit merci de vos réponses qui m'ont donné matière à réfléchir

Dès que j'aurai trouve le bon algo, je vous en ferai part.

Quoiqu'il en soit merci pour tout