[SOLVED] Foreach question

[tqla]

Hello, I am using this code to print out an array and it works:

 

foreach ($leads as $lead){
	echo $lead;
	}

 

I get a list of the $leads array. It looks like this "RedBlueYellowOrange"

 

Then  I attempt to place that array into my DB like this:

 

$today = date("Y-m-d");
$date=time();   
$sql = "INSERT INTO DB 

(createDate,
leads
)

VALUES

('$today',
'$lead',
)";

 

The problem is that only the last item in the array is placed in the database (Orange).

 

How do I get them all to go into the DB? What am I doing wrong?

 

 

[cooldude832]

cause I don't see you querying in the loop

<?php
foreach($lead as $lead){
$q = "Insert into `DB` (createDate,leads) VALUES('".time()."','".$lead."')";
$r = mysql_query($q) or die(mysql_error()."<br /><br />".$q);
}
?>

 

try that

[tqla]

Actually I am querying it. I just left it out in my post. I am using

 

mysql_query($sql) or die(mysql_error());

 

The problem is the only the last array item is going into the database.

 

Just Orange.

 

I would think that RedBlueYellowOrange would populate the DB.

 

I don't get it.

[cooldude832]

do what I gave you, echoing it won't make it appear in the query and if your only getting it to go once its pretty obvious its happen after the foreach loop

[tqla]

Ah ha. Thank you. I see. Okay, I did it but now instead of the 1 DB record showing "RedBlueYellowOrange", four records were created! One for Red, one for Blue, one for Yellow, and one for Orange.    I want 1 record to be created and have "RedBlueYellowOrange" in the "leads" field.  ???

[cooldude832]

the better way then is to do this

<?php
$lead = implode("",$leads);
$q = "Insert into `DB` (createDate,leads) VALUES('".time()."','".$lead."')";
$r = mysql_query($q) or die(mysql_error()."<br /><br />".$q);
?>

 

Don't see why you would ever want a single field like this want to explain what this is for?

[tqla]

Implode! I was thinking about that but had no idea how do properly use it.

 

Don't see why you would ever want a single field like this want to explain what this is for?

 

lol, actually this is just an excerpt of the real code. I extracted it so that I can find help here.

 

Thanks cooldude832, I'll try this now.

[Barand]

@tqla

 

see

http://dev.mysql.com/tech-resources/articles/intro-to-normalization.html

[tqla]

 

Thanks cooldude832! Implode works perfectly.

 

Barand, thank you too, I've put this on my study list.

[cooldude832]

 

Thanks cooldude832! Implode works perfectly.

 

Barand, thank you too, I've put this on my study list.

 

It shouldn't be your "study" list it should be on your "OMG if I don't implement this correct now I will have a ton more work to do later" list.

[Barand]

OMG, we agree

[cooldude832]

i am just as surprised as you are