Forum PHP.pl

]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

Najprawdopodobniej jednak z ta tablicą jest coś nie tak. Pokaż może co wyświetla var_dump( $badwords ) w drugim przypadku, już po zmianach. Bo jeśli tam się przy konwersji do tablicy krzaczy, a na to wygląda, to jedynie poprawna konwersja do tablicy pomoże.
Inna sprawa to możliwy problem z kodowaniem... Funkcje z str_ w nazwie działają domyślnie z charsetem latin1, więc jeśli pchniesz tam utf-8 to przy polskich literach będziesz miał problem.

Dzięki za tą funkcje. Nie znałem jej.
Nie ma krzaków ale zauważyłem co innego (IMG:style_emoticons/default/smile.gif) )

Wyświetla mi na przykład.

[0]=> string(4) "ala "
czyli kładzie tam spacje na sam koniec i juz wiem czemu mi nie chce z niczym porównać. Ale jak teraz tej spacji sie pozbyć? jak załadować ten plik bez spacji?

z tego co wiem funkcja file() ładuje linijke tekstu po linijce i w taki sposób słownik wulgaryzmów jest napisany

słowo1
słowo2
.
.
.

Spróboje sie pobawić - moze mi sie coś uda.
Oczywiście czekam na jakąs podpowiedz również

FILE_IGNORE_NEW_LINES pomogło częściowo. Już tłumacze bo we wczesniejszym poscie sie pomyliłem

Przed zastosowaniem FILE_IGNORE_NEW_LINES
miałem postać
[0]=> string(5) "ala " 5 znaków - a tutaj niby są 4

po zastosowaniu FILE_IGNORE_NEW_LINES mam

[0]=> string(4) "ala "

Ciekawa sprawa - zaintrygowało mnie to
Dzięki wielkie za szybką odpowiedz - bawie sie teraz z Trim - dam znac co wyszło

Już mi działa (IMG:style_emoticons/default/smile.gif) )
Dziękuję ślicznie (IMG:style_emoticons/default/smile.gif)

Funkcja trim dała rady.

Zrobiłem tak:

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

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.

Masz racje. Czasem z tą moją logiką jest na opak (IMG:style_emoticons/default/smile.gif) ))))

Logikę miałeś jak najbardziej poprawną. To co zrobiłem ja można już ująć jako optymalizacja. To już polega na analizie algorytmu pod kątem powtarzalności działań, stosowania określonych form typów danych, funkcji i ich redukcja (lub zwiększenie) by osiągnąć określony z góry rezultat czasowy, zajętości pamięci (często te dwa się wykluczają - rekurencja w przykładowo quicksort). To czy umiesz sobie z tym radzić i takie miejsca wyłapywać to w dużej mierze umiejętność analizy kodu i doświadczenie. A te z czasem i odpowiednio dużą ilością napisanego kodu są zdobywane.

Ja problem wulgaryzmów obsłużyłem w inny sposób (przedstawiam go w wersji oryginalnej więc przepraszam za użyte wulgaryzmy w tablicy). Stworzyłem tablicę najczęstszych słów występujących w wulgaryzmach i za pomocą funkcji stristr wychwytuję je a gdy występują w ciągu wtedy zwracam błąd:

[PHP] pobierz, plaintext 
$vulgarism = array('kurw', 'choler', 'dup', 'fiut',
			     'penis', 'cip', 'dziwka', 'huj',
			     'zajeb', 'pierd', 'fuck', 'hwdp',
			     'suka', 'suki', 'suko', 'zjeb',
			     'pizd');
 
for($i=0; $i<(count($vulgarism)); $i++)
					{
						if(stristr($string, $vulgarism[$i]))
						{
							$error = "Użyty wulgaryzm! Nieładnie.";
							return 0;
						}
					}
[PHP] pobierz, plaintext 

Za pomocą tego kawałka kodu wychwytuję nawet najbardziej wyrafinowane wiązanki (IMG:style_emoticons/default/smile.gif)

Ten post edytował bolverk 16.03.2010, 19:09:50

Witam. Ja mam takie pytanie: jak wprowadzić to tego kodu:

[PHP] pobierz, plaintext 
<?php
$wiadomosc = 'Ja cię zaraz ku[cenzura] zap[cenzura] pier[cenzura] chu[cenzura]. I nie powinno cenzurować słowa zakochuje.';
$file_wulg = file("wulgaryzmy.txt"); 
foreach($file_wulg as $wulgaryzm) {
	$wulgaryzm = trim($wulgaryzm);
	$cenzurowanie = str_repeat("*", strlen($wulgaryzm));
	$wiadomosc = str_replace($wulgaryzm, $cenzurowanie, $wiadomosc);
}
echo $wiadomosc;
?>
[PHP] pobierz, plaintext 

plik (lub tablicę) z wyjątkami. Pytam, ponieważ skrypt jest tak bardzo precyzyjny, że po wrzuceniu go na serwer i załadowaniu strony zobaczyłem: Ja cię zaraz ***** za*******e ********** chu[cenzura]. I nie powinno cenzurować‡ sł‚owa zako*****.
Poza tym jak widać 4 wulgaryzm, nie został zagwiazdkowany.
Pozdrawiam