Reprise du message précédent :

Dire que je trouve que je bosse sur un truc mal codé et lent, avec une architecture dégueulasse... Ben là d'un coup je me sens presque privilégié

 
C'est le toString() à répétition qui est un peu con, StringBuffer étant un CharSequence tu peux faire sb.charAt(sb.length() -1) pour trouver le dernier caractère

Va pas marcher si t'as un path avec des séparateurs UNIX sous Windows.

Ça casse pas exactement des briques comme critique. Et comme le note __alt à la base il n'y a pas besoin des toString() en fait.

Mais je parlais plus line-by-line que les détails au sein des expressions. Tu dégages le StringBuffer tu gagnes une ligne, le reste reste, et je présume que c'est pas de ça dont jubi parlait en mentionnant cette section d'imbroglio de merde.

Franchement c'est gentillet comparé à aller lire un XML ligne par ligne pour trouver une regexp (extraite à grand coups de substring avec un startswith, quelqu'un définit un autre pattern et le met au dessus yapuquimarche), checker si un formatage correspond ou en utiliser un autre (complètement arbitrairement) si ça match pas, avec le bonus que le xsd est utilisé pour valider des dates à grands coups de regex au lieu d'utiliser xs::dateTime et d'utiliser des dates ISO-8601 standard. Et c'est fait à chaque fois qu'une date est sérialisée ofc. Tu sais jamais, des fois qu'un mec aille éditer la regex dans le XSD derrière ton dos pendant que le soft tourne, faut prévoir.

Bon ok on peut quand même noter que la méthode démarrait avec une version custom de java.util.Property, qui fait techniquement partie du snippet de jubi, que ça sentait un peu la mort.

Le check du dernier caractère peut être fait sur le xsdPath qui est déjà là et déjà un String

Certes. Mais dans le cas present j'arrive pas a imaginer un cas qui foire en utilisant File.separator.

Tu peux developper?

Si xsdPath se finit par un / et que tu testes avec endsWith(File.separator), sous windows tu vas te retrouver avec un second / ajouté parce-que le File.separator va pas matcher. Je sais pas si les API windows dealent correctement (et si elles dealent correctement, tu peux juste faire sauter la conditionnelle et toujours injecter ton /)

C'est une grosse critique d'un petit problème, vu ce qu'il y a autour c'est un peu se faire une fourmilière au napalm: techniquement, c'est indéniablement correct, mais pratiquement ça aurait pu être utilisé ailleurs.

Et quand on sait à quoi ça sert...
 
Historiquement, y'a 2 formats de date : celui en dur et un autre. Le code doit pouvoir prendre en charge les deux (cas qui devrait jamais arriver s'ils valident les fichiers avec le XSD, mais je sais pas s'ils le font)
 
En tout cas, le code Java généré - et peut être modifié après - ne le fait pas : il se tamponne de la Regexp.
 
Donc bon, il y a 2 formats, CONNUS, et ces cons là en mettent un en dur dans le code et vont chercher l'autre dans un XSD... Sans dec.  
 
A la base, tu mets tes 2 patterns dans ton fichier, et c'est marre. Si tu sais que le format peut encore évoluer, a la limite, tu utilises ton super système de paramétrage en DB pour les y coller, mais ça, c'est juste débile.

Je parlais de ce qu'a dit Alt : la séquence d'action me paraît un peu fouillie : je comprends pas la plus value d'aller chercher un chemin dans un fichier de conf, et lui append la fin du nom du fichier...
 
Pourquoi ne pas mettre le chemin complet (non du fichier inclut) directement dans le fichier de conf, et checker si le fichier existe

 
Ca fait partie des nombreuses whathefuckeries de ce code.
 

 
Rechercher un fichier avec des / dans le filesystem normal sous windows ne marche pas, donc dans les deux cas ca marche pas. C'est vrai que je suis parti du principe que c'est un fichier local, et que c'est pas une resource dans un jar ou une url.

Une seule clé de conf qui donne un répertoire et tu peux avoir une tétrachiée de fichiers dedans?

Toutes les APIs système et la majorité des UIs l'acceptent (y compris Explorer). Il y a un seul endroit où ça passe pas quand tu utilises correctement les APIs système, c'est après le \\? dans un chemin UNC. C'est accepté depuis DOS 2.0.

j'imagine que c'est la raison, mais ça rend un détail d'ops (la localisation du fichier de conf) hardcodée dans ton code

Non mais ça prend plus de place en mémoire ça

from topic imges :
 

Prem's.
 
Bonjour,
Aurevoir.

 
On parlait de java je croyais...

Nico : http://m.instructables.com/contest/cncchallenge/

J'ai fait ça hier soir https://www.youtube.com/watch?v=Gwrp5xb4kjc

