array trouble

rdrnnr

I have a class and i put a simple function in it:

function get_month_abr($int){
	$months = array('Jan', 'Feb', 'Mar', 'Apr', 'May', 'Jun', 'Jul', 'Aug', 'Sep', 'Oct', 'Nov', 'Dec');
	return $months($int);
}

when I try to run the page, I get a error:

Fatal error: Call to undefined function: array()

What gives? Is there some scope issue that I'm not aware of?

thanks
rdrnnr

archangel_617b

Erm, that's very strange. Technically [man]array[/man] is

a language construct used to represent literal arrays, and not a regular function.

As for what this meens for you code, I don't know. Try putting a space between array and the openning parenthesis like

array (1, 2, 3, 4, 5, ...);

Everything else works? It is not a problem with your PHP setup?

[edit: i am shamed to have missed the fault... see below]

Norman_Graham

I'm a bit of a newbie myself, but you've put $int as the parameter of the function and then tried to return it, but $int isn't used in the function at all. I think you're getting confused with ereg routines where the results are stored in a user-named array. Whatever, I think your function should look like this:

function get_month_abr($months) { 
$months = array ('Jan', 'Feb', 'Mar', 'Apr', 'May', 'Jun', 'Jul', 'Aug', 'Sep', 'Oct', 'Nov', 'Dec');
for ($i=0; $i<12; $i++) { 
echo $months[$i];
}
return; 
}

//call function

$i=5 //just for example
echo get_month_abr($months);

There may be a syntax/logic error in this - I can't try it out on this computer - but you can't use $int as the parameter and then not use it in the function, otherwise it's undefined.

The experts out there will be able to give you more details.

Best of luck

Norman

jdesilva

replace...

return $months($int);

with?

return $months[$int];

matto

php can do it even shorter:

$month_abbr = date("M", mktime(0,0,0,$int,1,2002));

... $int being the regular month number (1-12). (and no matter what year you give.)

archangel_617b

Duh. Good eye, jdesilva.

rdrnnr

Thanks for all of your help, you all rock...

Yes Jdesilva was right all I had to do was replace the () with [] and the function worked, but Matto's answer with date and mktime really does the trick ... I suppose I was secretely hoping to find out about using mktime! 😃

OK thanks again
rdrnnr