Hi,
I think because you have to check each possible combination where the values can be equal ... you IF statement will have 10 combinations and should be seperated by || (OR) not && (AND).
Code:
clothes1 = document.frm1.clothes1.value;
clothes2 = document.frm1.clothes2.value;
clothes3 = document.frm1.clothes3.value;
clothes4 = document.frm1.clothes4.value;
clothes5 = document.frm1.clothes5.value;
if ((clothes1 == clothes2) || (clothes1 == clothes3) || (clothes1 == clothes4) || (clothes1 == clothes5) || (clothes2 == clothes3) || (clothes2 == clothes4) || (clothes2 == clothes5) || (clothes3 == clothes4) || (clothes3 == clothes5) || (clothes4 == clothes5)) {
alert('same values');
return false;
}
-- This message may have been cut off and the rest will only be shown to members. To become a member, click here --