bitte um hilfe bei der if logik

loco12

hi moin ich komme immer und immer und immer nicht mit der if logik klar ich kann es mir sogar aufmalen und dennnoch ist es immer falsch

einfache geschichte(anscheind nicht für mich) alle felder meines form müssen foll sein frage muss richtig sein und checkbox muss sein dann los...

habs jetzt so habs auch schon ganz anders und hoch und runter und drüber aber immer nur syntax error, unexpected '}'

PHP

$hostname = gethostbyaddr($_SERVER['REMOTE_ADDR']);
$ip=getenv("REMOTE_ADDR");
$frage = $_POST['frage'];
$antwort = "2";
$user = $_POST['Name'];
$user = htmlentities($user);
$inhalt = $_POST['inhalt'];
$inhalt = htmlentities($inhalt);
$inhalt = str_replace("\n", "<br>", $inhalt);
$email = $_POST['EMail'];
$email = htmlentities($email);
$accept = $_POST['accept'];
 
if ($inhalt == "" or $user == "" or $email== "" or $accept =="" or $frage =="")
   {
   echo "Sie müssen das Feld \"Namen\"
   und \"Inhalt\" ausfüllen und Sie müssen die Regeln lesen und akzepieren!";

      if ($frage == $antwort)
 {
   $datum= date("d.m.Y H:i:s");
   $eintrag="$email|$user|$datum|$inhalt|$ip|$hostname";
   $datei = fopen("gaestebuch.txt", "a");
   fwrite($datei, "\n".$eintrag);
   fclose($datei);
   echo "Ihr Eintrag wurde erfolgreich gespeichert";
   }
   }
   else{
   echo "antwort falsch"
   }

Alles anzeigen

Tobse

Die Zeile wäre interresant. Aber meine Vermuting ist, dass du nicht die Logik falsch begreiffst, sondern die Fehlermeldung und den Syntax noch nicht draufhast. Bei dem code hier nach echo "Ihr Eintrag wurde erfolgreich gespeichert" ist ein } zuviel.

EDIT: Ach, und gewöhn dir bitte an, deinen Code sauber einzurücken. Das wirst du dir später danken, wenn du dann mal Dateien mit 500+ Zeilen hast. Und dann hättest du das hier auch auf anhieb gesehen.

PHP

if (...) {
    // code
    if (...) {
        // code
    }
} else {
    // code
}
function bla() {
    // code
}

Alles anzeigen

Immer einen tabulator oder 4 leerzeichen.

Bandit

Zitat von Tobse

Bei dem code hier nach echo "Ihr Eintrag wurde erfolgreich gespeichert" ist ein } zuviel.

Nö, ordentlich eingerückt sieht das so aus:

PHP

<?php
$hostname = gethostbyaddr($_SERVER['REMOTE_ADDR']);
$ip=getenv("REMOTE_ADDR");
$frage = $_POST['frage'];
$antwort = "2";
$user = $_POST['Name'];
$user = htmlentities($user);
$inhalt = $_POST['inhalt'];
$inhalt = htmlentities($inhalt);
$inhalt = str_replace("\n", "<br>", $inhalt);
$email = $_POST['EMail'];
$email = htmlentities($email);
$accept = $_POST['accept'];
 
if ($inhalt == "" or $user == "" or $email== "" or $accept =="" or $frage =="")
{
   echo "Sie müssen das Feld \"Namen\" und \"Inhalt\" ausfüllen und Sie müssen die Regeln lesen und akzepieren!";

   if ($frage == $antwort)
   {
      $datum= date("d.m.Y H:i:s");
      $eintrag="$email|$user|$datum|$inhalt|$ip|$hostname";
      $datei = fopen("gaestebuch.txt", "a");
      fwrite($datei, "\n".$eintrag);
      fclose($datei);
      echo "Ihr Eintrag wurde erfolgreich gespeichert";
   }
}
else
{
   echo "antwort falsch"
}  
?>

Alles anzeigen

Bei dem Quellcode kommt kein Parse-Error, weil hinter dem letzten echo ein Semikolon fehlt

loco12

danke digger des geht rückwärts und rückwäts alter DICKES DANKE ich depp.... JUHUUUU

loco12

Torbens danke auch dir für die überaus schnelle antwort. bin halt noch anfänger!!!! aber jetzt gehts halt noch weiter mit dem ganzen if jetzt muss da noch hinter wenn neuer eintrag dann mail "neuer eintrag"

