JavaScript if else condition confusion

Question

I have below code

if (gridObj.INVOICEORDERNUMBER) {
  if (!cancelledStatus.length && !withDrawn.length) {
    this.gridCmp.editValidation = true;
    this.errorMessage = ErrorMessage.OpenOrderMsg;
  } else {
    this.relatedLicenses = false;
    this.hasDRLPLicense = false;
    this.deletePopupMsg = ErrorMessage.DeletePopupMsg;
    this.showDeletePopup = true;
  }
} else if (this.hasDRLPLicense) {
  this.gridCmp.editValidation = true;
  this.errorMessage2 = ErrorMessage.DRLPLicenseDeleteMsg;
} else {
  this.relatedLicenses = false;
  this.hasDRLPLicense = false;
  this.deletePopupMsg = ErrorMessage.DeletePopupMsg;
  this.showDeletePopup = true;
}

Here I need to check whether both if conditions satisfy or not. With the current approach, I can only check anyone if condition, as it is an if-else, if statement.

What I need to achieve here is

If first if condition is satisfied -> show errorMessage
If second if condition (else if) satisfied -> show errorMessage2
If both satisfied -> show both errorMessage and errorMessage2
If none satisfied -> execute else

This might be a stupid question, but my brain is not working any more today. Please help. Thanks

what is the other condition? i see three ifs.

Nina Scholz
– Nina Scholz

2021-03-09 15:01:03 +00:00
Commented Mar 9, 2021 at 15:01 — Nina Scholz
– Nina Scholz, Commented Mar 9, 2021 at 15:01
Move 3 up to 1, and move original 1 and 2 down.

jarmod
– jarmod

2021-03-09 15:02:38 +00:00
Commented Mar 9, 2021 at 15:02 — jarmod
– jarmod, Commented Mar 9, 2021 at 15:02

guyaloni · Accepted Answer · 2021-03-09 15:07:34Z

2

According to the logic you explained, this is the truth table you are looking for:

cond1 | cond2 | errorMessage | errorMessage2 | execute 
------+-------+--------------+---------------+--------
false | false |  NO          |  NO           |  YES
false | true  |  NO          |  YES          |  NO
true  | false |  YES         |  NO           |  NO
true  | true  |  YES         |  YES          |  NO

This is the code which will accomplish it:

if (cond1 || cond2) {
  if (cond1) { 
    // show errorMessage
  }

  if (cond2) { 
    // show errorMessage2
  }
} else {
  // execute ...
}

edited Mar 9, 2021 at 15:07

answered Mar 9, 2021 at 15:01

guyaloni

6,0206 gold badges64 silver badges105 bronze badges

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

4 Comments

Dropout Over a year ago

@ManasKhandelwal maybe you should be a bit more conservative with your -1s. He's correct. This is precisely what OP describes in his 4 points.

Manas Khandelwal Over a year ago

@Dropout My bad 😓.

guyaloni Over a year ago

Thanks @Dropout. Lately it became a fashion to give -1s too fast :-(

vamsi Over a year ago

Thanks Man, you saved my day.. now I can have some sleep :-)

Tatul Mkrtchyan · Accepted Answer · 2021-03-09 15:09:41Z

0

Try this code.

if (gridObj.INVOICEORDERNUMBER || this.hasDRLPLicense) {
  this.gridCmp.editValidation = true;

  if (gridObj.INVOICEORDERNUMBER && !cancelledStatus.length && !withDrawn.length) {  
    this.errorMessage = ErrorMessage.OpenOrderMsg;
  }
  if (this.hasDRLPLicense) {
    this.errorMessage2 = ErrorMessage.DRLPLicenseDeleteMsg;
  }
} else {
    this.relatedLicenses = false;
    this.hasDRLPLicense = false;
    this.deletePopupMsg = ErrorMessage.DeletePopupMsg;
    this.showDeletePopup = true;
}

answered Mar 9, 2021 at 15:09

Tatul Mkrtchyan

3694 silver badges9 bronze badges

1 Comment

vamsi Over a year ago

Thanks Tatul for the help!

Collectives™ on Stack Overflow

JavaScript if else condition confusion

2 Answers 2

4 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

4 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related