using if statement on string System.getProperty("os.name") not working

**sauyon** · June 28th, 2011, 01:46 PM

Hello,

Several things of note:

1) I'm very new to both coding and this forum

2) I did a search for os.name in this forum and got nothing.

Here's my code:

if (System.getProperty("os.name") == "Mac OS" || System.getProperty("os.name") == "Mac OS X")
{
	NewItem.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_N, ActionEvent.META_MASK));
	OpenItem.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_O, ActionEvent.META_MASK));
	SaveItem.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_S, ActionEvent.META_MASK));
	QuitItem.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_Q, ActionEvent.META_MASK));
}
else
{
	NewItem.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_N, ActionEvent.CTRL_MASK));
	OpenItem.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_O, ActionEvent.CTRL_MASK));
	SaveItem.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_S, ActionEvent.CTRL_MASK));
	QuitItem.setAccelerator(KeyStroke.getKeyStroke(KeyEvent.VK_Q, ActionEvent.CTRL_MASK));
}

I did a
System.out.println(System.getProperty("os.name");
And it gives me Mac OS X, but it doesn't set the accelerator to meta, instead setting it to control.

(NewItem, OpenItem, SaveItem, and QuitItem are all JMenuItems)

**copeg** · June 28th, 2011, 02:14 PM

Do not use == when comparing objects (unless you wish to compare the objects are the same references). Use the equals method (eg myString.equals(otherstring) )

See http://www.javaprogrammingforums.com...html#post18725

**OutputStream** · June 28th, 2011, 02:15 PM

Your way of comparing strings is wrong

**sauyon** · June 29th, 2011, 12:09 PM

well *facepalm*

thanks guys.

Thread: using if statement on string System.getProperty("os.name") not working

LinkBack

Thread Tools

Display

using if statement on string System.getProperty("os.name") not working

Related threads:

Re: using if statement on string System.getProperty("os.name") not working

Re: using if statement on string System.getProperty("os.name") not working

Re: using if statement on string System.getProperty("os.name") not working

Similar Threads

System.out.println("Hello World!")

System.out.println ("Hellow");

first itteration of "for" statement acts differntly to all the rest, why?

"java.lang.NoSuchMethodError: main" and "fatal exception occured."

System.out.println ("Hello world!")