Nascondere div se l'utente non è loggato

Salve a tutti!
volevo chiedervi se è possibile nascondere un div se l'utente non esegue il login.
Ho provato a scrivere il codice ma sembra non funzionare...infatti mi mostra la pagina completamente bianca

<?php
if (session_id() == "")
{
   session_start();
}

if (!isset($_SESSION['username']))
{
  ?>
      <script language="javascript">
        alert("ok");        
            document.getElementByClass("login").style.display = "none";
        
        </script>
      <?php
   
   exit;
}

if (isset($_SESSION['expires_by']))
{
   $expires_by = intval($_SESSION['expires_by']);
   if (time() < $expires_by)
   {
      $_SESSION['expires_by'] = time() + intval($_SESSION['expires_timeout']);
   }
   else
   {
      unset($_SESSION['username']);
      unset($_SESSION['expires_by']);
      unset($_SESSION['expires_timeout']);
      ?>
     <script language="javascript">
        alert("ok");        
            document.getElementByClass("login").style.display = "none";
        
        </script>
      <?php
      
      exit;
   }
}
?>

Cosa ce di sbagliato?

[socialedge]

session_start();                                        //Avvio la sessione
    require_once 'config.php';                                //Richiedo il file per la connessione al database
    
    if( !isset($_SESSION['user']) ) {                        //Se l'utente non è loggato
        header("Location: $baseurl");                        //lo rimando alla home
        exit;                                                //e interrompo lo script
    } 

Prova questo

Ma io non volevo reindirizzarlo sulla home...volevo visualizzare un div nascosto!

[socialedge]

struttura tutto in due pagine rende il caricamento più veloce.
Almeno questo è il mio consiglio.

[mzanella]

Con un redirect il client deve ricaricare una nuova pagina, il che è tutt'altro che più veloce.

La cosa più semplice è definire un predicato che indichi se l'utente è connesso o meno:

function is_logged() {
    if (!isset($_SESSION['username'])) {
        return false;
    }
    ...
} 

ed usarla per deciedere se mostrare o meno certi contenuti:

<?php if (is_logged()): ?>
  <div>utente collegato</div>
<?php else: ?>
  <div>utente non collegato</div>
<?php endif; ?>

Oppure per assegnare classi diverse (in modo che sia poi il CSS a determinarne la visualizzazione):

<div class="<?php echo (is_logged() ? "visible" : "hidden"); ?>">
  ...
</div>

ok...il problema adesso è che il codice php che ho scritto all'inizio non è corretto per individuare se l'utente è loggato.

[mzanella]

Dipende da come hai gestito le sessioni.
Ad occhio sembra che controllare che $_SESSION['username'] sia sufficiente a stabilire se un utente sia autenticato...

Riguardo al resto del codice, se credi che ci siano errori dovresti provare ad inserire

error_reporing(E_ALL); 

all'inizio dello script, in modo da avere qualche informazione in più.

ok ho risolto facendo piccole modifiche al codice :)
volevo introdurre il tuo codice:

<div class="<?php echo (is_logged() ? "visible" : "hidden"); ?>">
  ...
</div>

qui:

<article id="post-46377" class="post-46377 post type-post status-publish format-standard has-post-thumbnail hentry category-cura category-divertimento category-download tag-download-2 tag-ita tag-pc tag-prey <?php echo (!is_logged() ? "visible" : "hidden"); ?>" itemscope="" itemtype="http://schema.org/Article">

è giusto cosi?
In modo tale che se non ha eseguito il login non visualizza l'articolo

[mzanella]

Togli la negazione prima di is_logged() altrimenti fa il contrario

Dovrebbe diventare così:

<article id="post-46377" class="post-46377 post type-post status-publish format-standard has-post-thumbnail hentry category-cura category-divertimento category-download tag-download-2 tag-ita tag-pc tag-prey <?php echo (is_logged() ? "visible" : "hidden"); ?>" itemscope="" itemtype="http://schema.org/Article">

Ricorda che questo si limita ad aggiungere una classe chiamata "visible" o "hidden", per avere l'effetto finale devi indicare nel CSS:

.visible {
    display: block;
}
.hidden {
    display: none;
}

[mzanella]

Hai un link ad una pagina col problema?

Www.mondo-tech.it/prova2/TrucchiAndroid.php

La funzione la richiama correttamente...il problema sono i css

Non so come mai ma al posto di inserire hidden o visible sull if, metto style="display:none" riesce a leggerlo

ho provato in questo modo:

<?php if (!is_logged()){?>style="display:none"<?php } ?>

però analizzando la pagina non me lo rileva come stile css.
Ecco come me lo interpreta:
host image

[mzanella]

Se noti bene il tag style è finito dentro al tag class:

class="post-46377 post ... style="display: none""

mentre dovrebbe essere

class="post-46377 post ..". style="display: none"

Quindi...cosa devo modificare?

Aggiungo un doppio apice prima di style?

[mzanella]

Sì, esatto.

Funziona!!! Grazie mille :)
Un ultima domanda...per rendere la funzione accessibile in tutta la pagina la inserisco nel body?

[mzanella]

La funzione is_logged()? Potresti metterla in un file PHP a parte ed includerlo dove necessario, per esempio prima dell'HTML stesso.