Wie iteriere ich über eine Zeichenfolge in C?

Momentan versuche ich das:

#include <stdio.h>

int main(int argc, char *argv[]) {

    if (argc != 3) {

        printf("Usage: %s %s sourcecode input", argv[0], argv[1]);
    }
    else {
        char source[] = "This is an example.";
        int i;

        for (i = 0; i < sizeof(source); i++) {

            printf("%c", source[i]);
        }
    }

    getchar();

    return 0;
}

Auch das geht NICHT:

char *source = "This is an example.";
int i;

for (i = 0; i < strlen(source); i++){

    printf("%c", source[i]);
}

Ich bekomme den Fehler

Unbehandelte Ausnahme bei 0x5bf714cf (msvcr100d.dll) in Test.exe: 0xC0000005: Zugriffsverletzung beim Lesen an Position 0x00000054.

(frei übersetzt aus dem Deutschen)

Also, was ist falsch an meinem Code?

Sie wollen:

for (i = 0; i < strlen(source); i++) {

sizeof gibt Ihnen die Größe des Zeigers, nicht die Zeichenfolge. Es hätte jedoch funktioniert, wenn Sie den Zeiger als Array deklariert hätten:

char source[] = "This is an example.";

aber wenn Sie das Array an function übergeben, zerfällt auch das zu einem Zeiger. Für Saiten verwendest du am besten immer strlen. Beachten Sie, was andere über das Ändern von printf zur Verwendung von %c gesagt haben. Und auch unter Berücksichtigung von mmyers Kommentaren zur Effizienz wäre es besser, den Aufruf von strlen aus der Schleife zu verschieben:

int len = strlen(source);
for (i = 0; i < len; i++) {

oder die Schleife umschreiben:

for (i = 0; source[i] != 0; i++) {

Eine gängige Redewendung ist:

char* c = source;
while (*c) putchar(*c++);

Ein paar Anmerkungen:

• In C sind Strings nullterminiert. Sie iterieren, während das gelesene Zeichen nicht das Nullzeichen ist.
• *c++ Schritte c und gibt die dereferenzierte zurück alt Wert von c.
• printf("%s") gibt eine nullterminierte Zeichenfolge aus, kein Zeichen. Dies ist die Ursache Ihrer Zugriffsverletzung.

Anstatt wie oben vorgeschlagen strlen zu verwenden, können Sie einfach nach dem NULL-Zeichen suchen:

#include <stdio.h>

int main(int argc, char *argv[])
{
    const char *const pszSource = "This is an example.";
    const char *pszChar = pszSource;

    while (pszChar != NULL && *pszChar != '\0')
    {
        printf("%s", *pszChar);
        ++pszChar;
    }

    getchar();

    return 0;
}

Ein optimierter Ansatz:

for (char character = *string; character != '\0'; character = *++string)
{
    putchar(character); // Do something with character.
}

Die meisten C-Strings sind nullterminiert, was bedeutet, dass sobald das Zeichen zu a wird '\0' die Schleife sollte aufhören. Das *++string bewegt den Zeiger um ein Byte, dereferenziert ihn dann und die Schleife wiederholt sich.

Der Grund, warum dies effizienter ist als strlen() liegt daran, dass strlen bereits die Saite durchläuft, um die Länge zu finden, sodass Sie effektiv zweimal (einmal mehr als nötig) mit loopen würden strlen().

sizeof(source) gibt die Anzahl der vom Zeiger benötigten Bytes zurück char*. Sie sollten es durch ersetzen strlen(source) Dies ist die Länge der Zeichenfolge, die Sie anzeigen möchten.

Außerdem sollten Sie wahrscheinlich ersetzen printf("%s",source[i]) mit printf("%c",source[i]) da Sie einen Charakter anzeigen.

1. sizeof() enthält das abschließende Nullzeichen. Sie sollten strlen() verwenden (aber den Aufruf außerhalb der Schleife platzieren und in einer Variablen speichern), aber das ist wahrscheinlich nicht der Grund für die Ausnahme.
2. Sie sollten in printf “%c” und nicht “%s” verwenden – Sie drucken ein Zeichen, keine Zeichenfolge.

Das sollte funktionieren

 #include <stdio.h>
 #include <string.h>

 int main(int argc, char *argv[]){

    char *source = "This is an example.";
    int length = (int)strlen(source); //sizeof(source)=sizeof(char *) = 4 on a 32 bit implementation
    for (int i = 0; i < length; i++) 
    {

       printf("%c", source[i]);

    }


 }