Kontrol over if sætning

Du kan skrive (!$post_empty) i stedet for ($post_empty == 0).

Kan vi få lidt mere kode? Hvis vi skal hjælpe, så er det nemmest når vi kender navne på alle variabler mm.

Men som udgangspunkt kan du tjekke, om der er noget i en variabel, ved blot at skrive :

if ($var1 && $var2 && $var2) {
    echo "Alle variabler indeholder et eller andet";
} else {
    echo "Mindst én af variablerne var tomme";
}

Det var for ikke at gentage at jeg lavede en link til det andeet spørgsmål. Der er koden der checker om felterne er udfyldt. Jeg prøver lige at lege med dit eksempel.

(!$post_empty) giver det samme. Den gemmer i basen når jeg trykker opdater. Det vil jeg meget gerne undgå.

Nu har jeg forsøgt lidt forskelligt igen, og enten skriver den i databasen når den skal, og så gør den det også ved opdatering, eller også det modsatte. Jeg kan ikke finde en kombination der ikke gemmer ved opdatering.

Du kan gøre det på denne måde

1) Din form bruger POST
2) Den går til en side, der gemmer data i databasen
3) Denne side redirecter til en side, der viser et svar til brugeren, evt ved at hente noget i databasen

På grund af denne redirect vil en opdater kun opdater "vis et svar" siden, og ikke
den side, der gemmer i databasen.

(hvis jeg ellers har forstået dig korrekt)

Ja du har forstået det, bortset fra at jeg ikke har flere sider. Det foregår på een side. Sidst jeg lavede noget lignende, blev jeg nødt til, at lave det på to sider for netop at undgå, at den gemmer i utide.

Men jeg ville gerne have det til at virke på een side. Jeg kan bare ikke finde en kombination der udelukker opdatering og så gemmer når alle felter er udfyldt og der trykkes på submit knappen.

Siden i 2) kan sagtens være samme side som i 3) - det er redirecten der er vigtig.

Ok. Form og det hele er på samme side. og jeg kan ikke finde nogen redirect.

Det starter sådan her:

if ((isset($_POST["MM_insert"])) && ($_POST["MM_insert"] == "form1")) {

if (!isset($_POST['form1']))
{
  $post_empty = 0;
  foreach ($_POST as $key => $value) {
      if ($_POST[$key] == "" || $_POST[$key] == NULL)
      {
        $post_emptykey[] = $key;
        $post_empty++;
      }
  }
  for ($i = 0; $i < $post_empty; $i++)
  {
  echo "<center><font color='#FF0000'>";
  echo 'Du skrev ikke noget i '.$post_emptykey[$i].'!<br><br>';
  echo "</font></center>";
  }
}
if ($post_empty == 0) {


if ((isset($_POST["MM_insert"])) && ($_POST["MM_insert"] == "form1")) {


if (!isset($_POST['form1']))
{
  $post_empty = 0;
  foreach ($_POST as $key => $value) {
      if ($_POST[$key] == "" || $_POST[$key] == NULL)
      {
        $post_emptykey[] = $key;
        $post_empty++;
      }
  }
  for ($i = 0; $i < $post_empty; $i++)
  {
  echo "<center><font color='#FF0000'>";
  echo 'Du skrev ikke noget i '.$post_emptykey[$i].'!<br><br>';
  echo "</font></center>";
  }
}

og det går galt her:

if ($post_empty == 0) {
  $insertSQL = sprintf("INSERT INTO ...... en masse linier kode

Nå det gik vist helt galt her

Sådan skulle det have været:

if ((isset($_POST["MM_insert"])) && ($_POST["MM_insert"] == "form1")) {

if (!isset($_POST['form1']))
{
  $post_empty = 0;
  foreach ($_POST as $key => $value) {
      if ($_POST[$key] == "" || $_POST[$key] == NULL)
      {
        $post_emptykey[] = $key;
        $post_empty++;
      }
  }
  for ($i = 0; $i < $post_empty; $i++)
  {
  echo "<center><font color='#FF0000'>";
  echo 'Du skrev ikke noget i '.$post_emptykey[$i].'!<br><br>';
  echo "</font></center>";
  }
}
if ($post_empty == 0) {
  $insertSQL = sprintf("INSERT INTO ...... en masse linier kode

Så skal du ca. sådan her

if ($post_empty == 0) {
  $insertSQL = sprintf("INSERT INTO ...... en masse linier kode
  header("Location: sammefil.php?vis=$id");
  exit;
}

hvor $id er nummeret på den indsatte, og vis gør, at der kommer et passende
svar til kunden (du skal have en ny if, der checker på om vi kommer ind på denne
måde).

Jeg har ikke kode, jeg kan vise dig nu, men jeg bruger det selv for tiden ;)

Mener du at:

  header("Location: sammefil.php?vis=$id");
  exit;

Skal ind i bunden lige før denne:

}

Giver det ikke en header already sent .... fejl ? Jeg synes jeg har prøvet noget lignende, men jeg prøver lige efter kaffepausen :-)

Hmmm, det virker, men ikke som det skal. Den glemmer simpelthen alle variabler ved at kalde den. Jeg har de fleste af felterne i sessions og dem glemmer den ved at bruge den header i bunden.

Rolig nu. Det er vist mig der er et fjols. Tester lige lidt mere.

Det ser ud til at virke Erik. TAK. Vil du svare, eller ? ;-)

Men jeg får en fejl i denne:

header("Location: sammefil.php?vis=$id");

når alle felter ikke er udfylt og den ryger i det check. Så skriver den:

Warning: Cannot modify header information - headers already sent by .....

Hvis alle felter ikke er udfyldt skal det vel ikke sættes ind i databasen, og så
kommer du ikke dertil...?

Men ellers skal dit script selvfølgelig overholde en passende struktur, der sikrer,
at der ikke kommer noget "ud" før det skal. Beregning først, og så al det der HTML
bagefter.

Det er jo en formular og hvis man glemmer et felt ryger den i:

if (!isset($_POST['form1']))
{
  $post_empty = 0;
  foreach ($_POST as $key => $value) {
      if ($_POST[$key] == "" || $_POST[$key] == NULL)
      {
        $post_emptykey[] = $key;
        $post_empty++;
      }
  }
  for ($i = 0; $i < $post_empty; $i++)
  {
  echo "<center><font color='#FF0000'>";
  echo 'Du skrev ikke noget i '.$post_emptykey[$i].'!<br><br>';
  echo "</font></center>";
  }
}

og så stopper den med den der warning, hvor den før kom med formularen igen og man kunne udfylde resten. Jeg kan ikke se hvorfor den lander helt nede i bunden hvor jeg satte den header ind.

Jo så er den her:
if ($post_empty == 0) {
  $insertSQL = sprintf("INSERT INTO ...... en masse linier kode

jo ikke 0 og den springer gem rutinen over, men lander så i header linien. Den skal nok flyttes uden for den rutine ?

Nej, det gav samme advarsel at sætte den udenfor.

Nu fandt jeg vist det rigtige sted :-)

Det er simpelthen ren luxus nu :-)

TAk for hjælpen. Hvordan nu med at svare ?

Nej tak, jeg samler slet ikke på point.

Jamen tusind tak for hjælpen.