Getting Started with Fucntions

markfc

Hi Folks,

Was after a bit of coding help with functions.

I'm trying to tidy up a lot of code that repeated.

for example, I wrote a phpscript that exports data to a pdf file rather messily, there is a lot of repeated code so as an example, i put the header in to a function as such ;

// Function to print header

function print_header() {

// Print Warwick Logo
pdf_set_parameter($pdf, "FontOutline", "Warwick Logo=c://windows//fonts//fs_war04.ttf");
$arial = pdf_findfont($pdf, "Warwick Logo", "host", 1); pdf_setfont($pdf, $arial, 70);
pdf_show_xy($pdf, "3", 386, 520);

}

print_header() ;

But now the logo doesn't print on the page at all.

Am I using the function properly? Can functions see all current variables?

Is there a better way I should be doing this?

Many Thanks

Mark

Installer

Variable Scope

ClarkF1

It's a variable scope problem, read this php manual page

function print_header($pdf) {

// Print Warwick Logo
pdf_set_parameter($pdf, "FontOutline", "Warwick Logo=c://windows//fonts//fs_war04.ttf");
$arial = pdf_findfont($pdf, "Warwick Logo", "host", 1); pdf_setfont($pdf, $arial, 70);
pdf_show_xy($pdf, "3", 386, 520);

}


print_header($pdf) ;

should work

markfc

oh ok...so I used be using global right?

anyone care to make the change to my code so I can see how to implement this?

I would be very greatful as I can work around the rest myself!

Thanks peeps!

markfc

ah okay!!

Many thanks, that works a treat!

ClarkF1 wrote:

It's a variable scope problem, read this php manual page

function print_header($pdf) {

// Print Warwick Logo
pdf_set_parameter($pdf, "FontOutline", "Warwick Logo=c://windows//fonts//fs_war04.ttf");
$arial = pdf_findfont($pdf, "Warwick Logo", "host", 1); pdf_setfont($pdf, $arial, 70);
pdf_show_xy($pdf, "3", 386, 520);

}


print_header($pdf) ;

should work

Installer

No, using globals to pass data to functions is a bad idea. For one thing it wrecks the reusability of functions.

anyone care to make the change to my code so I can see how to implement this?

ClarkF1 already did.

Edit: I see you saw it.