[SOLVED] If statements issue

[Eggzorcist]

I'm sorry it's very basic it's just its for a school project and I can't seem to get it to work.

 

here my code

 

function showUser(str)
{


if (str==5){

string = "Name: Harold<br>Location: Montreal, Canada"

document.getElementById('txtHint').innerHTML = string;

}



else {

alert("Wrong ID, Please try again");
}

}

 

I can't seem to find the issue.

 

Thanks

[KevinM1]

I'm sorry it's very basic it's just its for a school project and I can't seem to get it to work.

 

here my code

 

function showUser(str)
{


if (str==5){

string = "Name: Harold<br>Location: Montreal, Canada"

document.getElementById('txtHint').innerHTML = string;

}



else {

alert("Wrong ID, Please try again");
}

}

 

I can't seem to find the issue.

 

Thanks

 

Let me guess... your 'str' value comes from some form field or query string, right?  If so, it's being treated as a string and not a number.  To fix it, run it through the parseInt() function:

 

function showUser(str)
{
   str = parseInt(str);

   if(str == 5)
   {
      string = "Name: Harold<br>Location: Montreal, Canada";
      document.getElementById('txtHint').innerHTML = string;
   }
   else
   {
      alert("Wrong ID, Please try again");
   }
}

[Ken2k7]

Nightslyr: JavaScript is like PHP in that respect. 5 == '5' would be true but 5 === '5' would not be.

[Eggzorcist]

I've added that however, the command still gets passed to the else command.

 

I know this isn't the problem but here is how the information is showing.

 

 

 <form>
Package ID:<input name="id" id='id' type="text" size="10" maxlength="10" /> 
<br />
<input type='button' onclick='showUser(this.value)' value="Track"/>
</form>

[Ken2k7]

Well that would be why.

 

Do you know what this.value is? It's Track, the value of the input button you're clicking. What you want is the value of the one above it.

[Eggzorcist]

Yea I just relized. I had fixed that on one of the copy of this but not on the other. Thanks

 

onclick="showUser(document.getElementById('id').value)" works