Le pire bout de code que vous ayez vu

[Paul TOTH]

Envoyé par kolodz

Voici comment on vérifie les codes postaux sur le site que je maintiens :

Code SQL :

SELECT * FROM code_posteau

Le résultat de la requête est utilisé pour pour faire ceci :

Code php :

if(array_key_exists($codepostal,$list))

Étant donné la lenteur d'affichage du site associé, il a été décidé de mettre en cache le résultat de la requête...

La personne qui a produit ce code était payé plus cher que moi.
Et avait les droits d’accès à toutes les machines.

3 semaines que j'ai vue l'erreur, j'ai toujours pas eu le droit de faire la modification sur la prod.

Cordialement,
Patrick Kolodziejczyk.

tient...personnellement j'aurais faut un "SELECT COUNT(*) FROM code_posteau WHERE code = '$codepostal' "...mais peut-être as-tu une série de CP à vérifier

[kolodz]

J'ai fait un

Code SQL :

SELECT * FROM code_posteau WHERE code = '$codepostal' limite 1

Il y a peut-être mieux, mais ça reste plus propre.

[Charvalos]

T'es vraiment obligé de mettre le LIMIT ?

Normalement, il ne peut pas y avoir plusieurs même codes postaux non ?

[kolodz]

Soit le code postal existe, soit il n'existe pas.
Je ne vais pas m'amuser à parcourir l'ensemble des code postaux de France, si j'ai la réponse à la première ligne.
C'est une table à 62000 entrées...

Edit : Notre base a une entrée pour chaque ville d'un code postal.

[Bebel]

Envoyé par Charvalos

T'es vraiment obligé de mettre le LIMIT ?

Normalement, il ne peut pas y avoir plusieurs même codes postaux non ?

C'est possible. Regarde par exemple le 31140, il correspond a 3/4 villes.

[Paul TOTH]

Envoyé par Bebel

C'est possible. Regarde par exemple le 31140, il correspond a 3/4 villes.

et souvent 97400 n'existe pas j'étais le mois dernier à Paris pour envoyer un colis...le soft me demande mon adresse, mon code postal était refusé, j'ai du pu passer par le guichet

[le y@m's]

Code java :

1
2
protected static String pckStr = (new Exception()).getStackTrace()[0].getClassName();
protected static Logger s_logger = Logger.getLogger(pckStr);

Pourquoi faire simple quand on peut faire compliqué...

[Deadpool]

Envoyé par le y@m's

Code java :

Pourquoi faire simple quand on peut faire compliqué...

Ah oui, là il a été le chercher loin son nom de classe.

[Médinoc]

Minute, que contient la stacktrace pendant une init de variables statiques, en fait?

[le y@m's]

Envoyé par Médinoc

Minute, que contient la stacktrace pendant une init de variables statiques, en fait?

La première ligne de la stacktrace correspond à là où est instancier l'exception. Donc cela revient à recupérer la classe dans laquelle est écrit ce bout de code (que le contexte soit statique ou pas) .
En fait c'est une manière compliquée et méga crado (création d'une exception avec toute la stacktrace derrière ) de faire MaClass.class.getName().
La création "usuelle" d'un logger étant donc :

Code java :

private static Logger LOG = Logger.getLogger(MaClass.class.getName());

[Jidefix]

Oui mais le truc de l'exception tu peux le copier/coller dans toutes les classes, tandis que

Code :

MaClass.class.getName()

Tu es obligé de le modifier à chaque fois, et donc de risquer un violent claquage de doigt.

Tout ça pour finir de toute façon dans l'appender par défaut dans la majorité des applis...

(les templates de classe? c'est quoi ça?)

[kolodz]

Pas forcement si stupide...
Si tu renomme la classe avec un IDE, il te fera le changement aussi là...