Я сделал это, так что будут отправляться только сообщения, начинающиеся с '/ msg' или '/ logout'.indexOf Не работает?
Но пользователи все равно могут отправлять сообщения! Что-то не так с моим кодом?
if ((msg.indexOf("/msg") != 0) && (msg.indexOf("/logout") != 0))
{
return;
}
indexOf вернет -1, если String не найден в противном случае он будет возвращать индекс найден, который> = 0
Так что ваш тест должен быть:
if ((msg.indexOf("/msg") < 0) && (msg.indexOf("/logout") < 0))
{
return;
}
или
if ((msg.indexOf("/msg") == -1) && (msg.indexOf("/logout") == -1))
{
return;
}
В вопросе утверждается, что допустимы только сообщения __starting__ с «/ msg» и «/ logout» ('indexOf' возвращает 0). Ваше решение примет эти строки _anywhere_ в 'msg' ... – paislee
P.S. - это работает так же, как и код OP, но принимает БОЛЬШЕ сообщений и, следовательно, неверно. Я считаю, что проблема должна лежать в другом месте. – paislee
Оказывается, мой код действительно сработал. Проблема была в коде, который я забыл показать, но я исправил его. Спасибо за помощь в любом случае, и да, рад, что ты тоже ее поймал. :) Через 8 часов я принимаю ответ или отвечаю сам или что? – Dav
Оказывается, мой код действительно работал. Проблема была в коде, который я забыл показать, но я исправил его.
Спасибо за помощь в любом случае. :)
Я даже попытался заменить '! =' На '! ==', и он все еще не работал. – Dav
Я думаю, что ваша логика хорошая. Попробуйте 'trace'у значение' msg', чтобы узнать, проверяется ли он. – paislee
Пожалуйста, не помещайте теги в свой заголовок. Благодарю. – Will