Kleiner Fehler !

jesenwanger · 13.10.2003, 14:11

Hallo,

im folgenden Script wird der Text:

"Die Kontingente wurde angelegt"

immer angezeigt.

Aber ich möchte dies erst anzeigen lassen. Wenn er das Formular den Script abgeschickt hat und die Daten wirklich eingetragen wurden.

Wer findet ihn bitte !?

Grüße und DANKE

Thomas

<?
//Kontingente eintragen
$sql = "INSERT INTO `Kalpen` ( `ispowinterez` , `ispowinterdz` , `ispowinteropt` , `letzteAenderung` ) ";
$sql .= "VALUES ('".$_POST["ispowinterez"]."','".$_POST["ispowinterdz"]."','";
$sql .= $_POST["ispowinteropt"]."', NOW( ))";
$okay = mysql_query($sql,$db);

if ($okay)
echo "Die Kontingente wurden angelegt";
else
echo "Beim Anlegen ist ein Fehler aufgetreten";
?>
</font><font face="Arial, Helvetica, sans-serif">
<p> </p>
<p> </p>
<p> </p>
<p> </p>
</font>
<p> </p>
<p> </p>
<p> </p>
<p> </p>
<p> </p>
<?
require_once("leer.htm");
?>
</body>
</html>

prefix · 13.10.2003, 14:32

hi!

grundlegend falsch in der syntax ist:

PHP-Code:


			
<?php

if ($okay)

echo "Die Kontingente wurden angelegt";

else

echo "Beim Anlegen ist ein Fehler aufgetreten";

?>

richtig ist (achte mal auf die klammern!):

PHP-Code:


			
<?php

if ($okay) {

  echo "Die Kontingente wurden angelegt";

}

else {

  echo "Beim Anlegen ist ein Fehler aufgetreten";

}

?>

wie gesagt: syntaxfehler!

mike · 13.10.2003, 14:55

Grundlegend falsch von Prefix ist, dass das falsche nicht richtig ist

auch der oben genannte Teil ist syntaktisch richtig, sofern nur ein befehl in der jewieligen Condition ausgeführt wird.

dein Code arbeitet somit korrekt. auch wird die Ausgabe nur dann ausgeführt, wenn der Insert korrekt war.

Ich vermute, dass dir da VORHER irgendwo eine Abfrage abgeht, ob denn nun das Formular angezeigt oder die Query ausgeführt werden muss.
Das macht man ansich mit der Abfrage einer Übergabevariable
if($_POST['doquery'])
{
query abfragen, zueuchs ausgeben
}
else
{
formular ausgeben...
}

prefix · 13.10.2003, 14:59

häää? man darf das ohne klammern bauen, echt?? (naja, hätte ich mir fast denken können, nachdem jesenwanger ja offensichtlich keinen parse-error bekam)...

aber die antwort von dir ist gut, longfang

mike · 13.10.2003, 15:21

so ist es, werter herr Prefix.
Das konstrukt würde sich sogar noch 'vereinfachen' lassen

PHP-Code:


			
echo ($okay)? "Die Kontingente wurden angelegt":

"Beim Anlegen ist ein Fehler aufgetreten";

geht auch in c so c++ auch in asp sowieso.
nur perl nervt da vor sich hin. Das braucht die Klammern unbedingt.

p.s. es ist aber auch nicht falsch Klammern zu setzen. Vermeidet sogar blöde Fehler bei einer nachträglichen Änderung des Codes

prefix · 13.10.2003, 15:23

*michverbeuge-und-weiterlern* - ich weiß, dass ich nichts weiß

thx, jetzt bin ich wirklich wieder schlauer geworden!

jesenwanger · 13.10.2003, 17:51

Hallo,

ich habe es ein bisschen anders gemacht, da ich es wie iben nicht hinbekommen habe.

Siehe:

<?php

