¿Cómo iterar sobre una cadena en C?

Question

En este momento estoy tratando esto:

#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; 
} 

Esto también no funciona:

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

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

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

consigo el error

excepción no controlada en 0x5bf714cf (msvcr100d.dll) en Test.exe: 0xC0000005: infracción de acceso al leer en la posición 0x00000054.

(vagamente traducido del alemán)

Entonces, ¿qué hay de malo en mi código?

Answer 1

que desee:

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

sizeof le da el tamaño del puntero, no la cadena. Sin embargo, habría funcionado si se hubiera declarado el puntero en forma de matriz:

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

pero si se pasa la matriz a funcionar, eso también decaerá a un puntero. Para cuerdas, es mejor usar siempre strlen. Y tenga en cuenta lo que otros han dicho sobre el cambio de printf para usar% c. Y también, teniendo mmyers comentarios sobre la eficiencia en cuenta, que sería mejor para mover la llamada a strlen fuera del circuito:

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

o reescribir el bucle:

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

Answer 2

Simplemente cambie sizeof con strlen.

De esta manera:

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

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

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

} 

Answer 3

sizeof(source) devuelve el número de bytes requeridos por el puntero char*. Debe reemplazarlo por strlen(source), que será la longitud de la cadena que está tratando de mostrar.

Además, probablemente debería reemplazar printf("%s",source[i]) con printf("%c",source[i]) ya que está mostrando un personaje.

Answer 4

Reemplace sizeof con strlen y debería funcionar.

Answer 5

Necesita un puntero al primer carácter para tener una cadena ANSI.

printf("%s", source + i); 

hará el trabajo

Además, por supuesto que debería haber significado strlen(source), no sizeof(source).

Answer 6

Esto debería funcionar

#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]); 

    } 


} 

Answer 7

• sizeof(source) está volviendo a que el tamaño de un char*, no la longitud de la cadena.Debería usar strlen(source), y debería moverlo fuera del ciclo, o de lo contrario, volverá a calcular el tamaño de la cadena en cada ciclo.
• Al imprimir con el modificador de formato %s, printf está buscando char*, pero en realidad está pasando char. Debe usar el modificador %c.

Answer 8

1. sizeof() incluye el carácter nulo de terminación. Debería usar strlen() (pero coloque la llamada fuera del ciclo y guárdela en una variable), pero probablemente eso no sea lo que está causando la excepción.
2. debe usar "% c", no "% s" en printf; está imprimiendo un carácter, no una cadena.

Answer 9

Un idioma común es:

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

Unas pocas notas:

• En C, las cadenas son null-terminated. Usted itera mientras que el carácter leído no es el carácter nulo.
• *c++ incrementa c y devuelve el viejo sin referencia valor de c.
• printf("%s") imprime una cadena terminada en nulo, no es un char. Esta es la causa de su violación de acceso.

Answer 10

En lugar de utilización STRLEN como se sugirió anteriormente, sólo se puede comprobar el carácter nulo:

#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; 
} 

Answer 11

sizeof(source) vuelve sizeof un puntero como fuente es declarado como char *. La forma correcta de usarlo es strlen(source).

siguiente:

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

espera cadena. es decir,% s espera una cadena pero está iterando en un bucle para imprimir cada carácter. Por lo tanto, use% c.

Sin embargo, su forma de acceder (iterar) una cadena utilizando el índice i es correcta y, por lo tanto, no hay otros problemas en ella.