Eliminar caracteres pares o impares en C

[AlmaVagabunda]

Hola, de vuelta estoy aqui, a ver si alguien me puede ayudar.
Estoy haciendo un programa en C, para hacer diversas funciones con cadenas, una de ella es eliminar los caracteres pares o los impares, es decir:

Hola---> par: oa || impar: hl

He conseguido que me cambie los caracteres a eliminar, por "_", pero necesito que los elimine, y no deje espacio en blanco donde ha eliminado una letra.
He probado "\0", 0, -1, '', ' ', ... y alguna locura mas, pero sigo sin poder hacer nada.
En la cadena original pueden haber espacios, asi que no es eliminar todos los espacios que hayan.

Os dejo el codigo (que no se si sera el mejor) de la funcion en concreto, la funcion recibe la referencia de la cadena, y ya realiza todo el proceso.

Código C:

void EliminarParesImpares(char cadena1[]){
    int i,l,a;
    
    l=strlen(cadena1);
    printf("\nEliminar pares o impares <p/i>: ");
    a=getche();
    if (a=='p'){
        for(i=0;i<l;i++){
            cadena1[i+1]='_';
            i++;
        }
    }
    if (a=='i'){
        for(i=0;i<l;i++){
            i++;
            cadena1[i-1]='_';
        }
    }
    system("cls");
    return;
}
 

Muchas gracias de antemano

[sam90]

Decidi cambiarte entera tu funcion, porque lo que propones se puede llegar a complicar mas.

Código C++:

#include <stdio.h>
#include <stdlib.h>
char * EliminarParesImpares(char * cadena )
{
    char * nueva,a;
    int l=strlen(cadena), init,i,j;
 
    nueva = (char *) malloc (l * sizeof(char));
    printf("\nEliminar pares o impares <p/i>: ");
    a=getchar();
    if (a=='p') init = 0;
    else init = 1;
    j=0;
    for (i=init;i < l;  i = i +2)
        nueva[j++] = cadena[i];
    nueva[j] = '\0';
    return nueva;
}
 
int main()
{
    char * c = EliminarParesImpares("0123456789");
    printf("%s\n",c);
    return 0;
}
 

La idea es la siguiente, en vez de modificar esa cadena, creamos una nueva y vamos a ir copiando los elementos ahi.
Uso malloc para reservear memoria para la cadena. un char nueva[50]; me crearia problemas ya que al devolver la funcion esta deja de existir, ademas tendria una limitacion.

Luego lo que hago es recorrer la cadena de dos en dos, empezando por cero si es par o por 1 si es impar. Asi voy copiando cada elemento a la cadena nueva. Esta va aumentando, por supuesto, de uno en uno...

Supongo que hay muchas cosas que no vas a entender ahora... pero con el tiempo vas a encontrarle el sentido.

La funcion devuelve un puntero a la nueva cadena.

Cualquier cosas pregunta.

[AlmaVagabunda]

No consigo hacer que funcione, te muestro la funcion que llama a esta funcion:

Código C:

//Funcion para eliminar pares o impares
char * EliminarParesImpares(char * frase )
{
    char * nueva,a;
    int l=strlen(frase), init,i,j;
 
    nueva = (char *) malloc (l * sizeof(char));
    printf("\nEliminar pares o impares <p/i>: ");
    a=getchar();
    if (a=='p') init = 0;
    else init = 1;
    j=0;
    for (i=init;i < l;  i = i +2)
        nueva[j++] = frase[i];
    nueva[j] = '\0';
    return nueva;
}
------------------------------------------------------------------------------------------------
void MenuPrincipal (char frase[]){
    system("cls");
    char seleccion;
    printf("\nCadena introducida : %s\n\n",frase);
    do{
        printf("\nMENU:");
        printf("\n1) Pasar a mayusculas o a minusculas");
        printf("\n2) Cambiar caracteres aleatoriamente");
        printf("\n3) Invertir cadena");
        printf("\n4) Eliminar caracteres pares e impares");
        printf("\n5) Estadisticas");
        printf("\n6) Volver a introducir una cadena");
        printf("\n7) Salir");
        printf("\nSelecciona una opcion: ");
        fflush(stdin);
        seleccion=getche();
        printf("\n");
        switch(seleccion){
        case '1':
            MayusculasMinusculas(frase);
            printf("\n\nCadena Actual : %s\n\n",frase);
            break;
        case '2':
            aleatorio(frase);
            printf("\n\nCadena Actual : %s\n\n",frase);
            break;
        case '3':
            Invertir(frase);
            printf("\n\nCadena Actual : %s\n\n",frase);
            break;
        case '4':
            EliminarParesImpares(frase);
            printf("\n\nCadena Actual : %s\n\n",frase);
            break;
        case '5':
            Estadisticas(frase);
            break;
        case '6':
            repetir(frase);
            break;
        case '7':
            break;
        }
    }while(seleccion!='7');
    return;
}
 

No se como ni donde poner la parte del main (en mi codigo, funcion del menu principal)

He intentado poner lo de:

Código C:

char * c = EliminarParesImpares("0123456789");
 

Justo donde en el menu principal se llama a la funcion eliminar pares, pero logicamente me da fallos.
Tambien he probado a cambiar esa variable c, por el nombre de la cadena. Tambien cambiar lo del parentesis por el nombre de la cadena.

