[PHP] problema con l'echo

Non riesco a far funzionare questo script, cioè se non sono loggato da admin esce "non sei un admin", ed è giusto, però se sono loggato non esce la scritta "form".
Come risolvo???

<?php

//includiamo il file di configurazione

@include "config.php";
session_start();


if(!isset($_SESSION['login']))
{
echo "Non sei admin";
}
else
{

if(isset($_POST['submit'])){
  if(isset($_POST['autore'])){
    $autore = addslashes($_POST['autore']);
  }
  if(isset($_POST['titolo'])){
    $titolo = addslashes($_POST['titolo']);
  }
  if(isset($_POST['articolo'])){
    $articolo = addslashes($_POST['articolo']);
  }

  // popoliamo i campi della tabella articoli con i dati ricevuti dal form
  $sql = "INSERT INTO articoli (art_autore, art_titolo, art_articolo, art_data) VALUES ('$autore', '$titolo', '$articolo', now())";

  // se l'inserimento ha avuto successo inviamo una notifica
  if (@mysql_query($sql) or die (mysql_error())){
    echo "Articolo inserito con successo.";
   }
   
else
{
echo "form";

  
}


}
}

?>

[alemoppo]

Se indenti correttamente, si capisce il problema.

<?php 

//includiamo il file di configurazione 

@include "config.php"; 
session_start();

if(!isset($_SESSION['login'])) 
{ 
    echo "Non sei admin"; 
} 
else 
{
    if(isset($_POST['submit']))
    { 
          if(isset($_POST['autore'])) { 
            $autore = addslashes($_POST['autore']); 
          } 
          if(isset($_POST['titolo'])){ 
            $titolo = addslashes($_POST['titolo']); 
          } 
          if(isset($_POST['articolo'])){ 
            $articolo = addslashes($_POST['articolo']); 
          } 

          // popoliamo i campi della tabella articoli con i dati ricevuti dal form 
          $sql = "INSERT INTO articoli (art_autore, art_titolo, art_articolo, art_data) VALUES ('$autore', '$titolo', '$articolo', now())"; 

          // se l'inserimento ha avuto successo inviamo una notifica 
          if (@mysql_query($sql) or die (mysql_error())){ 
            echo "Articolo inserito con successo."; 
          } 
          else 
          { 
            echo "form"; 
          } 
    } 
} 
?>

form lo scrive soltanto se fallisce la query (ma non lo scirverà comunque, perché eseguirà die(mysql_query())...

Ciao!

Il mio else doveva servire a se non ha postato if(isset($_POST['submit'])), come faccio ad aggiustarlo?

[alemoppo]

appunto, quindi dovrai fare:

<?php 

//includiamo il file di configurazione 

@include "config.php"; 
session_start();

if(!isset($_SESSION['login'])) 
{ 
    echo "Non sei admin"; 
} 
else 
{
    if(isset($_POST['submit']))
    { 
          if(isset($_POST['autore']))
          { 
            $autore = addslashes($_POST['autore']); 
          } 
          if(isset($_POST['titolo'])){ 
            $titolo = addslashes($_POST['titolo']); 
          } 
          if(isset($_POST['articolo'])){ 
            $articolo = addslashes($_POST['articolo']); 
          } 

          // popoliamo i campi della tabella articoli con i dati ricevuti dal form 
          $sql = "INSERT INTO articoli (art_autore, art_titolo, art_articolo, art_data) VALUES ('$autore', '$titolo', '$articolo', now())"; 

          // se l'inserimento ha avuto successo inviamo una notifica 
          if (@mysql_query($sql) or die (mysql_error())){ 
            echo "Articolo inserito con successo."; 
          }
    }
    else 
    { 
          echo "form"; 
    }  
} 
?>

no?

Ciao!

grazie, sono nuovo del php e non sapevo che l'indentazione facesse la differenza, grazie.

[alemoppo]

L'indentazione NON FA la differenza, ma le parentesi graffe sì
L'indentazione serve soltanto per capire... e leggere meglio.

Ciao!

ah ho visto ora l'altra parentesi, grazie ancora