jpgraph accbarplot issue - only 1 plot displayed

duanep

Ok, I'm baffled. Isn't:

$n = 0;

foreach($gens as $gen)
{
  $plot = new BarPlot($uze[$gen]);
  $plot->SetFillColor=$colors[$n];
  $abp[]= $plot;
  $n++;
}
$abp1plot  = new AccBarPlot ($abp); 
 $graph->Add($abp1plot);

equivalent to:

// Create the bar plots
$b1plot = new BarPlot($data1y);
$b1plot->SetFillColor("orange");
$b2plot = new BarPlot($data2y);
$b2plot->SetFillColor("blue");

// Create the grouped bar plot
$gbplot = new AccBarPlot(array($b1plot,$b2plot));

// ...and add it to the graPH
$graph->Add($gbplot);

If so, why does my graph only display 1 series? I have verified the content of each array (yes, matching quantity of data points and datapoint values >0), and of the $abp array as containing a plot (print_r shows the content of the plot, including valid data point values). I get one set of bars instead of 12. $colors[] has a list of hex color values, and $gen[] has a list of the names of the series. What am I missing?

Oh yeah, I'm doing it this way because the number of series can (and does) change so I can't hard-code the arrays.

I've tested a simpler version of this with a couple of small arrays and it worked so I know the installation is good. There is something about this particular code that is causing an issue but I can't seem to connect the dots.

Thanks,

Duane

bradgrafelman

duanep wrote:
Oh yeah, I'm doing it this way because the number of series can (and does) change so I can't hard-code the arrays.

Why should this matter? Arrays are dynamic - there's no need to "hard-code" anything related to their length. Can you show us how you get the data?

Whenever you start seeing variables such as $foo1, $foo2, etc. then it's almost always a case where arrays would be a better (and easier) choice.

As for the difference, no, at first glance the two code snippets don't appear to be vastly different, assuming you have indeed confirmed that all of the data is populated correctly.

EDIT: Wait, are you saying that you would rather have the first code snippet (using arrays), or would rather have the second (using multiple variables)? I think I might've been a wee bit confused...