Fork me on GitHub






CPascal
news 1.42
#1
Semi pro
Inscrit: 06/09/2005 01:34
De Royan
Messages: 609
Bonjour,

En essayant d'étudier le php j'ai remarquer cette synt dans le fichier "visit.php"
------------------------------------------------------------
}
echo "<html><head><meta http-equiv=\"Refresh\" content=\"0; URL=".$myts->oopsHtmlSpecialChars($url)."\"></meta></head><body></body></html>";
exit();
?>
----------------------------------------------------------
URL=".$myts->oopsHtmlSpecialChars <- ne manquerai t'il pas un "X" à oops ?
Merci à tous !

Posté le : 10/11/2005 03:51

bergers aux yeux de ciel
Partager Twitter Partagez cette article sur GG+
Rapport Haut
denisdlu
Re: news 1.42
#2
Supporter Xoops
Inscrit: 24/11/2004 07:56
De Marseillan - Cap d'Agde
Messages: 687
Salut,

On retrouve effectivement cette notation dans pas mal de modules :

Citation :

D:\DevWEB\Portail\xoops\xoops_install_modules\AMS\visit.php
1 echo "<html><head><meta http-equiv=\"Refresh\" content=\"0; URL=".$myts->oopsHtmlSpecialChars($url)."\"></meta></head><body></body></html>";
1 match in D:\DevWEB\Portail\xoops\xoops_install_modules\AMS\visit.php
D:\DevWEB\Portail\xoops\xoops_install_modules\news\visit.php
1 echo "<html><head><meta http-equiv=\"Refresh\" content=\"0; URL=".$myts->oopsHtmlSpecialChars($url)."\"></meta></head><body></body></html>";
1 match in D:\DevWEB\Portail\xoops\xoops_install_modules\news\visit.php
D:\DevWEB\Portail\xoops\xoops_install_modules\news_1.4\visit.php
1 echo "<html><head><meta http-equiv=\"Refresh\" content=\"0; URL=".$myts->oopsHtmlSpecialChars($url)."\"></meta></head><body></body></html>";
1 match in D:\DevWEB\Portail\xoops\xoops_install_modules\news_1.4\visit.php
D:\DevWEB\Portail\xoops\xoops_install_modules\news_old\visit.php
1 echo "<html><head><meta http-equiv=\"Refresh\" content=\"0; URL=".$myts->oopsHtmlSpecialChars($url)."\"></meta></head><body></body></html>";
1 match in D:\DevWEB\Portail\xoops\xoops_install_modules\news_old\visit.php
D:\DevWEB\Portail\xoops\xoops_install_modules\smartsection\visit.php
1 echo "<html><head><meta http-equiv=\"Refresh\" content=\"0; URL=".$myts->oopsHtmlSpecialChars($fileObj->getFileUrl())."\"></meta></head><body></body></html>";
1 match in D:\DevWEB\Portail\xoops\xoops_install_modules\smartsection\visit.php
D:\DevWEB\Portail\xoops\xoops_install_modules\smartsection_old_1.03\visit.php
1 echo "<html><head><meta http-equiv=\"Refresh\" content=\"0; URL=".$myts->oopsHtmlSpecialChars($fileObj->getFileUrl())."\"></meta></head><body></body></html>";
1 match in D:\DevWEB\Portail\xoops\xoops_install_modules\smartsection_old_1.03\visit.php
D:\DevWEB\Portail\xoops\xoops_install_modules\smartsection_old_1.03RC1\visit.php
1 echo "<html><head><meta http-equiv=\"Refresh\" content=\"0; URL=".$myts->oopsHtmlSpecialChars($fileObj->getFileUrl())."\"></meta></head><body></body></html>";
1 match in D:\DevWEB\Portail\xoops\xoops_install_modules\smartsection_old_1.03RC1\visit.php
D:\DevWEB\Portail\xoops\xoops_install_modules\smartsection_old_v1\visit.php
1 echo "<html><head><meta http-equiv=\"Refresh\" content=\"0; URL=".$myts->oopsHtmlSpecialChars($fileObj->getFileUrl())."\"></meta></head><body></body></html>";
1 match in D:\DevWEB\Portail\xoops\xoops_install_modules\smartsection_old_v1\visit.php
D:\DevWEB\Portail\xoops\xoops_install_modules\xfguestbook\class\xfgbformdhtmltextarea.php
1 $ret .= "<img onclick='xoopsCodeSmilie(\"".$this->getName()."\", \" ".$smiles[$i]['code']." \");' onmouseover='style.cursor=\"hand\"' src='".XOOPS_UPLOAD_URL."/".$myts->oopsHtmlSpecialChars($smiles['smile_url'])."' border='0' alt='' />";
1 match in D:\DevWEB\Portail\xoops\xoops_install_modules\xfguestbook\class\xfgbformdhtmltextarea.php
D:\DevWEB\Portail\xoops\xoops_install_modules\xoopsmembers\index.php
1 $hiddenform .= "<input type='hidden' name='".$myts->oopsHtmlSpecialChars($k)."' value='".$myts->makeTboxData4PreviewInForm($v)."' />\n";
1 match in D:\DevWEB\Portail\xoops\xoops_install_modules\xoopsmembers\index.php
TOTAL: 10 matches in 10 files (7258 other files without matches not listed)