Creo que ando algo perdido jaja.

Como deberia hacerlo?

Muchas gracias sam90 ;)
Voy a tener que pagarte, eres como un profesor particular jajaja
un abrazo


Edit:
Seria posible mantener el codigo tal como lo tenia al principio?
Lo digo por que aun no consigo entender la instruccion malloc, y si enseño eso y me preguntan que es lo que hace y como funciona, no voy a saber responder.
Me gusta mucho tu solucion, supongo que fiunciona, tiene pocas lineas, y eso es importante, pero si se puede arreglar con el codigo que tenia al principio, mejor.

Dejo aqui el codigo desde que se pide una frase en adelante, asi se me entendera mejor:

Código C++:

#include<conio.h>
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
//Funcion para eliminar pares o impares
void EliminarPI(char frase[]){
    system("cls");
    int i,l,a;
 
    l=strlen(frase);
    printf("\nEliminar pares o impares <p/i>: ");
    a=getche();
    if (a=='p'){
        for(i=0;i<l;i++){
            frase[i+1]='_';
            i++;
        }
    }
    if (a=='i'){
        for(i=0;i<l;i++){
            i++;
            frase[i-1]='_';
        }
    }
    system("cls");
 
 
    return;
}
//Funcion MenuPrincipal, donde podremos elegir una opcion//
void MenuPrincipal (char frase[]){
    system("cls");
    char seleccion;
    printf("\nCadena introducida : %s\n\n",frase);
    do{
        printf("\nMENU:");
        printf("\n1) Pasar a mayusculas o a minusculas");
        printf("\n2) Cambiar caracteres aleatoriamente");
        printf("\n3) Invertir cadena");
        printf("\n4) Eliminar caracteres pares e impares");
        printf("\n5) Estadisticas");
        printf("\n6) Volver a introducir una cadena");
        printf("\n7) Salir");
        printf("\nSelecciona una opcion: ");
        fflush(stdin);
        seleccion=getche();
        printf("\n");
        switch(seleccion){
        case '1':
            MayusculasMinusculas(frase);
            printf("\n\nCadena Actual : %s\n\n",frase);
            break;
        case '2':
            aleatorio(frase);
            printf("\n\nCadena Actual : %s\n\n",frase);
            break;
        case '3':
            Invertir(frase);
            printf("\n\nCadena Actual : %s\n\n",frase);
            break;
        case '4':
            EliminarParesImpares(frase);
            printf("\n\nCadena Actual : %s\n\n",frase);
            break;
        case '5':
            Estadisticas(frase);
            break;
        case '6':
            repetir(frase);
            break;
        case '7':
            break;
        }
    }while(seleccion!='7');
    return;
}
//Funcion para introducir cadena y llamar a menu principal//
void IntroducirCadena(){
 
    char frase[100];
    printf("\nIntroduzca una frase:\n");
    gets(frase);
    fflush(stdin);
    MenuPrincipal(frase);
    return;
}
//Funcion principal donde llamamos a IntroducirCadena//
int main(void){
    IntroducirCadena();
    return(0);
}
 

P.D: En cada proceso que se realiza, la variable frase tiene que quedarse modificada para volver a trabajar con ella, es decir:
hola----(pares)--->oa---(invertir)---ao----.....etc.
Con las demas funciones, va perfecto, pero con la de eliminar pares o impares, con mi codigo inicial, queda con "_" lo caracteres "eliminados" de la frase, y como ya sabes, necesito que elimine el caracter realmente y no sustituirlo.
Los for del codigo que me has pasado, los entiendo perfectamente, pero no consigo ver como maneja la cadena, se que recibe la cadena, y trabaja sobre una auxiliar (nueva), pero luego esta no se asigna a la variable del comienzo (frase)

P.D2: Si, he cambiado la variable a frase, no es un fallo mio, asi me acuerdo mejor.

[sam90]

El problema es que mi funcion no modifica a frase... aca tenes una variante acotado a tu ejemplo

Código C:

void  EliminarParesImpares(char * cadena )
    {
        char  nueva[100] , a;
        int l=strlen(cadena), init,i,j;
        printf("\nEliminar pares o impares <p/i>: ");
        a=getchar();
        if (a=='p') init = 0;
        else init = 1;
        j=0;
        for (i=init;i < l;  i = i +2)
            nueva[j++] = cadena[i];
        nueva[j] = '\0';
        strcpy(frase,nueva);
    }
 

[AlmaVagabunda]

No se que haria sin ti sam90!!
Funciona perfectamente.

De verdad, muchas gracias.
Cada vez aprendo mas, voy lento, pero aprendo.
Un abrazo, y como las demas veces +karma ;)

[sam90]

Lo importante es, notaste las modificaciones que le hice? las entendiste??
Declaro una nueva cadena, copia ahi todo lo necesario y luego copia la auxiliar a la cadena primaria con la funcon strcpy.

Saludos

[AlmaVagabunda]

Perdona por no haberte contestado, no vi que me dijiste algo mas :P

Si que lo entendí. Cada vez aprendo mas, y con la ayuda que me prestas, perfecto ;)

Un saludo