class konti {

function konti(){
global $action;
global $search;

if(!$action && !$search):
$this->add($cbrez,$cbrezp,$cbrdz,$cbrdzp,$cbropt,$msg);
$this->such_feld();

echo $this->show('');
elseif($search):
$this->add($cbrez,$cbrezp,$cbrdz,$cbrdzp,$cbropt,$msg);

$this->such_feld();

echo $this->show($search);
elseif($action == add):
global $cbrez;
global $cbrezp;
global $cbrdz;
global $cbrdzp;
global $cbropt;

if($cbrez && $cbrdz):
$query2 = mysql_query("SELECT * FROM Kalpen WHERE chrez = '$cbrez'");// bundesland suchen...
$result2 = mysql_fetch_array($query2);
mysql_query("INSERT INTO `Kalpen` (`cbrez`, `cbrezp`, `cbrdz`, `cbrdzp`, `cbropt`) VALUES ('$cbrez', '$cbrezp', '$cbrdz', '$cbrdzp', '$cbropt')");
echo mysql_error();
?>
<script language="JavaScript" type="text/JavaScript">

</script>
<title>Kontingent</title><body bgcolor="#ccffee" text="#000000" link="#FF0000" vlink="#FF0000">
<p> </p>
<table width="392" border="0" cellpadding="0" cellspacing="0">
<tr>
<td height="40"><div align="center"><strong><font face="Geneva, Arial, Helvetica, sans-serif">Vielen
Dank, </font></strong><font face="Geneva, Arial, Helvetica, sans-serif"><br>
Die Kontingente wurden hinzugefügt!</font></div></td>
</tr>

</table>
<?
else:
$msg = "Es sind nicht alle ausgefüllt!";
$this->add($cbrez,$cbrezp,$cbrdz,$cbrdzp,$cbropt,$msg);
endif;
endif;
}

function splitt_pages($anzahl,$aktuelleseite,$link){
$anzahl++;
for($i = 1; $i<$anzahl; $i++){
if($i == $aktuelleseite):
$output.= "$i ";
else:
$output.= "".$link."seite=".$i."'>$i</a> ";
endif;
}
$output.= "<br>";

return $output;
}
function show($search){
global $seite;
global $l;
$proseite = 20;
if($search):
$SQL = "SELECT * FROM Kalpen";
endif;

$query2 = mysql_query($SQL);
$anzahl = mysql_num_rows($query2);
$anzahl = $anzahl/$proseite;
if(!$seite OR $seite == 0 OR $seite == 1):
$limit = "limit 0,$proseite";
$seite = 1;
else:
$seite2 = $seite;
$seite2--;
$s = $seite2*$proseite;
$limit = "limit $s,$proseite";
endif;

$query = mysql_query("$SQL $limit");
if($search):
$link2go ="<a href='".$PHP_SELF."?name=Reviews&search=$search&";
elseif($l):
$link2go ="<a href='".$PHP_SELF."?name=Reviews&l=$l&";
else:
$link2go ="<a href='".$PHP_SELF."?name=Reviews&";
endif;
$seiten = $this->splitt_pages($anzahl,$seite,$link2go);

while($result = mysql_fetch_array($query)): // gibt alle aus
?>

<?
endwhile;

}
function such_feld(){
?>

<?
}

function add($cbrez,$cbrezp,$cbrdz,$cbrdzp,$cbropt,$msg){
?>
<form name="form1" method="post" action="<?=$PHP_SELF;?>?name=Reviews">

<table width="977" border="1" cellspacing="0" cellpadding="0">
<tr bgcolor="#006600">
<td colspan="9"><font size="3" face="verdana, geneva, arial, sans-serif"><font color="#FFFF00">Neues
Kontingent eintragen:</font><strong>
<input name="action" type="hidden" id="action" value="add">
<input name="name2" type="hidden" id="name2" value="Reviews">
</strong></font></td>
</tr>
<? if($msg): ?>
<tr bgcolor="#000000">
<td colspan="9" valign="top" bgcolor="#FF0000">
<div align="center"><font size="2" face="verdana, geneva, arial, sans-serif"><font color="#FFFFFF"><?=$msg;?></font>
</font></div></td>
</tr>
<? endif; ?>
<tr>
<td align="left" valign="top"><strong><font size="2" face="Arial, Helvetica, sans-serif">Nr.</font></strong></td>
<td align="left" valign="top"><strong><font size="2" face="Arial, Helvetica, sans-serif">Buchungsstichwort</font></strong></td>
<td><strong><font size="2" face="Arial, Helvetica, sans-serif">Messetage</font></strong></td>
<td><strong><font size="2" face="Arial, Helvetica, sans-serif">Kontingent Zeitraum</font></strong></td>
<td><strong><font size="2" face="Arial, Helvetica, sans-serif">EZ-Anzahl</font></strong></td>
<td><strong><font size="2" face="Arial, Helvetica, sans-serif">EZ-Preis</font></strong></td>
<td><strong><font size="2" face="Arial, Helvetica, sans-serif">DZ-Anzahl</font></strong></td>
<td><strong><font size="2" face="Arial, Helvetica, sans-serif">DZ-Preis</font></strong></td>
<td><strong><font size="2" face="Arial, Helvetica, sans-serif">Option 18.00 Uhr</font></strong></td>
</tr>
<tr align="left" valign="middle">
<td width="29"><div align="left"><font size="2" face="Arial, Helvetica, sans-serif">1</font></div></td>
<td width="131"><div align="left"><font size="2" face="Arial, Helvetica, sans-serif">C-B-R</font></div></td>
<td width="96"><div align="left"><font size="2" face="Arial, Helvetica, sans-serif">3</font></div></td>
<td width="164"><div align="left"><font size="2" face="Arial, Helvetica, sans-serif">14.02. - 18.02.2004</font></div></td>
<td width="103"><div align="left"><font size="2" face="Arial, Helvetica, sans-serif">

<input name="cbrez" type="text" id="cbrez" value="<?=$cbrez;?>" size="10" maxlength="6">
  </font></div></td>
<td width="98"><div align="left"><font size="2" face="Arial, Helvetica, sans-serif">
<input name="cbrezp" type="text" id="cbrezp" value="<?=$cbrezp;?>" size="10" maxlength="6">
  </font></div></td>
<td width="109"><div align="left"><font size="2" face="Arial, Helvetica, sans-serif">

<input name="cbrdz" type="text" id="cbrdz" value="<?=$cbrdz;?>" size="10" maxlength="6">
  </font></div></td>
<td width="93"><div align="left"><font size="2" face="Arial, Helvetica, sans-serif">

<input name="cbrdzp" type="text" id="cbrdzp" value="<?=$cbrdzp;?>" size="10" maxlength="6">
  </font></div></td>
<td width="134"><div align="left"><font size="2" face="Arial, Helvetica, sans-serif">

<input name="cbropt" type="text" id="cbropt" value="<?=$cbropt;?>10.02.04" size="14" maxlength="8">
  </font></div></td>
</tr>
<tr align="left" valign="middle">
<td><font size="2" face="Arial, Helvetica, sans-serif">2</font></td>
<td> </td>
<td> </td>
<td> </td>
<td> </td>
<td> </td>
<td> </td>
<td> </td>
<td> </td>
</tr>
</table>
<p>
<input type="submit" name="Submit" value="Abschicken">
</p>
</form>

<?

}

}

?>

<p> </p>
<p> </p>