Mais ce n'est pas une erreur .
Il existe bien dans Xoops une fonction avec ce nom .
Par exemple dans la version Xoops 2.0.13.2 on a cela :

Citation :

D:\DevWEB\Portail\xoops\xoops_2.0.13.2fr\html\class\module.textsanitizer.php
1 function &oopsHtmlSpecialChars($text)
1 match in D:\DevWEB\Portail\xoops\xoops_2.0.13.2fr\html\class\module.textsanitizer.php
D:\DevWEB\Portail\xoops\xoops_2.0.13.2fr\html\class\xoopsform\formdhtmltextarea.php
1 $ret .= "<img onclick='xoopsCodeSmilie(\"".$this->getName()."\", \" ".$smiles[$i]['code']." \");' onmouseover='style.cursor=\"hand\"' src='".XOOPS_UPLOAD_URL."/".$myts->oopsHtmlSpecialChars($smiles['smile_url'])."' border='0' alt='' />";
1 match in D:\DevWEB\Portail\xoops\xoops_2.0.13.2fr\html\class\xoopsform\formdhtmltextarea.php
D:\DevWEB\Portail\xoops\xoops_2.0.13.2fr\html\include\xoopscodes.php
1 echo "<img src='".XOOPS_UPLOAD_URL."/".$myts->oopsHtmlSpecialChars($smiles['smile_url'])."' border='0' alt='' onclick='xoopsCodeSmilie(\"".$textarea_id."\", \" ".$smiles[$i]['code']." \");' onmouseover='style.cursor=\"hand\"' />";
1 match in D:\DevWEB\Portail\xoops\xoops_2.0.13.2fr\html\include\xoopscodes.php
D:\DevWEB\Portail\xoops\xoops_2.0.13.2fr\html\install\class\textsanitizer.php
1 function &oopsHtmlSpecialChars($text)
1 match in D:\DevWEB\Portail\xoops\xoops_2.0.13.2fr\html\install\class\textsanitizer.php
D:\DevWEB\Portail\xoops\xoops_2.0.13.2fr\html\modules\mydownloads\visit.php
1 echo "<html><head><meta http-equiv=\"Refresh\" content=\"0; URL=".$myts->oopsHtmlSpecialChars($url)."\"></meta></head><body></body></html>";
1 match in D:\DevWEB\Portail\xoops\xoops_2.0.13.2fr\html\modules\mydownloads\visit.php
D:\DevWEB\Portail\xoops\xoops_2.0.13.2fr\html\modules\xoopsmembers\index.php
1 $hiddenform .= "<input type='hidden' name='".$myts->oopsHtmlSpecialChars($k)."' value='".$myts->makeTboxData4PreviewInForm($v)."' />\n";
1 match in D:\DevWEB\Portail\xoops\xoops_2.0.13.2fr\html\modules\xoopsmembers\index.php
TOTAL: 6 matches in 6 files (932 other files without matches not listed)


Posté le : 10/11/2005 05:29

Cordialement
Denis Open in new window
Partager Twitter Partagez cette article sur GG+
Rapport Haut
CPascal
Re: news 1.42
#3
Semi pro
Inscrit: 06/09/2005 01:34
De Royan
Messages: 609
Bonjour denis,

Merci de ta réponse, si cela est normal donc je laisse tel que.
J'ai une autre petite question: J'ai lu sur un site spécialisé dans le PHP que beaucoup de programmateur laisser des blancs entre les syntaxes et que cela diminué la rapidité de la lecture du script.
exemple:

