Juste comme ça si le code ressemble bien à ce que tu as posté, tu devrais peut être évité la longue série de if pour déterminer le type de case.
Citation : A la place tu mets Code PHP :
Tu as juste à renommer tes images de 1 à n et c'est plus simple que de faire n tests successifs. Ou au moins remplace le par un switch. Apres ta carte est peut être mal générée. Déjà tu fais une requête sur toute ta table pour générer un tableau que tu ne vas pas parcourir en entier. Effectue plutot une requete pour récupérer seulement les éléments que tu vas afficher. Code PHP :
Et sinon il y a un problème de fermeture de balise dans ton exemple. Et avoir une carte qui fait 600x600 bah ça dépend juste de la taille de tes images et du nombre d'images affichées.
22-11-2009, 07:01 PM
Merci,je vais utilisez tes conseils.
Anthono
22-11-2009, 07:13 PM
(Modification du message : 22-11-2009, 07:14 PM par jo_link_noir.)
Salut,
Tu réaffectes une valeur à $decors et ça ne devrait pas être le cas. Au début c'est une ressource et ensuite une chaine de caractère donc la boucle ne ce fait plus car mysql_fetch_array attend une ressource. Au passage tu pourrais réduire l'enchainement de if en mettant tous dans un tableau, un peu comme ça :
Pour agrandir la carte suffi d'avoir les casses en bdd et récupéré que la partit qui intéresse. Par exemple si tu veux un carré de 11*11 cases, il faut mettre une condition dans la requête : Citation :pos_x > $pos_x - 5 && pos_x < $pos _x+ 5 && pos_y > $pos_y - 5 && pos_y > $pos_y + 5 Par rapport au début - Ne pas mettre @ mais utiliser isset pour vérifier l'existence de la variable - Tu peux éliminer ereg pour directement utiliser explode
Bon bah Crimson à été plus rapide...
22-11-2009, 07:14 PM
Merci, j'aprécies vos conseils mais je ais utilisé la maniére de crimson pour les images
Anthono
22-11-2009, 07:36 PM
Oui mais en fait il vient de te trouver le pourquoi tu n'as qu'une case d'affichée.
Parce que tu parcours les enregistrements contenu dans la variable $decors et qu'en fait tu stockes le chemin de l'image dans la variable $decors; Donc forcément il ne va pas parcourir qu'un enregistrement. Bien vu j'avais pas fait gaffe.
22-11-2009, 07:38 PM
Ouai je m'en suis apercu aussi a l'instant donc tous marche sauf la condition pour mon select que je suis en train de faire.
Anthono
23-11-2009, 10:24 AM
anthono :
deux petites remarques: -le premier c'est que si tu nomes tes images de terrain avec le numéro d'identifiant de celui-ci, cela permet d'afficher le terrain en mettant directement dans le src de l'image $num_terrain.gif... (plus besoin de tout ces if...) -Mais si tu tente a garder des trucs inutile, je te conseil cependant, pour ce qui concerne t'a belle rangé de if les un a la suite des autres, de faire un if suivi de elseif et finir par un else Vilou, bonne chance.
Dévotion, jeu multijoueur gratuit par navigateur de stratégie et de conquête
The Magic Institute, le jeu de magie médieval fantastique gratuit en ligne Rapture Studio : créateur de divertissement pour tous JePolitique.fr - débattons ensemble JécrisLaConstitution.fr - ne laissons pas les Hommes aux pouvoirs écrire les règles du pouvoir Je Deviens Citoyen (Association à but non lucratif)
23-11-2009, 04:23 PM
Un switch c'est quand même plus beau qu'une longue ligne de elsif. :ange:
Oui mais argorate aime l'optimisation ^^
HS: Mais les barres de progression n'ont pas été remis, ce sera fait ce soir. |
|