Pas mal :)


}else if ($this->operateur==='+'){
}else if ($this->operateur==='-'){

et encore à d'autres endroits, pourais se reduire à un switch qui serais bien plus lisible.

for ($i=0; $i<($l=strlen($in));$i++){

il ne serais pas possible de traduire en : for ($i = 0, $l = strlen($in); $i < $l; $i++) ?

function __construct($in){

il est passer où sa visibilité ?
En utilisant les singletons ont doit pouvoir ameliorer les performances du code.


9/10  

Il faut fermer les handle de dossiers, avec closedir().
Tu ne verifie pas la retour des fonctions (problemes de droits etc).
Tu aurais put utiliser une array() pour decrire les extensions que tu souhaite copier, et les autres seront detruite.

La verification sur l'IP.. chez certains fai (internet mobile compris) ils peuvent être plusieurs par proxy, à ce que me disent mes collegues, l'IP des AOL-iens change à chaque socket (possible), sans parler des IP en boites.

La seul chose d'utilise ça serais de changer l'ID de sessions entre chaque page.
Limite.. mais limite.. l'user-agent (a voir).

Si tu pouvais fournir un screenshot pour voir ce que ça donnes, sa serais chouete.
Tu t'est inspirer de ce code-ci :
http://www.cppfrance.com/codes/TRAITEMENT-IMAGE-SEGMENTATION-CROISSANCE-REGION_41140.aspx
or not ?

Bonne idée d'utiliser #, la methode que j'ai trouvé est de faire un htmlentities(..,.., 'utf-8'), de faire une array('&Agrave;' => 'a' ...), suivit d'une grose boucle foreach() et de str_replace(), pour terminer avec un html_encode(.., .., 'utf-8').
Car le truc "dangeureux" c'est d'encoder mal son script php (ça peut arriver par inavertance) et de ce retrouvé avec quelque chose de pas encodé ou pire de mal encodé.
C'est pourqoui je n'aime pas du tout laisser des choses non ASCII hors des commentaires.
Si ya mieu je prend :)

"Correction et ajout d'un exemple pour certain."
La moindre des choses est d'expliquer le fonctionnement d'un code, non ? tu auras bien codé des lignes spaghetti, si tu explique pas les entrés & sorties à qoui bon partager le code ? !

"=> Non je ferais une classe."
Pour mettre la sauce Boulognaise dedans ? deja met une fonction qui entoure le code c'est deja ça de gagner ! et ensuite tu tu veux enfile ça dans une class.

Deux mauvais points:
- Tu aurais put partager ton code en une ou plusieurs fonction, pour faire un code "portable".
- Où est la fonction de verification ?

Sinon oué l'idée as du bon.
Debug inutile, ta source doit deja etre apte a etre utilisé en production quand elle est publié.

Bonne continuation.

Non non tu verifie bien $resultat avant $tab= mysql_fetch_array($req3);, car là si ton $resultat vaut <= 0, ça sera pas tres jolie :)

# $resultat = mysql_num_rows($req3);
#
#
# $tab= mysql_fetch_array($req3);

if (mysql_num_rows($req3) <= 0) { // ou !== (int)1 si ont prefere :)
   // Rien
} else {
   $tab = mysql_fetch_array($req3);
}

Le genre de code qui as dut etre poster 10-20-30 fois sur PHPCS :(
Pas de nouveautés, c'est le genre de code qu'ont ne reutiliserais meme pas sur son site, vu qu'il est pas du tout "portable", autemps tout reecrire soit meme pour ces besoins.

En vrac :

# if(isset($_POST['Inom'])) $Inom=$_POST['Inom'];
# else $Inom="";

Je traduit ça sur une seul ligne (bien que sa soit plus lent):
$Inom = isset($_POST['Inom']) ? $_POST['Inom'] : null;
http://fr.php.net/manual/fr/language.operators.comparison.php#language.operators.comparison.ternary

#
# $Idate="$Iyear"."-"."$Imois"."-"."$Ijour";

$Idate = $Iyear . '-' . $Imois . '-' . $Ijour;

# $sql3 = "SELECT * FROM public_tbl WHERE Nom_Utilisateur='$user_name'";
# $req3 = mysql_query($sql3) or die('Erreur SQL !

Tu selectionne touts les champs, c'est peut etre pas nessesaire ? et si un jour ta table ait beaucoups plus de champs ?

$resultat = mysql_num_rows($req3);

Tu ne verifit pas si $resultat vaut quelque chose avant mysql_fetch_array()

$nm=$tab["Nom"];

Regles de nomage et d'indentation.

# $db = mysql_connect('localhost', 'root', '') or die('Erreur de connexion '.mysql_error());

Tu aurais dut mettre les identifiants de connexions dans un fichier à part, pour faciliter ce qui aurons envie d'essayer le code.

#  $sql = "INSERT INTO public_tbl(Nom_Utilisateur, Mot_De_Passe, Nom, Prenom, Date_Naissance,Sexe, Commentaire) VALUES('$Iuser_name','$Ipassword','$Inom','$Iprenom','$Idate','$Isexe','$Itxt')";

Ce n'est pas securiser, voir mysql_real_escape_string() , get_magic_quotes_gpc(), htmlentities() .

Avec les sessions PHP sa serais mieu : http://fr.php.net/manual/fr/ref.session.php

Un index auto-incrementable sur la table membre sera trés utile.

Bref, bon courage, n'hesite sourtout pas à te rendre sur php.net pour la documentation detaillé des fonctions dont je vient d'en parlé.

nacedo57400 > "Par contre j’ai entendu qu’on a pas trop le droit de stocker l’IP des visiteurs, info ou intox ???" <<< si ya une declaration CNIL où est mentionné 'données de connexions' ça passe, pour les blogs (et site perso ?) il me semble que rien n'est à declarer.