= (false != $admin_view) ? $title.' '.$this->_statusText[$thread[$key]['obj']->getVar('com_status')] : $title;
----------------BLANC-----------------
// MLPatch hack by marcan //
$title = $myts->makeTareaData4Show($title);
// End of MLPatch hack by marcan //
-------------BLANC--------------------
$replies[] = array('id' => $key, 'prefix' => $current_prefix, 'date_posted' => formatTimestamp($thread[$key]['obj']->getVar('com_created'), 'm'), 'title' => $title, 'root_id' => $thread[$key]['obj']->getVar('com_rootid'), 'status' => $this->_statusText[$thread[$key]['obj']->getVar('com_status')], 'poster' => $this->_getPosterName($thread[$key]['obj']->getVar('com_uid')));
$current_prefix .= $prefix;
}
if (isset($thread[$key]['child']) && !empty($thread[$key]['child'])) {
$depth++;
-----------------------------------------------------------
J'ai pris n'importe quel script dans xoops, est il vraiment néssécaire de supprimé ces blancs/espaces ?
si oui cela aurra t'il une insidence sur le site (dysfonctionnement ou rapidité de lecture du script augmenter)? comme j'ai pu lire sur le site PHP.
Surtout ne prennaient pas ma question comme une critique LOIN DE LA, je souhaiterai juste savoir, ont lie tellement de chose qui se contre dise ont n'arrive plus a disocier le vrai du faut.
Merci à tous!

Posté le : 10/11/2005 16:47

bergers aux yeux de ciel
Partager Twitter Partagez cette article sur GG+
Rapport Haut
denisdlu
Re: news 1.42
#4
Supporter Xoops
Inscrit: 24/11/2004 07:56
De Marseillan - Cap d'Agde
Messages: 687
Salut Pascal ,


Oui dans l'absolu c'est vrai qu'en principe lire des blancs ca prend du temps ( comme tout autre caractere).
Tout est relatif , on parle ici de milli secondes pour un programme et meme de micro secondes .Donc s'il faut choisir de gagner quelques micro-secondes pour lire un programme et avoir un code bien présenté je n'hesite pas une seconde a garder mon source bien aéré et présentable .

Quand on fait des applis vraiment temps reel tout est bon pour gagner des secondes et meme le nom des variables est important dans un language interprété ( d'ailleurs pour du temps reel on prefera du compilé) .Ca va plus vite de lire la variable "I" que "CoucouRoucoucouLaPaloma" .
Mais pour du PHP sur un serveur on en est pas encore la .
Si un jour on a un serveur qui doit repondre a 100 000 personnes dans la meme seconde et que le delai de reponse est important alors il faudra se poser la question , mais pour le moment je pense qu'on est loin de ces contraintes .

J'ai eu l'occasion d'avoir a optimiser des applis dans ce sens , et meme a re ecrire des instructions de micro code qui geraient des instruction d'assembleur .Mais a ma connaissance ce sont vraiment des cas tres tres rares .

Pour résumer , je pense donc que vraiment le temsp de lecture des blancs est tellement négligeable qu'il ne faut pas s'en inquieter mais qu'avoir un code tres lisible est bien plus important (Pour la maintenance du programme par soi meme ou les copains qui travailleront un jour dessus ).

Posté le : 10/11/2005 17:23

Cordialement
Denis Open in new window
Partager Twitter Partagez cette article sur GG+
Rapport Haut
CPascal
Re: news 1.42
#5
Semi pro
Inscrit: 06/09/2005 01:34
De Royan
Messages: 609
Merci Denis pour toutes ces explications si bien détailler,cela ma aider à bien comprendre c'est trés sympas de ta part.
Merci vraiment.
cordialement

Posté le : 10/11/2005 17:43

bergers aux yeux de ciel
Partager Twitter Partagez cette article sur GG+
Rapport Haut

 Haut   Précédent   Suivant



Vous pouvez voir les sujets.
Vous ne pouvez pas débuter de nouveaux sujets.
Vous ne pouvez pas répondre aux contributions.
Vous ne pouvez pas éditer vos contributions.
Vous ne pouvez pas effacez vos contributions.
Vous ne pouvez pas ajouter de nouveaux sondages.
Vous ne pouvez pas voter en sondage.
Vous ne pouvez pas attacher des fichiers à vos contributions.
Vous ne pouvez pas poster sans approbation.
Propulsé avec XOOPS | Graphisme adapté par Tatane, Grosdunord, Montuy337513

111 Personne(s) en ligne (62 Personne(s) connectée(s) sur Forum) | Utilisateur(s): 0 | Invité(s): 111 | Plus ...