Buchstabenersetzung aus Datei

[Julian (jollepe)]

Hallo,

ich habe ein Problem mit einer Aufgabe.
Ich möchte ein Programm zeilenweise einlesen.
Dann wird jede Zeile in ein Array kopiert und anschließend
überprüft ob ein Arrayelement mit einem Char vergleichen.
Wird an einer Stelle eine Übereinstimmung gefunden, so wird diese Stelle
gegen einen anderen Buchstaben ausgetauscht.
Anschließend werden alte und neue Zeile mittels printf ausgegeben und
die Gesamtanzahl der Buchstabenersetzungen ausgegeben.

Ich weiß leider nicht wo mein Fehler liegt, da die Textersetzung
nicht funktioniert.

Hier mein Programmcode:

#include <stdio.h>
#include <stdlib.h>
int main ()

{
    FILE *input;
    int     n   = 0;    // Aufzählungsvaribale
    char    bs1 = 0;    // Buchstabe zu ersetzen
    char    bs2 = 0;    // Buchstabe ersetzt
    int     i   = 0;    // Laufvariable (for)-Schleife
    int     ers = 0;    // Gesamt an Ersetzungen
    char    feld [80];  // Feld aus Datei


    printf  ("Buchstabe 1: ");
    scanf   ("%s", &bs1);
    printf  ("Buchstabe 1: %c \n",bs1);

    printf  ("Buchstabe 2: ");
    scanf   ("%s", &bs2);

    printf  ("Buchstabe 2: %c \n\n\n",bs2);


    // Öffnet die Quelldatei
    input = fopen( "beispieltext.txt", "r" );

        // Überprüfen ob Datei gelesen wurde
        if (input == NULL)
        {
            printf ("Quelldatei nicht lesbar.\n");
            return -1;
        }

        // Datei wurde erfolgreich gelesen -> Zeile in String speichern
        else
        {
            // Kopiert jede Zeile in das feld
            while ( fgets (feld , 80 , input) != NULL )
                {
                    n++;
                    printf ("%d. Zeile Original:\t %s",n, feld);

                    // Das Feld wird mit den Buchstaben verglichen und ggf ersetzt
                    for (i=0; i>=80; i++)
                        {
                            if (feld[i]==bs1)
                                {
                                feld[i]=bs2;
                                ers++;
                                }
                            else
                                {
                                continue;
                                }
                        }
                    printf ("%d. Zeile Neu:\t\t %s",n, feld);
                    printf ("--------------------\n\n\n");
                }
        printf ("Insgesamt %dx Ersetzungen!\n",ers);
        fclose (input);
        }
    return 0;
}

[Peter II (Gast)]

> for (i=0; i>=80; i++)

hier würde ich noch mal drüber nachdenken.

[Michael S. (captain-stone)]

> Ich weiß leider nicht wo mein Fehler liegt, da die Textersetzung
> nicht funktioniert.

Das ist ein sehr großer Oberbegriff "nicht funktioniert"


>                     for (i=0; i>=80; i++)
>                         {
>                             if (feld[i]==bs1)
>                                 {
>                                 feld[i]=bs2;
>                                 ers++;
>                                 }
>                             else
>                                 {
>                                 continue;
>                                 }
>                         }

Ich nehme an, Du hast an die for Schleife Deinen Breakpoint gesetzt, 
oder? Was passiert danach in den Variablen? Was zeigt die Variable i 
denn an, wenn sie hochzählt? Kommen gleiche Zeichen zustande bei bs1 und 
bs2?

Sag mal genau, an welcher Stelle es im SingleStep nicht funktioniert.

[Julian (jollepe)]

Michael Steinbauer schrieb:
> Ich nehme an, Du hast an die for Schleife Deinen Breakpoint gesetzt,
> oder? Was passiert danach in den Variablen? Was zeigt die Variable i
> denn an, wenn sie hochzählt? Kommen gleiche Zeichen zustande bei bs1 und
> bs2?

Die for-Schleife soll am Ende des Feldes also i==79 aufhören.
Demnach (sollte) die Funktion die Felder von 0-79 durchlaufen und 
jeweils das Element mit dem Char-Wert (ein Buchstabe) vergleichen.

