[SQL] Script PHP qui marche pas ! [Fait] [Résolu]

[Diabless6]

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";?>

[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');

[julp]

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

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>&nbsp;"; // Pour obtenir un menu vertical ajouter la balise <br> après :&nbsp;
	} else{
		echo "<a href=web.php?magik2=$p>$p</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("$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.

[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>&nbsp;';

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

[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.

[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

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

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

Merci pour vos réponses.

[Diabless6]

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>&nbsp;';

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

[Diabless6]

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

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

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

[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

[Diabless6]

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