Insert array of colors into foreach loop

Question

$legend = array();
$color[] = ('000000', 'ff0000', 'fasd3f');
    foreach ($stats as $row) {

        if ($row->countofmsg > 0) {

            array_push($legend, "<div class='legend_label'><div class='color-block' style='background:#".$color.";'></div>".ucwords($row->msg)."<div class='legend_count'>$row->count</div></div><div class='clear'></div>");

        }

    }

Here is my code, what I would like to do is set an array of colors $color[], and then inside the foreach loop array, call the first color in the array and then call the second and third and so forth with each thing the foreach spits out. And then repeat at the beginning of the colors array when it reaches the last color in the array.

Would kick out something like:

(color1) msg - count
(color2) msg - count
(color3) msg - count
etc..

Please let me know if there is a duplicate question, I tried researching for it.

Jeff Lambert · Accepted Answer · 2012-05-09 16:19:53Z

2

This assumes that your $stats array is integer indexed

$legend = array();
$color[] = ('000000', 'ff0000', 'fasd3f');
$colorCount = count($color);
foreach ($stats as $k => $row) {
    if ($row->countofmsg > 0) {
            $legend[] = "<div class='legend_label'><div class='color-block' style='background:#".$color[ ($k % $colorCount) ].";'></div>".ucwords($row->msg)."<div class='legend_count'>$row->count</div></div><div class='clear'></div>";
    }
}

answered May 9, 2012 at 16:19

Jeff Lambert

24.7k4 gold badges72 silver badges98 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Jeff Lambert Over a year ago

@Jamund 'cause its the "best" way :P

Jeff Lambert Over a year ago

@K_G no problem, glad we could help.

klye_g Over a year ago

I was stuck at the I++ area and how to make it repeat.

shadyyx · Accepted Answer · 2012-05-10 14:09:37Z

2

$legend = array();
$color[] = ('000000', 'ff0000', 'fasd3f');
$colorSize = count($color);

foreach ($stats as $row) {
    if ($row->countofmsg > 0) {
        array_push($legend, "<div class='legend_label'><div class='color-block' style='background:#".$color[(3 % $colorSize)].";'></div>".ucwords($row->msg)."<div class='legend_count'>$row->count</div></div><div class='clear'></div>");
    }
}

Edited by @watcher advise but I know the right answer(s) were posted much before.

edited May 10, 2012 at 14:09

answered May 9, 2012 at 16:15

shadyyx

16.1k7 gold badges62 silver badges98 bronze badges

4 Comments

Jeff Lambert Over a year ago

What happens when $i gets to be 3? $color[3] is not defined.

shadyyx Over a year ago

@watcher : I thought that there will be as many colors as many rows is present... That was a bad thought... Sorry for this.

Jeff Lambert Over a year ago

@shaddyyx no worries. you can easily make your code work, just grab the count of the colors array before you enter the loop and instead of referencing $color[$i], reference $color[($i % $colorSize)] instead.

shadyyx Over a year ago

@watcher Edited but I'm not very happy with this one... I'd rather delete it :-)

Community · Accepted Answer · 2017-05-23 11:48:45Z

1

Use modulo as explained here: https://stackoverflow.com/a/7237074/496735

Except in your case do %3 instead of %2

$color = $colors[$i % 3]; // where $i is the current row

edited May 23, 2017 at 11:48

CommunityBot

11 silver badge

answered May 9, 2012 at 16:16

Jamund Ferguson

17k3 gold badges47 silver badges51 bronze badges

Collectives™ on Stack Overflow

Insert array of colors into foreach loop

3 Answers 3

3 Comments

4 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

4 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related