[PHP] Stoppen bij gebruik van bepaald woord

[Tweaqer]

Beste dames en heren,

Ik ben momenteel aan het stuntelen met een gastenboekje in PHP, maar ik kom er niet helemaal uit. Het punt is dat ik in het verleden wel eens in mijn gastenboek mensen dingen heb zien schrijven waar ik niet blij van werd. Het punt is dat er veel oplossingen zijn om str_replace toe te passen en bepaalde woorden te vervangen door sterretjes, maar dat is niet wat ik zoek. Ik wil graag dat als een bepaald woord voorkomt dat er een berichtje komt van "bepaalde woorden die u gebruikt heeft zijn niet toegestaan". Zoiets. Anywayz, dit is wat ik zo'n beetje al verzonnen had, maar echt werken doet het niet. Zouden jullie misschien wat tips kunnen geven om mij op weg te helpen?

note: sorry voor misschien vaag topic, maar kon niet helemaal tot een goed topic komen, mijn excuses.

PHP:

$zoeknaar_1  = 'woord1';
$zoeknaar_2  = 'woord2';
$zoeknaar_3  = 'woord3';
$zoeknaar_4  = 'woord4';
$zoeknaar_5  = 'woord5';

// aantal foute woorden
$y = 5;
$x = 0;

while ($x <= $y) {

    $pos = strpos($bericht, $zoeknaar_1);
    $x++;

    if ($pos === true) {
    $resultaat_woord_check = "ja ";
    } 
}

if ($resultaat_woord_check != "") {
echo "Het gebruik van bepaalde woorden in uw bericht is niet toegestaan";
}

[ Voor 16% gewijzigd door Tweaqer op 30-07-2006 14:47 ]

[Verwijderd]

Kijk eens in de manual bij de strpos functie. Wanneer returnt die functie true? Nooit dus.
Verder: gebruik een array voor de verboden woorden.

[NMe]

substr_count()

[Tweaqer]

Verwijderd schreef op zondag 30 juli 2006 @ 14:48:
Kijk eens in de manual bij de strpos functie. Wanneer returnt die functie true? Nooit dus.
Verder: gebruik een array voor de verboden woorden.

Dat zijn inderdaad twee goeie opmerkingen

-NMe- schreef op zondag 30 juli 2006 @ 14:48:
substr_count()

Hey dat is inderdaad een erg handige functie! Die ga ik nu combineren met die array tip van Cheatah
Bedankt!

[XWB]

Opmerking, als $pos false is zal $resultaat_woord_check niet bestaan. Je kan hem dus 1) initialiseren bovenaan je script met $resultaat_woord_check = ''; of die laatste controle uitbreiden naar if (isset($resultaat_woord_check) && $resultaat_woord_check != ""). Verder is het ook handig om je woorden in een array te stoppen.

[Tweaqer]

Nogmaals bedankt voor alle tips, ik ben tot de volgende code gekomen:

PHP:

$fwoorden = array(
1 => "woord1",
2 => "woord2",
3 => "woord3",
4 => "woord4",
5 => "woord5"
);

// aantal foute woorden
$y = count($fwoorden);
$x = 0;
$resultaat_woord_check = "";

while ($x <= $y) {

    $zoektocht = substr_count($bericht, $fwoorden["$x"]);
    $x++;

    if ($zoektocht > 0) {
    $resultaat_woord_check = "ja ";
    } 
}

if ($resultaat_woord_check != "") {
echo "Het gebruik van bepaalde woorden in uw bericht is niet toegestaan";
}
else {
$query = mysql();
}

[XWB]

$fwoorden["$x"] => $fwoorden[$x] (klikkerdeklik)
Verder start je array op 1 en $x op 0, zet error_reporting(E_ALL) eens bovenaan je script

[ Voor 33% gewijzigd door XWB op 30-07-2006 15:29 ]

[ReverendBizarre]

Angelgamer schreef op zondag 30 juli 2006 @ 14:44:

PHP:

    if ($pos === true) {
    $resultaat_woord_check = "ja ";
    }

Waarom gebruik je eerst een boolean constant in je if-statement en dan vervolgens een string "ja" in de regel daaronder? Dat laatste is echt not-done. Ja/Nee waarden sla je op als boolean variabelen en niet als een string met "ja" of "nee" of iets dergelijks erin. Dat is enorm foutgevoellig. De test die je vervolgens op die variabele doet zou bijvoorbeeld ook gewoon true teruggeven als je er "nee" of "Hallo" in had gezet. Het is niet een erg logische constructie. Maak een variabele en zet die op false voordat je de loop ingaat, en zet hem binnen de loop op true als je iets gevonden hebt. En na de loop test je de waarde van deze variabele.

