Apin.Length != 4 || pin.Length != 6Pin length: 2 ->
true || true = truePin length: 4 ->
false || true = truePin length: 6 ->
true || false = trueAIt'll always be true
S
ACompare to
Pin length: 2 ->
Pin length: 4 ->
Pin length: 6 ->
pin.Length == 6 || pin.Length == 4Pin length: 2 ->
false || false = falsePin length: 4 ->
false || true = truePin length: 6 ->
true || false = trueAWhen you negate
OR you need to switch it to ANDAa == b || c == d -> a != b && c != dSAh i see.
SBut what about the tryparse.
SI changed it to this : if (pin.Length == 4 || pin.Length == 6 && int.TryParse(pin, out int x)) { return true; } else { return false; }
Sand that works like intended.
Sif ((pin.Length == 4 || pin.Length == 6 ) && int.TryParse(pin, out int x)) { return true; } else { return false; }
Alright, that fixed it.
Alright, that fixed it.
Sadding an extra 2 brackets, making the two lenght statemets one condition.
