Antworten zum Thema „PHP include() Frage“

[aaaaldi]

Hallo zusammen.

Ich bin noch ein PHP-Anfäger und hab deswegen eine Frage:

Ich habe folgenden Quellcode erstellt:

<table width="512">
  <tr>
    <td><a href="index.php?action=news">News</a></td>
    <td><a href="index.php?action=impressum">Impressum</a></td>
    <td><a href="javascript:history.back()">Zur&uuml;ck</a></td>
  </tr>
</table>
<table width="512" height="128">
  <tr>
    <td align="left" valign="top">
	  <?php 

if ($_GET['action'] == "")
{
include"index.php";
}
elseif ($_GET['action'] == "news") include"news/news.html";
elseif ($_GET['action'] == "impressum") include"impressum/impressum.html";

     ?>
   </td>
  </tr>
</table>

Wenn ich dann die Datei und ".../index.php" öffne, passiert wohl ein Selbstbezug und es taucht etliche Male auf.
Wenn ich die Datei unter ".../index.php?action=news" öffne klappt alles in ordnung.

Wie kann ich diesen Selbstbezug vermeiden? (aber trotzdem den link ".../index.php" öffne)

 

[NicoB]

Der Code ist nicht sehr sauber geschrieben, sodass bei der IF-Abfrage möglicherweise einige Fehler auftreten könnten:

<?php switch ($_GET['action']) {
case news:
   include('news/news.html');
case impressum:
   include('impressum/impressum.html');
default:
   include('index.php');
} ?>

Achja: Wenn diese Datei als index.php abgespeichert wurde ist es kein Wunder, dass die Seite mehrmals erscheint.
Die Logik stimmt dann nämlich nicht: Wenn action nicht definiert ist, rufe diese Datei nochmals auf und stelle wieder fest dass action nicht definiert ist. Das macht keinen Sinn.

 

[aaaaldi]

nach deinem Code sieht es aber genau so aus



also müsste es nach deiner meinung so besser sein:

if ($_GET['action'] == "index")

Problem gelöst.

DAnke

 

[Duddle]

Das könnte mit

include_once("index.php");

funktionieren, aber eleganter fände ich es eigentlich, den tatsächlichen Inhalt von index.php in eine neue datei auszulagern (z.B. supertollerinhalt.php) und die dann zu includen.


Duddle

 

[NicoB]

Inhalt von index.php in eine neue datei auszulagern (z.B. supertollerinhalt.php) und die dann zu includen.

Ja genau, das meinte ich - konnte es nur nicht so schön in Worte fassen
Man muss immer aufpassen, dass durch bedingungsabhängige includes (oder includes generell) keine endlos-schleifen auftreten.

 

[Chickenshooter]

AW: PHP include() Frage

Moin, Moin

Auch eine gute Lösung wäre diese

Bastel Dir eine Datei Namens functions.inc.php
in diese schreibst du dir all deine funktionen die du so brauchst,
du hast dann nämlich nur noch eine Datei die du includen musst.
für ein kleineres Projekt denk ich völlig ausreichend.

Code deiner index.php

 <?php 
require_once("functions.inc.php");

switch ($_GET['action']) {
   case news:
     echo get_news();
   case impressum:
     echo get_impressum();
   default:
     echo get_default_Page();
} 
?>

 

[LowkaZ]

AW: PHP include() Frage

ich würd nicht die inhalte in die index laden sondern eher den kopf und den fuß in ne gesonderte index, dass is wesentlich suchmaschinen freundlicher

 

[CIX88]

AW: PHP include() Frage

^^^ fehlt das nicht ein break; bei switch ?
Bzw. müsste es nicht case 'news': sein, oder ist news eine Konstante ?

 

[Chickenshooter]

AW: PHP include() Frage

@Cix88
Jo hast Recht

so muss das nartürlich aussehen

 <?php 
require_once("functions.inc.php");

switch ($_GET['action']) {
   case news:
     echo get_news();
   break;
   case impressum:
     echo get_impressum();
   break;
   default:
     echo get_default_Page();
   break;
} 
?>

MfG