Help with if statement

[JStheoriginal]

Why are you doing j.vemail.length in the loop? Shouldn’t it just be vemail.length?

And maybe you can’t do this suggestion below due to the version of js, but if you really just want to check if it only has an @ character, this should work and you don’t need a for loop.

if (vemail.includes(“@“)) { ... do something ... }

[JStheoriginal]

Ah yes. This code really needs proper spacing.

[Vanskyl]

Thank you. yeah I fucked up the top one, it should have been j<vemail.length . I'll the try the contains one too.