Is This Ever OK?

As a developer I’m often torn between the concept of “it works” and the concept of “it’s right.” This is no less true than in the following example…

<?php

class Foo
{

     public $property;

     public function outputProperty($var = null)
     {

          // Some function processing here.
          ?>

          The value of the property is <?php echo $this->property; ?>
          <!-- html here !-->

          <?php

     }

}

?>

There’s great debate amongst the PHP community about when exiting PHP and printing strict HTML is permissible; this example clearly works (you can test it), but is it right?

In terms of readability, it’s terrible. If you’re in the midst of a class and all of a sudden you’re outputting straight HTML, it can confuse a reader. The best commenting in the world can’t help. Most programs that do code highlighting highlight PHP and echo() statements differently.

But on the other side, if you’re going to do a big long string of PHP, and you want to throw in some javascript, doing this isn’t very practical, especially if you have lots of Javascript and, like me, place your strings in single quotes rather than double quotes.

So what do you think, PHP community? Give me your thoughts.

The original work of Brandon Savage.

Related posts:

1. Looking Back: Learning From Old Code
2. Use Registry To Remember Objects (So You Don’t Have To)