http://www.php.net/manual/en/language.types.boolean.php

[Tweaqer]

Ik had trouwens nog een foutje gemaakt, die substr_count is case-sensitive dus heb ik
$zoektocht = substr_count($bericht, $fwoorden["$x"]); veranderd naar $zoektocht = substr_count(strtolower($bericht), $fwoorden["$x"]);

edit en de "" weggehaald thanks hacku

[ Voor 15% gewijzigd door Tweaqer op 30-07-2006 15:37 ]

[XWB]

Nu we toch bezig ziijn, een array kan je simpelweg met foreach() aflopen. Dan heb je $x en $y niet nodig. Voor ja/nee gebruiken we beter een bool:

PHP:

$fwoorden = array(
1 => "woord1",
2 => "woord2",
3 => "woord3",
4 => "woord4",
5 => "woord5"
);

$resultaat_woord_check = false;
$bericht = "woord1";

foreach ($fwoorden as $key => $value) 
{
    $zoektocht = substr_count($bericht, $value[$key]);

    if ($zoektocht > 0) 
    {
        $resultaat_woord_check = true;
        break;
    } 
}

if ($resultaat_woord_check === true) 
{
    echo "Het gebruik van bepaalde woorden in uw bericht is niet toegestaan";
}

[Tweaqer]

Hacku schreef op zondag 30 juli 2006 @ 15:39:
Nu we toch bezig ziijn, een array kan je simpelweg met foreach() aflopen. Dan heb je $x en $y niet nodig. Voor ja/nee gebruiken we beter een bool:

Was een goede tip om eventjes error_reporting(E_ALL) te adden aan mijn eerder gemaakte code Trouwens deze code is inderdaad een stuk schoner, overzichtelijker en logischer! Thanks

[ Voor 3% gewijzigd door Tweaqer op 30-07-2006 15:49 ]

[Noork]

Ik heb het op deze manier gedaan voor mijn weblog en gastenboek:

PHP:

// check verboden woorden
$berichtcheck=strtolower($bericht);
$dezedoennietmee = "(casino|viagra|baccarat|slots|gambling|medication|rmine|phente|phentermine|dirare|yellow|online|yellowpages|levitra|cialis|centralbest|lmb.dp.ua|halpinos|pokerpacific|bible|poker|wrkplace|hold-em|croupier|roulette|busntech|ringtone)"; 

if (eregi($dezedoennietmee,$berichtcheck)) { echo "error"; exit(); }

De foutafhandeling is wel erg banaal, maar het gaat hier toch om spammers.

[ Voor 3% gewijzigd door Noork op 30-07-2006 16:57 ]

[Tweaqer]

Ik heb trouwens nog wel een probleem met de code ;

Warning: substr_count() [function.substr-count]: Empty substring. in /var/www/hosts/bericht.php on line 14
Bij $zoektocht = substr_count($bericht, $value[$key]);
Hoe moet dat oplossen? Ik gebruik de code zoals Hacku die net gepost heeft.

[ Voor 11% gewijzigd door Tweaqer op 30-07-2006 16:37 ]

[Icekiller2k6]

Angelgamer schreef op zondag 30 juli 2006 @ 16:36:
Ik heb trouwens nog wel een probleem met de code ;

Warning: substr_count() [function.substr-count]: Empty substring. in /var/www/hosts/bericht.php on line 14
Bij $zoektocht = substr_count($bericht, $value[$key]);
Hoe moet dat oplossen? Ik gebruik de code zoals Hacku die net gepost heeft.

je kunt de warning onderdrukken door @ voor te voegen.. of bedoel je een _echte_ oplossing

[Tweaqer]

Icekiller2k6 schreef op zondag 30 juli 2006 @ 16:40:
[...]


je kunt de warning onderdrukken door @ voor te voegen.. of bedoel je een _echte_ oplossing

hehe ten minste 'n beetje echt

[Michali]

$value[$key] moet gewoon $value zijn (of $fwoorden[$key], maar dat is minder imo). Het $key => gedeelte in de foreach kan er ook helemaal uit trouwens.

En nog iets, array items worden automagisch geindiceerd, dus het is een beetje overbodig om ze zelf zo een nummertje te geven.

[ Voor 32% gewijzigd door Michali op 30-07-2006 16:46 ]

[Mr. Bondt]

PHP:

foreach ($fwoorden as $key => $value) 
{
    $zoektocht = substr_count($bericht, $value[$key]);

    if ($zoektocht > 0) 
    {
        $resultaat_woord_check = true;
        break;
    } 
}

Gaat ook echt werken, hè?
Dit is beter:

PHP:

