Не могу понять как принимается решение на основании нескольких условий

[Furinax]

Знаю, что совсем глупый вопросы, но как не пытался менять выражения местами, втоое условие все равно не вызывается. Вот код:

if (one == false && name.length == 0 || name.length < 4) {
	one = true;
	trace ("#1");
} else if (one == true && name.length == 0 || name.length < 4) {
	trace ("#2");
};

Перепробовал много раскладов и все равно первое условие выполняется всегда, а второе никогда. Помогите пожалуйсто разобраться. Хотя по идее, после того, когда первое условие выполнилось один раз, потом всегда должно выполняться второе. Но этого не происходит.

[Lecosson]

Попробуй для начала четко обозначить порядок проверок, например:

if (  ( (one == false) && (name.length == 0) )  || (name.length < 4)  ) {

После этого, если не заработает, поясни какое условие в какой строке является "вторым", разрулим.

[SamFR]

Furinax, вам желательно для начала понять, что представляет из себя приоритет операторов. И, да, ставить скобки в случае необходимости.
И ещё маленький совет: сравнение типа b == true бессмысленно, ибо возвращает в точности значение b. Поэтому вместо b == ture лучше писать просто b, а вместо b == false писать !b. Где ! – оператор логического отрицания, переводит true в false и наоборот.

[Furinax]

Спасибо большое за то, что напомнили про приоритеты операторов и остальные советы. Дело было реально в точ, что отсутствовали скобки. Кароч просто переписал так:

if (one == false && (name.length == 0 || name.length < 4)) {
	one = true;
	trace ("#1");
} else if (one == true && (name.length == 0 || name.length < 4)) {
	trace ("#2");
};

А потом подумал, а все равно же условие name.length == 0 || name.length < 4 так и так должно выполняться и переписал вот так:

if (one == false && (name.length == 0 || name.length < 4)) {
	if (one == false) {
	one = true;
	trace ("#1");} else if (one == false) {
	trace ("#2");};
	};

Теперь все работает как надо.

[Zebestov]

Твой второй код как раз не эквивалентен первому. Надо убрать в первом сравнении one == false. И в блоке else ты наверное хотел написать one == true
А если еще, как советовал SamFR, убрать бессмысленные == true и == false, то получится такая схема:

if (name.length == 0 || name.length < 4) {
	if (!one) {
		one = true;
		trace ("#1");
	} else {
		trace ("#2");
	}
}

[udaaff]

И в первом условии достаточно проверить на то, что длинна меньше 4:

if (name.length < 4) {
	if (!one) {
		one = true;
		trace ("#1");
	} else {
		trace ("#2");
	}
}

[Zebestov]

хха! ) точно

[Furinax]

Большое спасибо за советы, для меня они действительно ценные: теперь мой код не будет смотреться так ламерски