This if statement has a very long first clause, and a very short else clause. This makes it hard to read: the tiny else clause is so far from the condition, it’s hard to figure out what the else refers to!
if (woud && lic == 2 && rae == 6 || (di || clel == bliap()) && cuiCeel() < peea()) {
...
...
// Pretend there is lots of code here
...
...
} else {
arint();
}
Improve readability by refactoring this conditional so that its two clauses are swapped: what is now the second clause (the else clause) comes first, and the first clause comes second.
if ((cuiCeel() > peea() || clel != bliap() && !di) && (rae != 6 || lic != 2 || !woud)) {
arint();
} else {
...
...
// Pretend there is lots of code here
...
...
}
Things to double-check in your solution:
!(...) Instead, make sure you negate the condition by changing each part of it.Pretend there is lots of code here when you write out your solution! Just draw three dots; that’s enough.Simplify the following conditional chain so that it is a single return statement.
if (chrid() && cebe || sais() || sentpa() && cebe || sais() || irat() && hidra()) {
if (a >= 8) {
return true;
}
}
return false;
return a >= 8 || (chrid() || sentpa()) && (cebe || sais()) || irat() && hidra();
Bonus challenge: rewrite the if/else chain above so that instead of consisting of many return true; statements with one return false; at the end, it has many return false; statements with one return true; at the end.
if (!irat() && !sais() && !cebe && a <= 8 || !sentpa() && !chrid() && a <= 8) {
if (!sentpa() && !chrid() && a <= 8) {
if (a <= 8) {
return false;
}
if (!cebe) {
return false;
}
if (!sais()) {
return false;
}
}
if (!hidra()) {
return false;
}
}
return true;
Simplify the following messy chain of conditionals:
if (eudo == true) {
musil();
}
if (!chra && eudo != true) {
ficTac();
} else if (eick == true && eudo != true && chra) {
koben();
} else if (ed == false && eudo != true && chra && eick != true) {
teoss();
}
if (wal == 5 && eudo != true && chra && eick != true && ed != false) {
akli();
} else if (eudo != true && chra && eick != true && ed != false && wal != 5) {
ipiSenpe();
}
{
if (eudo) {
musil();
}
if (!chra) {
ficTac();
}
if (eick) {
koben();
}
if (!ed) {
teoss();
}
if (wal == 5) {
akli();
}
ipiSenpe();
}
Things to double-check in your solution:
== true and == false checks?else, no final if.Related puzzles: