TSTCBAN V.2 , SIMPLE GÉNÉRATEUR DE BANNIÈRES

Commentaire de i_robot le 21/10/2006 18:57:02

J'attend vos commentaires

Commentaire de jean84 le 22/10/2006 18:15:06

Re-

j'ai regarder le code :
- rien de bien complique, c'est bien commente, l'utilisation du tableau natif $_FILES[] est judicieuse (d'ailleurs je vais la retenir)
- tu utilises des ' partout la ou il faut, c'est bien pour les debutants qui regarderont ton code
- par contre, mieux vaut utiliser <?php plutot que <?, question de securite (si tu as plusieurs types de languages sur la meme page, sa peut creer des conflits)
- dans veriftstcban.php, y a un truc bizarre, c'est dans la generation des liens (apercu) :

"<p>Un Apercu :</p>
<a href="<? echo $_POST['site'] ?>" title="<? echo $_POST['titre'] ?>" target=_blanc>
<img src="<? echo $_POST['urlimage'] ?>" height="<? echo $_POST['dimlargeur'] ?>" width="<? echo $_POST['dimtaille'] ?>" ></a>
<br><br><br>Le code source :<textarea    name="textarea" rows="10"><a href="<? echo $_POST['site'] ?>" title="<? echo $_POST['titre'] ?>">
<img src="<? echo $_POST['urlimage'] ?>" height="<? echo $_POST['dimtaille'] ?>" width="<? echo $_POST['dimlargeur'] ?>" ></a></textarea><br><br><br><a href="tstcban.php">Retour au générateurs de banière</a><br>
<a href="http://tstc.awardspace.com/index.php">Retour a l'accueil</a>"

Desole mais al c'est horrible (quoi que j'exagere). Tu aurais gange en lisibilite si tu avais utiliser l'operateur de concatenation (le point .) afin d'eviter d'utiliser une mirriade de <?. J'en suis pas sur mais a mon avis, question vitesse, sa doit pas etre la joie... A reviser donc

Et le dernier point sera quand a l'interet pure du code :
En lisant "generation de banniere", on s'attend reellement a generer une banniere sauf que la, ce n'est aps le cas. Tu crees juste un lien sur une image (en plus tu oublies de mettre border="0" ce qui fait qu'il y a un liseret bleu tout autour de l'image ce qui est du plus mauvais effet).
Mon conseil : tu crees une image vide siuvant les dimensions que l'utilisateur t'as indiques, tu redimmensionnes l'image (en gardant le ratio) et tu la copies dans ta nouvelle image vide. Tu ecris ensuite le lien qui a ete transmis a cote de l'image (a droite, a gauche ou au milieu) et tu renvoi l'image ainsi cree. Tu obtiendras a ce moment une VRAI banniere, avec un logo et un texte dans l'image.

C'est en tout cas l'idee de base propre a ce type de code :-)

@++

Commentaire de jean84 le 23/10/2006 17:35:48

Kankrelune > je le suis plante dans le choix du mot, c'etait bien le mot compatibilites que je voulais dire.

"après c'est une question d'habitudes et de gouts... certains trouvent ça plus clair d'autre moin... .. ."
Mouais.. moi je trouve sa immonde ^^

Commentaire de Mastronic le 14/11/2006 18:59:02

De mon coté, quand je charge un fichier de type jpg... ton programme m'indique qu'il ne connait pas au format jpg!
idem pour le format png.
pour gif, c ok.

PS: j'ai vue le meme effet de ce que tu a fait sur un autre site:
http://www.cybertoutou.com/fairebanniere.html
il utilise le javascript.

idem que Jean84, j'attendais a voir des appels au GD
Regarde les sources utilsant la librairie Gd. Utilise le moteur de recherche de ce site ( en haut) et tapes minatures.. tu pourra t'amusé avec volonté.

bonne continuation.