Jedoch sieht es in der Praxis so aus, dass anscheinend die Ersetzung 
nicht durchläuft, da das feld == feld' ist.

[Peter II (Gast)]

Julian R. schrieb:
> Jedoch sieht es in der Praxis so aus, dass anscheinend die Ersetzung
> nicht durchläuft, da das feld == feld' ist.

nein wie schon oben geschrieben - der fehler ist hier:

> for (i=0; i>=80; i++)

[Julian (jollepe)]

Peter II schrieb:
> Julian R. schrieb:
>> Jedoch sieht es in der Praxis so aus, dass anscheinend die Ersetzung
>> nicht durchläuft, da das feld == feld' ist.
>
> nein wie schon oben geschrieben - der fehler ist hier:
>
>> for (i=0; i>=80; i++)

Ich finde da leider nicht meinen Fehler.
Habe es mit i==79 probiert, jedoch die gleiche Problematik

[Peter II (Gast)]

Julian R. schrieb:
> Ich finde da leider nicht meinen Fehler.

dann schau noch mal genau hin wie eine for schleife funktioniert.

[Julian (jollepe)]

Peter II schrieb:
> Julian R. schrieb:
>> Ich finde da leider nicht meinen Fehler.
>
> dann schau noch mal genau hin wie eine for schleife funktioniert.

Asche über mein Haupt. Manchmal sieht man den Wald vor lauter Bäumen 
nicht.

Es wird nun die Gesamtanzahl der Ersetzungen angezeigt, jedoch wird das 
feld' nicht verändert. Zudem erhalte ich wahrscheinlich 
Speicher-garbage...

[Peter II (Gast)]

Julian R. schrieb:
> Es wird nun die Gesamtanzahl der Ersetzungen angezeigt, jedoch wird das
> feld' nicht verändert. Zudem erhalte ich wahrscheinlich
> Speicher-garbage...

du überschreibst dir ja auch fleißig den speicher

 char    bs2 = 0;
 scanf   ("%s", &bs1);

%s steht für string, du willst aber ein zeichen einlesen.

[Julian (jollepe)]

Peter II schrieb:
> Julian R. schrieb:
>> Es wird nun die Gesamtanzahl der Ersetzungen angezeigt, jedoch wird das
>> feld' nicht verändert. Zudem erhalte ich wahrscheinlich
>> Speicher-garbage...
>
> du überschreibst dir ja auch fleißig den speicher
>
>  char    bs2 = 0;
>  scanf   ("%s", &bs1);
>
> %s steht für string, du willst aber ein zeichen einlesen.

Die Problematik ist dann für ein %c, dass die erste Scanf-Abfrage 
durchgeht und die zweite komplett überspringt und gleich in den 
Algorithmus geht

[Dirk B. (dirkb2)]

Das einlesen der Buchstaben ist auch verkehrt.
%s dient zum einlesen von C-Strings. Und die sind immer nullterminiert.
Darum brauchst du Platz für zwei zeichen.

Der richtige Formatspecifier dafür ist %c.
Und damit du beim zweiten Buchstaben nicht das '\n' von der Entertaste 
einliest, gehört noch ein Leerzeichen davor:

scanf(" %c", &bs1);
       ^ wichtig

[Julian (jollepe)]

Vielen Dank an Alle!
Jetzt läuft es endlich.

[Dirk B. (dirkb2)]

Noch eine Frage:
Warum willst du die Datei zeilenweise einlesen?

Zeichenweise mit fgetc wäre weniger aufwendig.

[Julian (jollepe)]

Dirk B. schrieb:
> Noch eine Frage:
> Warum willst du die Datei zeilenweise einlesen?
>
> Zeichenweise mit fgetc wäre weniger aufwendig.

Aufgabenstellung des Dozenten ;-)

[Dirk B. (dirkb2)]

Ach, noch etwas.

Bis jetzt ging es ja nur um das "Vorzeichen" von dem Vergleich (< statt 
>=).

Allerdings ist die Bedingung hier auch verkehrt. Wenn die Zeile nur ein 
Zeichen enthält, prüfst du trotzdem alle 80. Auch den Müll der vorher 
drin stand.

Du liest einen C-String ein und diese sind ja nullterminiert.
Darum:

for (i=0; feld[i]!='\0'; i++)