What Is Wrong?

[Anon]

Whats Wrong With This Code:

for($i >= 0; $i <= $count; $i++){
$to = "$address[$i]";
mail($to,$subject,$message,$from);}

Here Is What The Message Looks Like When It is Sent:

Subject:
Date: Tue, 5 Sep 2000 15:44:11 -0600 (MDT)
From: Unprivileged user <[e-mail]>
To: [e-mail]

Subject: Testing
From:[e-mail]

Testing The New Newsletter Script.

All the variables are set correctly, what am I doing wrong?

[Anon]

I'm not familiar with the scripting language you are using, but it looks strange to see this:

for($i >= 0; $i <= $count; $i++)

when I would expect it to look something like this:

for($i = 0; $i <= $count; $i++)

[Anon]

The language is called php, hence the name PHPbuilder in the website name :-)

anyway, you were correct about the for statement,
it does need to look like that and not like Owen did it, that was his problem.

[Anon]

Still dont work guys...any other ideas?

[Anon]

What's the code you are using now?

[Anon]

$address = file("$addressfile");
$count = sizeof($address);
for($i = 0; $i <= $count; $i++){
$to = "$address[$i]";
$from = "From:$email";
mail($to,$subject,$message,$from);
$success = "yes";}
if($success = "yes"){
print"<BR><BR><BR><CENTER>Your Message Was Sent Successfully.<BR><BR><BR></CENTER>";
}

[Anon]

Problem:

PHP does not like you to put array statements between quotes.

instead of

$to = "$address[$i]";

try

$to = $address[$i];

The quotes are not required here anyway, because you are not including any normal text.

if you wanted to add normal text to your $to variable you should use the concatenation sign, a dot, to paste the two together like this:

$new_line = "Hello, I am sending this message to ".$address[$i]." and I like it";

[Anon]

I was a little quick on the send button.

You don't have to quote variables if you only want to use the variable itself.

And this:

if($success = "yes")

does not check if $success equals "yes"
Instead, is checks if it was possible to put "yes" into variable $success.

Read the manual about comparison operators.

[Anon]

I just thought I'd let you know that I got it workin by using,
$to = trim($address[$i]);

Now to just get the unsubscribe portion of the script working,
Owen

[Anon]

And this:

if($success = "yes")

does not check if $success equals "yes"
Instead, is checks if it was possible to put "yes" into variable $success.
Read the manual about comparison operators.

Where can I get more information on the "possible to put into" phenom you mentioned?

Can you quote a url? I'm lazy tonight. =)

Michael

[Anon]

It's as very basic thing in many programming languages.

Resultcode of operations.