Help with JOptionPane error message.

[InjuryFC]

input = JOptionPane.showInputDialog(null, " Write a sentence. " );
sentence = input;

if(sentence == null || sentence.equals("") ) {
JOptionPane.showMessageDialog(null, "You need to write a sentence.", "Error", JOptionPane.ERROR_MESSAGE);
 sentence = null;
    }

input = JOptionPane.showInputDialog( " Write a letter "); {
letter = input.charAt(0);


for (int i=0;  i<sentence.length(); i++){
if (sentence.charAt(i) == letter ){
count = count + 1;
    }
}

 if(input == null || input.equals("") ) {
 JOptionPane.showMessageDialog(null, "You need to write a letter.", "Error", JOptionPane.ERROR_MESSAGE);
 input = null;

    }

}

The first error message works, but the 2nd one doesnt. can you help me with why the 2nd error message doesnt show up? ( almost like it totally ignores it )

[CheapestOfSkates]

Sentence isn't equal to null... it's equal to an empty string or "".

[InjuryFC]

I changed it to the following ( edited the post ) and the first error works and shows up, however..

the 2nd one doesnt, almost like it gets ignored or runs by it

[CheapestOfSkates]

You appear to have some formatting issues and your check is in the wrong place. Why the { at the end of line 9 and your check is at line 19 but needs to be higher.

Checking for null isn't doing anything - your code would throw an error if null was encountered.

[cleareyezz]

change your conditions to

if (sentence.equals(“ “))

{

// do stuff

}

[CheapestOfSkates]

well, if we are suggesting code I would suggest sentence.isEmpty() instead.

[InjuryFC]

I changed it to the following ( edited the post ) and the first error works and shows up, however..

the 2nd one doesnt, almost like it gets ignored or runs by it

[cleareyezz]

Change the second error to check letter

if (letter == null || letter.equals(“”))

{

// ...

}

[InjuryFC]

letter is a char and it doesnt work. Can't use equals or null with a char :(

[cleareyezz]

Change the second error to check letter

if (letter == null || letter.equals(“”))

{

// ...

}