displaying results of array not functioning as expected…

Bilbozilla

Hi there,

Here's the problem - I've got a series of tables (table1, table2, table3, table4) that store data entered for 4 independent data sets. To possible make things easier, imagine table1-table4 each being the vehicle models for 4 separate vehicle manufacturers.

Now, my problem arises when I go through this loop and output the data using the implode function. If it pulled data from table1 and table2, the data from table1 might, on its own, appear as

535i, 550i, M3, Z8

and the table2 data might be

911S, 911 Turbo, GT3, Boxster S

. However, when using the implode function as I have done, they are combined and appear as

535i, 550i, M3, Z8911S, 911 Turbo, GT3, Boxster S

Notice how there is no comma between Z8 and 911S. How do I add fix this?

foreach (array(1, 2, 3, 4)as $counter_value) {
if($row['id']==$counter_value){
$end_digit=$row['id'];
$_name="table".$end_digit."[]";
$table_name = "table";
$table_name .= $end_digit;
$sql="………
……
"
$result = mysqli_query($connection, $sql) or die(mysql_error());
while($row = mysqli_fetch_array($result_spec,MYSQLI_BOTH))
{
$arrayspec[] = $row_spec['name']; 
}
echo implode(", ",$arrayspec);
}
unset($row_spec, $arrayspec);
}

Weedpacket

Have an array of result sets. Imploding each element with ", " gives you an array of comma-and-space-separated strings. Imploding that with ", " gives you a single string.

Bilbozilla

Weedpacket;11001737 wrote:
Have an array of result sets. Imploding each element with ", " gives you an array of comma-and-space-separated strings. Imploding that with ", " gives you a single string.

Where would I set that up? I see what you'd be doing in theory, but in practice, how would you define it?

Weedpacket

It's a bit hard for me to tell based on what you've got. (Are the names of the tables really "something1", "something2", "something3" and "something4"? Doesn't sound very "independent" to me...)

An alternative of course is to just use one query that is an SQL UNION of the others.

Bilbozilla

Weedpacket;11001739 wrote:
It's a bit hard for me to tell based on what you've got. (Are the names of the tables really "something1", "something2", "something3" and "something4"? Doesn't sound very "independent" to me...)

An alternative of course is to just use one query that is an SQL UNION of the others.

"Something1" contains data unique and specific to "client1", while "something2" is unique and specific to "client2", etc. There is no overlap, but the looping query pulls 0 or multiple data points from 1 or several "somethingX" tables.

Back to your first response, how would I write the code such that each imploded list of arrays with commas is then placed into an array which is then further imploded to include the ", " between the multiple lists to avoid the issue I had in my original statement?

And I'm looking for the union of the values, not the keys, so short of rewriting code, I don't believe the Union will work.

Bilbozilla

I tried this, but got the same results.

foreach (array(1, 2, 3, 4)as $counter_value) { 
if($row['id']==$counter_value){ 
$end_digit=$row['id']; 
$_name="table".$end_digit."[]"; 
$table_name = "table"; 
$table_name .= $end_digit; 
$sql="……… 
…… 
" 
$result_spec = mysqli_query($connection, $sql) or die(mysql_error()); 


while($row_spec = mysqli_fetch_array($result_spec,MYSQLI_BOTH))
{
$arrayspec[] = $row_spec['name']; 
}
$arrayspec2=array();  // I define the new array to contain the array of imploded strings
$arrayspec2[]= implode(", ",$arrayspec); // Is this the correct syntax?
echo implode(", ",$arrayspec2); // This outputs the strings, but misses the ", " between them.
}

bradgrafelman

Bilbozilla;11001753 wrote:
"Something1" contains data unique and specific to "client1", while "something2" is unique and specific to "client2", etc.

Are the tables otherwise identical, e.g. they all share the same structure/schema?

If so, it seems you should instead have a single table with an additional column that identifies which 'client' a given row is associated with.

Bilbozilla

bradgrafelman;11001765 wrote:
Are the tables otherwise identical, e.g. they all share the same structure/schema?

If so, it seems you should instead have a single table with an additional column that identifies which 'client' a given row is associated with.

So then, for my own edification, with the current setup, how would I create the array so that the first time the implode function is called, the results are entered into a new array which, after looping, is imploded and displayed?

Weedpacket

$array_of_arrays = array(array('1', '2', '3', '4'), array('a', 'b', 'c', 'd'));

$array = array_map(function($inner_array) {
    return join(",", $inner_array);
}, $array_of_arrays);
//...or use a foreach loop to iterate over the elements of $array_of_arrays, but that seems messier

$string = join(',', $array);

var_dump($array_of_arrays);
var_dump($array);
var_dump($string);