Commentaires sur une source : [PHP5] NOTIMEOUT PACKAGE

cs_mrfox

Messages postés: 89
Date d'inscription: samedi 25 mai 2002
Statut: Membre
Dernière intervention: 5 décembre 2009

19 mai 2008 à 21:20

Ola
Ca peut fonctionner pour la génération d'un mega fichier zip ?

malalam

Messages postés: 10839
Date d'inscription: lundi 24 février 2003
Statut: Modérateur
Dernière intervention: 2 mars 2010

25
23 mai 2007 à 19:39

Oui tu as raison, mais au moment où j'ai codé cette classe, je voulais voir ce que cela donnait de gérer les exceptions interception comprise dans la classe.
En fait, ça ne sert à rien si ce n'est à perdre des ressources pour rien...faudra que je recode ça.
Alakhnor => l'utilisation de ces classes passent nécessairement par une revisite de ton code. Il faut le voir autrement : la boucle est gérée par du javascript, c'est tout.
Poutr php4, tu dois pouvoir réécrire la classe php...à priori. Rien de vital ne tire partie des spécificités PHP5.

FhX

Messages postés: 2350
Date d'inscription: mercredi 13 octobre 2004
Statut: Membre
Dernière intervention: 18 avril 2015

3
6 mai 2007 à 17:08

Etrange, je ne l'aurais pas codé comme ca cette classe.
Enfin si, mais au niveau des try{}catch() non.

J'aurais dégagé les try{}catch() des méthodes pour commencer.
Théoriquement, le try{}catch() se fait lors d'un appel de méthode et pas dans la méthode elle même. Sauf si c'est pour appeler une méthodes privée/protégée.

Ex :
# public function __set ($sType, $sVal) {
# try {
# if (!array_key_exists ($sType, $this -> aProps)) {
# throw new Exception ($sType.' is not a valid property');
# }
# } catch (Exception $e) {
# echo $e -> getMessage ();
# }

Ce que j'aurais fait :
public function __set($var, $val) {
if ( !array_key_exists($var, $this -> aProps) )
throw new GetSetException($var, 'set');
// suite...
}

Et quand tu appèles ta classe de l'extérieur :
try {
$maclasse = new noTimeOut;
$maclasse->timeout = 360;
// etc..
} catch ( GetSetException $gs ) {
die ( $gs->getMessage() );
} catch ( Exception $e ) {
die ( $e->getMessage() );
}

Par contre (voila la suite ^^) :
#public function flushMe ($aWork = null) {
# try {
# switch ($this -> aProps['TYPE']) {
# case 'DB':
# if (false === self::isNull ($this -> aProps['DB'], $this -> aProps['HOST'], $this -> aProps['LOGIN'], $this -> aProps['PWD'], $this -> aProps['QUERY'], $this -> aProps['DBSERVER'], $this -> aProps['START'], $this -> aProps['STEP'])) {
# throw new Exception ('DB properties have not been fully defined');
# }
# $mTmp = $this -> getDB ();
# break;

switch ($this -> aProps['TYPE']) {
case 'DB':
if ( !self::isNull ($this -> aProps['DB'], $this -> aProps['HOST'], $this -> aProps['LOGIN'], $this -> aProps['PWD'], $this -> aProps['QUERY'], $this -> aProps['DBSERVER'], $this -> aProps['START'], $this -> aProps['STEP']) )
throw new IssetException('DB');
try {
$mTmp = $this -> getDB ();
} catch ( Exception $e ) {
die ( $e->getMessage() );
}
break;
// Etc...

try {
$maclasse = new noTimeOut;
$maclasse->timeout = 360;
$maclass->flushMe();
// etc..
} catch ( GetSetException $gsE ) {
die ( $gsE->getMessage() );
} catch ( IssetException $iE ) {
die ( $iE->getMessage() );
} catch ( Exception $e ) {
die ( $e->getMessage() );
}

Voila comment je l'aurais joué. J'aurais étendu la classe d'exception pour chaque cas :)
Autrement, ton try{}catch() ne sert pas à grand chose dans une classe :)

