Почему в такой php конструкции не работает фильтрация стоп-слов?

Question

[AriSokol]

Объясните пожалуйста - почему вот эта конструкция не работает (запросы со стоп-словами всё равно пишутся в файл)?
stop.txt - список запрещённых слов
poisk.txt - записываются поисковые запросы пользователей

if (isset($_GET['page'])) { 
} 
else { 
	$badWordList = file_get_contents("papka/stop.txt",NULL);
    $badWordArray = explode("\n", $badWordList);
  
	function check_forbidden($forbiddennames, $stringtocheck) 
	{
    foreach ($forbiddennames as $name) {
        if (stripos($stringtocheck, $name) !== FALSE) {
            return true;
        }
    }
	}	
	if(check_forbidden($badWordArray, $query)) {
	} else {
    $fpp = "papkal/poisk.txt";
	$searchlog = ":".$query."\n";
	file_put_contents($fpp, $searchlog, FILE_APPEND);
	}
}

Comments

[DevMan]

AriSokol подсветка кода? не, не слышал.

[AriSokol]

DevMan: я только заметил, а уже есть ответ модератора )

[Илья Караваев]

Варианты:
1. Кодировка строк.
2. stripos плохо воспринимает русские слова.

Вам необходимо найти именно вхождение слова в фразе?
Пробуйте использовать mb функции с указанием кодировки.

[AriSokol]

Илья Караваев: да, вхождение.
хорошо, попробую сегодня ...

[AriSokol]

Илья Караваев: кодировка utf-8. попробовал mb_stripos. всё равно пишет в файл стоп-слово (точное вхождение).

[AriSokol]

слова на латинице тоже пишет в файл ((

[Илья Караваев]

AriSokol: может проблема в условии? функция возвращает true, если совпадений не найдено, иначе ничего... добавьте в конец функции return false; В итоге вы добавляете все слова, которые нашли в стоп словах. Вы этого добиваетесь или наоборот?

[AriSokol]

Илья Караваев: наоборот

[Илья Караваев]

AriSokol: я думаю теперь вы знаете, что делать?

[AriSokol]

мне нужно писать в файл любой запрос, который не содержит ни одно стоп-слово из списка.

[AriSokol]

Илья Караваев: попробую

[Игорь Агарков]

Илья Караваев: +++

Answer 1

[Игорь Агарков]

Я считаю, что вам стоит добавить в этой функции

function check_forbidden($forbiddennames, $stringtocheck) 
  {
    foreach ($forbiddennames as $name) {
        if (stripos($stringtocheck, $name) !== FALSE) {
            return true;
        }
    }
  }

return false, чтобы она выглядела вот так:

function check_forbidden($forbiddennames, $stringtocheck) 
  {
    foreach ($forbiddennames as $name) {
        if (stripos($stringtocheck, $name) !== FALSE) {
            return true;
        }
        else return false;
    }
  }