4. Badwordlist ...

Badwordlist ...

  • D

    Hallo,
    also folgendes ich habe meinen IRC Bot um eine Badwortliste erweiter ok das klappt auch fast einigstes Problem ich bräcute eine vollständige Badwordliste meine beinhaltet nur 4 Testwörter. 2tes Problem ich mäcute gerne das Casesensitive raus haben. Jemand eine Idee ?

    Hier der Code :

    FILE *datei;
		datei = fopen("badword.cfg","r");
		if(datei!=NULL)
		{
		char zeile[1024];
		while (!feof(datei))
		{
		fgets(zeile,1024,datei);

		if(strncmp(zeile,Nachricht,14) == 0) {

		sscanf(Absender1,"~%s",Absender1);
		printf("Kicke %s (BadWord) !\n",Absender1);
		strcpy(buf,"KICK ");
		AddToStringC(buf, Channel);
		AddToStringC(buf, " ");
		AddToStringC(buf, Absender1);
		AddToStringC(buf, " BadWord!");
		AddToStringC(buf, "\r\n");
		send(s,buf,strlen(buf),0);
		}
		}
		}
		else {
			printf("Fehler beim oeffnen der Datei!\n");
		}

    Optimierungsvorschläge sidn bei mir auch gerne wilkommen 😃

    Gruß

    0
  • M

    Mach alle Buchstaben klein/groß und vergleiche dann mit der Liste (da müssen sie natürlich auch so stehen). Optimierungsvorschläge gibts bei mir nur in C++ oder bei einer einmaligen Einrichtungsgebühr für nur 99,99€. Sie sparen 30% 😃

    0
  • I

    non case sensitive Vergleich:
    http://www.unet.univie.ac.at/aix/libs/basetrf2/strcmp.htm

    unter Win:
    http://msdn.microsoft.com/library/default.asp?url=/library/en-us/vclib/html/_crt__strnicmp.2c_._wcsnicmp.2c_._mbsnicmp.asp

    Was du mit den "nur 4 Wörtern" meinst ist mir nicht klar...

    0
  • ?

    Caseinsentives Vergleichen ist langsam, viel schneller gehts wenn du einfach alles beim Einlesen GROSSSCHREIBST.

    Also mit:

    #include<ctype.h>
void to_uppercase(char*begin,char*end){
  for(;begin!=end;++begin)
    *begin=tolower(*begin);
}

    Drüber gehst.

    0
  • I

    Irgendwer schrieb:

    Caseinsentives Vergleichen ist langsam, viel schneller gehts wenn du einfach alles beim Einlesen GROSSSCHREIBST.

    Also mit:

    #include<ctype.h>
void to_uppercase(char*begin,char*end){
  for(;begin!=end;++begin)
    *begin=tolower(*begin);
}

    Drüber gehst.

    Ich bezweifle, dass es bei einem IRC Bot auf Geschwindigkeit ankommt 😉

    0
  • D

    Das mit den 4 Wörtern war so gemeint das meine eigene BadWordliste zum Testen nur 4 Wörter beinhaltet daher brauche ich unbedingt eine vernünftige die mehr besitzt aber 5 sind auch noch zu wenig 😃
    Gruß

    0
  • I

    Dummie schrieb:

    Das mit den 4 Wörtern war so gemeint das meine eigene BadWordliste zum Testen nur 4 Wörter beinhaltet daher brauche ich unbedingt eine vernünftige die mehr besitzt aber 5 sind auch noch zu wenig 😃
    Gruß

    Dann schreib doch einfach eine längere... 🙄

    0
  • D

    Soviele fallen mir aber nicht ein, gibts da nichts fertiges ?

    Naja das mti case sensitive klappt jedenfalls

    Gruß

    Danke

    0
  • D

    Ich habe nciht bemerkt das ,dass ganze nur funktioniert wenn es am anfang steht wenn es mit in einem Satz steth klappt das ganze nicht mehr noch jemand ne Idee ? 🙄

    Gruß

    EDIT :

    if(strnicmp(zeile,Nachricht,14)) {
...
}
    0
  • H

    evtl. ist strstr eher für Dich geeignet.

    strstr sucht nach einem String innerhalb des strings

    strstr liefert einen Zeiger an die Stelle, an der der Anfang der Teilzeichenkette gefunden wurde oder NULL falls er nicht gefunden wurde.

    strstr(zeile, "Blödmann");
    0
  • D

    Ja das hab ich auch gedacht nur dann haben wir wieder das Problem ?

    Genau wir haben wieder das Problem mit dem Case Sensitive es msus eine andere möglichkeit geben die beides zusammen bietet

    Da brauche ich nun wirklich eure hifle

    0
  • ?

    Die Lösung hab ich doch bereits gepostet! Zuerst alles Uppercase (oder Lowercase wenn du willst) machen und dann casesenitiv vergleichen.

    @interpreter
    Trotzdem ist das kein Grund Geschindigkeit zu verschwenden, denn nach jeder Eingabe alles Uppercase zu machen ist ja nicht sonderlich schwierig oder (programierer)zeitraubend also ist es meiner Meinung wert.

    0
  • I

    Irgendwer schrieb:

    @interpreter
    Trotzdem ist das kein Grund Geschindigkeit zu verschwenden, denn nach jeder Eingabe alles Uppercase zu machen ist ja nicht sonderlich schwierig oder (programierer)zeitraubend also ist es meiner Meinung wert.

    Schön. Und warum umständlich wenns ne Funktion gibt, die das alles schon macht und effizient implementiert wurde?

    0
  • S

    interpreter schrieb:

    Schön. Und warum umständlich wenns ne Funktion gibt, die das alles schon macht und effizient implementiert wurde?

    Welche denn?
    stricmp ist kein Standard. Warum dass so ist, habe ich mich aber auch schon oft gefragt...

    0
  • D

    Selbst wenn ich strstr nehme und das Problem mit dem case sensitvie gelöst habe gibts immernoch ein Problem : SChreibt man z.B Ich habe gesterb einen Basch*
    gefangen dann wird man aus dem Channel ohne Grund geschmissen. Das macht mir zurzeit mehr Sorgen 😞

    Gruß

    EDIT :
    Mir ist aufgefallen das dies Forum das gleiche Problem hat ! 😮

    0
  • H

    Dann geh doch die Zeile Wort für Wort durch (' ' ist schließlich ein Trennzeichen zwischen 2 Worten).

    0
  • H

    Shade Of Mine schrieb:

    Welche denn?
    stricmp ist kein Standard

    strcmpi auch nicht? 😕

    /Edit: Tatsache... scheint so als ob es keinen cse-insesitive Compare gibt im Standard... Sachen gibt's...

    0
Anmelden zum Antworten