Duda en pequeño ejercicio

[IVICP3]

Tengo un ejercicio que le paso una cadena y me la edita a capicua, un ejemplo:

Le paso la palabra jamón, pues me saldria jamonnomaj.

Código C:

#include <stdio.h>
#include <string.h>
 
void main(){
    char cadena[20];
    int i, x, cont=0, pepe=0;
 
    printf("Introduce una palabra: ");
    gets(cadena);
 
    for(i=0;pepe!=5;i++){
        cont=i;
        if(cadena[i]==NULL){
            for(x=cont;x!=0;x--,i++){
                cadena[i]=cadena[x-1];
                pepe=5;
            }
        }
    }
 
    printf("CADENA %s", cadena);
 
 
 
    fflush(stdin);
    getchar();
 
 
 
}
 

No sabia como salir del primer for, he puesto eso de pepe. Esta bien?

Porque si que me sale, pero el resultado del programa no es jamonnomja, es jamonnomja con signos raros. Me hace el capicua bien, pero las celdas restantes del char me salen rellenadas con signos.

Que esta mal?


Un saludo.

[Instru]

Que relajo.

Incluso es difícil seguir el código. Antes de poder llegar a la solucion de tu duda hay que limpiar el código:

Primero, realmente no necesitas la variable cont. Ya tienes i:

Código C++:

for (i = 0; pepe != 5; i++)
    {
        if (cadena[i] == NULL)
        {
            for (x = i; x != 0; x--, i++)
            {
                cadena[i] = cadena[x - 1];
                pepe = 5;
            }
        }
    }
 

Si te das cuenta, no necesitas 2 fors anidados. Puedes sacar el interior y todo se simplifica. Ya no necesitas a pepe:

Código C++:

printf("Introduce una palabra: ");
    gets(cadena);
 
    for (i = 0; ; i++)
    {
        if (cadena[i] == NULL)
        {
            break;
        }
    }
 
    for (x = i; x != 0; x--, i++)
    {
        cadena[i] = cadena[x - 1];
    }
 
    printf("CADENA %s", cadena);
 

Ya se ve mucho mas simple no?

Aun se puede simplificar mas. Podemos unir el if con el for:

Código C++:

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

Y como veo que ya incluyes string.h, entonces puedes puedes sustituir por:

Código C++:

i=strlen(cadena);
 

Ahora, en el segundo for: Tienes un pequeño detalle:

Código C++:

for (x = i; x != 0; x--, i++)
    {
        cadena[i] = cadena[x - 1];
    }
 

La posición 0 también se debe incluir por lo que mejor usa:

Código C++:

for (x = i; x >=0; x--, i++)
    {
        cadena[i] = cadena[x - 1];
    }
 

Finalmente, para que no te salga basura, debes inicializar tu arreglo char con todos sus elementos a 0. Porque printf va a leer el arreglo hasta encontrar un 0. Entonces si no lo encuentra, se sigue y te imprime otras cosas.

Aun no es un código limpio y legible, pero esta mucho mejor que como estaba. No crees?.

El punto es que no necesitabas fors anidados.

Saludos

[IVICP3]

Gracias por la respuesta!!


He inicializado el char tal que asi

Código C:

char cadena[20] = "";
 

y me sigue saliendo basura, pero ahora solo me sale un signo, antes me salian hasta que se acababan los espacios del array (hasta que llegaba a 20 celdas) ahora solo me sale uno... que cosa mas rara , no?

[eferion]

Iniciado por IVICP3

He inicializado el char tal que asi

Código C:

char cadena[20] = "";
 

y me sigue saliendo basura, pero ahora solo me sale un signo, antes me salian hasta que se acababan los espacios del array (hasta que llegaba a 20 celdas) ahora solo me sale uno... que cosa mas rara , no?

Con eso no has inicializado el arreglo con todos los elementos a 0... únicamente has indicado que el primer elemento sea 0 mientras que el resto seguirán con la basura habitual.

Una forma de inicializar la memoria a 0 bastante limpia es usar, por ejemplo, memset:

Código C++:

memset(cadena,0,sizeof(cadena));
 

Otra opción para evitar errores es finalizar manualmente la cadena, añadiendo un '\0' después de finalizar la copia. Quizás algo tal que:

Código C++:

cadena[i*2]='\0';
 

Un saludo.

[aguml]

Además gets guarda el salto de línea así que en vez de comparar con 0 tendría que comparar con \n.
Sobre lo de setear una cadena, hace unos días vi por casualidad que en string.h existe strset que si no entendí mal servía para eso pero yo la probé y al final acabé usando memset porque no me funcionó. ¿strset no es para eso?

[eferion]

strset no te sirve. La razón es que esa función sustituye el contenido de la cadena por un carácter concreto hasta que se encuentra con un nulo. Esto es así porque esa función no conoce el tamaño del buffer.

Usar esa función en un string no inicializado o bien corrompe memoria al no encontrar un nulo antes de salirse del buffer, o bien no setea todo el buffer por encontrar un nulo antes de llegar al final.

Un saludo