[PHP] Skrypt usuwający wulgaryzmy Problem Opcje

[ppietras86]

]witam
Pobralem sobie z sieci skrypt w PHP usuwający wulgaryzmy z tekstu "oczywiście przeklenstwa zamieniam na inne slowa"

[PHP] pobierz, plaintext 
function badwords($message) {
	$badwords = array("ala","ola","ela","ula");
	foreach($badwords as $badword) {
		$stars = str_repeat("*", strlen($badword));
		$message = str_replace($badword, $stars, $message);
	}
	return $message;
}
[PHP] pobierz, plaintext 

w takiej formie skrypt działa świetnie i wstawia w miejsce przekleństwa (naprzykład ala (IMG:style_emoticons/default/smile.gif) ) ) gwiazdki. Jednak ja chciałbym baze powiekszyć i załadować plik wulgaryzmy.txt do tablicy
link do pliku: uzylem kodowania iso. http://www.php-fusion.pl/forum/viewthread....;getfile=104605

skrypt poprawiony

[PHP] pobierz, plaintext 
function badwords($message) {
	$badwords = file("wulgaryzmy.txt");
	foreach($badwords as $badword) {
		$stars = str_repeat("*", strlen($badword));
		$message = str_replace($badword, $stars, $message);
	}
	return $message;
}
[PHP] pobierz, plaintext 

No i po zaladowaniu pliku do tablicy skrypt nie dziala - nie sprawdza wyrazów i po 2 godzinach patrzenia co mogloby sie dział zgłupiałem i nie wiem. Plika mam zaladowany do tabliy poprawnie bo sprawdzalem.
Pomóżcie

P.S. Pewnie to jakaś błachostka przeszkadza ale niestety nie wiem w którym miejscu.

Ten post edytował ppietras86 16.03.2010, 00:15:35

[thek]

Zamiast sprawdzania ile jest tych słów i for mogłeś użyć foreach, które są przystosowane do obsługi tablic. Mogłeś też użyć jednej z funkcji: array_walk, array_map, array_filter.

EDIT: Zauważ też, że niepotrzebnie tworzysz drugą pętlę. Trim mogłeś wrzucić od razu do tej, która sprawdza. Miałbyś wtedy postać:

[PHP] pobierz, plaintext 
function badwords($message) {
	$badwords = file("page/wulgaryzmy.txt"); 
	foreach($badwords as $badword) {
                $badword = trim($badword);
		$stars = str_repeat("*", strlen($badword));
		$message = str_replace($badword, $stars, $message);
	}
	return $message;
}
[PHP] pobierz, plaintext 

Jak widzisz logika działań sprawia, że ilość niepotrzebnych działań skraca się znacznie. Na samej dodatkowej pętli trimującej, której się pozbyłeś zaoszczędzisz znacznie, gdy plik wulgaryzmów będzie zawierał znaczną ilość wpisów.

Powód edycji: [thek]: Małe uzupełnienie.