loco12

oh man jetzt bekomme ich immer immer antwort falsch ich mag net mehr ifen....

PHP

<?php
$hostname = gethostbyaddr($_SERVER['REMOTE_ADDR']);
$ip=getenv("REMOTE_ADDR");
$frage = $_POST['frage'];
$antwort = "2";
$user = $_POST['Name'];
$user = htmlentities($user);
$inhalt = $_POST['inhalt'];
$inhalt = htmlentities($inhalt);
$inhalt = str_replace("\n", "<br>", $inhalt);
$email = $_POST['EMail'];
$email = htmlentities($email);
$accept = $_POST['accept'];
$betreff = "Neuer eintrag im Gaestebuch";
$from = "From: test.test.tets<info@test.com>";
$text = $inhalt;
$empfaenger = "test@test.com";
 
if ($inhalt == "" or $user == "" or $email== "" or $accept =="" or $frage =="")
{
   echo "Sie müssen das Feld \"Namen\" und \"Inhalt\" ausfüllen und Sie müssen die Regeln lesen und akzepieren!";

   if ($frage=$antwort)
   {
      $datum= date("d.m.Y H:i:s");
      $eintrag="$email|$user|$datum|$inhalt|$ip|$hostname";
      $datei = fopen("gaestebuch.txt", "a");
      fwrite($datei, "\n".$eintrag);
      fclose($datei);
      mail($empfaenger, $betreff, $text, $from);
      echo "Ihr Eintrag wurde erfolgreich gespeichert";

 
   }
}
else
{
   echo "antwort falsch";
}  
?>

Alles anzeigen

Tobse

Schau nochmal in dein Tutorial/Buch/Anleitung und les den Teil mit den If-Abragen nochmals genau durch.

Zunächst überprüfts du, ob alle Felder asugefüllt wurden:

PHP

