[RESOLVED] parsing question

[gueland]

I have created a script which adds "y" or "n" to a database for over 50 fields, then emails the results to a specified email address.

I was wondering how I could just email the results that have "y" in them, such as:

if $field = y add to email message with description in front then . chr(13) . chr(10) .
else
skip to next field

repeat
Code:

$to = '[e-mail]';
$subject = $required_field . " - " . "Description: $required_field2";
$headers = 'MIME-Version: 1.0' . "\r\n";
$headers .= 'Content-type: text/plain; charset=iso-8859-1' . "\r\n";
$message (4 required fields then the added fields that have y separated by . chr(13) . chr(10) . );

$mail_sent = @mail( $to, $subject, $headers, $message,'[e-mail]');

Can anyone suggest a solution?😕

[gueland]

Will this work?

$message (4 required fields then the added fields that have y separated by . chr(13) . chr(10) . );

if ( $field1 == y {
	$message .=" . chr(13) . chr(10) . chr(13) . chr(13) . 'Field1 Descrip: $field1 ' " 
	}
	else
	$message .=""
	})

Will I then be able to just next state:

if ( $field2 == y {
	$message .=" . chr(13) . chr(10) . chr(13) . chr(13) . 'Field2 Descrip: $field2 ' " 
	}
	else
	$message .=""
	})

To add to the $message field unless it is not y? And keep going with these statements until there are no more fields to add?

[bradgrafelman]

The general idea you've got will work, yes; the code itself won't work since you've got several parse errors in it.

Also if there is nothing to do if the field doesn't equal 'y', then why have an 'else' branch at all.. especially one that contains useless commands, e.g. appending the empty string onto the end of a string?

[gueland]

So.... if my statement is

if ( $field1 == "y" ) {
           $message .=" . chr(13) . chr(10) . chr(13) . chr(13) . 'Field Descrip: $field1'";
   }

Then... just do statements for each field like above, it will continue on until there is no more fields/statements, then my statement at the end

$mail_sent = @mail( $to, $subject, $headers, $message,'[e-mail]');

will do what I want it to do?

[gueland]

is my syntax correct on the first statement? Should it be:

if ( $field1 == "y" ) {
           $message .= . chr(13) . chr(10) . chr(13) . chr(13) . "Field Descrip: $field1";
   }

????

[bradgrafelman]

You've got an extra '.' concatenation operator in there just before the first chr(13). Since there isn't a string on the left hand side of that '.', you're going to get a parse error.

[gueland]

Thanks... this got me on the right track... Here is the code sample:

// Statement for y or n responses

if ( $field == "y" ) {
           $message .= chr(13) . chr(10) . "Descrip: $field";
   }

//Statement for empty comment fields

if ( $field == "" ) {
           $message .= "";
   }
   else {
   	   $message .= chr(13) . chr(10) . "Descrip: $field";
   }