Emular una funcion de una cadena(Resuelto)

[Dark Invader]

Buenas a todos,estoy intentando emular la cadena strtok para que pida un caracter de la frase y lo elimine de esa cadena,quedando la cadena final sin ese caracter,lamentablemente me sale este error:

Código [Seleccionar] Expandir


In function 'void cortarcadena(char*, char*)':
incompatible types in assignment of 'char' to 'char [30]'

He ido cambiando cosas pero no se donde está el fallo,tiene que ver con asignar una cadena a un caracter,pero lo que asigno es un caracter a un caracter,pego el codigo:

Código (c) [Seleccionar] Expandir


#include <stdio.h>

void cortarcadena(char *frase,char *caracter){
   char *p = frase;
   char *q = caracter;
char c_temp;
char c_ori[30];
   while(*p!='\0'){
                      if(*q==*p){
                                 c_temp= *p;
                                 *p++;
                                 
                                 }else{
                                       c_ori=*p;
                                       *p++;
                                       
                                       }
                                       }
                                      puts(c_ori);
                                       }
                                       int main (void){
                                           char frase [30];
                                           char caracter;
                                           printf("vamos a quitar un caraacter a la frase");
                                           printf("escriba la primera frase:  ");
                                           gets(frase);
                                           printf("escriba el caracter a eliminar: ");
                                           scanf("%c",&caracter);
                                           cortarcadena(frase,&caracter);
                                           getchar();
                                           return 0;
                                           }
                                       

Gracias al que pueda ayudarme.

Salu2

[leogtz]

El problema es que no puedes asignar un array a un puntero (o indireccionando el puntero). Dinos que querías hacer para poder decirte qué hacer.

Saludos.

[Dark Invader]

Gracias por contestar Leo,

Mira lo que queria hacer es asignar un puntero al principio de la cadena,e ir recorriendola con el *p++,es decir,que fuera al siguiente elemento,el problema es ese,que no se a que es debido,porque he hecho otra funcion,la strcmp y me funciona perfecto:

Código (c) [Seleccionar] Expandir

#include <stdio.h>

void compararcadenas(char *cadena,char *cadena2){
     char *p,*q;
     
     p=cadena;
     q=cadena2;
     while(*p!='\0' || *q!='\0'){
                   
                    if(*p==*q){
                              *p++;
                              *q++;
                              printf("\n\ntiene el mismo caracter,pasando al siguiente caracter...");
                              if (*p!='\0') continue;
                              if (*q!='\0') continue;
           
           
            printf("\n\n!!Las cadenas son las mismas!!");
                             
                              }else{
                             
                                    printf("\n\nse presenta un caracter de diferencia,las cadenas no son las mismas");
                                   
           
           
       
                                    if(*p =='\0' && *q !='\0'){
                                          printf("\n\nla segunda cadena tiene mas longitud");
                                          break;
                                          }
                                         if(*p !='\0' && *q =='\0'){
                                               printf("\n\nla primera cadena tiene mas longitud");
                                               break;}
                                               
                                                    printf("\n\n!!Las cadenas son distintas,pero tienen la misma longitud!!");break;
                                               
                                                     
                                    }
                                   }
                             
                                   
                                   }
int main (void){
                                   
   char cadena[40];
   char cadena2[40];
   
   printf("escriba la primera cadena:  ");
   gets(cadena);
   printf("escriba la segunda cadena:  ");
   gets(cadena2);
   compararcadenas(cadena,cadena2);
   getchar();
   return 0;
}

¿A qué crees que puede ser debido?

Salu2

[El_Java]

Hola, en un rato de aburrimiento te he hecho una función como la que pides porque creo que has hecho una montaña de un granito de arena.
Todo esta hecho con las librerias standar del http://www.cplusplus.com/, si quieres info del codigo preguntame o miralo ahi.
Mi código lo que hace es eliminarte un sólo caracter de la frase que le pases, se puede hacer con mas de una letra, pero no me apetecia complicarme.

