par
moogli » 21 juin 2011, 20:11
il te faut penser à bien utiliser les balise html, donc mettre les " pour tous les attributs htmlt (<img src="image" /> et non <img src=image>)
qui plus est tu peux largement simplifier le code en supprimant les tableaux dans la boucle while (ils sont inutile tu dispose déjà des infos).
j'ai testé rapidement ton script :
- faut vraiment que tu revois ta classe de "réduction" des images, car celle ci fait un "crop" et non une réduction, pire quand l'image est trop petite elle l'augmente ... tuto sur le sujet =>
Comment faire une galerie de miniature
- faire un code html valide
- utiliser
mysql_free_result() après une requête sql.
un code fonctionnel
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="fr" lang="fr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
<link rel="stylesheet" href="css/admin.css" type="text/css" media="screen" />
</head>
<body>
<div id="carrousel">
<?php
// include('params.inc.php');
$connect=mysql_connect(MYHOST,MYUSER,MYPASS) or die ('erreur de connexion');
mysql_select_db(MYDB) or die ('erreur de connexion à la base');
$requeteselect = 'SELECT id,lienimage,url FROM images';
$result = mysql_query($requeteselect, $connect);
$tableau = array();
while($image_courante = mysql_fetch_array($result,MYSQL_ASSOC)){
//on met toutes les infos dans un tableau
echo '<div>
<a href="'.$image_courante['url'].'">
<img src="'.$image_courante['lienimage'].'"> </a></div>';
$image_courante++;
}
mysql_free_result($result);
mysql_close($connect);
?>
</div>
</form>
</body>
</html>
ça serait plutot sympa si tu pouvais utiliser la balise php (y a un bouton juste au dessus de la boite de saisie) le code en couleur c'est vachement mieux
@+
il te faut penser à bien utiliser les balise html, donc mettre les " pour tous les attributs htmlt (<img src="image" /> et non <img src=image>)
qui plus est tu peux largement simplifier le code en supprimant les tableaux dans la boucle while (ils sont inutile tu dispose déjà des infos).
j'ai testé rapidement ton script :
- faut vraiment que tu revois ta classe de "réduction" des images, car celle ci fait un "crop" et non une réduction, pire quand l'image est trop petite elle l'augmente ... tuto sur le sujet => [url=http://www.lephpfacile.com/howto/12-comment-faire-un-script-de-galerie-photos-miniatures-en-php]Comment faire une galerie de miniature[/url]
- faire un code html valide
- utiliser [url=http://www.php.net/mysql_free_result]mysql_free_result()[/url] après une requête sql.
un code fonctionnel
[php]
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="fr" lang="fr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
<link rel="stylesheet" href="css/admin.css" type="text/css" media="screen" />
</head>
<body>
<div id="carrousel">
<?php
// include('params.inc.php');
$connect=mysql_connect(MYHOST,MYUSER,MYPASS) or die ('erreur de connexion');
mysql_select_db(MYDB) or die ('erreur de connexion à la base');
$requeteselect = 'SELECT id,lienimage,url FROM images';
$result = mysql_query($requeteselect, $connect);
$tableau = array();
while($image_courante = mysql_fetch_array($result,MYSQL_ASSOC)){
//on met toutes les infos dans un tableau
echo '<div>
<a href="'.$image_courante['url'].'">
<img src="'.$image_courante['lienimage'].'"> </a></div>';
$image_courante++;
}
mysql_free_result($result);
mysql_close($connect);
?>
</div>
</form>
</body>
</html>[/php]
ça serait plutot sympa si tu pouvais utiliser la balise php (y a un bouton juste au dessus de la boite de saisie) le code en couleur c'est vachement mieux ;)
@+
