Organizzare codice

**Raulken** · 28-10-2011, 12:47

ragazzi io quando devo far navigare l'utente per il sito richiamo una variabile get

www.sito.com/page=nomepagina

<html>

<head>
<?php if(isset($_GET['page'])){
if($_GET['page']=="nomepagina"){
?>
<script src="percorso.js"></script>
<?php
}
}
?>
</head>

<body>
if(isset($_GET['page'])){
if($_GET['page']=="nomepagina"){
include("percorso.php");
}
}

</body>
</html>

questo per me è sembrato un vantaggio perchè non dovevo riscrivere tutto il corpo delle pagina ma semplicemente includevo i contenuti nei vari punti che mi interessavano

pero' mi rendo conto anche che se le pagine sono tante ci saranno una marea di if else

quindi domanda:
conviene fare come me che utilizza una sola pagina principale e poi richiama i vari contenuti

oppure per ogni pagina devo creare una pagina apposita per essa?

**philbert** · 28-10-2011, 12:54

Ciao,
io ti consiglio di Crere una pagina per ogni contenuto.
E' la cosa migliore per una serie di motivi, non utimo il posizionamento suo motori di ricerca.

Poi se hai una gran quantità di materiale che compare sempre uguale in tutte le pagine, mettilo in un file a parte che poi includi in tutte le pagine.

Bytez
P.

**Simo990** · 28-10-2011, 13:16

Ciao, va benissimo quello che fai, ovviamente sarebbe molto più faticoso e ingestibile riscrivere ogni volta il corpo della pagina... metti caso che hai 2000 pagine e ti accorgi che devi cambiare un piccolo dettaglio, che fai?

Comunque prima di tutto non c'è bisogno che controlli due volte se $_GET['page'] esiste.
Poi dovresti controllare se la pagina effettivamente esiste...
io di solito uso uno switch

Codice PHP:


<head>

<?php

$p = isset($_GET['page']) ? trim($_GET['page']) : '';

?>

</head>



<body>

<?php

switch ($p):

   case 'home':

      include_once 'home.php';

      break;

   case 'contatti':

      include_once 'contatti.php';

      break;

   default:

      include_once '404.php';

endswitch;

?>

</body>

così stai sicuro che nessuno va a scrivere nell'url una pagina che non esiste.

**wartpro** · 28-10-2011, 13:32

Qui mi fu dato un consiglio utilissimo sul come organizzare il codice: http://forum.html.it/forum/showthrea...readid=1479427

Da quel giorno per ogni elemento di pagina (intendo dire: box login, colonna centrale, ecc..) creo un file e poi li includo a seconda delle esigenze.

Sinceramente il metodo $_GET non l'ho mai usato anche perchè...non l'ho capito. Utilizzo switch basati sul valore di PHP_SELF.

**Raulken** · 28-10-2011, 13:55

Grazie simo mi piace la tua idea

wartpro mi puoi fare un esempio pratico di come gestisci tu la cosa? Grazie

**wartpro** · 28-10-2011, 14:00

Box Login: login.php

Codice PHP:


<?php

# Modulo log in

?>

<div class="form-login">

 <form>

   <label for="user">Username: </label>

   <input id="user" type="text" value="" />

    <input type="submit" value="Invia" />

 </form>

</div>

..e poi lo includi nella pagina..

Pagina: index.php

Codice PHP:


<html>

<head></head>

<body>

 <?php

   include_once('login.php');

 ?>

</body>

</html>

..e fai così per tutto il resto.

**philbert** · 28-10-2011, 14:35

Originariamente inviato da Simo990
Ciao, va benissimo quello che fai, ovviamente sarebbe molto più faticoso e ingestibile riscrivere ogni volta il corpo della pagina... metti caso che hai 2000 pagine e ti accorgi che devi cambiare un piccolo dettaglio, che fai?

ma se hai 2000 pagine cosa fai? un switch con 2000 case?????

**Raulken** · 28-10-2011, 15:03

Originariamente inviato da wartpro
Box Login: login.php

Codice PHP:


<?php

# Modulo log in

?>

<div class="form-login">

 <form>

   <label for="user">Username: </label>

   <input id="user" type="text" value="" />

    <input type="submit" value="Invia" />

 </form>

</div>

..e poi lo includi nella pagina..

Pagina: index.php

Codice PHP:


<html>

<head></head>

<body>

 <?php

   include_once('login.php');

 ?>

</body>

</html>

..e fai così per tutto il resto.

a si questo metodo già lo utilizzo

**Raulken** · 28-10-2011, 15:03

Originariamente inviato da philbert
ma se hai 2000 pagine cosa fai? un switch con 2000 case?????

hai ragione anche tu...
per questo sono combattuto

**LuckySevenRoX** · 28-10-2011, 15:12

la soluzione alle "2000 pagine" è questa:

codice:

<?
$pag = $_GET['pagina'];
include($pag . ".php");
?>

così eviti lo switch e includi direttamente la pagina che ti interessa.. poi ovviamente devi star attento a eventuali problemi di sicurezza che possono nascere da questo codice.. ma almeno è un inizio

Discussione: Organizzare codice

Strumenti discussione

Ricerca discussione

Visualizza

Organizzare codice

Permessi di invio