Aqui te dejo el codigo:

Código (cpp) [Seleccionar] Expandir


#include <iostream>
using namespace std;

char* cortarcadena(char *frase, char caracter){
   string str(frase);
   string::iterator it;
   unsigned int num;
   num = str.find(caracter);
   it = str.begin() + num;
   if(it == str.end()) return NULL;
   str.erase(it);
   return (char*)str.c_str();

}

int main(){
   char* frase = "HOLAS";
   frase = cortarcadena(frase, 'A');
   cout << frase << endl;
   return 0;
}


Un saludo!

[Dark Invader]

Hola El_Java  y gracias por contestar tambien,

El problema es que no domino muy bien C++ todavia,y queria que mi codigo fuera en C,porque las cadenas en C++ no la tengo muy controladas,solo queria ver exactamente donde esta el fallo de mi codigo.

De todas maneras muchas gracias.

Salu2

[leogtz]

Mira lo que queria hacer es asignar un puntero al principio de la cadena,e ir recorriendola con el *p++,es decir,que fuera al siguiente elemento

Así es, pero bueno, el error está aquí:

Código [Seleccionar] Expandir

c_ori=*p;
¿Qué es lo que querías hacer?

[Dark Invader]

Resuelto,era lo que habias dicho Leo,esta poniendo una cadena pero no le estaba poniendo el numero de argumentos,y no los incrementaba,por eso he hecho un nuevo puntero a la cadena origen y ahora si compila,muchas gracias colega 

Código (c) [Seleccionar] Expandir


#include <stdio.h>

void cortarcadena(char *frase,char *caracter){
    char *p;
    char *q;
    char *r;
    p=frase;
    q=caracter;
char c_temp;
char c_ori[30]="";
r =c_ori;
    while(*p!='\0'){
                       if(*q==*p){
                                  c_temp= *p;
                                  *p++;
                                 
                                  }else{
                                       
                                        *r=*p;
                                        *p++;
                                        *r++;
                                        }
                                       
                                        }
                                       puts(c_ori);
                                       getchar();
                                        }
                                        int main (void){
                                            char frase [30];
                                            char caracter;
                                            printf("vamos a quitar un caraacter a la frase");
                                            printf("escriba la primera frase:  ");
                                            gets(frase);
                                            printf("escriba el caracter a eliminar: ");
                                            scanf("%c",&caracter);
                                            cortarcadena(frase,&caracter);
                                            getchar();
                                            return 0;
                                            }
                                       
                       
Tema resuelto 

Salu2
   

[leogtz]

@Dar Invader

He probado la función, funciona (valga la redundancia), hay cosas que comentar, en primera está el indentado del código, está muy mal, debes corregirlo.

Y la función se puede acortar a esto (usando tu manera):

Código (c) [Seleccionar] Expandir

void cortarcadena(char *frase,char *caracter)
{
   char *p;
   char *q;
   char *r;
   p = frase;
   q = caracter;
   char c_temp;
   char c_ori[30]="";
   r = c_ori;
   while(*p)
       if(*q == *p)
           c_temp= *p++;
       else
           *r++ = *p++;
   puts(c_ori);
   getchar();
}


Código (c) [Seleccionar] Expandir

void cortarcadena(char *frase,char *caracter)
{
   char *p = frase;
   char *q = caracter;
   char *r;
   char c_temp;
   char c_ori[30]="";
   r = c_ori;
   while(*p)
       *q == *p ? c_temp = *p++ : (*r++ = *p++);

   puts(c_ori);
   getchar();
}

[Dark Invader]

Tienes toda la razon,se puede acortar muchisimo,y parece mucho mas ordenador,gracias de nuevo colega 

Salu2

[leogtz]

Aún te puedes ahorrar una variable, no hace falta que maniobres el caracter con un puntero ya que es una sola unidad, lo puedes omitir.