clumsy code

[kralspace]

I have a survey with textboxes where the user tends to list titles using returns like:

Title 1
Title 2
Title 3

the php I wrote to send the results through email sends it that way also, I wrote a script that addresses each of the 10 textboxes that strips the returns and puts in // instead which lets me import it properly, but is there an easier way to approach this and address them all at one time?
thanks, Kathy

<php
//Purpose: replace carriage returns & line feeds with double-slashes in each comment.

$quest1Com = str_replace("\r","",str_replace("\n"," // ",$quest1Com));

$quest2Com = str_replace("\r","",str_replace("\n"," // ",$quest2Com));

$quest3Com = str_replace("\r","",str_replace("\n"," // ",$quest3Com));

$quest4Com = str_replace("\r","",str_replace("\n"," // ",$quest4Com));

$quest5Com = str_replace("\r","",str_replace("\n"," // ",$quest5Com));

$quest6Com = str_replace("\r","",str_replace("\n"," // ",$quest6Com));

$quest7Com = str_replace("\r","",str_replace("\n"," // ",$quest7Com));

$quest8Com = str_replace("\r","",str_replace("\n"," // ",$quest8Com));

$quest9Com = str_replace("\r","",str_replace("\n"," // ",$quest9Com));

$quest10Com = str_replace("\r","",str_replace("\n"," // ",$quest10Com));

?>

[Nixon]

I have tackled this problem in the past using double variables. I use a while statement and from thier make up the variable in different sectinos for example:

$i=0;

while ($i<11)

{

$i++;

$variable="Quest";
$variable.="$i";
$variable.="Com";

$$variable = str_replace("\r","",str_replace("\n"," // ",$$variable)); 

}

That code should work for your example. Hope this helps!

[kralspace]

Thanks, makes sense now that I see it written out like that, can't wait to try it out,

Kathy

[kralspace]

EUREKA!!!!

just changed the "Q" to "q" and it works like a charm! You've made my day.........thanks,

regards, Kathycangohomenow

[mrhappiness]

why not using

<textarea name ="questcom[]"></textarea>
<textarea name ="questcom[]"></textarea>
<textarea name ="questcom[]"></textarea>

instead of

<textarea name ="quest1com"></textarea>
<textarea name ="quest2com"></textarea>
<textarea name ="quest3com"></textarea>

? now you could do

$count = count($_POST['questcom']);
for ($i=0; $i<$count; $i++)
  $_POST['questcom'][$i] = str_replace("\r","",str_replace("\n"," // ",$_POST['questcom'][$i]));
$all_textboxes = implode("--|--", $_POST['questcom']);

now you should have all values enetered per textbox seperated by // and the values of each textbox seperated by --|-- all together in $all_textboxes

[Nixon]

Glad I could help 🙂

[kralspace]

wow, thanks also, mrhappiness.....

that works too. It's amazing to see the different ways it can be done. Wish I could think more like a programmer and less like an artist.

You guys are great.
Kathy

[BuzzLY]

Actually, a good programmer does think like an artist. Programming can be compared with art in many ways.

[Weedpacket]

Originally posted by BuzzLY
Actually, a good programmer does think like an artist. Programming can be compared with art in many ways.

"I think you'll find the answer to that in Knuth".