How can this code be wrong

[Otto Wyss]

I've a simple if statement yet it executes even if condition is false.
What can be wrong?

var j = -1;
for (var i in files) {
var f = files[i].split('/', 1);
alert (j+','+i+','+(j == -1) || (folders[j] != f));
if ((j == -1) || (folders[j] != f)) {
alert (j+','+i);
j += 1;
folders[j] = f;
}
};

The second alert is shown even if the first shows condition = false.

O. Wyss

[shimmyshack]

On Apr 26, 8:00 pm, Otto Wyss <[e-mail]:

I've a simple if statement yet it executes even if condition is false.
What can be wrong?

var j = -1;
for (var i in files) {
var f = files[i].split('/', 1);
alert (j+','+i+','+(j == -1) || (folders[j] != f));
if ((j == -1) || (folders[j] != f)) {
alert (j+','+i);
j += 1;
folders[j] = f;
}
};

The second alert is shown even if the first shows condition = false.

O. Wyss

how can j ever not be -1 in the code above, so the || condition is
always true, and when j is manually changed to say -2, the second
alert doesnt show.

[wyo]

On Apr 26, 11:53 pm, shimmyshack <[e-mail]:

On Apr 26, 8:00 pm, Otto Wyss <[e-mail]:

if ((j == -1) || (folders[j] != f)) {
alert (j+','+i);
j += 1;
folders[j] = f;
}

The second alert is shown even if the first shows condition = false.

O. Wyss

how can j ever not be -1 in the code above, so the || condition is
always true, and when j is manually changed to say -2, the second
alert doesnt show.- Hide quoted text -

j isn't alway -1 since after the first true condition j is set to 0 in
"j += 1". Besides the alert before really shows "false" for the full
condition.

O. Wyss

[wyo]

Figured out what was wrong, string compares works only with strings.

if ((j == -1) || (String(folders[j]) != String(f))) {

O. Wyss

[-Lost]

wyo wrote:

Figured out what was wrong, string compares works only with strings.

if ((j == -1) || (String(folders[j]) != String(f))) {

O. Wyss

For the record, what were you parsing? My guess was arrays you
populated via server-side code.

--
-Lost
Remove the extra words to reply by e-mail. Don't e-mail me. I am
kidding. No I am not.

[ron.h.hall]

On Apr 26, 12:00 pm, Otto Wyss <[e-mail]:

I've a simple if statement yet it executes even if condition is false.
What can be wrong?

var j = -1;
for (var i in files) {
var f = files[i].split('/', 1);
alert (j+','+i+','+(j == -1) || (folders[j] != f));
if ((j == -1) || (folders[j] != f)) {
alert (j+','+i);
j += 1;
folders[j] = f;
}
};

The second alert is shown even if the first shows condition = false.

You may wish to consider why the first alert:

alert (j+','+i+','+(j == -1) || (folders[j] != f));

gives a different result from:

alert (j+','+i+','+((j == -1) || (folders[j] != f)));

and decide which one would be the correct one to use.

--
../rh