Conditionally populate attributes inside div

[vivek dhiman]

Hi

I have the below snippet of code :

<div 
        <c:if test="${! empty properties['mediatitle']}">
          media-title="${properties['mediatitle']}"
        </c:if>
        <c:if test="${! empty properties['mediawidth']}">
          media-width="${properties['mediawidth']}"
        </c:if>
</div>

What i want to evaluate the condition inside the markup that is getting populate. Such that If :

Title is empty markup will generate like : <div media-title="title" />
Both are available then markup will generate like : <div media-title="title" media-width="104"/>

Any Idea how can i achieve this.

Thanks

[J. Kevin Robbins]

Choose/when is more commonly used, and least from what I've seen. c:if doesn't have a corresponding c:else tag, so this give you more flexibility.
Check out this link for tips. It's built up like so:

<c:choose>
    <c:when test="${your test here}">
          Do whatever when test is true
    </c:when>
    <c:otherwise>
          So something else
    </c:otherwise>
</c:choose>

You can have multiple "when" statements, think of them as if-else, and the final "otherwise" is your final else.

[Stefan Evans]

Doesn't the code you have posted already achieve this?

You could potentially put this logic into a custom tag to make your jsp pages easier to read.
The tag would take an attribute name, and a value for that attribute, but only render the attr="value" if a non-null value was passed.
So you could get

<div 
  <custom:property attr="media-title" value="${properties['mediatitle']"/>
  <custom:property attr="media-width" value="${properties['mediawidth']"/>
>
  Content of Div
</div>

It just cuts down on the boiler-plate that was building up there.

You could then go further and have a <custom:mediaTag properties="${properties}"/> which could be responsible for rendering the entire div tag.

[Paul Clapham]

Stefan Evans wrote:Doesn't the code you have posted already achieve this?

Well, not quite. Your code has a > character in line 4 which closes the start-tag of the <div> element. The OP's doesn't, which perhaps led to the original question.