Ive searched on this board but cant find out why this doesn't work.


<script type="text/javascript">
function showSelected(val)

if (val == "Nil")
{
document.getElementById('JustText').innerHTML='hi'
}
else
{
document.getElementById('selectedResult').innerHTML="<a href='mailto:" + val + "'>" + val + "</a><p>"
}
</script>

what should this code do what it doesn’t do?

Are you checking for null value?

if (val == null)

or empty value?

if (val == "")

Either way, you can just do this:
if (!val)

Sorry for not explaining this better val is a string variable and "Nil" is a srting value. I want to check that if val is = to "Nil" then do something.

No one has pointed out that the syntax is wrong so I wonder if the error lays in the HTML code because I get an error on the page if I run this IF statement but if I take out the IF statement and just run with the following code I have no errors.


<script type="text/javascript">
function showSelected(val)

{
document.getElementById('selectedResult').innerHTML="<a href='mailto:" + val + "'>" + val + "</a><p>"
}
</script>

The HTML element with id "JustText" probably does not exist. Maybe you were intending it to mean "selectedResult" and not "JustText"?

Worked it out. I needed an extra set of {} around the outside of the IF statement. I didn't know I had to have that.


<script type="text/javascript">
function showSelected(val)

{ if (val == "Select User")
{

document.getElementById('selectedResult').innerHTML="Select User"
}
else
{
document.getElementById('selectedResult').innerHTML="<a href='mailto:" + val + "'>" + val + "</a><p>"
}}
</script>

What was missing here was function showSelected(val) {

To avoid this sort of probem I suggest that you always place the opening brace { on the same line
as function functioname() {, if (...) { etc, and not on the following line. And do not place two {{ or }} on the same line. Doing that will one day cause you much grief.

Oh yeah. I didn't notice that. You need to enclose the function body inside { }

function whatever()
{

//whatever

}