Posts dans "A ne pas faire"

Oopppsss - oui, encore.

Ajouté le 13.01.2009 - 23:04  •  Commentaires (1)

Suite à une fausse manip, j'ai légèrement giclé la bdd... J'ai ensuite remarqué que le backup de ladite bdd était pas fait.

On remerciera postgresql et la non compatibilité entre les versions (en l'occurence, pg_dumpall-8.3 tentant de faire un dump d'une bdd-8.2...)
On remerciera aussi les gugus qui font en sorte que pg_dumpall ne tienne pas compte des contraintes sur les tables, ce qui pose un léger problème lors d'une restauration. A ce sujet, j'ai pas réussi à faire gober le dump de pg_dumpall à pg_restore.... l'a fallu que je me le fasse à la main, bdd par bdd...
Happy.

Bref, y a juste le poste sur HG-TV et les commentaires associés qui manquent. mais c'est un peu lourd quand même. Va falloir que je fasse des tests de tout ce bordel.
De ce point de vue, mysql est nettement plus simple à gérer :/

Sur ce, vais me coucher ^^'

Lire plus

Quand on ne sait pas employer mysql...

Ajouté le 26.08.2008 - 22:44  •  Commentaires (0)

... on arrive à faire ce genre d'horreur : 
<?php
// connexion mysql. du blabla inutile et pour finir ceci:
  $tri=' critere desc, users.nickname asc';
  if (! $_GET['tri']) $_GET['ordre']='desc';
  switch ($_GET['tri'])
  {
  case 'inscription': 
    $tri='users.date_inscription '.$_GET['ordre']; 
  break;
  case 'connexion': 
    $tri='users.last_connected '.$_GET['ordre']; 
  break;
  case 'turnover': 
    $tri='users_turnover.turnover '.$_GET['ordre']; 
  break;
  case 'nom': 
    $tri='users.nickname '.$_GET['ordre']; 
  break;
  case 'moyenne' : 
    $tri = 'moyenne '.$_GET['ordre'].' , users.nickname asc, new.nombre_notations desc'; 
  break;
  default: break;
  }

  $query = " 
    select users.nickname, users_turnover.turnover,
    users.pr_en_archive, optim_users_pages.title,
    new.nombre_notations,

    case when ((new.note_communication + 
    new.note_documentation + new.note_qualite +
    new.note_timing)/4)=10  && new.nombre_notations=0 
    then 0 
    else ((new.note_communication + 
    new.note_documentation + new.note_qualite + 
    new.note_timing)/4) end as moyenne,

    case when (new.nombre_notations=0 && 
    users_turnover.turnover <> 0) 
    then users_turnover.turnover 
    when (new.nombre_notations <> 0 && 
    users_turnover.turnover = 0) then (new.note_communication + 
    new.note_documentation + new.note_qualite + 
    new.note_timing)/4
    else (((new.note_communication + new.note_documentation + 
    new.note_qualite + new.note_timing)/4) * 
    users_turnover.turnover) end as critere

    from users, users_turnover, users_id_nickname, 
    optim_users_pages, new where 
    users.nickname=users_id_nickname.nickname and 
    users_id_nickname.id=users_turnover.id 
    and optim_users_pages.id=users_id_nickname.id and 
    users.nickname=new.nickname order by $tri ";
 
  $res=mysql_query($query);

  while ($row=mysql_fetch_row($res)) {
    $user=new CUser($row[0]);
    if ((stristr(strip_tags($user->etudes), $keyword)) ||
      (stristr(strip_tags($user->exp_professionnelle), $keyword)) ||
      (stristr(strip_tags($user->connaissances_info), $keyword))) {
        echo "<tr><td><center>";
        echo "<a href='".$row[3]."'>".$user->nickname."</a></center></td><td> </td>";
        echo "<td><center>".get_date($user->date_inscription)."</center></td><td> </td>";
        echo "<td><center>".count(str_to_vect($row[2]))."</center></td><td> </td>";
        echo "<td><center>".$row[1]."€</center></td><td> </td>";
        if ($row[4]!=0) {
          $nb = substr($row[5], 0, 4);
          echo "<td><table cellpadding=\"0\" cellpadding=\"0\"  width=\"50%\"><tr>";
          for ($i = 1; $i <= 10; $i++) {
            if ($i <= $nb) echo "<td><img src=\"images_news/bar_" . $i . ".gif\" border=\"0\"  width=\"16px\" height=\"6px\"></td>";
            else echo "<td><img src=\"images_news/bar_no.gif\" border=\"0\" width=\"16px\" height=\"6px\"></td>";
          }
          echo "<td>(".substr($row[5], 0, 4)."/10)</td>";
          echo "</tr></table></td>";
        }
        else {
          echo "<td><img src=\"images_news/bar_no.gif\" border=\"0\" width=\"16px\" height=\"6px\"> ?</td>";
        }
        echo "</tr>";
      }
  }
  echo "</table>";
  mysql_close();
?>


J'ai un peu coupé les parties "html" inutiles (et fausses...), le code qui est intéressant est au niveau du php et du mysql.
Il faut savoir que la classe CUser reprend une bonne partie voire tous les éléments sélectionnés précédemment dans la requête...
La-dite requête prend environ 13s à elle seule.
Je vous laisse imaginer ensuite la boucle while() qui suit.... Sur plus de 3'000 résultats, c'est toujours bon ;)

Sans compter les injections SQL possibles...

Encore un code foireux que j'ai joyeusement taillé à coups de hache. D'autant que, sur le même site, j'avais déjà prévu une table pour les moteurs de recherche, permettant, via des index FullText, de rendre les recherches plus efficaces (même si encore lourdes)...

Happy coding!

Lire plus

Apple...

Ajouté le 10.02.2008 - 00:00  •  Commentaires (1)

Un pote a raconté sa fabuleuse histoire d'amour avec Apple et principalement le SAV

Je me permet donc de vous copier son blabla, qui est plutôt long je dois avouer, mais fort instructif quant aux déboirs possibles avec un SAV.

Bonne lecture!

14:12 <+Aelia> alors bon, un joli cadeau d'anniv/noel : un ipod mini, moi tout content ^^
14:12 <+Aelia> enfin content pendant 1 mois environ...
14:12 <+Aelia> tout d'un coup un jour de grand froid paf hdd mort dans le ipod
14:13 <+Aelia> alors bon je fais jouer la garantie, colis UPS qui arrive direct à la maison, le livreur repart avec la bête, 2 semaines plus tard nouveau ipod mini, la vie est belle
14:13 <+Aelia> sauf que... d'usine le ipod avec un défaut sur l'écran
14:13 <+Aelia> une ligne entière morte
14:14 <+Aelia> alors bon ça me fait chier d'envoyer celui-là direct en réparation, j'attends la fin de la garantie pour l'envoyer
14:14 <+Aelia> là le colis UPS arrive chez moi, et le livreur se casse sans que j'aie pu lui donner le renvois... déjà j'ai trouvé con de sa part... mais bon j'appelle UPS pour qu'ils viennent chercher le colis
14:15 <+Aelia> il part, et c'est là que ça se corse....
14:15 <+Aelia> selon UPS le ipod est arrivé a destination, selon apple il serait arrivé mais ils savent pas où
14:16 <+Aelia> alors bon moi je le savais pas au début alors j'ai attendu 3 semaines et toujours pas de ipod à la maison...
14:16 <+Aelia> j'appelle apple pour savoir, ils me disent "no prob il sera bientôt réparé"
14:17 <+Aelia> 1 semaine plus tard, je rappelle, "ah ben le ipod est pas chez le technicien, je vais voir où il est"
14:17 <+Aelia> 1 semaine plus tard... "ah ben on arrive pas à retrouver le ipod... mais selon UPS il est arrivé à destination, sauf qu'on sait pas vraiment où c'est..."
14:18 <+Aelia> puis après plus de 2 mois d'attente, je tombe enfin sur une personne pas trop conne : "bon on va pas le chercher indéfiniement, je vous propose de vous envoyer un autre ipod à la place"
14:19 <+Aelia> et c'est comme ça que j'ai eu un ipod nano, tout neuf avec une garantie de 1 an comme excuse d'apple pour sa grosse gaffe
14:19 <+Aelia> moi tout content avec le tout dernier joujou, en plus c'était le tout début du nano quasiment personne ne l'avait
14:19 <+Aelia> mais ça a pas duré longtmp... ce ipod n'arrêtait pas de planter, il me faisait des merdes sans arrêt etc...
14:20 <+Aelia> et là j'en ai eu ras-le-bol et après discution avec un tech apple agréé (dont je ne donnerai pas le nom) je suis arrivé à la conclusion que le faire griller serait le seul moyen d'en avoir un remplacement
14:21 <+Aelia> ce que j'ai fait, et cette fois la garantie n'a pas posé de prob j'ai eu un autre nano à la place
14:21 <+Aelia> et c'est celui avec lequel je me balade encre (et qui commence à me faire les même coups que l'autre...)
14:21 <+Aelia> chapitre sur les ipods fini ^^
14:22 <+Tengu> :]
14:22 <+Aelia> maintenant il y a mon macbook noir : Fate
14:22 <+Aelia> et là... ben ça commence dès l'achat ^^
14:23 <+Tengu> c'est la série noire
14:23 * Tengu court
14:23 <+Aelia> pour acheter apple avec neptune, il faut payer avec soit carte de crédit soit virement banquaire
14:24 <+Aelia> évidemment j'ai pas de carte de crédit, et je suis à la poste mais qui veux croire que quand c'est écrit banquaire et précisé que c'est pas possible avec un compte postal c'est pas des conneries...
14:24 <+Aelia> ben oui... le système de payement n'est pas compatible avec la poste... faut pas chercher...
14:24 <+Tengu> uhuhuhu
14:25 <+Tengu> ca, je l'avais pas su :P
14:25 <+Aelia> mais ça m'a valu d'attendre 1 semaine supplémentaire le temps qu'ils retrouvent mon payement chez apple...
14:25 <+Aelia> bon n'empèche que je l'ai eu mon macbook
14:25 <+Aelia> tout content... enfin pendant 1 mois
14:26 <+Aelia> et là paf l'écran qui ne s'allume plus
14:27 <+Aelia> et comme ces chers amis de chez apple ne foutent pas un connecteur standard pour l'écran externe, et ne filent pas l'adaptateur gratos... (l'est 40.-) je pouvais pas tester pour savoir si c'était l'écran ou la carte mère
14:27 <+Aelia> le technicien non plus.... alors les deux ont étés changés
14:27 <+Aelia> re-content, pendant encore 1-2 mois
14:27 <+Aelia> puis pouf la batterie qui n'a plus que 2h d'autonomie et passe de 40% à 7% en 2 sec
14:28 <+Aelia> alors bon, changement de batterie no prob
14:28 <+Aelia> puis encore qqes semaines plus tard cette fois c'est le hdd qui crève direct sans signes précurseur...
14:28 <+Aelia> encore une fois pas de prob changement en 3 jours
14:29 <+Aelia> et puis vient le gros morceau
14:29 <+Tengu> Aelia|> ca mériterait presque une mise en log sur le net ce truc...
14:30 <+Aelia> mon top-case s'est fissuré... et moi j'ai pensé que ça allait jamais passer sous garantie alors j'ai laissé, puis 3 semaines plus tard j'ai "perdu" ma led de mise en veille, en fait elle a commencé à faire un faux-contact
et ne marchait plus que quand on appuyait sur le coin droit du portable
14:30 <+Aelia> alors là ça m'a fait chier... et puis j'ai payé l'applecare alors je me suis dit que j'allais la faire travailler
14:31 <+Aelia> je suis allé à art-computer à genève (grosse MAIS _GROOOSSSSSSEEEEEE_ erreur...)
14:32 <+Aelia> j'arrive ils me demande mon num de tel etc... tout se passe nickel il me dit "dans 2 semaines" le boitier cassé et la led allaient être remplacée sans prob
14:32 <+Aelia> deux semaines plus tard... je me rend compte que cet abruti avait mal compris mon num de tel
14:32 <+Aelia> et il m'avait pas demandé confirmation ni rien...
14:33 <+Aelia> alors je retourne au magasin pour faire changer mon numéro et je demande si mon portable est prêt, il me dit "il est prêt mais il est pas ici"
14:33 <+Aelia> en fait le magasin envoie tout chez des techniciens qui sont ailleurs... juste à 1min à pied de chez moi...
14:34 <+Aelia> alors bon il me dit que le lendemain il sera là, le lendemain j'y retourne et oui il était là, top-case tout neuf joli et tout
14:34 <+Aelia> sauf que... la led avait toujours le faux contact
14:35 <+Aelia> alors le vendeur (un autre) me dit "ni prob, je le fais passer en priorité, il sera prêt en qqes jours, au pire une semaine prochaine"
14:35 <+Aelia> *dans une semaine
14:36 <+Aelia> alors bon je le crois pas tellement, mais je le laisse quand même
14:37 <+Aelia> la semaine suivante, je reviens, "heu nan il a pas été réparé j'appelle les techniciens" il s'éloigne ... fini son appel, "alors en fait il y a un _autre_ problème et ils attendent la confirmation d'apple pour savoir si
c'est pris sous garantie" moi : "et c'est quoi le problème" lui : "je sais pas"
14:38 <+Aelia> alors là j'étais pas très content... j'ai trouvé l'adresse des techniciens, et j'y suis allé directement
14:38 < AnalogicDot> tu leux as cassé la geule?
14:38 <+Tengu> meme po :'(
14:38 <+Aelia> les tech qui me disent "bah en fait le bottom-case des mac n'ont jamais été sous garantie"
14:38 <+Aelia> moi : et le rapport avec la led qui marche pas ?
14:39 <+Aelia> "bah en fait la led est DANS le bottom-case, alors que selon apple il n'y a rien du tout d'électronique dans le bottom-case, ça nous embète pas mal"
14:40 <+Aelia> alors bon je dis que je veux que ce soit réparé, il dit qu'il va insister auprès d'apple, mais promet rien
14:41 <+Aelia> 3 jours plus tard, sans nouvelles je reviens, il me dit : "ça a été accepté ce matin, on va commander la pièce plus tard" (c'était 14h)
14:41 <+Tengu> mdr
14:42 <+Aelia> "la pièce sera là, dans 1-2 jours, après c'est 1h de boulot pour la changer, ce sera prêt soit vendredi, ou dans le pire des cas lundi"
14:42 <+Aelia> "quoiqu'il en soit on vous appelle pour vous tenir au courant"
14:42 <+Aelia> lundi après-midi je me re-pointe, "ah on a pas reçu la pièce"
14:43 <+Aelia> "mais bon une fois qu'on la reçoit, c'est 1h de boulo, on vous appelle pour vous prévenir"
14:43 <+Tengu>
14:43 <+Aelia> le jeudi je me RERE-pointe très énervé parceque bien évidemment ils m'ont jamais appelés mais pas une seule fois à chaque fois qu'ils me l'ont dit
14:44 <+Aelia> et là le tech a une tasse de café dans les mains et est en train de boire, au moment ou il me voit, il manque d'avaler de travers
14:45 <+Aelia> il vient même pas me dire bonjour il commence direct à chercher s'ils ont pas la pièce... il me fait non de la tête
14:45 <+Aelia> il vient vers moi, il me dit qu'ils ont pas reçu la pièce... moi : "et vous avec pas un numéro de tracking pour savoir où elle est ?"
14:46 <+Aelia> et là il va vers son ordi, cherche un peu, et reste bouche-bée devant son écran... moi je sais direct que c'est pas bon...
14:46 <+Aelia> il donne à son collègue le numéro de référence de la réparation, qui regarde... et selon apple l'intervention est close, "pièce reçue"
14:47 <+Aelia> alors bon il me dit que c'est la première fois qu'il voit ça etc... mais j'en ai un peu rien à faire...
14:48 <+Aelia> alors pour finir je lui ai fait remonter mon laptop, et me le rendre, il l'a fait "alors que normalement il a pas le droit"
14:48 <+Aelia> il va se débrouiller pour qu'apple lui envoie vraiment la pièce...
14:48 <+Aelia> et moi en attendant j'ai un bottom-case qui est défectueux...
14:49 < AnalogicDot> achetez apple... (certified troll free ;-) )
14:49 <+Aelia> mais avec tout ça, ça fait 4 réparations sur Fate, et normalement c'est à la 5eme que j'ai droit à un laptop neuf si je demande
14:50 <+Aelia> mais si un macbook avec une carte wifi potable sort, je leur fais chier jusqu'au dernier moment pour en avoir un neuf, autant que c'est humainement possible
14:50 <+Aelia> alors pour l'instant j'attends, j'ai pas trop le choix quoi
14:51 <+Aelia> surtout que la led ne fais même plus contact quand on appuye sur le coin du portable... donc elle marche vraiment plus depuis qu'ils l'ont démonté

Lire plus

Quand on part en boucle...

Ajouté le 01.02.2008 - 00:00  •  Commentaires (0) 

<?php
@mysql_close(); @mysql_close(); @mysql_close();

(mysql_connect($GLOBALS['db_host'],$GLOBALS['db_user'],GLOBALS['db_password']))
or die(mysql_error());
(mysql_select_db($GLOBALS['db_name'])) or die(mysql_error());
 
for ($z=1; $z<12; $z++) {
  $query="
  update `table` set `champ1`='".$_POST['field1']."', 
 `champ2`='".$d."', 
 `champ3`='15', 
  `champ4`='".$_GET['field2']."' where 
  `champ5`=".$_GET['field3'];
  
  ($result=mysql_query($query))  or die("Erreur: enregistrement des données dans la base de données");
  $query_2="select * from `table` where `champ5`='".$_GET['field3']."'";
	($res_2=mysql_query($query_2)) or die(mysql_error());
  if ($res_2) {
    $row_2=mysql_fetch_array($res_2);
  }
  @mysql_close(); @mysql_close(); @mysql_close();
  (mysql_connect($GLOBALS['db_host'],$GLOBALS['db_user'],$GLOBALS['db_password']))
  or die(mysql_error());
  (mysql_select_db($GLOBALS['db_name'])) or die(mysql_error());
}
?>


Ce code a été trouvé dans une page php... Lorsque j'ai demandé à mon client "pourquoi cette boucle", la réponse fut : "Le dév précédant arrivait pas à faire passer l'update, et pour finir on a vu que ça marchait comme ça. Surtout tu touches pas, ça marche comme ça !!".

... oué... bin le code a giclé 1 jour plus tard, et depuis la page se charge à une de ces vitesses, zavez pas idée.

Pour la petite histoire, ce code fait :
1- fermer 3 fois la connexion mysql
2- ouvre une nouvelle connexion, sélectionne la database
for {
3- met à jour un enregistement dans la table (notez: il n'y a AUCUNE sécurité sur les données, tout est pris depuis l'URL....)
4- récupère un truc inutilisé par la suite (même en dehors de la boucle)
5- ferme 3x la connexion mysql
6- rouvre la connexion, sélectionne la database
}
Et itère donc 11x le contenu de la boucle for()

On peut aussi remarquer que la syntaxe n'est pas du tout propre et optimisée : en effet, en PHP comme dans d'autres langages, il y a une sacrée différence entre les ' et les ".
Là, le "dév" a planté des " en faisant, en plus, des concaténations... ce qui est complètement absurde et prend des ressources pour rien, PHP allant parser le contenu se trouvant entre " " pour trouver des variables....
Soit on concatène, soit on plante des double-quotes! Pas les deux.
Et pour les variables bizarres :
"ceci est {$_GET} une variable"
ou alors 'ceci est '.$_GET.' une variable'

Plus encore l'absence du moindre test et de la moindre sécurité sur les éléments envoyé dans mysql... Ce code n'est plus en activité après mon passage, mais il aurait permis de faire vraiment mal. Même pas de limit ni rien. Une porte ouverte aux injections SQL en tous genres.

Bravo -.-

Lire plus