[SQL] Script PHP qui marche pas !

Version imprimable

Bonsoir,

Voici un code qui affiche un menu en PHP : page1 - page2 - page3 - page4
en cliquant sur les pages.

Le problème c'est quil ne marche pas !
et je ne vois pas ce qui ne fonctionne pas.
Merci pour votre aide.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
<?php
// liste des rubriques
$magik1 = array("page1","page2","page3","page4");
// page affiché à l'ouverture du menu
if($magik2){}else{$magik2="page1";}
$magik3 = sizeof($magik1);
for($magik4=0;$magik4 < $magik3;$magik4++){
if ($magik2 == $magik1[$magik4]){
echo"<a href=web.php?magik2=$magik1[$magik4]>$magik1[$magik4]</a>&nbsp;";}//pour obtenir un menu vertical ajouter la balise <br> après :&nbsp;
else{echo"<a href=web.php?magik2=$magik1[$magik4]>$magik1[$magik4]</a>&nbsp;";}}//pour obtenir un menu vertical ajouter la balise <br> après :&nbsp;
echo"<br><br>"; // séparation entre les liens du menu et du contenu des pages
include"$magik2.php";?>

11/02/2007, 02h08
poof65

C'est quand même ignoble ton code, tu pourrais trouver des noms de variable plus explicites.

J'utiliserais plutot

Code:

$magik2 = (!empty($_GET['magik2']) ? $_GET['magic2'] : 'page1');

Obfuscateur intégré :mouarf: ? C'est clair qu'on fait difficilement plus illisible 8O.

J'ai une préférence pour ce code que j'ai testé :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 <?php // Page par défaut define('PAGE_PAR_DEFAUT', 'page1'); // Liste des rubriques $pages = array( 'page1', 'page2', 'page3', 'page4' ); // Page affichée à l'ouverture du menu $page_courante = isset($_GET['magik2']) ? $_GET['magik2'] : PAGE_PAR_DEFAUT; foreach ($pages as $p) { if ($page_courante == $p){ echo "<a href=web.php?magik2=$p>$p</a> "; // Pour obtenir un menu vertical ajouter la balise <br> après :  } else{ echo "<a href=web.php?magik2=$p>$p</a> "; // Pour obtenir un menu vertical ajouter la balise <br> après :  } } echo "<br/><br/>"; // Séparation entre les liens du menu et du contenu des pages include("$page_courante.php"); ?>
Les variables passées par l'URL (méthode GET donc), se récupèrent par le tableau superglobal $_GET (sauf si register_globals est à On, ce qui n'est pas conseillé).

Julp.

11/02/2007, 08h01
Sub0

Il vaut mieux toujours utiliser des simples quotes pour les echo et réserver les doubles pour les propriétés des balises HTML ou les requêtes SQL éventuellement. Exemple pour ton 1er lien :

Code:

echo '<a href="web.php?magik2='.$magik1[$magik4].'">'.$magik1[$magik4].'</a> ';

C'est expliqué pourquoi ici :
http://pbnaigeon.developpez.com/tuto...phe-guillemet/
11/02/2007, 10h20
s.n.a.f.u

Bonjour,

Franchement, quand je vois du code comme ça, je n'ai pas franchement envie de me plonger dedans. Et il fera peur même à son dévellopeur lorsque celui-ci reviendra dessus 3 mois après.
Imaginez un bouquin écrit de la sorte, vous l'achetez ?

Bref, avant de résoudre la syntaxe, il faut mettre son code en forme !

Allez hop Diabless, je vais te mettre dans un camps de travaux forcés en Python, tu vas apprendre ce qu'est l'indentation !
Et si tu ne comprends pas pourquoi c'est illisible, félicitations, tu peux te mettre à Perl.
11/02/2007, 14h37
Diabless6

Bonjour à tous,

Merci pour vos réponses, je tiens a dire que je n'ai pas écrits ce code ! Je l'ai télécharger sur un site de script php :oops:

Pour ce qui est de la lisibilité je suis entièrement d'accord, jy comprends rien. Enfin avec le peu que je sais d'habitude je pige le fonctionnement et je modifie a mon gré, mais là oups :aie:

Donc je vais étudier ce que vous m'avez donner ;)

Merci pour vos réponses.
11/02/2007, 14h40
Diabless6

Citation:

Envoyé par Sub0

Il vaut mieux toujours utiliser des simples quotes pour les echo et réserver les doubles pour les propriétés des balises HTML ou les requêtes SQL éventuellement. Exemple pour ton 1er lien :

Code:

echo '<a href="web.php?magik2='.$magik1[$magik4].'">'.$magik1[$magik4].'</a> ';

C'est expliqué pourquoi ici :
http://pbnaigeon.developpez.com/tuto...phe-guillemet/

Merci pour le lien car c'est très confus, un coup je vois des ' ' et parfois des "", alors je sais quand rapidité d'execution il vaut mieux les ' '.
Merci pour la doc, je vais la téléchargée pour comprendre ;)
11/02/2007, 14h53
Diabless6

Merci Julp pour la correction du script, il fonctionne bien maintenant :king:

Je préfère les scripts aux bouquets de fleurs :mrgreen: On va dire que c'est ma ST Valentin :mouarf:

Merci encore @tous ;)

Ps: Ha! oui j'utilise PsPad pour la fabrication de mes pages web, j'ai trouvé les fonctions pour l'indentation soit en CSS ou le code entier. Mais pour la sélection d'une partie du code je n'ai pas trouvé comment faire. Quelqu'un aurait une astuce ?
Merci
12/02/2007, 08h54
Dia_FR

ben... chez moi, sélectionner la partie de code voulue puis "reformater" ça marche très bien
par contre, les formatages de code de pspad sont pas toujours très judicieux, pense à repasser derrière pour refaire un peu le boulot (ni fait ni à faire quoi)

enfin, si je peux me permettre j'ai laché depuis quelques mois pspad pour easyeclipse for php
autant sur des fichiers isolés, pspad fait bien l'affaire, autant sur de gros projets, ee4php est largement mieux
12/02/2007, 16h28
Diabless6

Citation:

Envoyé par Dia_FR

ben... chez moi, sélectionner la partie de code voulue puis "reformater" ça marche très bien
par contre, les formatages de code de pspad sont pas toujours très judicieux, pense à repasser derrière pour refaire un peu le boulot (ni fait ni à faire quoi)

enfin, si je peux me permettre j'ai laché depuis quelques mois pspad pour easyeclipse for php
autant sur des fichiers isolés, pspad fait bien l'affaire, autant sur de gros projets, ee4php est largement mieux

Bonjour,

oui c vrai que faut tjours repasser derrière c un peu chiant, je pensais que ça venait d'une option..

Je vais regarder le lien que tu mas mis. Merci