Problème valeur par default de l'include

Version imprimable

Comme a dit un jour un grand codeur "un petit code vaut mieux qu'un long discourt"

Donc voici mon code

Code:

1
2
3
4
5
6
7
8
9
10
   <div id="menu_top">
       <ul>
            <li><a href="index.php?page=accueil">Accueil</a></li>
            <li><a href="index.php?page=news">News</a></li>
            <li><a href="index.php?page=portfolio">Portfolio</a></li>
            <li><a href="#">Forum</a></li>
            <li><a href="index.php?page=livre dor">Livre d'Or</a></li>
            <li><a href="index.php?page=contact">Contact</a></li>
        </ul>
    </div>

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
 
<div id="contenu_body">
				<?
				switch($page)
					{
						case 'accueil' :
							include 'accueil.php';
							break;
 
						case 'news' :
							include 'news.php';
							break;
 
						case 'portfolio' :
							include 'portfolio.php';
							break;
 
						case 'livre dor' :
							include 'livre dor.php';
							break;
 
 
						case 'livre dor' :
							include 'contact.php';
							break;
 
					default : 
						include 'accueil.php';
							break;
					}
 
				?>
 
        </div>

Donc voici le code source.(utile si vous avez besoin de voir tout le code source faite mois signe :ccool:)
Je ne sait pas pourquoi ma page m'affiche TOUJOURS la valeur de la variable $page par default

Voila mon problème s'il vous plait aidez moi :calim2:

04/09/2009, 17h31
Amara

Slt,

Est-ce qu'un $_GET['page'] au lieu de $page change quelque chose ?

Sinon par ailleurs pour le livre dor je pense qu'il est préférable de ne pas avoir d'espace ca peut être source de problèmes.

Oki c'est bon sa marche nikel maintenant

Donc voici le code source ;)

Code:

1
2
3
4
5
6
7
8
9
10
   <div id="menu_top">
       <ul>
           <li><a href="?page=accueil">Accueil</a></li>
            <li><a href="?page=news">News</a></li>
            <li><a href="?page=portfolio">Portfolio</a></li>
            <li><a href="?page=forum">Forum</a></li>
            <li><a href="?page=livre">Livre d'Or</a></li>
            <li><a href="?page=contact">Contact</a></li>
        </ul>
    </div>

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
<?php
    if ($_GET['page'] == "accueil")
    {
        include("accueil.php");
    }
 
    if ($_GET['page'] == "news")
    {
        include("news.php");
    }
 
	if ($_GET['page'] == "portfolio")
    {
        include("portfolio.php");
    }
 
	if ($_GET['page'] == "forum")
    {
        include("forum.php");
    }
 
   	if ($_GET['page'] == "contact")
    {
        include("contact.php");
    }
 
	if ($_GET['page'] == "livre")
    {
        include("livre.php");
    }
 
    ?>

Par contre j'ai 2 petites questions:
Es-ce que la niveau sécurité je ne risque pas de me faire avoir par la faille include?
J'aimerai bien faire un else

Code:

1
2
3
4
	else
	{
		print "Désoler,mauvaise page";
	}

mais lorsque je mets mon else a la fin de mes if sa me marque en permanence(sur toutes les pages) "Désoler,mauvaise page"
Pourtant j'ai bien mis else se qui fait que normalement sa devrait s'afficher que si on choisi une mauvaise page(ou qu'on tente une RFI ou LFI)

question sécurité, tu peux faire ca:
Code:

1 2 $page = addslashes($_GET['page']); if (strlen($page) >=10) unset($page);

05/09/2009, 09h28
tham_
pour ce qui est de ton else c'est normal que ça ne donne pas le résultat espéré puisque avec ta syntaxe il ne concerne que le dernier if.

il faudrait faire comme ça:
Code:

1 2 3 4 5 6 7 8 if ($page == 'accueil') { ... } elseif ($page == '...') { ... } ... else { print "désolé mauvaise page"; }
ou bien utiliser un switch comme tu avais commencer à le faire (il me semble qu'en terme de rapidité c'est mieux en plus)

Donc j'ai fait avec le switch voila se que sa donne et sa marche niquel ^^

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
 
    <?php
	if (isset ($_GET['page']))
		{
		$page = $_GET['page'];
		}
	else
		{
		$page ="";
		}
 
switch ($page)
         {
            case "accueil":
               include 'accueil.php';
               break;
            case "news" :
               include 'news.php';
               break;
            case "portfolio" :
               include 'portfolio.php';
               break;
            case "forum" :
               include 'forum.php';
               break;
            case "livre" :
               include 'livre.php';
               break;
            case "contact" :
               include 'contact.php';
               break;
            default :
               include 'accueil.php';
         }
      ?>

La je croit que c'est "parfait" ?

Par contre j'ai un autre problème je fait un nouveau post ;)

06/09/2009, 17h44
Niki59

Salut ptifou78,

La solution est simple, on ne met pas "break;" en fin de ligne de d'un "default:" ;)