Beau boulot : http://www.instructables.com/id/DI [...] per-motor/

http://www.dailykos.com/story/2015 [...] s-amazing#

Je m'en lasse pas..

private Query constructCriteria(final CriteresRechercheDossier criteres) {
        
        // passe les recherches en mode caseInsensitive et accentsInsensitive
        Utils.setCaseInsensitiveSession(em);
 
        String qlString = "SELECT d FROM " + persistentClass.getName() + " d ";
 
        /*
         * Sous-requête pour récupérer les dernières versions des dossiers
         */
        String subQlString = "(SELECT max(ld.numVersion) " +
                "FROM " + persistentClass.getName() + " ld " +
                "WHERE ld.numDossier = d.numDossier)";
        
        qlString += "WHERE d.numVersion = " + subQlString + " ";
 
        /*
         * Requête pour récupérer les dossiers correspondant au contexte
         */
        if(criteres.getContexte().equals("suppression" )){
            qlString += "AND d.dateDebutPlacement > :today ";
        }
        else if(criteres.getContexte().equals("archivage" )){
            qlString += "AND d.etatMesure.code = '" + Constants.ETAT_MESURE_EN_COURS + "' "; 
        }
        else if(criteres.getContexte().equals("desarchivage" )){
            qlString += "AND d.etatMesure.code = '" + Constants.ETAT_MESURE_CLOS + "' ";
        }
 
        /*
         * Requête de recherche par identifiant
         */
        if (TYPE_RECHERCHE_IDENTIFIANT.equals(criteres.getTypeRecherche())) {
            qlString += "AND d.numDossier = " + criteres.getIdentifiant() + " ";
        }
 
        /*
         * Requête de recherche par identité
         */
        else if (TYPE_RECHERCHE_IDENTITE.equals(criteres.getTypeRecherche())) {
            qlString += "AND d.individu IN ";
            String subQlStringIndividu = " (SELECT i FROM Individu i ";
 
            // Critère sur le nom (pouvant être un nom partiel)
            if (com.equipelogiciel.fwk.utils.StringsUtils.containsEtoile(criteres.getNom())) {
                subQlStringIndividu += "WHERE i.nomFamille LIKE '" + ConvertUtil.convetEtoileToPourcent(criteres.getNom()) + "' ";
            } else {
                subQlStringIndividu += "WHERE i.nomFamille LIKE '" + criteres.getNom() + "' ";
            }
 
            // Critère sur le prenom (pouvant être un prenom partiel)
            if (!StringUtils.isEmpty(criteres.getPrenom())) {
                if (com.equipelogiciel.fwk.utils.StringsUtils.containsEtoile(criteres.getPrenom())) {
                    subQlStringIndividu += "AND i.prenoms LIKE '" + ConvertUtil.convetEtoileToPourcent(criteres.getPrenom()) + "' ";
                } else {
                    subQlStringIndividu += "AND i.prenoms LIKE '" + criteres.getPrenom() + "' ";
 
                }
            }
            if (!StringUtils.isEmpty(criteres.getLimiteSexe())) {
                subQlStringIndividu += "AND i.sexe.code LIKE '" + criteres.getLimiteSexe() + "' ";
            }
            if (com.equipelogiciel.fwk.utils.StringsUtils.isDateValide(criteres.getJourNaissance(), criteres
                    .getMoisNaissance(), criteres.getAnneeNaissance())) {
                subQlStringIndividu += "AND i.dateNaissance LIKE :dateNaissance";
            }
            subQlStringIndividu += " )";
            qlString += subQlStringIndividu;
        }
 
        // Restriction disp (évolution)
        if (criteres.getDispId() != null
                && !criteres.getContexte().equals("consultation" )
                && !criteres.getContexte().equals("enquetePlace" )
                && !criteres.getContexte().equals("majPlace" )) {
            final Disp disp = dispDao.getById(criteres.getDispId());
            qlString += "AND d.disp LIKE '" + disp + "' ";
        }
 
        // Interdiction des actions sur les dossiers archivés (évolution)
        if (!criteres.getContexte().equals("desarchivage" )
                && !criteres.getContexte().equals("consultation" )
                && !criteres.getContexte().equals("enquetePlace" )
                && !criteres.getContexte().equals("majPlace" )) {
            qlString += "AND d.indicateurArchivageBase LIKE 'N' ";
        }
 
        if(LOG.isDebugEnabled() && qlString!=null){
            LOG.debug("Critere de recherche : " + qlString);
        }
        
        qlString += "ORDER BY d.individu.nomFamille, d.individu.prenoms";
        Query q = em.createQuery(qlString);
        
        if(criteres.getContexte().equals("suppression" )){
            final java.sql.Date sqlDate = new java.sql.Date(new Date().getTime());
            q.setParameter("today", sqlDate);
        }
        if (com.teamlog.fwk.utils.StringsUtils.isDateValide(criteres.getJourNaissance(), criteres
                .getMoisNaissance(), criteres.getAnneeNaissance())) {
            q.setParameter("dateNaissance", construireCritereDate(criteres));
        }
        
        return q;
    }

