¿Por qué este código para modificar una cadena no funciona?

Question

Con cadenas estilo c, ¿cómo se asigna un carácter a una dirección de memoria a la que apunta un puntero de carácter? Por ejemplo, en el ejemplo siguiente, quiero cambiar num a "123456", así que intenté establecer p en el dígito donde está ubicado '0' e intento sobrescribirlo con '4'. Gracias.

#include <stdio.h> 
#include <stdlib.h> 

int main() 
{ 
    char* num = (char*)malloc(100); 
    char* p = num; 

    num = "123056"; 

    p = p+3; //set pointer to where '4' should be 
    p = '4'; 

    printf("%s\n", num); 

    return 0; 
} 

Answer 1

Ese código no funcionará, simplemente porque la línea:

num = "123056"; 

cambia num a apuntar hacia afuera de la memoria asignada (y sigue siendo p que apunta a la memoria de modo que ya no son la misma ubicación) a lo que probablemente sea la memoria de solo lectura. No está permitido cambiar la memoria que pertenece a los literales de cadena, es un comportamiento indefinido.

Es necesario lo siguiente:

#include <stdio.h> 
#include <stdlib.h> 
int main (void) { 
    char *num = malloc (100); // do not cast malloc return value. 
    char *p = num; 

    strcpy (num, "123056"); // populate existing block with string. 

    p = p + 3;    // set pointer to where '0' is. 
    *p = '4';     // and change it to '4'. 

    printf ("%s\n", num); // output it. 

    return 0; 
} 

Answer 2

Solo use *p = '4'; ...!

Answer 3

En primer lugar, cuando lo hace:

num = "123056"; 

No está copiando la cadena "123056" a la zona del montón asignado por malloc(). En C, la asignación de un puntero char * un valor literal de cadena es equivalente a establecer como una constante - es decir, idéntica a:

char str[] = "123056"; 

Por lo tanto, lo que acaba has logrado no has abandonado su única referencia a la Área de almacenamiento de 100 bytes asignada por malloc(), por lo que su código posterior no imprime el valor correcto; 'p' todavía apunta al área de pila asignada por malloc() (ya que num lo señaló en el momento de la asignación), pero num ya no lo hace.

Supongo que en realidad tenía la intención de hacer era copiar la cadena "123056" en esa área del montón. He aquí cómo hacerlo:

strcpy(num, "123056"); 

Aunque, esto es mejor práctica para una variedad de razones:

strncpy(num, "123056", 100 - 1); /* leave room for \0 (null) terminator */ 

Si acababa de hacer:

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

int  main(void) { 
     char *num = malloc(100); 
     char *p = num; 

     strncpy(num, "123056", 100 - 1); 

     p = p + 3; 
     *p = '4'; 

     printf("%s\n", num); 

     return 0; 
} 

Usted habría conseguido la resultado correcto:

123456 

Usted puede contraer esta operación:

p = p + 3; 
*p = '4'; 

... y evitar la iteración del puntero, por deferencing de la siguiente manera:

*(p + 3) = '4'; 

Algunas otras notas:

• Aunque estilística común práctica, arrojar el valor de retorno de malloc() a (char *) es innecesario. La conversión y alineación del tipo void * está garantizada por el lenguaje C.

• SIEMPRE verifique el valor de retorno de malloc(). Será NULO si la asignación del montón falló (es decir, no tiene memoria) y en ese punto su programa debería salir.

• Dependiendo de la implementación, el área de memoria asignada por malloc() puede contener basura obsoleta en ciertas situaciones.Siempre es una buena idea para poner a cero a cabo después de la asignación:

  memset(num, 0, 100); 
  

• nunca se olvide de free() su montón! En este caso, el programa se cerrará y el sistema operativo limpiará su basura, pero si no adquiere el hábito, tendrá pérdidas de memoria en muy poco tiempo.

Por lo tanto, aquí está la "mejor práctica" versión:

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

int  main(void) { 
     char *num, *p; 

     /* 
     * Don't take 1-byte chars for granted - good habit to get into. 
     */ 

     num = malloc(sizeof(char) * 100); 

     if(num == NULL) 
       exit(1); 

     memset(num, 0, sizeof(char) * 100); 

     p = num; 

     strncpy(num, "123056", 100 - 1); 

     *(p + 3) = '4'; 

     printf("%s\n", num); 

     free(num); 

     return 0; 
} 

Answer 4

bueno, ya sabes p es un tipo de puntero. Almacena la dirección de char '0'. Si asigna p el valor de '4'. Tomará '4' como la dirección. Sin embargo, la dirección '4' es ilegal. Puede usar el operador '*' para obtener el valor de la dirección almacenada en p almacenada.

Answer 5

Además del problema * p que otros han señalado, también tiene problemas de uso de memoria. Tienes un buffer de 100 bytes, con contenido desconocido. Tiene otro búfer de 7 bytes, que contiene la cadena "123056" y un terminador nulo. Estás haciendo esto:

1. num se establece en el punto en el búfer de 100 bytes
2. p se establece para que apunte a num; es decir, apunta a la memoria intermedia de 100 bytes
3. que restablece num para apuntar a la memoria intermedia de 7 bytes; p todavía está apuntando a la memoria intermedia 100 bytes
4. Utilice p para modificar el buffer de 100 bytes
5. continuación, utiliza num para imprimir el buffer de 7 bytes

Por lo que no está imprimiendo el mismo tampón que estás modificando