function blabla()

[AndyR]

Hi leute hab mal wieder ein Prob. Ich habe in einer einer function, function schreiben(), eine mysql anweisung. Die mysql anweisung schreibt Daten in meine DB die über function schreiben() übergeben werden. Aber irgendwie klappt das nicht so wie ich will. Ich hab mir Tutorials angeschaut und alles probiert doch ohne erfolg. Hoffe ihr könnt mir helfen.

write_news.php:

if (isset($_POST['senden']))
{
 $schreib=schreiben($_POST['text'], $_POST['subject'], $_POST['name'], $_POST['email']);
 if($schreib==true)
  {
   echo"<center>News erfolgreich gespeichert!</center>";
  }
  else
  {
   echo"<center>Speichern von News fehlgeschlagen!</center>";
  }
}
else 

..........
und hier ein Auszug aus der functions.inc.php

function schreiben($text, $header, $name, $email)
{
  global $time;
  global $datum;
  global $day;
  $insert="INSERT INTO $db_table1 (newstext, newsheader, time, date, tag, name, email) VALUES 
  ('$text', '$header', '$time', '$datum', '$day', '$name', '$email')";
  mysql_query($insert);
} 

Hoffe ihr könnt mir einen Typ geben.
Mfg Andy

[Abraxax]

änder das mal ab

// alt
mysql_query($insert);


// neu
$res = mysql_query($insert) or die(mysql_error()); 

ausgabe?

[AndyR]

You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near '(newstext, newsheader, time, date, tag, name, email) VALUES

Das ist die Ausgabe. Wenn ich die mysql Anweisung direkt in mein Script einbaue klappts prächtig. nur über die function wills nicht.

[MelloPie]

ja is ja auch klar in der Funktion ist $db_table1 nicht bekannt.

[AndyR]

Stimmt...hab ich glatt übersehen. leider gehts immer noch nicht!!!
Ich dreh hohl...

[AndyR]

Ok Problem gelöst....geht doch obwohl er sagt "Speichern von News fehlgeschlagen!" Muss ich da wohl was verändern.
Danke für die Hilfe.
Mfg Andy

[MelloPie]

wie wärs auch mit nem return wert?
Du solltest mal Deine Nase in ein Buch stecken...

[Abraxax]

... oder bei www.schattenbaum.net/php vorbei schauen....

[AndyR]

ja durch Schattenbaum hab ich ja erst angefangen zu programieren. Leider steht da nichts von functions. Für ein buch hab ich leider kein Geld und wenn ihr mich auf solche fehler hinweist ist das kein beinbruch für euch und auch keiner für mich.

Mfg

[AndyR]

Hab return() eingefügt unds klappt jetzt super. danke euch
Mfg

[Abraxax]

du solltest schon in der funktion unterscheiden, ob es korrekt oder nicht korrekt funktioniert hat und entsprechend true oder false zurückgeben. anderenfall kann dein aufrufendes script auch nicht erkennen, ob da ein fehler war.