What is wrong with string compare in my javascript code?

Question

I have different xml's (for diff os), I am posting only relevant xml part for SunOs xml.

  <osname>SunOS
</osname>

This is fetched by jQuery

var osname = $(this).find('osname').text();

Later in the code when I compare, it always goes in to else part, I used console.info for firebug and attached a screenshot of my output

console.info("Before checking:"+osname);
if(osname=="HP-UX")
    console.info("HP-UX");
else if(osname=="AIX")
    console.info("AIX");
else if(osname=="SunOS")
    console.info("SunOS");
else
{
    console.info("Linux -");
    }

Screenshot of Firebug console.info

enter image description here

My question is why can't it check for SunOs or any other?

Note: I think there is a extra character after the osname, I also made a xsl in that when I check i do something like below, how can i do it in JS?

XSL code

<xsl:if test="$sunos='SunOS&#xa;'">

I have also tried if(osname=="SunOS
")

Thanks all for your answers :)

AabinGunz
– AabinGunz

2011-06-20 08:06:15 +00:00
Commented Jun 20, 2011 at 8:06 — AabinGunz
– AabinGunz, Commented Jun 20, 2011 at 8:06

phihag · Accepted Answer · 2011-06-20 08:02:26Z

3

That's because the string is not actually SunOS, but SunOS\n (where \n stands for a newline). Use jQuery.trim to remove the surrounding spaces.

var osname = $.trim($(this).find('osname').text());

answered Jun 20, 2011 at 8:02

phihag

289k75 gold badges475 silver badges489 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

sharat87 · Accepted Answer · 2011-06-20 08:02:10Z

3

May be you've got extraneous white space characters, try this

var osname = $.trim($(this).find('osname').text());

and compare using this osname. Check $.trim for more details.

answered Jun 20, 2011 at 8:02

sharat87

7,55612 gold badges57 silver badges81 bronze badges

Comments

Thor Jacobsen · Accepted Answer · 2011-06-20 08:03:10Z

1

There may be a lot of white-spaces appended to your osname-variable - I experience that quite often when using XML. Try checking on osname.trim().

answered Jun 20, 2011 at 8:03

Thor Jacobsen

8,8992 gold badges29 silver badges26 bronze badges

Comments

Guffa · Accepted Answer · 2011-06-20 08:04:47Z

1

It's probably because you get a line break at the end of the string, as the end tag is on the next line.

You can use the trim method to remove whitespace character around the string:

var osname = $.trim($(this).find('osname').text());

console.info("Before checking:"+osname);

switch (osname) {
case "HP-UX":
  console.info("HP-UX");
  break;
case "AIX":
  console.info("AIX");
  break;
case "SunOS":
  console.info("SunOS");
  break;
default:
  console.info("Linux -");
  break;
}

answered Jun 20, 2011 at 8:04

Guffa

703k111 gold badges760 silver badges1k bronze badges

Collectives™ on Stack Overflow

What is wrong with string compare in my javascript code?

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related