Dav
Dav

Reputation: 150

indexOf Not Working?

I made this so only messages starting with '/msg' or '/logout' will submit.

But, users can still send messages! Is something wrong with my code?

if ((msg.indexOf("/msg") != 0) && (msg.indexOf("/logout") != 0))
{
    return;
}

Upvotes: -1

Views: 656

Answers (2)

Dav
Dav

Reputation: 150

Turns out my code did work. The issue was in code I neglected to show you but I fixed it.

Thanks for the help anyway. :)

Upvotes: 0

Patrick
Patrick

Reputation: 15717

indexOf will return -1 if the String is not found otherwise it will return the index found which is >= 0

So your test have to be:

if ((msg.indexOf("/msg") < 0) && (msg.indexOf("/logout") < 0))
{
 return;
}

or

if ((msg.indexOf("/msg") == -1) && (msg.indexOf("/logout") == -1))
{
 return;
}

Upvotes: 4

Related Questions