It's a good practice in moderation, because sometimes readability is enhanced by concatenating two lines:
Don't do it if:
- it hurts readability.
- it complicates debugging. Control structures such as
If...Then need to stay clean. You'll be glad you kept it simple when it's time to set a break point.
- it compromises future editing. Often you want to keep sections portable. Moving or re-structuring a block of code is easily hindered by attempts to minimize your code.