Gaffe y'a des noms de package identifiables.

on dirait du code d'ici, mais en mieux commenté

m'en fous, equipelogiciel a fermé

 
sandekonaÿ ????

Même, je trouve que c'est pas classe de taper sur un partenaire/client/fournisseur, ne serait-ce que pour ne pas être toi même reconnu et te faire une mauvaise pub

...

 
les PreparedStatement c'est pour les faibles...
 
y'a 2-3 trucs bien, mais ça va pas au bout : par ex les passages taggués "évolution", je trouve ça bien, mais ça aurait pas couté plus cher de mettre la référence du ticket (à moins que ce soit dans le commentaire de commit, mais là comme ça je doute)

Lisieux. un responsable info où il ya du dev, de infrastructure et des gros automat' !!

 
ça va se finir en larmes tout ça

 
C'est p-ê l'occasion de proposer un document avec des règles de codage (ou des améliorations du document existant). Et ensuite propose une réunion d'exposition des règles, avec ce qui est bien, ce qui ne va pas, et ce qui doit être amélioré. Une telle démarche me paraîtrait positive de la part d'un dev senior.

C'est une reprise de TMA : ce code, on l'a pour 4 ans, mais il est pas à nous à la base.

Evidemment qu'on a des règles de codage, et qu'on va virer tout ce merdier.

Ou alors on fait comme le titulaire actuel : non seulement on améliore rien, mais on écrit de la merde aussi, dans le même esprit, histoire qu'il y ait une certaine cohérence.

Ben non, l'idée, c'est justement d'éviter ça.

Ouais enfin ça c'est la vision du développeur.

Le PM lui il voit qu'on lui dit "il faut que ça soit fini pour vendredi, heureusement un truc dans le genre a déjà été fait, reprend le gros machin là, ça marche pareil" et le budget qui va avec.
Enfin je suis pas dans le projet hein, j'imagine juste.

Pour m'être déjà bouffé des TMA bien pourries je peux te dire que même quand tu sais qu'il faut tout péter et réécrire c'est parfois juste impossible pour cause de facteur temps / budget / pression politique / connerie des décisionnaires.
Alors tu bricoles avec ce qu'on te donne en te disant "et merde, dire que j'associe mon nom à ça, je me demande ce que va se dire à mon propos le pauvre gars qui reprendra cette merde dans quelques mois".

 
... Fallait vraiment un redface?

 
En l'occurence, on a 1.5 ETP sur 4 ans pour une appli de 50k lignes, dont au moins 50% de gras (et une bonne partie qu'on peut virer à peu de frais), or ici ils bossaient avec 1ETP qui avance comme un escargot paralytique, et un stagiaire.
 
Donc pour 1.5 ETP les premier mois, on va tout refaire bien propre, et sur les 40 mois qui restera, on aura 0.5 ETP occupé, et le reste en fonction des besoins..  
Au passage, le client va pouvoir constater l'amélioration des perfs, des métriques de qualité (que le présent presta ne fournissait pas), et des temps de résolution d'erreur, tout le monde sera  gagnant.

Au moins le client y met les moyens, effectivement ça a l'air prometteur

Perso j'ai viré le peu de code jpql que j'avais et j'ai remplacé par du jooq.  
En plus tu peux générer les classes jooq à partir des classes JPA annotées.
 
Et ça oblige pas à casser ce que t'as déjà, tu remplace petit à petit.

 
C'est une petite TMA qui peut rapporter gros, ouais. Surtout qu'il y en a une autre avec le même client à gagner, pour 20 ETP pendant 4 ans.

Plus de TVA à 5 sur les ebooks

Une idée d'un stockage online pour un fichier de 700 Mo, gratuit, là tout de suite?

[edit] Sur Google Drive, y'a une taille limite pour un fichier?

(bon, en fait c'est bon : la DSI du titulaire de la TMA m'annoncait 4h pour copier 700 Mo de dump SVN (déjà, 700 Mo de dump... Pour savori comment arriver à ce résultat, il faut regarder dedans, et voir qu'on n'a pas la même notion de copie, branche et tag)

 
dl.free.fr ?

Genre il avait checkouté tout le repo localement et c'était ça son dump ou quoi?

 
nooon..  
 
Mais une copie, des fois, c'est "export de la branche n-1, reimport"