foreach ($fwoorden as $value) 
{
    $zoektocht = substr_count($bericht, $value);

    if ($zoektocht > 0) 
    {
        $resultaat_woord_check = true;
        break;
    } 
}

@hieronder: Dus. Ik gebruik nooit foreach

[ Voor 30% gewijzigd door Mr. Bondt op 30-07-2006 16:51 ]

[Michali]

Dat gaat ook niet werken. $key[$value] slaat nog op minder dan $value[$key]. Gewoon $value los, of $fwoorden[$key]. Ik zou die $key helemaal laten zitten, want direct met $value werken is veel makkelijker. $key => in de foreach kun je dan weghalen, omdat je die niet nodig hebt foreach ( $fwoorden as $value ) moet het dan worden, en dan $value gewoon los gebruiken. Wel een beetje erg basic dit hoor.

[ Voor 69% gewijzigd door Michali op 30-07-2006 16:49 ]

[XWB]

@Mr. Bondt: klopt, ik zat met een multi dim tabel in gedachten

[Tweaqer]

Nou het werkt inmiddels als een tiet!

[_Thanatos_]

Is substr_count niet een beetje overkill? Als ik 500x een vies woord in een tekst zet, gaat substr_count ze alle 500 opzoeken en uittellen. Terwijl 1 voorkomen van zo'n woord al voldoende is. strpos is dus voldoende, alleen moet je wel kijken wat de return value daarvan is, wanneer een dergelijk woord gevonden is.

[Scott]

Is dit een niet nog makkelijkere manier dan ?

PHP:

$fwoorden = array();

$pattern = implode('|', $fwoorden);

if (preg_match('/('.$pattern.')/i', $bericht)) {

$resultaat_woord_check = true;

}

echo (isset($resultaat_woord_check)) ? 'Goed' : 'Fout';

Of is een regular expression hier 'not-done' ?

[BalusC]

_Thanatos_ schreef op zondag 30 juli 2006 @ 22:29:
Is substr_count niet een beetje overkill? Als ik 500x een vies woord in een tekst zet, gaat substr_count ze alle 500 opzoeken en uittellen. Terwijl 1 voorkomen van zo'n woord al voldoende is. strpos is dus voldoende, alleen moet je wel kijken wat de return value daarvan is, wanneer een dergelijk woord gevonden is.

Ik zou eerder strstr() gebruiken.

PHP:

if (strstr($message, $vieswoord)) {
    echo 'geen vieze woorden gebruiken';
}

[mcdronkz]

BalusC schreef op zondag 30 juli 2006 @ 23:00:
[...]

Ik zou eerder strstr() gebruiken.

PHP:

if (strstr($message, $vieswoord)) {
    echo 'geen vieze woorden gebruiken';
}

stristr() lijkt me beter, anders pakt 'ie TeRiNGLijER niet hè

[Oid]

PHP:

function scheldwoorden($tekst) 
{
    //tekst uit elkaar halen
    $geschreven_tekst = explode(" ", $tekst);
    $qMGS = mysql_query("SELECT woord FROM mod_guestbook_scheldwoorden");

    $woorden = array();
    while($qMGSR = mysql_fetch_assoc($qMGS))
    {
        $woorden[] = $qMGSR['woord'];
    }

    // $woorden = array ("woord1","woord2","etc");
    // Aantal woorden tellen
    $aantal = count($woorden);
    $aantal_tekst = count($geschreven_tekst);
    // Array aflopen
    for ($woord=0;$woord<$aantal_tekst;$woord++)
    {
        for ($a=0;$a<$aantal;$a++)
        {
            if($woorden[$a] == $geschreven_tekst[$woord])
                return true;
            else
                return false;
        }
    }
}

PHP:

if( ( scheldwoorden($_POST['onderwerp'] ) ) 
                OR ( scheldwoorden($_POST['naam'] ) ) 
                OR ( scheldwoorden($_POST['email'] ) )
                OR ( scheldwoorden($_POST['locatie'] ) )
                OR ( scheldwoorden($_POST['bericht'] ) ))
            {
                echo "Uw bericht bevat niet toegestane woorden";

Ik heb dit, of het ook werkt op kleine letters en HOOFDLETTERS weet ik niet, gaat me meer om het beetje controle en als het dan maf word, gewoon verwijderen van bericht en Ban

[Michali]

Sorry hoor, maar die code slaat nergens op. Ga die loop eens in gedachten doorlopen. Dan kom je er achter dan in de eerste iteratie van beide loops je gelijk een waarde returned. Oftewel, je loopt helemaal niet. Hoe dan ook, als je daar achter bent is het ook goed te bedenken dat dit behoorlijk inefficient is en je beter met stristr oid kan werken.