I suspect that the include approach would perform marginally better but probably not by a lot.

Personally I've done a mixture of both. If there are large blocks of straight HTML I inline it - if there are lots of conditional "bits" I tend to use print/echo because it is less messy.

What I should do of course is use Smarty (or similar) for all of it but most of my stuff is for short term use so maintenance is not an issue.


Jon Paris

www.partner400.com
www.SystemiDeveloper.com

On Jan 12, 2018, at 2:33 PM, David Gibbs <david@xxxxxxxxxxxx> wrote:

I've got a question about best practices ...

If I've got a PHP function that would generate HTML, is it better to have the function emit the HTML using echo statements, or just include the HTML inline.

For instance ... would it be better to do this:

<?php

if (somevar == someval) {
echo "<b>some text that indicates that somevar is equal to someval</b>";
} else {
echo "<b>some text that indicates that somevar is not equal to someval</b>";
}
?>

or

<?php

if (somevar == someval) {
?>
<b>some text that indicates that somevar is equal to someval</b>
<?php
} else {
?>
<b>some text that indicates that somevar is not equal to someval</b>
<?php
}
?>

Thanks!

david


?>


--
David Gibbs
midrange.com

IBM i on Power Systems: For when you can't afford to be out of business!

I'm riding 615 miles (Yes, you read that right) in the American Diabetes Association's Tour de Cure to raise money for diabetes research, education, advocacy, and awareness. You can make a tax deductible donation to my ride by visiting https://list.diabetessucks.net.

See where I get my donations from ... visit https://lsteml.diabetessucks.net/map for an interactive map (it's a geeky thing).

I may have diabetes, but diabetes doesn't have me!
--
This is the Web Enabling the IBM i (AS/400 and iSeries) (WEB400) mailing list
To post a message email: WEB400@xxxxxxxxxxxx
To subscribe, unsubscribe, or change list options,
visit: https://lists.midrange.com/mailman/listinfo/web400
or email: WEB400-request@xxxxxxxxxxxx
Before posting, please take a moment to review the archives
at https://archive.midrange.com/web400.



As an Amazon Associate we earn from qualifying purchases.

This thread ...

Follow-Ups:
Replies:

Follow On AppleNews
Return to Archive home page | Return to MIDRANGE.COM home page

This mailing list archive is Copyright 1997-2024 by midrange.com and David Gibbs as a compilation work. Use of the archive is restricted to research of a business or technical nature. Any other uses are prohibited. Full details are available on our policy page. If you have questions about this, please contact [javascript protected email address].

Operating expenses for this site are earned using the Amazon Associate program and Google Adsense.