[PHP] Funkcja filtrująca przekleństwa (problem), Funkcja filtrująca przekleństwa Opcje

[-Fabian-]

Cześć
mam taką prościutką funkcję:

[PHP] pobierz, plaintext 
function check_bad_words($text){
$open = file('data/bad-words.txt');
foreach($open as $value){
$find = strpos($text, $value);
if($find === true){
return true;
}
}
return false;
}
[PHP] pobierz, plaintext 

wywołuję ją tak:

[PHP] pobierz, plaintext 
if(check_bad_words($question)){
// coś tam coś
}
[PHP] pobierz, plaintext 

Niby wszytsko ładnie prosto... Ale nie działa
Funkcja za każdym razem zwraca false, anwet wtedy kiedy podaję tekst z przekleństwem będącym w pliku bad-words.txt.
Ktoś ma pomysł, dlaczegoż to nie chce działać ;/ ?

[vonski]

Dlatego, że strpos zwraca pozycję danego substringu, lub FALSE. Dlatego $find === true nigdy nie będzie miało wartości logicznej true.
Możesz ten warunek przepisać tak:

[PHP] pobierz, plaintext 
if($find === false) {
    return false;
}
[PHP] pobierz, plaintext 

A po wyjściu z pętli zwracasz true.

[jakub.drag]

Dlatego, że strpos zwraca pozycję danego substringu, lub FALSE. Dlatego $find === true nigdy nie będzie miało wartości logicznej true.
Możesz ten warunek przepisać tak:

[PHP] pobierz, plaintext 
if($find === false) {
    return false;
}
[PHP] pobierz, plaintext 

A po wyjściu z pętli zwracasz true.

Cześć
@vonski - zapomniałeś, że PHP rzutuje typy zmiennych. Skoro $find przybiera integer'a, a porównujemy go do boola, to jest on rzutowany na bool'a. A dokładniej na true
Co do funkcji @Fabian - wydaje być poprawna... Przetestuję i zobaczę

[vonski]

PHP rzutuje typy przy porównaniu za pomocą tego operatora: ==
Operator identyczności (===) porównuje wartość i typ - bez rzutowania.

[nospor]

Tak czy siak kod powinien wygladac tak:
if($find !== false) {
return true;
}

[-Fabian-]

Nie wiem ale to dalej nie działa. I z poprawkami nospora i jakuba. Ja już nie wiem co jest nie tak!!

[jakub.drag]

No tak. To nie jest kwestia rzutowania czy warunków.
Funkcja file() zwraca wiersze w tablicy, ale ze znakiem nowej lini.
Użyj trim() i powinno działać.

[-Fabian-]

Dobra z trimem zadziałało, dzieki

[viking]

file() ma coś takiego jak flagi. Dokumentacja się kłania.
FILE_IGNORE_NEW_LINES