alakhnor

Messages postés: 2
Date d'inscription: dimanche 4 mars 2007
Statut: Membre
Dernière intervention: 5 mai 2007

5 mai 2007 à 20:36

J'aimerais pouvoir utiliser ces classes. Malheureusement, je me heurte à 2 problèmes:
- je dois développer en php4
- mon programme est une boucle effectuant un certains nombre de traitements incluant des requêtes vers FlickR calculées par le reste. Je ne vois pas trop comment déporter des traitements et les traiter dans la classe php.

malalam

Messages postés: 10839
Date d'inscription: lundi 24 février 2003
Statut: Modérateur
Dernière intervention: 2 mars 2010

25
2 nov. 2006 à 18:06

Merci, c'est très gentil :-)

malalam

Messages postés: 10839
Date d'inscription: lundi 24 février 2003
Statut: Modérateur
Dernière intervention: 2 mars 2010

25
13 oct. 2006 à 11:40

Ps : c'est pour PHP5, les exceptions, hein.

Sinon, je me rends compte que pour bien utiliser ce package, il faut comprendre ce qu'il fait...j'incluerai un tuto dans le package un de ces jours.
Mais je vais donner un exemple :
sans utiliser la classe PHP, qui ne servira à rien dans ce cas. On peut utiliser la classe JS, elle facilitera les choses.
Il suffit d'utiliser sa méthode getDefault (donc, type = DEFAULT).
On met un STEP de 100 par exemple, un START à 0, et pas de limite (dans ce cas, virer le if (j < LIMIT) et toujours rappeler récursivement la méthode xmlhttp), ou une très grande limite, genre 1 milliard. Dans le fichier de traitement PHP, on récupère START, et STEP. Et on fait un truc du genre : for ($i = START; $i < START + STEP; $i ++) et on affiche $i. (je n'ai pas utiliser les bonnes variables hein, ce sera $_POST['iStart'] par exemple, si on reste dans le cadre de mon package). On met dans ce fichier, et même le fichier d'appel pour être convaincu, un set_time_limit (5) par exemple.
Tu verras que PHP affichera les chiffres, sans sourciller. Il pourra ainsi tourner indéfiniment, jusqu'à arriver à LIMIT s'il y en a une, et affichera : 0, 1, 2...500...10000...99999999, 999999999999999 etc... durant des jours et des jours si tu le veux.
Essaye de faire ça sans mon package, ou du moins, sans ce qu'il montre (avec un time_limit évidemment hein), tu auras du mal.

Ce package peut aussi permettre d'éviter le dépassement de la mémoire allouée à PHP dans certain cas. Une énorme requête par exemple, qui va chercher des tonnes d'enregistrements, peut planter à ce niveau là, avant le timeout. Ben là c'est pareil, vu que je vais récupérer mes enregistrements en plusieurs passes, la mémoire allouée ne sera pas dépasser (si on calcule bien hein).
Bien sûr, ce n'est pas valable pour, par exemple, le redimensionnement d'une image énorme! On ne peut - à priori - pas faire ça en plusieurs passes. Donc...niet.

J'espère que ça éclaircit un peu le bin's :-)

malalam

Messages postés: 10839
Date d'inscription: lundi 24 février 2003
Statut: Modérateur
Dernière intervention: 2 mars 2010

25
13 oct. 2006 à 11:21