if ($inhalt==""...

Und danach kommt code für erfolgreiches eintragen!? Das KANN nicht funktionieren. Denn wenn du dann etwas einträgst ist die erste Bedingung ja false und der else block wird ausgeführt.

Ich seh gerade nochwas:

PHP

if ($a=$b) {

ist IMMER true bzw. $b
richtig:

PHP

if ($a==$b) {
// bw in deinem fall
if ($frage==$antwort) {

loco12

bitte nicht ich hab schon genug grundlagen intuss beck´s warsteiner und gilde. nein scherz bei seite habs überprüft bei $frage wird nur die antwort übertragen. allso in meinem fall "2" bitte hier der code

PHP

<form action="neu.php" Method="post">
Ihr Name:&nbsp;<input name="Name" size="40"><br>
E-Mail:&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;<input name="EMail" size="40"><br>


Inhalt:<br>
<textarea name="inhalt" cols="40" rows="12"
wrap="physical"></textarea><br>
<br>
10 Minus 8 =<input name="frage"><br>
<br>
<input type="submit" value="Absenden"> <input type="checkbox" name="accept" value="accept"> 
Mit den <a href="gbr.html" target="_blank">Regeln</a>  einverstanden!</form>

Alles anzeigen

loco12

ja sorry ich sag ja ich komme immer wider mit der logik durcheinander oder es haut einfach nicht hin das ist zu meinem bedauern leider nicht das erste mal deswegen hab ich es mir schon wirklich mit zettel und bleistift augeschrieben doch es hat nie hingehauen auser bei den standarts

PHP

if (anweisung){
echo test
}else{
echo test2
}

nein
ich will halt nur abfragen o
b alle felder ausgefüllt sind
nein bitte alle felder
dann ob die frage richtig ist
nein frgae richtig
dann ausführen

aber wenn dannwenn dann das kann ich nicht zumal hab ich gelehrnt in php ist kein nicht sonder nur "==" IST HALT AUCH LEICHTER! mann muss nur die logik verstehen das kann ich halt nicht ach mann allso doch wider bis zum bitteren ende probieren und nix draus lehrnen
mist

Tobse

kk, dann versuchs nochmal. Schreib auf, was passiert (das macht man bei Komplizierteren sachen auf hin und wieder so, weils manchmal wirklich nötig ist)

Code

Sind alle Felder ausgefüllt?
Ja:
    Ist die Antwort richtig?
    Ja:
        Eintrag machen, E-Mail schicken und meldung ausgeben
    Sonst:
        "Antwort Falsch" ausgeben
Sonst:
    "Alle felder müssen ausgeüllt sein" ausgeben

loco12

ich bokomme es nicht hin habs jetzt nach dem schema gemacht. jetzt bekomme ich halt nur sie müssen alle felder ausfüllen als antwort......

PHP

if ($inhalt == "" or $user == "" or $email== "" or $accept =="" or $frage =="")
{


   if ($frage=="2")
   {
      $datum= date("d.m.Y H:i:s");
      $eintrag="$email|$user|$datum|$inhalt|$ip|$hostname";
      $datei = fopen("gaestebuch.txt", "a");
      fwrite($datei, "\n".$eintrag);
      fclose($datei);
      mail($empfaenger, $betreff, $text, $from);
      echo "Ihr Eintrag wurde erfolgreich gespeichert";
 }else{
   echo "antwort falsch";}
   }else{
   echo "Sie müssen alle Felder ausfüllen und Sie müssen die Regeln lesen und akzepieren!";
}  
?>

Alles anzeigen

alxy

Das heißt eine der variablem aus der ersten reihe hat keinen wert bzw den wert ""

Tobse

Nein, hast du nicht. Schau dir nochmal diese If-Bedingung an:

PHP

if ($inhalt=="" OR $user=="" OR $email=="" OR $accept=="" OR $frage=="")

Auf Duetsch heisst das doch

Code

Wenn
     Der Inhalt leer ist
oder Der Benutzername leer ist
oder Die Checkbox nicht gehakt wurde
oder Die Antwort leer ist
Dann prüfe die antwort und sende die mail.

Was du warscheinlch ausdrücken wolltest:

Code

Wenn
    alle Felder asugefüllt sind
Dann prüfe die antwort und sende die Mail

Aber mit dem Satz der im Code steht bewirkst du doch was gaaanz anderes.

synaptic

nur um dir das mal zu vereinfachen..
ich hab bei solchen dingern meist nen fehler-array, leer initialisiert.
ich prüfe die felder in einer foreach auf die abgesendeten daten und wenn einer der werte nich passt, dann schreibe ich den schlüsselwert in das fehler-array.
am ende lese ich das fehler-array aus und wenns keinen inhalt hat mache ich weiter.

so hab ich eine bzw 2 schleifen, statt ne verkettung von if-abfragen.
und so haste dann dein formular auch für ne anständige validierung vorbereitet.

kannst ne funktion schreiben, die ne error-klasse setzt oder wat weiß ich

und wenn du das so vorhast, wie oben, wirste nie erfahren, wo der fehlende wert war.

abgesehen davon fragst du ja wie tobse schon sagt nach ODER und willst aber nen UND

wenn feld1 nicht leer UND feld2 nicht leer UND feld 3 nicht leer etc pp

loco12

jo betsens leute ihr seit große klasse vielen vielen dank für die hilfe und die ratschläge. ich habs jetzt so gelöst und es funzt einwandfrei... ja du hast rechts mit den arrys aber so gehts halt auch bei kleineren sachen. Und lehrne immer was draus...

"torbens" vielen dank für die überaus gute und freundliche und kontinuierliche unterstützung

PHP

if ($inhalt != "" or $user != "" or $email != "" or $accept !="" or $frage !="")
{


   if ($frage=="2")
   {
      $datum= date("d.m.Y H:i:s");
      $eintrag="$email|$user|$datum|$inhalt|$ip|$hostname";
      $datei = fopen("gaestebuch.txt", "a");
      fwrite($datei, "\n".$eintrag);
      fclose($datei);
      mail($empfaenger, $betreff, $text, $from);
      echo "Ihr Eintrag wurde erfolgreich gespeichert";
 }else{
   echo "antwort falsch";}
   }else{
   echo "Sie müssen alle Felder ausfüllen und Sie müssen die Regeln lesen und akzepieren!";
}

Alles anzeigen

jetzt möchte ich noch das der eintrag erst durch freischaltung ausgegeben wird. wie ich das löse weis ich schon ich mach einfach ein 0 und 1 dazu und fehlt mir der denkanstoss für die überprüfung und freischaltung ohne die datei manuel zu öffnen und von hand zu bearbeiten.... hat jemand vieleicht ein kleinen ultra heissen tip?

Bandit

Zitat von loco12

es funzt einwandfrei

Garantiert nicht! Denn mit

PHP

if ($inhalt != "" or $user != "" or $email != "" or $accept !="" or $frage !="")

muss nur ein Feld ausgefüllt sein, und das willst du doch nicht.

Zitat von loco12

hat jemand vieleicht ein kleinen ultra heissen tip?

Wie du schon sagtest, hänge eine 0 beim speichern an den Datensatz. Dann schreibst du ein Script, dass dir die Daten in einem Formular anzeigt, in dem die 0 bzw. 1 als Checkbox symbolisiert wird. Dann kannst du Beiträge sperren oder freischalten. Nach einem Klick auf Speichern muss du dann die Checkboxen überprüfen und die Datei entsprechend neu schreiben.

Das ganze wäre mit einer Datenbank deutlich angenehmer.

loco12

moinsen jo hast recht war nur die freude die hats mich übersehen lassen anstatt or sollte doch && funktionieren oder?
ja datenbank hab ich schon gelesen aber noch nix mit gemacht ist ja auch wieder ne stange imput die datenbank funtionen...

ja danke ist ne gute idee mit der checkbox.

probiere es jetzt mit der checkbox leider klapt es noch nicht so wirklich

PHP

<?php
$beitrag = file("gaestebuch.txt");
krsort($beitrag);
foreach($beitrag as $ausgabe)
   {
   $ausgabe = stripslashes($ausgabe);
   $zerlegen = explode("|", $ausgabe);
    if($zerlegen[6]==1)  $freigabe="checked" ;
   echo "
   <table align=\"center\"
 border=\"1\" cellspacing=\"0\"
   cellpadding=\"5\" bordercolorlight=\"black\"
   bordercolordark=\"black\" width=\"100%\">

   <tr>
   <td>
   Freigeben:  
<form action=\"gbg.php\" Method=\"post\"><input type=\"checkbox\" name=\"freigeben\" value=\"1\" checked=\"$freigabe\"> <input type=\"submit\" value=\"Absenden\"></form>
   </td>
   </tr>
   </table><br>
   ";

   }

Alles anzeigen

Bandit

Meinst du nicht, dass du ein paar Formulare zuviel einbaust?

Ungetestet:

PHP

<?php
   $data = file("gaestebuch.txt");
   echo "<form action='gbg.php' Method='post'>"
         <table align='center' border='1' cellspacing='0' cellpadding='5' bordercolorlight='black' bordercolordark='black' width='100%'>

   $i = 0;

   foreach($data as $line)
   {     
      $values = explode("|", $line);

      $checked = $zerlegen[6] ? "checked='checked'" : "";

      echo "<tr>
              <td>$values[0]</td>
              <td>$values[1]</td>
              <td>$values[2]</td>
              <td>$values[3]</td>
              <td>$values[4]</td>
              <td>$values[5]</td>
              <td>Freigabe <input type='checkbox' name='freigabe[$i]' $checked></td>
            </tr>\n";

     $i++;
   }

   echo "<tr>
        <td colspan='7'><input type='submit' name='submit' value='Speichern'></td>
         </table>
         </form>";
?>

Alles anzeigen

PHP

<?php
  $data = file("gaestebuch.txt");

  $count = count($data);

  for ($i = 0; $i < $count; $i++)
  {
     $line = substr($data[$i], 0 , -1);

     if (isset($_POST['freigabe' . $i]))
       $line .= "1";
     else
       $line .= "0";

     $data[$i] = $line;
  }

  file_put_contents("gaestebuch.txt", implode("\n", $data));
?>

Alles anzeigen

loco12

ja hast recht sind halt immer so viele forms wie beiträge aber es war halt das erste was mir eingefallen ist. (simpel)
jo cool danke für das beispiel das funzt leider nicht.
Parse error: syntax error, unexpected T_STRING, expecting ',' or ';' in line 4
ich muss doch das <table> auch mit ins echo setzen? allso halt alle html in ein echo"" oder bin ich da auf dem holzweg?

Bandit

Wenn du jeden Parse-Error in deiner PHP-Karriere hier posten willst, dann wirst du hier Dauergast! Sowas darf man auch mal alleine finden!!

PHP

echo "<form action='gbg.php' Method='post'>
      <table align='center' border='1' cellspacing='0' cellpadding='5' bordercolorlight='black' bordercolordark='black' width='100%'>";

Tags