[SOLVED] loop help plz

[gevo12321]

ok i have this code but it doesnt work how can i change it to make it work?

else if (isset($edit))
{
  foreach ($_POST['button'] as $namekey => $namevalue) 
  {
  $name[$namekey] = $namevalue;
  }
  foreach ($_POST['buttonsrc'] as $sourcekey => $sourcevalue)
  {
  $source[$sourcekey] = $sourcevalue;
  }
  foreach ($_POST['id'] as $idkey => $idvalue)
  {
  $id[$idkey] = $idvalue;
  }
mysql_query("UPDATE layout SET 'name'='".$name[$namekey]."', 'source'='".$source[$sourcekey]."'
WHERE id='".$id[$idkey]."'");
}

[Neptunus Maris]

What is the error message?

[gevo12321]

no error the page just comes out blank

 

im sure that the problem is in the way i have set it up because the update needs to loop as well

[teng84]

where the echo or the print there

[Neptunus Maris]

where the echo or the print there

 

Yeah...do you have a success message...like "Data has been updated successfully!"...?

[teng84]

ok i have this code but it doesnt work how can i change it to make it work?

else if (isset($edit))
{
  foreach ($_POST['button'] as $namekey => $namevalue) 
  {
  $name[$namekey] = $namevalue;
  }
  foreach ($_POST['buttonsrc'] as $sourcekey => $sourcevalue)
  {
  $source[$sourcekey] = $sourcevalue;
  }
  foreach ($_POST['id'] as $idkey => $idvalue)
  {
  $id[$idkey] = $idvalue;
  }
mysql_query("UPDATE layout SET 'name'='".$name[$namekey]."', 'source'='".$source[$sourcekey]."'
WHERE id='".$id[$idkey]."'");
}

 

hmm your code is weird pass the variable into the array to be use while others get array content and pass to variables to be used hmm thats totally different 

[gevo12321]

?

 

would it be possible to put the update into a loop or to put all 3 arrays in the loop?

[teng84]

yes

[gevo12321]

how?

[teng84]

1st explain this

foreach ($_POST['button'] as $namekey => $namevalue)

  {

  $name[$namekey] = $namevalue;

  }

  foreach ($_POST['buttonsrc'] as $sourcekey => $sourcevalue)

  {

  $source[$sourcekey] = $sourcevalue;

  }

  foreach ($_POST['id'] as $idkey => $idvalue)

  {

  $id[$idkey] = $idvalue;

  }

[gevo12321]

ok im looping 3 arrays becuase i need all three for the update

[gevo12321]

bump

[tapos]

mysql_query("UPDATE layout SET 'name'='".$name[$namekey]."', 'source'='".$source[$sourcekey]."'
WHERE id='".$id[$idkey]."'");
}

 

You should use only name or `name` instead 'name'.

 

another thing this query will work only for last values of the arrays ($name,$source,$id)

 

--

Tapos Pal

[gevo12321]

well plz help me how would i make it work for all values?

[tapos]

first let me know:

the value and the length of the all three arrays.

 

--

Tapos Pal

[gevo12321]

well the arrays are defined as:

  foreach ($_POST['button'] as $namekey => $namevalue) 
  {
  $name[$namekey] = $namevalue;
  }
  foreach ($_POST['buttonsrc'] as $sourcekey => $sourcevalue)
  {
  $source[$sourcekey] = $sourcevalue;
  }
  foreach ($_POST['id'] as $idkey => $idvalue)
  {
  $id[$idkey] = $idvalue;
  }

i dont know if that was what u meant by value

 

as for the length, the number of elements in each array is variable

i dont know if thats what u meant by length

 

thx

[tapos]

You can try this:

$i = 0;
$info = array();
foreach ($_POST['button'] as $namekey => $namevalue) 
  {
   $info[$i++]['name'] = $namevalue;
  
  }
  $i=0;
  foreach ($_POST['buttonsrc'] as $sourcekey => $sourcevalue)
  {
  $info[$i++]['source'] = $sourcevalue;
  
  }
  $i=0;
  foreach ($_POST['id'] as $idkey => $idvalue)
  {
   $info[$i++]['id'] = $idvalue;
  
  }
//queries
for($i =0;$i<count($info);$i++)
{
mysql_query("UPDATE layout SET name='".$info[$i]['name']."', source='".$info[$i]['source']."'
WHERE id='".$info[$i]['id']."'");
}

--

Tapos Pal

[gevo12321]

thank you it worked

but can u please tell me what u did in english and not in php because im fairly new to this and i didnt quite understand what the code says.

 

thx a lot

[tapos]

First, u should see the foreach array.

 

2nd, u have three arrays of three fields of a table. then u should make the query properly.

 

3rd, u should handle arrays properly.

 

Thanks,

 

--

Tapos Pal