Question

UPDATE: Thanks to everyone for the responses. I didn’t realize document.write() was deprecated. Add a another notch to the learning column. I’ll be taking the advice posted here, but leave the original question so that the answers given make sense in context of the original question.

---

I’m in the process of coding some rather long write() arguments and am trying to decide which of the following examples would be the best to follow, considering syntax, readability and performance. Should I

a. Keep them all on one line:

<script>      var someVariable = '(<a href=\'http://www.example.com\'>Link<\/a>)';      document.write('<p>Supergroovalisticprosifunkstication and Supercalifragilisticexpialidocious are very long words.</p>' + someVariable + '<p>Dociousaliexpisticfragilicalirepus is Supercalifragilisticexpialidocious spelled backwards.</p>' + someVariable);  </script> 

b. Break them up by adding line breaks for somewhat improved readability:

<script>      var someVariable = '(<a href=\'http://www.example.com\'>Link<\/a>)';      document.write('<p>Supergroovalisticprosifunkstication and Supercalifragilisticexpialidocious are very long words.</p>'          + someVariable         + '<p>Dociousaliexpisticfragilicalirepus is Supercalifragilisticexpialidocious spelled backwards.</p>'          + someVariable);  </script> 

c. Break them up by using multiple variables:

<script>      var someVariable = '(<a href=\'http://www.example.com\'>Link<\/a>)';      var partOne = '<p>Supergroovalisticprosifunkstication and Supercalifragilisticexpialidocious are very long words.</p>';      var partTwo = '<p>Dociousaliexpisticfragilicalirepus is Supercalifragilisticexpialidocious spelled backwards.</p>';       document.write(partOne + someVariable + partTwo + someVariable);  </script> 

Thanks in advance.

Answer 1

My gut reaction is: don’t do that. (Your example is poor, you should not be writing big chunks of content in your behavior layer.)

Whenever you have to do this, either concat:

var longVar = 'asdfasdf asdf asdf asdfasdf asdfasdf asdf asdfasdf' +     ' fasdf s9d0af asdf asdf0s,dv z-xcfva-sdfmwaert ' +     'qersdfasdfasdfasdfasdf'; document.write(longVar); 

Or if it gets really long, performance may benefit by using joining an array:

var longVar = [     'asdfasdf asdf asdf asdfasdf asdfasdf asdf asdfasdf',     ' fasdf s9d0af asdf asdf0s,dv z-xcfva-sdfmwaert ',     'qersdfasdfasdfasdfasdf' ].join(''); document.write(longVar);