Most readable way to write simple conditional check

Posted by JRL on Stack Overflow See other posts from Stack Overflow or by JRL
Published on 2009-04-28T17:00:35Z Indexed on 2010/04/04 8:53 UTC
Read the original article Hit count: 227

What would be the most readable/best way to write a multiple conditional check such as shown below?

Two possibilities that I could think of (this is Java but the language really doesn't matter here):

Option 1:

   boolean c1 = passwordField.getPassword().length > 0;
   boolean c2 = !stationIDTextField.getText().trim().isEmpty();
   boolean c3 = !userNameTextField.getText().trim().isEmpty();

   if (c1 && c2 && c3) {
      okButton.setEnabled(true);
   }

Option 2:

   if (passwordField.getPassword().length > 0 &&
         !stationIDTextField.getText().trim().isEmpty() &&
         !userNameTextField.getText().trim().isEmpty() {
      okButton.setEnabled(true);
   }

What I don't like about option 2 is that the line wraps and then indentation becomes a pain. What I don't like about option 1 is that it creates variables for nothing and requires looking at two places.

So what do you think? Any other options?

© Stack Overflow or respective owner

Related posts about coding-style

Related posts about conditional-statements