votre avis sur mon premier script php (100 lignes)

Marsh Posté le 08-07-2005 à 14:30:08

voici mon premier script php: il tient sur une page en fonction du lien indiqué il est accompagné d'un fichier css très très court:
fichier index.php :

Code :

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1 Strict//EN" "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="fr" >
<head>
<meta http-equiv="content-type" content="text/html; charset=iso-8859-1" />
<meta name="author" content="Benoit de Biolley" />
<meta name="keywords" content="garden, jardin, jardins, detente, plaisir, architecture" />
<meta name="description" content="Un jardin pour la détente." />
<meta name="robots" content="all" />
<title>Un jardin pour la détente</title>
<script type="text/javascript"></script>
<style type="text/css" media="all">
@import "style.css";
</style>
</head>
<body onload="window.defaultStatus='Un jardin pour la détente';" link="#999999" vlink="#999999" alink="999999">
<script type="text/javascript">
var x,y;
if (self.innerHeight) // all except Explorer
{
x = self.innerWidth;
y = self.innerHeight;
}
else if (document.documentElement && document.documentElement.clientHeight)
// Explorer 6 Strict Mode
{
x = document.documentElement.clientWidth;
y = document.documentElement.clientHeight;
}
else if (document.body) // other Explorers
{
x = document.body.clientWidth;
y = document.body.clientHeight;
}
</script>
<center>
<div id="container">
<?
if (isset ($_GET['photo'])) //si on doit afficher une photo
{
?>
<script type="text/javascript">
document.write('<img src= <? echo "photos/".$_GET['photo'].".jpg" ?> alt="Photo jardin" width="' + (x/1024*600) + '" height="' + (y/600*450) + '" />')
</script>
<br>
<br>
<br>
<A HREF="Javascript:history.go(-1)">Retour à la page précédente</A>
<?
}
else //si on doit afficher la page avec les miniatures
{
$k=0;
if (isset ($_GET['mini']))
{
$nb=$_GET['mini'];
if ($nb<10) $nb*=10;
else if ($nb<100) $nb*=10;
$nb++;
}
else $nb=1;
mysql_connect("localhost", "root", "" ); // Connexion à MySQL
mysql_select_db("bdebiolley" ); // Sélection de la base coursphp
$repmini = mysql_query("SELECT DISTINCT type FROM miniatures WHERE nr>=$nb AND nr<($nb+9) ORDER BY nr LIMIT 9" ); // Requête SQL
mysql_close();
for($i=$nb;$i<($nb+9);$i++)
{
$miniature = mysql_fetch_array($repmini);
$k++;
?>
<A <?
switch ($miniature['type'])
{
case 1: echo "";
break;
case 2: echo 'href="?mini='.$i.'"';
break;
case 3: echo 'href="?photo='.$i.'"';
break;
}
?>
>
<script type="text/javascript">
document.write('<img src= <? echo "miniatures/mini_".$i.".jpg" ?> alt="Photo jardin" width="' + (x/1024*180) + '" height="'+ (y/600*180) +'" border="0" HSPACE="0" VSPACE="0"/>')
</script>
</A>
<?
if ($k%3==0)
{
?>
<br>
<?
}
}
if ($nb!=1)
{
?><A HREF="Javascript:history.go(-1)">Retour à la page précédente</A><?
}
}
?>
</div>
</body>
</HTML>

fichier style.css

Code :

body {
font: 8pt/16pt georgia;
color: #555753;
margin: 0px;
background: #fff url(blossoms.jpg) no-repeat bottom right;
background-color: #999999;
padding: 0px 0px 0px 0px;
}
a {
color: #ffffff;
}
/* specific divs */
#container {
background: url(zen-bg.jpg) no-repeat top left;
padding: 20px 60px 20px 20px;
margin: 0px;
}

---------------
-------------------

Reply

Marsh Posté le 08-07-2005 à 14:30:08

Reply

Marsh Posté le 08-07-2005 à 14:33:06

oui, et tu veux qu'on en fasse quoi? :??:

---------------
MZP est de retour

Reply

Marsh Posté le 08-07-2005 à 14:34:41

vos commentaires sur la structure
si c assé clair
vu que j'ai tout fait moi même en fesant quelques copiers collé.
si y a moyen de l'optimiser,...

---------------
-------------------

Reply

Marsh Posté le 08-07-2005 à 14:37:46

je dirais, vite fait :

<br /> pas <br>
<center> n'existe plus
pas de controle des vraiables passer en GET ( $nb=$_GET['mini']; ) là c'est pas tres grave, tu fait juste un $nb++ mais bon, on sais jamais pour les autres variables.

++

ps: verifie quand meme ton code parceque tu as un doctype en XHTML1.1 et je suis pas vraiment sur que tu sois valide ...

Message édité par bl@p_psx le 08-07-2005 à 14:38:19

Reply

Marsh Posté le 08-07-2005 à 15:42:54

il n'est pas valide du tout son code même ...

Dans l'ordre :
-link,vlink,et cie dans le <body>, incorrect, css
-nom des balises toujours en minuscules, pas un coup minuscule et un coup majuscule
- "border="0" HSPACE="0" VSPACE="0" "> exemple, et en plus, tout ca, c'est dans la css.
- "<script type="text/javascript"></script>", au début de ton script. Ca sert a rien si tu n'indiques pas de src

Enfin, le xhtml 1.1 doit être servi dans le format application/xhtml+xml, mais IE ne sait pas le lire. Donc passe à un doctype xhtml 1.0 Strict par exemple.

---------------
http://www.alsacreations.com , http://www.openweb.eu.org. Mon CV : http://cv.roane-irkana.net/. Exemple à ne surtout pas suivre : www.worldinternet.be

Reply

votre avis sur mon premier script php (100 lignes)

Sujets relatifs:

Leave a Replay