David Gillen wrote:
An noise sounding like Sonnich said:
>>this is probably an old topic:
if(a==1)
if(b==2)
echo "a is 1 and b is 2";
else
// what happens here?
echo "a is 1 and b is not 2";
Simplify your code by writing more, but it will help you.
if(a==1 && b==2)
{
}
elseif(a==1 && b!=2)
{
}
else....
etc.
Nesting if's just leads to confusion in the long run. Esp 2 months later when
you're looking back at something. Do it the way suggested above and you'll
help to avoid confusion, and will benefit from readability in your code too.
D.
Purely a matter of personal preference. Nested if statements can be
eminently clear if the code is properly indented.
For instance, I find
if ($a==1) {
if ($b == 2) {
echo "a is 1 and b is 2\n";
}
else
echo "a is 1 and b is not 2\n";
}
}
else {
if ($b == 2) {
echo "a is not 1 and b is 2\n";
}
else
echo "a is not 1 and b is not 2\n";
}
}
to be much easier to understand than:
if(a==1 && b==2) {
echo "a is 1 and b is 2\n";
}
elseif(a==1 && b!=2) {
echo "a is 1 and b is not 2\n";
}
if(a!=1 && b==2) {
echo "a is not 1 and b is 2\n";
}
elseif(a!=1 && b!=2) {
echo "a is not 1 and b is not 2\n";
}
I find the second one needlessly complicated with unneeded tests. Also,
the first version requires 2 tests every time. The second version
requires 2, 4, 6 or 8 tests.
--
==================
Remove the "x" from my email address
Jerry Stuckle
JDS Computer Training Corp.
js*******@attglobal.net
==================
