CORTANA PROJECT

Commentaire de putch le 02/03/2007 15:30:58

salut !

"Tout est dans le Zip" => le repertoire include est vide, donc pas de fichier conf.php ...

Commentaire de audayls le 02/03/2007 16:13:01

Don't Worry be happy les jeunes lol c'est pas la peine de se battre :-P

J'ai regardé vite fait et voici ce que je peux te dire :
-> (truc classique 1) Utilises toujours "<?php" et non pas "<?" (même si tu as utilisé "<?php" en premier)

-> (truc classique 2) Utilises les simples quotes ' à la place des doubles " (utilise la concaténation avec la virgule "," dans avec la fonction "echo" et le point "." pour les autres fonctions)

-> (truc classique 3) N'utilises pas "SELECT *" pour tes requêtes mais indiques les champs que tu souhaîtes récupérer (même si tu dois récupérer tous les champs) et n'oublies pas "mysql_close()" (je ne l'ai pas vu dans "admin\liste_news.php")

-> Remplaces tous les "if (...) {...} elseif (...) {...} etc..." par un switch (dù moins FhX crisera pas XD)

-> "$infos == ''" utilises plutôt "empty($infos)" (sa devrait être plus rapide enfin je crois ... à voir :-P)

-> Attention aux normes W3C ;-)

Commentaire de malalam le 03/03/2007 09:45:44 administrateur CS

Hello,

même si je duis du même avis que les 2 commentaires précédents, je tiens à émettre un bémol sur certains :
"Remplaces tous les "if (...) {...} elseif (...) {...} etc..." par un switch (dù moins FhX crisera pas XD)"
=> Ca reste une affaire de goût. Un switch est certes plus clair selon moi, mais bcp de monde préfèrent une suite de if elseif. Donc bon...et en terme de performance, on est peu ou prou à égalité (à dire vrai, les if... sont toujours un peu plus rapides).

"$infos == ''" utilises plutôt "empty($infos)"
=> ce n'est pas tout à fait pareil...voir ce petit test :
<?php
$a = '0';
if ($a == '') {
echo '$a == \'\'', '<br />';
}
if (empty ($a)) {
echo 'true === empty($a)', '<br />';
}
if ($a === '') {
echo '$a === \'\'', '<br />';
}
?>

Commentaire de FhX le 03/03/2007 19:48:23

Bah vi, l'imbrication de if/elseif/elseif/else doit se faire modérément.
Comment utiliser un switch à la place d'un if/elseif/else ?
Simple, tout dépend ce que tu veux analyser. Si tu n'analyses qu'une variable, ou une expression qui est identique, autant utiliser switch(). Autrement, il vaut mieux utiliser if/elseif/else.

Ex :

if ( $var === 3 )
     // code
elseif ($var ===5)
     // code
elseif ($var === 7)
     // code
else
     // code

Ce bloc peut facilement être transformé en :
switch ( $var ) {
   case 3:
         // code
         break;
   case 5:
         // code
         break;
   case 7:
         // code
         break;
   default:
         // code
}

Eventuellement, on peut aussi dégager le "break" dans le cas ou la variable a tester change en cours de route dans le switch. Cela peut toujours être pratique.

La où tu ne peux pas faire comme ca, c'est si tu testes différentes expressions (bien que...)
if ( isset($_COOKIE[]) ) {
     //code
} elseif ( isset($_SESSION[]) ) {
     //code
} elseif( isset($_GET[]) ) {
     // code
} else {
     // code
}

Ca, il faut le laisser telquel. Le remplacement logique serait de faire :
switch ( true ) {
   case ( isset($_COOKIE[]) ):
          // code
   case ( isset($_SESSION[]) ):
          // code
   case ( isset($_GET[]) ):
          // code
   default:
          // code
}

Mais j'adhère pas forcément le choix du switch(true). Car si on commence comme cela, un jour on va se retrouver un "switch(false)" dans son code et certaines expressions que l'on va tester vont retourner false au lieu de ce que l'on attend.

C'est une facon de programmer :)

Commentaire de coucou747 le 04/03/2007 15:54:00 administrateur CS

vive if et a mort switch, php est derive du perl, en perl, on n'a pas switch, et modifier un bloc de if est plus simple qu'un switch (genre ajouter une condition <)

Commentaire de coucou747 le 04/03/2007 17:19:44 administrateur CS

c'est moins modifiable...

Commentaire de nimaka83700 le 05/03/2007 09:39:44

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in /home.10.4/libertin/www/cortana/index.php on line 50

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in /home.10.4/libertin/www/cortana/index.php on line 99

voila les deux erreurs ou je bute sur le script
(j'en ai deja corrigé deux autres mais là je bute)

Commentaire de BTAJV le 05/03/2007 09:46:32

Ba la nimaka83700 je ne comprend pas T ereur tu as bien fait l'install avec le fichier d'install a l'adresse suivant
http://monsite.com/install
?????

Et euh attendez 8 petite heure et je renvoi un zip avec pas mal de bug corriger mais la j'ai pas le temp un max de boulot et peu de temp pour le faire

Commentaire de coucou747 le 05/03/2007 11:49:50 administrateur CS

ajoute or die apres tes mysql_query, ou au moins une verification d'echec de requette

Commentaire de BTAJV le 05/03/2007 11:58:06

nimaka83700
Je n'arrive pas a comprendre l'erreur que tu rencontre je suis désolé de ne pas pouvoir t'aider

Commentaire de BTAJV le 26/03/2007 14:22:00

Sa fait plaisir 20 commentaire 0 note
J ai 2 sentiment un de peur je me dit ma saouce c de la M.... enfin vous voyez qoi ou sinon ma source est si bien que personne veu noté

Et aussi personne ne m a raporté de gros bug des truc de malade qui fait que tout foire dois je pour autan conclure qui en a pas ???

Commentaire de coucou747 le 30/03/2007 13:36:41 administrateur CS

je ferais quand je trouverais le temps un bench sur les if et switch, mais a mon avis, c'est largement if qui gagne... De plus, cote modifications possibles, entre un if et un switch, la seule chose bien pour le switch c'est case a; .... casee b .... break;

Commentaire de BTAJV le 31/03/2007 07:23:37

Je voi pas le rapport avec ma source mais tant pis