Please Help Me With This

[intenseone345]

Hello, when i tested this php script below i got the following error message,

 

ERROR SHOWN ON BROWSER:

------------------------

Post message

Warning: Cannot modify header information - headers already sent by

(output started at /hermes/bosweb/web176/b1761/ipw.tomagr/public_html/maila.php:23)

in /hermes/bosweb/web176/b1761/ipw.tomagr/public_html/maila.php on line 39

------------------------

 

The code is in its own page just as its shown below, simple word check, then post the comments

to a flat file and emails me with posted comment.

the, $post = $_POST['comments']; reffers to the form on another page that the

comments are comming from. I want to know what im doing wrong in this script?

Any ideas i'd love to hear, thanks

 

the php script :

 

<?

$post = $_POST['comments'];

$words = array('murmer', 'frog', 'bat', );

 

$continue = true;

 

foreach ($words as $word) {

 

if (preg_match('/\b' . $word . '\b/i', $post)) {

$continue = false;

header("Location: mysite.html");

exit();

 

}

 

}

 

if (!$continue) {

echo 'Bad boy!';

}

 

else {

echo 'Post message';

}

$fc = fopen("comments.txt","a+b"); //opens the file to append new comment -

fputs($fc,$_POST['comments']."\n\n\nNewComment->"); //writes the comments followed by a

fclose($fc); //closes the files

 

if(sizeof($_POST)) {

$body = "";

while(list($key, $val) = each($HTTP_POST_VARS)) {

$body .= "$key: $val \n";

}

 

mail("[email protected]", // to

"Subject Line",

$body);

 

header("Location: mysite.html");

}

 

// end form processing

?>

[cags]

Have you read the sticky? Oh and 

[gevensen]

add ob_start(); as the first line of code and ob_end_flush(); as the last

[micah1701]

as the above said, use ob_start()

 

your code is working fine, but it would seem some thing else is being output to the browser before the header("Location: "); call.  Nothing can be sent to the browser before a header();

[teynon]

To answer your question even if you didn't read the sticky. You are outputting data then trying to redirect the browser using "header("location:blah");". But since you already echo'd something, headers were already sent. Either don't echo and redirect or put output buffering on. Personally, I don't see why you would ever echo something before you send a header anyways. (It doesn't work, and they wouldn't see it anyways.)

[GoneNowBye]

What does the foreach thing do? and how does it work?

i've never quite got it and this seems a good thread to ask

[teynon]

That's actually not relevant to the thread at all, but foreach cycles through every element in an array.

 

array(1,2,3);

 

foreach would get the values one at a time from the array.

[GoneNowBye]

how does that translate to named arrays?

[teynon]

http://uk.php.net/manual/en/control-structures.foreach.php PHP.net has a very useful manual.

[GoneNowBye]

thanks -.-