Ben évidemment que je ruse. Le timeout est celui de PHP : le max_execution_time. Il est fixé à 30, 60 etc secondes. Si on a pas la main sur le serveur, on ne peut généralement pas le changer.
Mais Ajax, c'est du js, donc ce n'est pas soumis au timeout. Mon package utilise Ajax pour effectuer un traitement lourd en plusieurs fois, en appelant un nombre de fois le traitement serveur, nbre de fois défini par le STEP et LIMIT, STEP se chargeant aussi d'incrémenter le traitement: par exemple, j'ai un fichier de 10000 lignes, je veux le lire en entier mais avec PHP ça prend trop de temps. Donc j'utilise ce package pour lire, par exemple, le fichier 100 lignes par 100 lignes. Ajax me l'affiche 100 lignes par 100 lignes (en gardant les lignes précédentes évidemment) : j'ai un affichage séquentiel, qui dure ce qu'il dure, mais qui n'est jamais arrêté avant la fin de la lecture complète.
Même principe pour n'importe quel autre traitement : je veux envoyer 50000 emails ? Pas de problème. Lire une table contenant 1000000 d'entrées ? No problemo... ;-)

vas voir sur la doc officielle, php.net, pour les exceptions :
http://fr2.php.net/manual/fr/language.exceptions.php

malalam

Messages postés: 10839
Date d'inscription: lundi 24 février 2003
Statut: Modérateur
Dernière intervention: 2 mars 2010

25
13 oct. 2006 à 09:12

Ma théorie est bonne. Il suffit simplement de bien calculer son coup et de ne pas faire faire plus de traitements au serveur qu'il n'en faut pour rester dans le temps limite.
Et elle est bonne parce que : xmlhttp est un objet javascript, donc exécuté côté client. On appelle grâce à lui un traitement côté serveur. A ce moment, le décompte du temps commence. Le traitement s'effectue. S'il dure trop longtemps, boum, timeout. D'où le STEP que j'ai mis dans mes classes. Une fois le traitement effectué, le serveur balance le résultat au client, et évidemment, vu qu'on quitte le serveur, le décompte s'arrête et le temps imparti revient à 0. Là, ma classe vérifie si on a fini (LIMIT), sinon, elle rebalance un traitement, en incrémentant grâce au STEP, au serveur. Le décompte recommence, etc.

Soit tu as mal utilisé ma classe, soit ton STEP est trop gros.

Chez moi ça marche parfaitement. Sauf dans un cas : si on fait une connerie dans l'utilisation qui provoque une erreur dans la classe PHP par exemple (et encore, normalement, plus maintenant, vu que j'intercepte tout).

Quant à avoir une idée, non, pas son le code utilisé.

ImmortalPC

Messages postés: 954
Date d'inscription: mardi 11 mai 2004
Statut: Membre
Dernière intervention: 11 novembre 2008

2
12 oct. 2006 à 08:13

Salut,

"PHP version 4.3.3, et ça marche sous IE...?
T'es sûr de ta version de PHP hein ? Parce que la classe noTimeOut PHP est écrite en PHP5."
>> Enfin quand je dis que ça marche ajax marche ^^
Il est évidant que la class php ne puisse marcher.
Mais ce que je voulais voir c' etait ta class en javascript donc peu importe la version de php ^^
Avec IE il m' envoie toutes les erreurs qui sont faites avec php, alors qu' avec FF rien :'(

"Bon enfin, quelle est l'erreur JS sous firefox ?"
>> Il n' y a aucune erreur !!!
C' est trop zarb :S

"Essaye en virant le parent devant document...mais je ne suis pas convaincu.
essaye aussi en gardant parent.blabla, mais à la place du appendChild (), tu mets un bon vieux non standard
parent.document.getElementById(TARGET).innerHTML += oXmlhttp.responseText"
>> J' y ai déjà pensé ^^
J' ai essayé:
parent.document.getElementById(TARGET).innerHTML = oXmlhttp.responseText;

document.getElementById(TARGET).innerHTML = oXmlhttp.responseText;
document.getElementById(TARGET).firstChild.appendData = oXmlhttp.responseText;

Et toujour rien :'(
Alors que si je fais :
alert(Xmlhttp.responseText);
Il m' affiche mon alert avec tout ce qu' il faut :S

