Palindrom: Problem beim entfernen von Sonderzeichen

[Genmutant]

Benutz zwei Zähler. Einmal deinen counter, der jedesmal hochgezählt wird wenn du ein Zeichen aus dem input[] liest. Und dann noch einen, der nur hochzählt, wenn du was in workingCopy schreibst. Damit kannst du dann lückenlos nach workingCopy schreiben. Am Ende wenn du mit dem schreiben fertig bist, packst du nach dem letzten Buchstaben noch eine 0 in workingCopy.

[loonie]

vielen dank für die schnelle Antwort, werde ich mich dann mal ans "basteln" machen und hoffen das es klappt^^

mfg

kannst du mir noch erklären warum ich zwei zähler brauche weil eigentlich haben die beide dann doch immer den gleichen Wert oder haben ich deine Idee nicht richtig verstanden.

und sorry bin noch ein noob in ANSI C

[;fricky]

loonie schrieb:

b) alle Zeichen außer Buchstaben aus dem Text entfernt,

dafür gibts: http://www.cplusplus.com/reference/clibrary/cctype/isalpha/

[Genmutant]

Das benutzt er doch sogar schon

[;fricky]

Genmutant schrieb:

Das benutzt er doch sogar schon

ok, ich war wieder zu schnell, hab den code nicht beachtet.

[Genmutant]

loonie schrieb:

kannst du mir noch erklären warum ich zwei zähler brauche weil eigentlich haben die beide dann doch immer den gleichen Wert

Also guck, wir nehmen mal als Beispiel des input "ab12cd".
Am Anfang sind die beiden Zähler, counter und counter_buchstaben nennen wir sie mal, beide gleich groß, weil du ja beide hochzählst, wenn du schreibst.
dann schauen deine beiden arrays so aus bis zu dem Wert vom counter:
input: ab
workingCopy: ab
so, jetzt kommen deine Zahlen. Da zählt der normale counter hoch. Wenn du da jetzt einfach nur nicht schreiben würdest, hättest du sowas:
input: ab12
workingCopy: ab[müll][müll]
Das würde dazu führen dass du am Ende, wenn du das auf die gleiche Stelle kopierst, sowas hast:
workingCopy: ab[müll][müll]cd

Jetzt machst du das aber mit 2 Zählern. Der counter_buchstaben wird nur hochgezählt, wenn du auch wirklich was nach workingCopy schreibst.
D.h. die Zuordnung wird: workingCopy[counter_buchstaben]=input[counter];

Drum hast du dann am Ende:
input(bis counter): ab12cd
workingCopy(bis counter_buchstaben): abcd

Verstanden?

[;fricky]

oder mach dir doch 'ne funktion zum rausfiltern von nicht-buchstaben. beispiel:

void letters_only (char *str)
{
    char *p = str;
    while (*str)    // string abklappern bis ende
    {
        if (isalpha(*str)) // buchstabe?
        {
            *p++ = *str;  // ja, dann übernehmen  
        }
        str++;            // naechstes zeichen
    }
    *p = 0;               // mit 0 abschliessen
}

void main()
{
    char test[] = "1234ha77.llo88 du da){";
    letters_only (test);
    puts (test);
}

[Genmutant]

Schöne Sache, sogar ohne zusätzliches Array

[;fricky]

Genmutant schrieb:

Schöne Sache, sogar ohne zusätzliches Array

danke, bin nun mal ein grosser freund von sparmassnahmen. naja, aber sollte ein zweites array nötig sein, lässt sich die funktion auch ganz leicht umbauen.

[SprechenderCode]

;fricky schrieb:

char *p = str;
    while (*str)    // string abklappern bis ende
    {
        if (isalpha(*str)) // buchstabe?
        {
            *p++ = *str;  // ja, dann übernehmen  
        }
        str++;            // naechstes zeichen
    }
    *p = 0;               // mit 0 abschliessen

[loonie]

jetzt hab ich kapiert warum zwei zaehler, besten dank dür die Erklärung Genmutant!
hatte jetzt noch keine Zeit es umzuschreiben.

und danke für die "Teilmusterlösung" aber werde mein Programm umschreiben... da steckt dann doch der ergeiz hinter, es zum lauifen zum bringen.

mfg

[loonie]

so hab das Programm jetzt fertig:

#include<stdio.h>

#include<string.h>

#include<ctype.h>

int main(void){

        int inputLenght = 0, counter = 0, counterWrite = 0 ;

	char input[255],  workingCopy[255], outputPalindrom[255], output[255];

	printf("Palindrom\n");

	printf("Geben Sie einen Text ein: ");

	fgets(input, sizeof(input) , stdin);

        inputLenght = strlen(input);

	for(counter=0;counter<inputLenght;counter++)

        {

          if (isalpha (input[counter]))

          {
	      	workingCopy[counterWrite]=input[counter];

		counterWrite++;

          }

        }

	workingCopy[counterWrite] = '\0';

	inputLenght = strlen(workingCopy);

        for(counter = 0; counter<inputLenght; counter++){
          if(islower(workingCopy[counter])){

            workingCopy[counter] = toupper(workingCopy[counter]);

          }
	  output[counter] = workingCopy[counter];
        }

	output[counterWrite] = '\0';

	workingCopy[counterWrite] = '\0';

	for(counter = 0; counter<inputLenght; counter++){

	  outputPalindrom[counter] = workingCopy[inputLenght-counter-1];

        }
        outputPalindrom[counter] = '\0';

        printf("\n");
        printf("Ihre Eingabe nachdem entfernen aller Zeichen ausser Buchstaben ist:\n");
        printf("%s\n", output);
        printf("\n");
        printf("Die neue laenge Ihrer Eingabe ist:\n");
        printf("%d\n", inputLenght);
        printf("\n");
        printf("Die Ausgabe in umgedrehter Reinfolge ist:\n");
        printf("%s\n", outputPalindrom);
        printf("\n");

	if(strncmp(output, outputPalindrom, inputLenght)==0)
        {
          printf("Bei der Eingabe handelt es sich um ein Palindrom!\n"); 
        }
        else
        {
          printf("Bei der Eingabe handelt es sich um kein Palindrom!\n");
        }

	printf("\n");

  	return 0;

}

und ein Dankeschön für die Hilfe

hab doch noch eine Frage und zwar bekomme ich wenn ich mit der Option -O2 (gcc -O2 Palindrom.c) compiliere bekomme ich diesen Fehler:
"Palindrom.c:25: warning: ignoring return value of ‘fgets’, declared with attribute warn_unused_result"