PHP function not returning a value from multidimensional array

Question

I have been working on a color palette system where a multidimensional array holds different color palettes. EX:

$palette['1']['borderColor'] = "black";
$palette['2']['borderColor'] = "white";

I have a function that should return a color palette item when called but for some reason it's not returning anything. Instead it says "Undefined variable: palette"EX:

$palette['1'] = array(
    "borderColor"=> "#222222",
    "divColor1"=> "#00f2ff",
    "divColor2"=> "#222222",
    "headerColor"=> "rgba(38, 38, 38, 0.5)",
    "color1"=> "#00f2ff",
    "color2"=> "#e9ffdd"
  );

function getItem($number, $item) {
  return $palette[$number][$item];
}

$borderColor = getItem('1', 'borderColor');
echo "borderColor is> " . $borderColor . " <.";

//returns "borderColor is>  <."

Any suggestions?

You need to add global $palette; to your function in order to have access to it. — Greg Klaus
– Greg Klaus, Commented Apr 15, 2018 at 2:11

Joseph_J · Accepted Answer · 2018-04-15 02:28:56Z

3

The function you wrote does not have the actual palette value.. Just pass the palette you want into the function.

$palette['1'] = array(
    "borderColor"=> "#222222",
    "divColor1"=> "#00f2ff",
    "divColor2"=> "#222222",
    "headerColor"=> "rgba(38, 38, 38, 0.5)",
    "color1"=> "#00f2ff",
    "color2"=> "#e9ffdd"
  );

function getItem($palette, $number, $item) {
  return $palette[$number][$item];
}

$borderColor = getItem($palette,'1', 'borderColor');
echo "borderColor is> " . $borderColor . " <.";

Another way to do this would be to make a simple class in which you could call your palettes.

class Palette {

  public function __construct($palette){

  $this->borderColor  = $palette['borderColor'];
  $this->divColor1    = $palette['divColor1'];
  $this->divColor2    = $palette['divColor2'];
  $this->headerColor  = $palette['headerColor'];
  $this->color1       = $palette['color1'];
  $this->color2       = $palette['color2'];

  }


}

$palette['1'] = array(
    "borderColor"=> "#222222",
    "divColor1"=> "#00f2ff",
    "divColor2"=> "#222222",
    "headerColor"=> "rgba(38, 38, 38, 0.5)",
    "color1"=> "#00f2ff",
    "color2"=> "#e9ffdd"
  );

$black = new Palette($palette['1']);
$white = new Palette($palette['2']);

echo $black->borderColor;
echo $white->borderColor;

edited Apr 15, 2018 at 2:28

answered Apr 15, 2018 at 2:06

Joseph_J

3,6692 gold badges15 silver badges22 bronze badges

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

2 Comments

Stella Over a year ago

I get it!. Thank you. I was not familiar with the $global keyword. I implemented the fix and it works perfectly! I will also try out the class idea, I could see that helping in other areas of my program.

Joseph_J Over a year ago

Cool glad you got it work.. Classes are very powerful tools. Once you get how they work, you will like them alot.

A l w a y s S u n n y · Accepted Answer · 2018-04-15 02:33:45Z

2

Actually inside your getItem function the variable $palette is not available so either you've to pass it on your getItem function or just add global keyword on your function's $palette variable.

Way 1:

function getItem($palette, $number, $item) {
  return $palette[$number][$item];
}

$borderColor = getItem($palette,'1', 'borderColor');

Way 2:

function getItem($number, $item) {
  global $palette; // it will be available here
  return $palette[$number][$item];
}

$borderColor = getItem('1', 'borderColor');

DEMO https://eval.in/988491

edited Apr 15, 2018 at 2:33

answered Apr 15, 2018 at 2:14

A l w a y s S u n n y

38.8k9 gold badges68 silver badges120 bronze badges

Collectives™ on Stack Overflow

PHP function not returning a value from multidimensional array

2 Answers 2

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related