Voilà la version de FF : Mozilla/5.0 (Windows; U; Windows NT 5.1; fr; rv:1.8.0.7) Gecko/20060909 Firefox/1.5.0.7
Javascript : On

Une idée ??

@+

malalam

Messages postés: 10839
Date d'inscription: lundi 24 février 2003
Statut: Modérateur
Dernière intervention: 2 mars 2010

25
11 oct. 2006 à 08:16

Hello,

plait-il...? Se passer des vérifications en se disant que de toutes façons, en prod, on ne doit pas avoir d'erreurs ?
Mais, si tu ne fais aucune vérification, donc aucun try catch ou autre...et que ton code, globalement, se retrouve avec une mauvaise variable quelquepart : tu auras une erreur, en prod aussi. Une erreur interne à PHP, mais qui sortira. quand même. Et ton code ne marchera pas.
Si on suit ce que tu dis, autant coder sans jamais faire de vérification. Aucune.
"En effet une fois le code mis en place, la classe utilisée a bon escient, on reverifie toujours tout, alors que le code ne change pas... ??"
=> mais le code change tout le temps...? Tu peux le faire évoluer sur la même page, changer les paramètres...etc. Sans parler de pages différentes.

Il ne faut pas confondre optimisation et laisser-aller! Tu ne peux JAMAIS être sûr à 100% que ton code recevras toujours les bons paramètres. Parce que généralement, un code est alimenté par un autre code, lui-même dépendant d'un 3ème etc...

Ma façon de coder est efficace...la preuve : en production, mes serveurs sont quand même en error_reporting (E_ALL). Et je fais des sites grand public, pour de grosses boîtes.

Donc non, je ne suis pas d'accord avec toi ;-) Ca n'alourdit pas le code, au contraire, à la lecture, on voit tout de suite ce dont on a besoin. Et ça ne le ralentit pas, parce qu'un try ou un if ( isset ()), ça ne bouffe pas trop de ressource. Et parce qu'à côté de ça, j'optimise quand même pas mal...

malalam

Messages postés: 10839
Date d'inscription: lundi 24 février 2003
Statut: Modérateur
Dernière intervention: 2 mars 2010

25
10 oct. 2006 à 08:19

Les try catch, c'était pour te faire plaisir en plus (si si!).
Pour la fonction 'null' oui tu as raison, c'est une bonne idée.
pour la propriété $aProps, nan, je préfère comme ça parce que ça me permet de centraliser mes propriétés pour tester le __set (). Et une constante de classe ne peut pas être un tableau, malheureusement.
Et le while (), oui, c'est un oubli. Ca ne marchait pas au début, alors pour tester j'ai mis la fonction en dur, puis j'ai trouvé ce qui n'allait pas, et j'ai oublié de la remettre lol.
Et je n'utilise pas d'objet db parce que c'est juste la 1ère version : à terme, je vais en mettre un. Mais là c'est juste que j'avais besoin de quelques fonctions bien précises. Je n'ai pas terminé ce code, je compte le rendre plus fonctionnel; mais je dois le tester à fond sur mes codes d'abord. Et trouver à quelles occasions il peut être utilisé, celles-ci mises à part.
Pi avant j'ai toujours un problème avec les fichiers XML. Le flushing par motif fonctionne bien...sauf dans le cas d'un XML complexe contenant des caractères invisibles : la taille du fichier, et le nombre de caractères renvoyés ne correspondent plus, du coup javascript et php ne sont plus d'accord, n'ont plus les mêmes offset, et j'ai des décalages :-(
Et comme à la base je veux l'utiliser pour du xml, ce truc...(mon debugger en fait : je sors le contexte de la page débuggée, et ce contexte peut-être énorme! Donc, cette appli me permettra de l'afficher quand même sans problème; mais je n'y suis pas encore tout à fait).

Merci en tous cas, je ferai tes modifs tout à l'heure :-)